-
Notifications
You must be signed in to change notification settings - Fork 1.7k
fix(ci): e2e dd logs failure #23038
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.
B 10000 y 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): e2e dd logs failure #23038
Conversation
081aa67
to
b522745
Compare
Datadog ReportBranch report: ✅ 0 Failed, 8 Passed, 0 Skipped, 25.45s Total Time |
ef4fc9d
to
463d266
Compare
463d266
to
35207ff
Compare
35207ff
to
5ac1977
Compare
5ac1977
to
b92e7e4
Compare
This comment was marked as outdated.
This comment was marked as outdated.
a5efc71
to
2766b67
Compare
ef7198f
to
59c0ab0
Compare
4e07efd
to
414690d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/ci-run-e2e-datadog-logs
4c2276a
to
a82d61d
Compare
a82d61d
to
efccefb
Compare
efccefb
to
8f4fab3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/ci-run-integration-datadog-logs
This didn't fix the original CI failure but it has multiple desirable changes that will help debugging in the future. Merged this manually and on the next failure I will investigate again. |
Summary
Changes:
dd_url
is specified in DD agent configscripts/int-e2e-test.sh
can now be used for local iterationsChange Type
Is this a breaking change?
How did you test this PR?
Run the following on my machine and on an EC2 instance:
Result:
Does this PR include user facing changes?
Notes
@vectordotdev/vector
to reach out to us regarding this PR.pre-push
hook, please see this template.cargo fmt --all
cargo clippy --workspace --all-targets -- -D warnings
cargo nextest run --workspace
(alternatively, you can runcargo test --all
)./scripts/check_changelog_fragments.sh
git merge origin master
andgit push
.Cargo.lock
), pleaserun
cargo vdev build licenses
to regenerate the license inventory and commit the changes (if any). More details here.References