sous-chefs / selinux

Development repository for the selinux cookbook

https://supermarket.chef.io/cookbooks/selinux

Apache License 2.0

58 stars 70 forks source link

Fix kitchen test failures on centos6 for chef16/17 in chef's CI pipeline #89

Closed neha-p6 closed 1 year ago

neha-p6 commented 1 year ago

Signed-off-by: Neha Pansare neha.pansare@progress.com

Description

Centos6 doesnot support policycoreutils packages. Update condition so as not to use them on centos6.

Issues Resolved

https://github.com/sous-chefs/selinux/issues/88

Check List

[ ] A summary of changes made is included in the CHANGELOG under ## Unreleased
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable.

Stromweld commented 1 year ago

Can you fix the readme errors so the mdl job will pass and run the integration tests?

xorima commented 1 year ago

Hi @neha-p6

Welcome to Sous Chef's cookbooks. Could you add an entry into the CHANGELOG.md file under the heading ## UNRELEASED please?

Stromweld commented 1 year ago

GitHub actions for this cookbook should be locked to chef 17 since 18 has the resources built in now. That will fix the error in the integration tests. Probably worth adding note in readme as well.

ramereth commented 1 year ago

I'm going to work on properly fixing this so that it also works on 18 (which it should)

kitchen-porter commented 1 year ago

Released as: 6.0.7

neha-p6 commented 1 year ago

Thank you all! I was OOO so couldn't respond.