-
Notifications
You must be signed in to change notification settings - Fork 204
[8.19] (backport #8605) [Synthetics] Add e2e test for synthetics deps in complete variants #8679
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
Conversation
|
Cherry-pick of f8c1f2e has failed: To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
|
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
|
Pinging @elastic/obs-ds-hosted-services (Team:obs-ds-hosted-services) |
|
@emilioalvap as author of the original PR #8605, could you please fix the backports ? Thank you |
|
💛 Build succeeded, but was flaky
Failed CI StepsHistory
cc @emilioalvap |
|
This pull request has not been merged yet. Could you please review and merge it @emilioalvap? 🙏 |
|
@pchila Thanks for the reminder. This should be fixed now, please review. |





What does this PR do?
Add tests to check synthetics dependencies for complete variant docker packages. These will prevent the possibility of an unrelated change removing the required dependencies.
Why is it important?
We have seen instances where unrelated changes to the docker template have caused the generated
-completevariants to be missing synthetics dependencies.Checklist
[] I have made corresponding changes to the documentation[ ] I have made corresponding change to the default configuration files[ ] I have added an entry in./changelog/fragmentsusing the changelog toolHow to test this PR locally
Related issues
This is an automatic backport of pull request [Synthetics] Add e2e test for synthetics deps in complete variants #8605 done by Mergify.