content icon indicating copy to clipboard operation
content copied to clipboard

Defined notes and rules for BSI APP.4.4.A19

Open benruland opened this issue 1 year ago • 9 comments
trafficstars

Description: Notes / Rules for BSI APP4.4.A19

Rationale: As we have multiple customers asking for a BSI profile to be included in the compliance-operator, we are contributing a profile. To provide a better review process, the individual controle are implemented as separate PRs.

This is a follow-up of https://github.com/ComplianceAsCode/content/pull/11659. It was breaken up for better reviewability.

benruland avatar Jul 15 '24 13:07 benruland

Hi @benruland. Thanks for your PR.

I'm waiting for a ComplianceAsCode member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

openshift-ci[bot] avatar Jul 15 '24 13:07 openshift-ci[bot]

Start a new ephemeral environment with changes proposed in this pull request:

ocp4 (from CTF) Environment (using Fedora as testing environment) Open in Gitpod

Fedora Testing Environment Open in Gitpod

Oracle Linux 8 Environment Open in Gitpod

github-actions[bot] avatar Jul 15 '24 13:07 github-actions[bot]

:robot: A k8s content image for this PR is available at: ghcr.io/complianceascode/k8scontent:12155 This image was built from commit: bd743c5ab68957ab9cba0d49512ab663b77c4543

Click here to see how to deploy it

If you alread have Compliance Operator deployed: utils/build_ds_container.py -i ghcr.io/complianceascode/k8scontent:12155

Otherwise deploy the content and operator together by checking out ComplianceAsCode/compliance-operator and: CONTENT_IMAGE=ghcr.io/complianceascode/k8scontent:12155 make deploy-local

github-actions[bot] avatar Jul 15 '24 13:07 github-actions[bot]

@benruland Hi, do you plan to act on @sluetze's feedback? What should we do here?

yuumasato avatar Sep 17 '24 14:09 yuumasato

@benruland Hi, do you plan to act on @sluetze's feedback? What should we do here?

Hey @yuumasato, sorry for the absence! I was swamped in projects and personal obligations... In two weeks, I will return and address the feedback.

benruland avatar Sep 20 '24 18:09 benruland

Hi @yuumasato, I adjusted this PR and also https://github.com/ComplianceAsCode/content/pull/12153 and https://github.com/ComplianceAsCode/content/pull/12154 according to the review.

Could you take a look if it looks fine?

benruland avatar Oct 04 '24 09:10 benruland

/packit test

yuumasato avatar Oct 14 '24 08:10 yuumasato

/ok-to-test

yuumasato avatar Oct 14 '24 08:10 yuumasato

/test 4.17-e2e-aws-ocp4-bsi /test 4.17-e2e-aws-ocp4-bsi-node /test 4.17-e2e-aws-rhcos4-bsi

yuumasato avatar Nov 28 '24 20:11 yuumasato

@benruland: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/4.17-e2e-aws-ocp4-bsi bd743c5ab68957ab9cba0d49512ab663b77c4543 link true /test 4.17-e2e-aws-ocp4-bsi

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

openshift-ci[bot] avatar Nov 28 '24 21:11 openshift-ci[bot]

Code Climate has analyzed commit a7b9d817 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 61.6% (0.0% change).

View more on Code Climate.

qlty-cloud-legacy[bot] avatar Jan 07 '25 08:01 qlty-cloud-legacy[bot]

/retest

yuumasato avatar Feb 13 '25 11:02 yuumasato

/ok-to-test

yuumasato avatar Feb 13 '25 11:02 yuumasato

/retest

yuumasato avatar Feb 13 '25 13:02 yuumasato

/ok-to-test

yuumasato avatar Feb 18 '25 15:02 yuumasato