Skip to content

[Defend Workflows][Fleet] Hide uninstall token for Cloud Agent Policy#167627

Merged
dasansol92 merged 1 commit intoelastic:mainfrom
gergoabraham:fix/hide-elastic-cloud-agent-policy-uninstall-token
Oct 2, 2023
Merged

[Defend Workflows][Fleet] Hide uninstall token for Cloud Agent Policy#167627
dasansol92 merged 1 commit intoelastic:mainfrom
gergoabraham:fix/hide-elastic-cloud-agent-policy-uninstall-token

Conversation

@gergoabraham
Copy link
Copy Markdown
Contributor

@gergoabraham gergoabraham commented Sep 29, 2023

Summary

fixes #164013

The API does not return uninstall token for the policy id policy-elastic-agent-on-cloud

questions

  • should it omit uninstall tokens for other managed policies, too?
  • or should we not generate uninstall tokens for managed policies at all?

@gergoabraham gergoabraham added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:Fleet Team label for Observability Data Collection Fleet team Team:Defend Workflows “EDR Workflows” sub-team of Security Solution ci:cloud-deploy Create or update a Cloud deployment labels Sep 29, 2023
@gergoabraham gergoabraham self-assigned this Sep 29, 2023
@kibana-ci
Copy link
Copy Markdown

kibana-ci commented Sep 29, 2023

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @gergoabraham

@gergoabraham gergoabraham marked this pull request as ready for review September 29, 2023 14:51
@gergoabraham gergoabraham requested a review from a team as a code owner September 29, 2023 14:51
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/fleet (Team:Fleet)

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/security-defend-workflows (Team:Defend Workflows)

field: `${UNINSTALL_TOKENS_SAVED_OBJECT_TYPE}.attributes.policy_id`,
size: bucketSize,
include,
exclude: 'policy-elastic-agent-on-cloud', // todo: find a better way to not return or even generate token for managed policies
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add a test for this?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will do in a follow-up pr

Copy link
Copy Markdown
Contributor

@juliaElastic juliaElastic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dasansol92 dasansol92 merged commit 0ae3b23 into elastic:main Oct 2, 2023
@gergoabraham gergoabraham deleted the fix/hide-elastic-cloud-agent-policy-uninstall-token branch October 9, 2023 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting ci:cloud-deploy Create or update a Cloud deployment release_note:skip Skip the PR/issue when compiling release notes Team:Defend Workflows “EDR Workflows” sub-team of Security Solution Team:Fleet Team label for Observability Data Collection Fleet team v8.11.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fleet]: Uninstall token is visible for Elastic Cloud agent policy under Uninstall tokens tab.

6 participants