Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(ci): fix upload test results to DD in int test workflows #17905

Closed
wants to merge 2 commits into from

Conversation

neuronull
Copy link
Contributor

The upload to DD in integration test workflows was broken because the retry GHA used, does not allow forwarding env vars to the shell that executes the command to retry.

@netlify
Copy link

netlify bot commented Jul 7, 2023

Deploy Preview for vector-project canceled.

Name Link
🔨 Latest commit c8fb952
🔍 Latest deploy log https://app.netlify.com/sites/vector-project/deploys/64a84c1b9760a600087114dc

@neuronull neuronull self-assigned this Jul 7, 2023
@netlify
Copy link

netlify bot commented Jul 7, 2023

Deploy Preview for vrl-playground canceled.

Name Link
🔨 Latest commit c8fb952
🔍 Latest deploy log https://app.netlify.com/sites/vrl-playground/deploys/64a84c1ba1f6de000868e42e

@neuronull neuronull added the domain: ci Anything related to Vector's CI environment label Jul 7, 2023
@neuronull neuronull marked this pull request as ready for review July 7, 2023 17:10
@datadog-vectordotdev
Copy link

datadog-vectordotdev bot commented Jul 7, 2023

Datadog Report

Branch report: neuronull/ci_fix_upload_test_results_to_dd
Commit report: 16a2453

vector: 0 Failed, 0 New Flaky, 1910 Passed, 0 Skipped, 1m 29.47s Wall Time

@neuronull
Copy link
Contributor Author

After looking into this more with Jesse, we discovered this will not actually fix the root cause.
I'l submit a separate PR soon.

@neuronull neuronull closed this Jul 7, 2023
@neuronull neuronull deleted the neuronull/ci_fix_upload_test_results_to_dd branch July 7, 2023 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain: ci Anything related to Vector's CI environment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants