-
Notifications
You must be signed in to change notification settings - Fork 21
updated the readme to include the security metrics integration part #1784
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
base: main
Are you sure you want to change the base?
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1784 +/- ##
=======================================
Coverage 78.49% 78.49%
=======================================
Files 147 147
Lines 12255 12255
=======================================
Hits 9619 9619
Misses 2260 2260
Partials 376 376
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| go build -o ./engine cmd/engine/engine.go | ||
| ``` | ||
|
|
||
| ## Security metrics integration |
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.
This section contains valuable info! Still, I think it is too detailed for the general Readme; let's move this part to the policies folder and instead just add a short paragraph about the metrics repo here.
| > Note: We are currently preparing a `v2` release of Clouditor, which will be somewhat incompatible with regards to storage to `v1`. The APIs will remain largely the same, but will be improved and cleaned. We will regularly release pre-release `v2` versions, but do not have a concrete time-frame for a stable `v2` yet. | ||
| > | ||
| > If you are looking for a stable version, please use the [v1.10.1](https://github.com/clouditor/clouditor/releases/tag/v10.10.1) release. | ||
| > If you are looking for a stable version, please use the [v1.10.1](https://github.com/clouditor/clouditor/releases/tag/v1.10.1) release. |
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.
| - granular report of detected non-compliant configurations | ||
| - quick and adaptive integration with existing service through automated service discovery | ||
| - descriptive development of custom rules using [Cloud Compliance Language (CCL)](clouditor-engine-azure/src/main/resources/rules/azure/compute/vm-data-encryption.md) to support individual evaluation scenarios | ||
| - curated security metrics integrated from the external Security Metrics repository |
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.
Please add a link to the security metrics repo
| ## Quick start with UI | ||
|
|
||
| In order to just build and run the Clouditor, without generating the protobuf file, one can use the `run-engine-with-ui.sh` script. This still requires Go and Node.js to be installed. For example, to run the engine in-memory with the Azure provider the following command can be used: | ||
| To quickly build and run Clouditor without generating protobuf files, you can use the `run-engine-with-ui.sh` script. This still requires Go and Node.js to be installed. For example, to run the engine in memory with the Azure provider, use: |
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.
Here, 'in-memory' must be used.
| git add policies/security-metrics && git commit -m "chore: bump security-metrics" | ||
| ``` | ||
|
|
||
| Notes: |
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.
I think the first bullet point is important, the other two we do not need here.

related to issue : #1763