-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Revert "[AWS Cloudwatch] Fix Tags for APIGW" #41109
Conversation
This reverts commit 5aca19c.
This pull request doesn't have a |
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
|
CI is green on this PR but I don't see the https://buildkite.com/elastic/beats-packaging-pipeline step being run on this PR. Pinged @rowlandgeoff off-PR to ask why that might be. Without knowing if this step succeeds on this PR, its hard to say with 100% confidence that merging this PR will make |
@axw We might need your review on this to unblock Beats CI |
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.
Approving to hopefully unblock CI, but there's nothing unusual in this PR, so seems like something more systemic needs addressing.
Thanks @axw. Yeah, nothing in the PR jumps out as the cause but this failure is (at least so far) only being observed with #40755. Since we've merged this revert PR here, we haven't seen the same failure on |
Reverts #40755 because since that PR was merged, we're seeing the https://buildkite.com/elastic/beats-packaging-pipeline consistently failing like so when it tries to run
mage package
on Agentbeat: