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 subscription file for ROSA installs #12004

Merged

Conversation

rhmdnd
Copy link
Collaborator

@rhmdnd rhmdnd commented May 17, 2024

Since ROSA clusters only have worker nodes, we need to modify the
subscription to allow the operator to schedule on worker nodes.

Without this, ROSA installations will spin waiting for master nodes to
schedule on.

This will get wired up to CI through the ocp4e2e test suite in a
separate PR:

ComplianceAsCode/ocp4e2e#41

Since ROSA clusters only have worker nodes, we need to modify the
subscription to allow the operator to schedule on worker nodes.

Without this, ROSA installations will spin waiting for master nodes to
schedule on.

This will get wired up to CI through the ocp4e2e test suite in a
separate PR:

  ComplianceAsCode/ocp4e2e#41
Copy link

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

Fedora Environment
Open in Gitpod

Oracle Linux 8 Environment
Open in Gitpod

Copy link

🤖 A k8s content image for this PR is available at:
ghcr.io/complianceascode/k8scontent:12004
This image was built from commit: 7e9e212

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:12004

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

Copy link

codeclimate bot commented May 17, 2024

Code Climate has analyzed commit 7e9e212 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 59.4% (0.0% change).

View more on Code Climate.

@rhmdnd rhmdnd requested review from yuumasato and Vincent056 May 17, 2024 03:59
Copy link
Contributor

@Vincent056 Vincent056 left a comment

Choose a reason for hiding this comment

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

/lgtm

@rhmdnd
Copy link
Collaborator Author

rhmdnd commented May 17, 2024

Note to reviewers:

You should be able to test this on a ROSA cluster by applying the subscription file and the operator should come up like it normally would.

rhmdnd added a commit to rhmdnd/release that referenced this pull request May 17, 2024
We've made some changes to the Compliance Operator so that it supports
running on managed offerings. This commit wires up testing CIS and
PCI-DSS node profiles on ROSA so that we can assess the compliance
posture of those profiles on the platform.

The following related PRs need to land before this change will wire
things up appropriately:

  ComplianceAsCode/content#12004
  ComplianceAsCode/ocp4e2e#41
rhmdnd added a commit to rhmdnd/release that referenced this pull request May 17, 2024
We've made some changes to the Compliance Operator so that it supports
running on managed offerings. This commit wires up testing CIS and
PCI-DSS node profiles on ROSA so that we can assess the compliance
posture of those profiles on the platform.

The following related PRs need to land before this change will wire
things up appropriately:

  ComplianceAsCode/content#12004
  ComplianceAsCode/ocp4e2e#41
@rhmdnd
Copy link
Collaborator Author

rhmdnd commented May 17, 2024

/packit retest-failed

1 similar comment
@rhmdnd
Copy link
Collaborator Author

rhmdnd commented May 17, 2024

/packit retest-failed

@yuumasato yuumasato added this to the 0.1.74 milestone May 17, 2024
@yuumasato yuumasato added the OpenShift OpenShift product related. label May 17, 2024
@yuumasato yuumasato merged commit a2f3828 into ComplianceAsCode:master May 17, 2024
113 of 114 checks passed
@yuumasato yuumasato self-assigned this May 17, 2024
rhmdnd added a commit to rhmdnd/release that referenced this pull request May 17, 2024
We've made some changes to the Compliance Operator so that it supports
running on managed offerings. This commit wires up testing CIS and
PCI-DSS node profiles on ROSA so that we can assess the compliance
posture of those profiles on the platform.

The following related PRs need to land before this change will wire
things up appropriately:

  ComplianceAsCode/content#12004
  ComplianceAsCode/ocp4e2e#41
rhmdnd added a commit to rhmdnd/release that referenced this pull request May 21, 2024
We've made some changes to the Compliance Operator so that it supports
running on managed offerings. This commit wires up testing CIS and
PCI-DSS node profiles on ROSA so that we can assess the compliance
posture of those profiles on the platform.

The following related PRs need to land before this change will wire
things up appropriately:

  ComplianceAsCode/content#12004
  ComplianceAsCode/ocp4e2e#41
rhmdnd added a commit to rhmdnd/release that referenced this pull request May 21, 2024
We've made some changes to the Compliance Operator so that it supports
running on managed offerings. This commit wires up testing CIS and
PCI-DSS node profiles on ROSA so that we can assess the compliance
posture of those profiles on the platform.

The following related PRs need to land before this change will wire
things up appropriately:

  ComplianceAsCode/content#12004
  ComplianceAsCode/ocp4e2e#41
rhmdnd added a commit to rhmdnd/release that referenced this pull request May 22, 2024
We've made some changes to the Compliance Operator so that it supports
running on managed offerings. This commit wires up testing CIS and
PCI-DSS node profiles on ROSA so that we can assess the compliance
posture of those profiles on the platform.

The following related PRs need to land before this change will wire
things up appropriately:

  ComplianceAsCode/content#12004
  ComplianceAsCode/ocp4e2e#41
rhmdnd added a commit to rhmdnd/release that referenced this pull request May 23, 2024
We've made some changes to the Compliance Operator so that it supports
running on managed offerings. This commit wires up testing CIS and
PCI-DSS node profiles on ROSA so that we can assess the compliance
posture of those profiles on the platform.

The following related PRs need to land before this change will wire
things up appropriately:

  ComplianceAsCode/content#12004
  ComplianceAsCode/ocp4e2e#41
rhmdnd added a commit to rhmdnd/release that referenced this pull request May 23, 2024
We've made some changes to the Compliance Operator so that it supports
running on managed offerings. This commit wires up testing CIS and
PCI-DSS node profiles on ROSA so that we can assess the compliance
posture of those profiles on the platform.

The following related PRs need to land before this change will wire
things up appropriately:

  ComplianceAsCode/content#12004
  ComplianceAsCode/ocp4e2e#41
rhmdnd added a commit to rhmdnd/release that referenced this pull request May 23, 2024
We've made some changes to the Compliance Operator so that it supports
running on managed offerings. This commit wires up testing CIS and
PCI-DSS node profiles on ROSA so that we can assess the compliance
posture of those profiles on the platform.

The following related PRs need to land before this change will wire
things up appropriately:

  ComplianceAsCode/content#12004
  ComplianceAsCode/ocp4e2e#41
rhmdnd added a commit to rhmdnd/release that referenced this pull request May 23, 2024
We've made some changes to the Compliance Operator so that it supports
running on managed offerings. This commit wires up testing CIS and
PCI-DSS node profiles on ROSA so that we can assess the compliance
posture of those profiles on the platform.

The following related PRs need to land before this change will wire
things up appropriately:

  ComplianceAsCode/content#12004
  ComplianceAsCode/ocp4e2e#41
openshift-merge-bot bot pushed a commit to openshift/release that referenced this pull request May 23, 2024
We've made some changes to the Compliance Operator so that it supports
running on managed offerings. This commit wires up testing CIS and
PCI-DSS node profiles on ROSA so that we can assess the compliance
posture of those profiles on the platform.

The following related PRs need to land before this change will wire
things up appropriately:

  ComplianceAsCode/content#12004
  ComplianceAsCode/ocp4e2e#41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OpenShift OpenShift product related.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants