-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Cloud Posture] Adding CSP 3P support callout for Wiz integration #196025
[Cloud Posture] Adding CSP 3P support callout for Wiz integration #196025
Conversation
💔 Build Failed
Failed CI StepsTest Failures
Metrics [docs]Module Count
Async chunks
Historycc @JordanSh |
…osture-fleet-wiz-callout
Pinging @elastic/kibana-cloud-security-posture (Team:Cloud Security) |
const LS_CLOUD_POSTURE_3P_SUPPORT_WIZ_INTEGRATIONS_CALLOUT_KEY = | ||
'fleet:cloudSecurityPosture:thirdPartySupport:wizIntegrationsCallout'; | ||
|
||
const CloudPostureThirdPartySupportCallout = () => { |
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.
What about having this component in its own file so we can add it to our team ownership? I would also pass the package name to it so that all the logic was inside the component, again to eliminate the risk smth being changed outside our our ownership. Ofc there always be the risk that the call of the component is deleted
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.
Sure, makes sense
Pinging @elastic/fleet (Team:Fleet) |
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.
Looks good, agree with Maxim's comment about moving it to a separate file.
Starting backport for target branches: 8.x |
Starting backport for target branches: 8.x |
Starting backport for target branches: 8.x |
Starting backport for target branches: 8.x |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
1 similar comment
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
Starting backport for target branches: 8.x |
Starting backport for target branches: 8.x |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
Starting backport for target branches: 8.x |
Starting backport for target branches: 8.x |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
1 similar comment
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…astic#196025) (cherry picked from commit ebe16fa)
…on (#196025) (#196274) # Backport This will backport the following commits from `main` to `8.x`: - [[Cloud Posture] Adding CSP 3P support callout for Wiz integration (#196025)](#196025) <!--- Backport version: 8.9.8 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Jordan","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-14T12:35:25Z","message":"[Cloud Posture] Adding CSP 3P support callout for Wiz integration (#196025)","sha":"ebe16fa467d6e35e2398ad724780db05c27779cf","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Fleet","v9.0.0","Team:Cloud Security","backport:prev-minor"],"number":196025,"url":"https://github.com/elastic/kibana/pull/196025","mergeCommit":{"message":"[Cloud Posture] Adding CSP 3P support callout for Wiz integration (#196025)","sha":"ebe16fa467d6e35e2398ad724780db05c27779cf"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196025","number":196025,"mergeCommit":{"message":"[Cloud Posture] Adding CSP 3P support callout for Wiz integration (#196025)","sha":"ebe16fa467d6e35e2398ad724780db05c27779cf"}}]}] BACKPORT--> --------- Co-authored-by: kibanamachine <[email protected]>
Summary
Part of #187230
Callout will only be displayed on Wiz integration and on the condition that the user has not dismissed the callout already
Screen.Recording.2024-10-13.at.15.33.34.mov