-
Notifications
You must be signed in to change notification settings - Fork 154
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
Remove use of artifacts-api.elastic.co and its client #5050
Remove use of artifacts-api.elastic.co and its client #5050
Conversation
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
Pinging @elastic/elastic-agent (Team:Elastic-Agent) |
This pull request does not have a backport label. Could you fix it @rdner? 🙏
NOTE: |
This pull request is now in conflicts. Could you fix it? 🙏
|
2a2ba2e
to
899434d
Compare
This includes removal of TestStandaloneDowngradeToSpecificSnapshotBuild integration test.
899434d
to
b8336a4
Compare
Quality Gate passedIssues Measures |
@cmacknz are we still doing this? |
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.
I'm fine with it, this leaves some infrequently and not critical path code without a test, but eliminates the most frequent test failure.
This includes removal of
TestStandaloneDowngradeToSpecificSnapshotBuild integration test.
Discussed with @pchila and @cmacknz
artifacts-api.elastic.co
and that's the only code that is still using this API.Closes #4347