Skip to content

[Cloud Posture] Compliance by CIS section table#145114

Merged
JordanSh merged 37 commits intoelastic:mainfrom
JordanSh:cis-section-table
Nov 14, 2022
Merged

[Cloud Posture] Compliance by CIS section table#145114
JordanSh merged 37 commits intoelastic:mainfrom
JordanSh:cis-section-table

Conversation

@JordanSh
Copy link
Contributor

@JordanSh JordanSh commented Nov 14, 2022

Summary

Resolves #143938

image

  • replaced colors for passed and failed statuses according to latest figma design
  • new compliance score table
  • compact view for cluster cards
  • sorted by compliance score, query fetching the 5 lowest scores
  • black text with underline on hover
  • view all button
  • tooltip on hover shows passed findings / total findings
  • always displays items, including only passed sections (used to be filtered out)
Screen.Recording.2022-11-14.at.20.58.23.mov

@JordanSh JordanSh self-assigned this Nov 14, 2022
@JordanSh JordanSh added release_note:skip Skip the PR/issue when compiling release notes Team:Cloud Security Cloud Security team related labels Nov 14, 2022
@JordanSh JordanSh requested a review from orouz November 14, 2022 19:11
@JordanSh JordanSh marked this pull request as ready for review November 14, 2022 19:11
@JordanSh JordanSh requested a review from a team as a code owner November 14, 2022 19:11
@elasticmachine
Copy link
Contributor

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

sort_by_score: {
bucket_sort: {
sort: {
score: 'asc' as 'asc',
Copy link
Contributor Author

@JordanSh JordanSh Nov 14, 2022

Choose a reason for hiding this comment

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

@orouz 🧙

Copy link
Contributor

@orouz orouz left a comment

Choose a reason for hiding this comment

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

LGTM 🚢

Comment on lines +72 to +73
it('return sorted array with the correct number of elements', () => {
expect(getTopRisks(mockData, 5)).toEqual([etcdAgg, apiAgg, podsAgg, serverAgg, systemAgg]);
Copy link
Contributor

Choose a reason for hiding this comment

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

i think this is tested in previous case too

height: euiThemeVars.euiSizeXS,
borderRadius: euiThemeVars.euiBorderRadius,
overflow: 'hidden',
gap: 1,
Copy link
Contributor

Choose a reason for hiding this comment

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

gap takes a <length> value, so 1 shouldn't be valid

@JordanSh JordanSh enabled auto-merge (squash) November 14, 2022 20:16
@kibana-ci
Copy link

💚 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 127.1KB 128.1KB +1.0KB
Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 59 65 +6
osquery 108 113 +5
securitySolution 441 447 +6
total +19

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 67 73 +6
osquery 109 115 +6
securitySolution 518 524 +6
total +20

History

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

cc @JordanSh

@JordanSh JordanSh merged commit b72a9a3 into elastic:main Nov 14, 2022
@kibanamachine kibanamachine added v8.6.0 backport:skip This PR does not require backporting labels Nov 14, 2022
jloleysens added a commit to jloleysens/kibana that referenced this pull request Nov 15, 2022
* main: (65 commits)
  Migrate server-side `Root` and `Server` to packages (elastic#144990)
  [Discover] Handle no data views state for `esQuery` alert (elastic#145052)
  [ML] Allow updates for number of allocations and priority for trained model deployments (elastic#144704)
  [api-docs] 2022-11-15 Daily api_docs build (elastic#145203)
  [Security solution] remove guided onboarding feature flag (elastic#144247)
  [DOCS] Automate final case APIs (elastic#145007)
  [Enterprise Search] Name and description flyout for connectors (elastic#143827)
  [Guided onboarding] Update header button logic (elastic#144634)
  [Lens] Multi metric partition charts (elastic#143966)
  [Dashboard] [Controls] Add unmapped runtime field support to options list (elastic#144947)
  [Security Solution] Add Task Metric Collection to New Tasks (elastic#145181)
  [TriggersActionsUi] disable jest config in CI (elastic#145186)
  [TableListView] Enhance tag filtering (elastic#142108)
  [Cloud Posture] Compliance by CIS section table (elastic#145114)
  [8.6][Session View] Fix hidden alert flyout  in session view (elastic#145141)
  [customIntegrations] async load all components (elastic#145166)
  Fix time for logs smoke tests in integration test (elastic#145130)
  [RAM] Update rule status (elastic#140882)
  Update babel (main) (elastic#145060)
  [Actionable Observability] Add context.alertDetailsUrl variable to action connector template for APM rule types (elastic#144791)
  ...
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 release_note:skip Skip the PR/issue when compiling release notes Team:Cloud Security Cloud Security team related v8.6.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Cloud Posture] Dashboard summary section - Compliance by CIS Section table

5 participants