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

OCP4: Don't use a list in platforms #8852

Merged
merged 1 commit into from
Jun 3, 2022

Conversation

jhrozek
Copy link
Collaborator

@jhrozek jhrozek commented May 29, 2022

Description:

The list notation doesn't work anymore, let's use a boolean "sentence"
instead.

Rationale:

Related: #8833

@github-actions
Copy link

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

Open in Gitpod

@jhrozek jhrozek force-pushed the no_list_in_platforms branch from 7a75beb to ffde1e4 Compare May 30, 2022 09:04
@jhrozek
Copy link
Collaborator Author

jhrozek commented May 30, 2022

/hold
we can't push this yet, it seems like the platform are still not generated correctly.

@openshift-ci openshift-ci bot added the do-not-merge/hold Used by openshift-ci-robot bot. label May 30, 2022
@JAORMX
Copy link
Contributor

JAORMX commented May 31, 2022

@jhrozek what's the issue?

@evgenyz
Copy link
Member

evgenyz commented May 31, 2022

@jhrozek what's the issue?

#8853

The list notation doesn't work anymore, let's use a boolean "sentence"
instead.

Related: ComplianceAsCode#8833
@jhrozek jhrozek force-pushed the no_list_in_platforms branch from ffde1e4 to c46cd0b Compare June 3, 2022 13:09
@jhrozek
Copy link
Collaborator Author

jhrozek commented Jun 3, 2022

/hold cancel
This should now work

@openshift-ci openshift-ci bot removed the do-not-merge/hold Used by openshift-ci-robot bot. label Jun 3, 2022
@codeclimate
Copy link

codeclimate bot commented Jun 3, 2022

An error occurred when fetching issues.

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 26.1% (0.0% change).

View more on Code Climate.

@jhrozek
Copy link
Collaborator Author

jhrozek commented Jun 3, 2022

/retest
cluster installation and remediation application timed out, respectively

@jhrozek
Copy link
Collaborator Author

jhrozek commented Jun 3, 2022

The code change is trivial and was previously acked.
Merging.

@jhrozek jhrozek merged commit 5a7b95a into ComplianceAsCode:master Jun 3, 2022
@jan-cerny jan-cerny added this to the 0.1.63 milestone Jun 15, 2022
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.

4 participants