Skip to content
This repository was archived by the owner on Oct 14, 2024. It is now read-only.

refactor: extend misconfiguration API and UI models #1172

Merged
merged 4 commits into from
Feb 10, 2024

Conversation

akijakya
Copy link
Contributor

@akijakya akijakya commented Feb 8, 2024

Description

As part of #1124 epic, updated some of the fields of the Misconfiguration object so they are generic enough to be used by the CIS Docker scanner as well.

These fields got updated in the database and the UI as well.

Screenshot of the UI changes:

Screenshot 2024-02-09 at 11 41 30

Type of Change

[ ] Bug Fix
[ ] New Feature
[X] Breaking Change
[X] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

@akijakya akijakya added the component:api Issues related to API label Feb 8, 2024
@akijakya akijakya self-assigned this Feb 8, 2024
@akijakya akijakya changed the title refactor: update misconfiguration model refactor: extend misconfiguration API model and update related UI components Feb 8, 2024
@akijakya akijakya force-pushed the extend-misconfiguration-model branch from cc0f6e8 to f6a9678 Compare February 8, 2024 11:48

This comment has been minimized.

@akijakya akijakya force-pushed the extend-misconfiguration-model branch from f6a9678 to c3453d6 Compare February 8, 2024 13:17

This comment has been minimized.

@akijakya akijakya force-pushed the extend-misconfiguration-model branch from c3453d6 to a3a851f Compare February 8, 2024 14:02
@akijakya akijakya marked this pull request as ready for review February 8, 2024 14:03
@akijakya akijakya requested a review from a team as a code owner February 8, 2024 14:03

This comment has been minimized.

@ramizpolic ramizpolic changed the title refactor: extend misconfiguration API model and update related UI components refactor: extend misconfiguration API models Feb 9, 2024
@ramizpolic ramizpolic changed the title refactor: extend misconfiguration API models refactor: extend misconfiguration API and UI models Feb 9, 2024
Copy link
Member

@ramizpolic ramizpolic left a comment

Choose a reason for hiding this comment

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

lgtm! thanks @akijakya 🚀

@ramizpolic ramizpolic force-pushed the extend-misconfiguration-model branch from a3a851f to 833e725 Compare February 9, 2024 15:29

This comment has been minimized.

@ramizpolic ramizpolic force-pushed the extend-misconfiguration-model branch from 833e725 to 04f8727 Compare February 9, 2024 17:24
@ramizpolic ramizpolic enabled auto-merge February 9, 2024 17:24
Copy link

github-actions bot commented Feb 9, 2024

Hey!

Your images are ready:

  • ghcr.io/openclarity/vmclarity-apiserver-dev:pr1172-c3e5c64257bb01fd846013bcb09a5141fcd843cf
  • ghcr.io/openclarity/vmclarity-orchestrator-dev:pr1172-c3e5c64257bb01fd846013bcb09a5141fcd843cf
  • ghcr.io/openclarity/vmclarity-ui-backend-dev:pr1172-c3e5c64257bb01fd846013bcb09a5141fcd843cf
  • ghcr.io/openclarity/vmclarity-ui-dev:pr1172-c3e5c64257bb01fd846013bcb09a5141fcd843cf
  • ghcr.io/openclarity/vmclarity-cli-dev:pr1172-c3e5c64257bb01fd846013bcb09a5141fcd843cf
  • ghcr.io/openclarity/vmclarity-cr-discovery-server-dev:pr1172-c3e5c64257bb01fd846013bcb09a5141fcd843cf

@ramizpolic ramizpolic added this pull request to the merge queue Feb 9, 2024
@chrisgacsal chrisgacsal removed this pull request from the merge queue due to a manual request Feb 9, 2024
Copy link
Contributor

@chrisgacsal chrisgacsal left a comment

Choose a reason for hiding this comment

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

Sorry for the -1, but I'd like to review this.

Copy link
Contributor

@chrisgacsal chrisgacsal left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for the patience! :shipit:

@akijakya akijakya added this pull request to the merge queue Feb 10, 2024
Merged via the queue into main with commit 84c711c Feb 10, 2024
37 checks passed
@akijakya akijakya deleted the extend-misconfiguration-model branch February 10, 2024 09:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
component:api Issues related to API
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants