Skip to content

Drop deprecated or removed packages in RHEL8#3632

Closed
yuumasato wants to merge 6 commits intoComplianceAsCode:masterfrom
yuumasato:rhel8_remove_deprecated_packages
Closed

Drop deprecated or removed packages in RHEL8#3632
yuumasato wants to merge 6 commits intoComplianceAsCode:masterfrom
yuumasato:rhel8_remove_deprecated_packages

Conversation

@yuumasato
Copy link
Copy Markdown
Member

Description:

  • Drop deprecated or removed packages from RHEL8 CSV's.

Rationale:

  • According to Changes to packages page in Beta release notes, these packages won't be part of RHEL8.

kernel-tools
kexec-tools
libcgroup
libcgroup-tools
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@yuumasato why were these dropped? Packages still exist.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

They're mentioned that they are removed in the docs https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/8-beta/html/8.0_beta_release_notes/changes_to_packages but they exist on my RHEL8 beta, so that means the docs are wrong.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Since these packages are still shipped, I'll put them back into the list.

@redhatrises
Copy link
Copy Markdown
Contributor

@yuumasato let's just remove these from the profiles for now until we are sure that the rpms don't end up in epel repo or some other repo.

@yuumasato
Copy link
Copy Markdown
Member Author

@yuumasato let's just remove these from the profiles for now until we are sure that the rpms don't end up in epel repo or some other repo.

I was wondering about EPEL repos, if they are available there we should still track these rpms.

These packages are not available for RHEL8
- pam_pkcs11 was removed from RHEL8
- piggy-backing fix: also enable pcsc-lite for Fedora
RHEL8 doesn't have pam_pkcs11 package
RHEL8 doesn't have pam_pkcs11 package
@yuumasato yuumasato force-pushed the rhel8_remove_deprecated_packages branch from c6fa060 to 737a42b Compare December 4, 2018 10:30
@yuumasato
Copy link
Copy Markdown
Member Author

@yuumasato let's just remove these from the profiles for now

@redhatrises Maybe I don't understand what you mean, you suggest to just remove rules from the profile but leave them on CSV files?

@scrutinizer-notifier
Copy link
Copy Markdown

The inspection completed: 1 new issues

@@ -9,6 +9,7 @@ libreswan
ntp
Copy link
Copy Markdown
Member Author

@yuumasato yuumasato Dec 4, 2018

Choose a reason for hiding this comment

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

I have kept ntp here because rule service_chronyd_or_ntpd_enabled relies on check for package_ntp_installed.
chronyd_or_ntpd rules need a chronyd only rule version before ntp can be removed from this CSV.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

That could be achieved by JInja macros in the rule, OVALs and remediations.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I could filter out checks for ntp and make it work, but the rule would still be called chronyd_or_ntpd, which would be confusing...

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We can rename the rule to clock_synchronization_enabled. But I agree with keeping ntp here, it's easier.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There is already a ticket to break out these rules which should be separate rules and not rules joined by jinja.

@redhatrises
Copy link
Copy Markdown
Contributor

@yuumasato let's just remove these from the profiles for now

@redhatrises Maybe I don't understand what you mean, you suggest to just remove rules from the profile but leave them on CSV files?

Yes to remove them from the profile, but keep them enabled. Of course, we can hold off on this PR for now.

@redhatrises redhatrises self-assigned this Dec 11, 2018
@jan-cerny
Copy link
Copy Markdown
Collaborator

Hi @yuumasato @redhatrises, what is the status of this?

@yuumasato
Copy link
Copy Markdown
Member Author

Closing in favor of #3742

@yuumasato yuumasato closed this Feb 7, 2019
@yuumasato yuumasato deleted the rhel8_remove_deprecated_packages branch August 24, 2022 08:13
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