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

Add rudimentary docs for compliance check pipeline #272

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mbuechse
Copy link
Contributor

@mbuechse mbuechse commented Nov 6, 2024

No description provided.

@mbuechse mbuechse requested a review from garloff November 6, 2024 15:42
@mbuechse mbuechse linked an issue Nov 6, 2024 that may be closed by this pull request
+ - ENCRYPTED_SECRET
```

Note that you have to use the same keys as in Step 3 (that is, with special characters replaced).
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
Note that you have to use the same keys as in Step 3 (that is, with special characters replaced).
Note that you have to use the same keys as in Step 1 (that is, with special characters replaced).


3. Add your subject to the results table. This is necessary so your subject shows up in the
[compliance monitor web-site](https://compliance.sovereignit.cloud/page/table). Add the following lines
(substituting all-caps parts):
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
(substituting all-caps parts):
(substituting all-caps parts except HM):

virtualenv .venv
. .venv/bin/activate
pip install -r requirements.txt
pip install passlib argon2_cffi # these are only needed for step 2
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
pip install passlib argon2_cffi # these are only needed for step 2
pip install passlib argon2_cffi # these are only needed for Step 2


```shell
git commit -asm "Add SUBJECT_NAME"
git push # the output of this command will show you the URL for creating the pull request
Copy link
Member

Choose a reason for hiding this comment

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

This will only work if you are part of the github SCS Organization. Ask the SCS team (or other members) to add you.
Otherwise you will need to create a fork in your own github project and create a PR for the SCS team to merge from there.

continuous-integration pipeline, you may want to use some dedicated credential store and mechanism for
injecting secrets. You may want to take inspiration from our own Zuul setup by looking at
[.zuul.d](https://github.com/SovereignCloudStack/standards/tree/main/.zuul.d) and
[playbooks](https://github.com/SovereignCloudStack/standards/tree/main/playbooks).
Copy link
Member

Choose a reason for hiding this comment

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

Do we want to mention that other job schedulers like Jenkins or even cron jobs can be used as well. This avoids scaring people that are overwhelmed by the capabilities that zuul offers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document compliance-check job
2 participants