-
Notifications
You must be signed in to change notification settings - Fork 706
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
Make Rawhide CI Green #12065
Make Rawhide CI Green #12065
Conversation
4bdac97
to
05b70ca
Compare
🤖 A k8s content image for this PR is available at: Click here to see how to deploy itIf you alread have Compliance Operator deployed: Otherwise deploy the content and operator together by checking out ComplianceAsCode/compliance-operator and: |
54f46c4
to
11a4dee
Compare
5bef7d5
to
63eba0d
Compare
Looking at the failing
it's suspicious that the conditional is expected to have different ordering of operands than the expression |
@evgenyz Got any ideas on the CI failure that Jan mentioned above? |
I think the produced Bash and Ansible code should have the same order of operands as the platform expression. Therefore, the test should be reworked. Then, I assume it will start failing everywhere except Rawhide. It's unclear to me how to fix that. There probably is used a set or a dictionary somewhere where we should use a tuple or a list or other structure that preserves order of the elements. But I haven't found where is the place. I think it needs to be somewhere in the bundled "expressions" library. |
I think that with introduction of new Python We can either try and fix boolean.py or stop comparing expressions as strings. |
Since we use normalization of boolean expressions we can't really depend on order (and even particular set of elements) of the initial expression. |
There is an example of how one could test features of the expression rather than its string representation contents. |
63eba0d
to
f24d745
Compare
It should be updated a bit more often DockerHub, hopefully
The id is not stable and should not be used for testing
d55fde6
to
15e7703
Compare
Code Climate has analyzed commit 15e7703 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. |
Description:
Use registry.fedoraproject.org/fedora:rawhide for rawhide test
Rationale:
It should be updated a bit more often DockerHub, hopefully.