-
Notifications
You must be signed in to change notification settings - Fork 122
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
Add permission checks to all install and upgrade integration tests #4761
Conversation
Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane) |
One of the issues here is that the Elastic Agent user was renamed from |
32f0120
to
403f33b
Compare
Quality Gate passedIssues Measures |
…4761) * Fix calling enroll from install. * Refactor installation checks. * Add detailed checks. * Improve permission validation. * Fix lint. * Fix windows lint. * More lint. * Adjustments for <=8.13. * Pass context into CheckSuccess. * Fix upgradetest to only check installation for specific versions. * Inverse logic. (cherry picked from commit 07f7921)
…4761) (#4778) * Fix calling enroll from install. * Refactor installation checks. * Add detailed checks. * Improve permission validation. * Fix lint. * Fix windows lint. * More lint. * Adjustments for <=8.13. * Pass context into CheckSuccess. * Fix upgradetest to only check installation for specific versions. * Inverse logic. (cherry picked from commit 07f7921) Co-authored-by: Blake Rouse <blake.rouse@elastic.co>
What does this PR do?
Improves the integration install and upgrade tests for both Linux and Windows to ensure that the permissions set during installation and installation + enrollment are correct.
Why is it important?
Ensures that any future changes that can cause bad permissions are caught during the PR stage.
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/fragments
using the changelog tool