-
Notifications
You must be signed in to change notification settings - Fork 696
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 support for Debian 11 #7715
Conversation
Hello @mdedonno1337! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2021-10-14 16:22:24 UTC |
Hi @mdedonno1337. 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 Once the patch is verified, the new status will be reflected by the 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/test-infra repository. |
The issue #7718 is blocking this PR; with the patch proposed applied or the commit reverted, the check on a Debian 11 machine is done without errors. |
What do you think about adding a github actions job here that would build the debian (9,10,11) content and run some tests... you can basically copy paste one of the |
Never done actions on Github (only on Gitlab usually), but I will try, thanks for the idea! |
Probably only an issue with the scanner. |
The compilation of this repo require the We could What do you advice on that ? |
You can rather use debian10 to build the content. That's not a problem. I imagine you would have to use container though, something like this:
|
Ok, works for me. The PR is OK to test/merge on my side. |
The problem in tests:
is reported here: ComplianceAsCode/content-test-filtering#20 It can be addressed later. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for your contribution.
@mdedonno1337 I believe we need to update the package selection in the build job to install YAML through pip to fix this problem(the package might be too old): https://github.com/ComplianceAsCode/content/pull/7758/checks?check_run_id=3927494067
|
I will have a look on that. |
I have just created #7760 it seems to work |
It fixed one problem, but then appear another ;( |
I will have a look. |
I'm installing the same version as the one in Fedora, let's see what happens. #7760 it was probably this: yaml/pyyaml#576 it installed the version 6 of pyyaml xD |
Description:
Rationale:
Content for Debian 11 does not exists
#### Notes:- Please note that the check for the security repository in the/etc/apt/sources.list
file is not done correctly at the moment. A commit shall be done to patch this before mergin in master (I'm fighting with somesegmentation fault
in the binary at the moment...)