-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Fleet] Do not allow user to add agent to managed policy #122676
[Fleet] Do not allow user to add agent to managed policy #122676
Conversation
Pinging @elastic/fleet (Team:Fleet) |
agentPolicyId={(agentPolicy && agentPolicy.id) || ''} | ||
showAgentText | ||
/> | ||
) : agentPolicy?.is_managed ? ( |
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.
Actually this is the only behavior change in that PR
@elasticmachine merge upstream |
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.
PR change is simple, and refactoring looks good. Thanks for this. 🚀
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]Module Count
Async chunks
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: cc @nchaulet |
) (cherry picked from commit 5f76e79)
💔 Some backports could not be created
How to fixRe-run the backport manually:
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
) (cherry picked from commit 5f76e79) # Conflicts: # x-pack/plugins/fleet/public/applications/fleet/sections/agent_policy/details_page/index.tsx
Hi @nchaulet In concern to above available PR merges, I would like to update you that, the functionality was broken on Integrations tab and not on Policy tab. So the merges above made are not effective, as on policy detail page on cloud env we already have a fleet server that is in offline mode, so no 'add agent' option is available. [Steps here used are: create a deployment with APM and fleet enabled... later go back to deployment page and delete apm and fleet section. Now UI will be available as shown below.] However, If we go to Integration tab, and look for Fleet server integration, From there under Integration policies tab, we are able to add agent under Managed policies. So could you please look into this. Thanks |
Hi @nchaulet Just a Reminder that this is still pending. Whenever, new merges will be available. We will retest this. Thanks |
Hi @dikshachauhan-qasource thanks for checking on this this should have been addressed with that PR #123160 merged in 7.17 and 8.0 too |
Hi @nchaulet Thanks for info on related new PR. Further, we have validated the ticket changes under related PR and provided observations there. Thanks |
Summary
Resolve #122298
This PR hide the add agent button for managed policy in the agent policy details.
I used this as the opportunity to do some refacto and break the agent details index component in multiple components, this way we can easily add unit tests here in the future.
UI Change
managed policy
non managed
storybook