Skip to content
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 Security]Added Beta tag + improvements for CIS GCP #163663

Merged
merged 21 commits into from
Aug 11, 2023

Conversation

animehart
Copy link
Contributor

@animehart animehart commented Aug 10, 2023

Summary

  • Added Beta tag for GCP option on CSPM
  • Fix a bug where setup_access is stuck on google_cloud_shell ( This bug occur when user clicks on Manual option, click the start of Project ID name, and then click google cloud shell option again)
  • Added unit test for getCspmCloudShellDefaultValue to help with refactor later
Screenshot 2023-08-10 at 2 02 46 PM

@animehart animehart linked an issue Aug 10, 2023 that may be closed by this pull request
@animehart animehart marked this pull request as ready for review August 10, 2023 21:19
@animehart animehart requested review from a team as code owners August 10, 2023 21:19
@animehart animehart marked this pull request as draft August 10, 2023 21:20
@animehart animehart removed the request for review from a team August 10, 2023 22:56
@animehart animehart marked this pull request as ready for review August 11, 2023 00:10
@animehart animehart added release_note:skip Skip the PR/issue when compiling release notes Team:Cloud Security Cloud Security team related labels Aug 11, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-cloud-security-posture (Team:Cloud Security)

@opauloh opauloh added the ci:cloud-deploy Create or update a Cloud deployment label Aug 11, 2023
Copy link
Contributor

@opauloh opauloh left a comment

Choose a reason for hiding this comment

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

we can use flex instead of a calculated css

Comment on lines 61 to 63
flexGrow: 1,
/* Set Maximum width ONLY for Integration type buttons, we know this because only Integration Type has icons */
...(option.icon && { maxWidth: `calc(${euiTheme.size.xxs} * 115)` }),
Copy link
Contributor

Choose a reason for hiding this comment

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

we can also use the flex CSS property to achieve that:

        style: {
          flex: '1 1 0',
        },

for context, flex: '1 1 0' is a shorthand representation of these three values in the order of flex-grow, flex-shrink, and flex-basis.

So, flex: '1 1 0' means that the flex item will grow and shrink proportionally with other items with an initial size of zero, making all of the elements achieve the same width.

@kibana-ci
Copy link
Collaborator

kibana-ci commented Aug 11, 2023

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
cloudSecurityPosture 254.9KB 255.1KB +173.0B

History

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

@@ -123,3 +129,126 @@ describe('getMaxPackageName', () => {
expect(result).toBe('kspm-1');
});
});

describe('getCspmCloudShellDefaultValue', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@animehart animehart requested a review from opauloh August 11, 2023 16:18
Copy link
Contributor

@opauloh opauloh left a comment

Choose a reason for hiding this comment

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

lgtm

opauloh

This comment was marked as duplicate.

@animehart animehart merged commit 349972c into elastic:main Aug 11, 2023
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Aug 11, 2023
benakansara pushed a commit to benakansara/kibana that referenced this pull request Aug 14, 2023
…3663)

## Summary

- Added Beta tag for GCP option on CSPM
- Fix a bug where setup_access is stuck on google_cloud_shell ( This bug
occur when user clicks on Manual option, click the start of Project ID
name, and then click google cloud shell option again)
- Added unit test for getCspmCloudShellDefaultValue to help with
refactor later
<img width="950" alt="Screenshot 2023-08-10 at 2 02 46 PM"
src="https://github.com/elastic/kibana/assets/8703149/ccd45ed1-8b6c-4631-8a01-22d35d1b62aa">
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit 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:Cloud Security Cloud Security team related v8.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Cloud Security]Add Beta tag for CIS GCP option for CSPM
6 participants