Modify

Ticket #1152 (accepted defect)

Opened 6 years ago

Last modified 5 years ago

RPM package verification failures on configuration files causes bcfg2 to reinstall package

Reported by: https://www.google.com/accounts/o8/id?id=AItOawn0s7RDAZJqmy8qhaS8PFtgoke6VxxZKSI Owned by: solj
Priority: major Milestone: Bcfg2 1.3.5 Release (Bugfix)
Component: bcfg2-client Version: 1.0
Keywords: YUM Cc: rjsm@…, jsbillin@…

Description

Hello,

For packages that have configuration files (specifically %config(noreplace) config files), when Bcfg2 runs in non-quick mode, Bcfg2 marks the package as needing a reinstall when a config file fails verification. When it reinstalls the package, it doesn't replace the file, because the package marks it as %config(noreplace). So, next time Bcfg2 runs, it will continue to mark the package as needing to be reinstalled. The only solution is to mark every config file as "ignored" Path entries.

Ideally, Bcfg2 should ignore files that are config files (or at least, %config(noreplace) config files, if that's possible) when evaluating which packages need to be reinstalled. This would match standard RPM/Yum behavior.

Attachments

0001-YUM-Add-a-check-to-ignore-verification-failures-on-c.patch (1.4 KB) - added by https://www.google.com/accounts/o8/id?id=AItOawn0s7RDAZJqmy8qhaS8PFtgoke6VxxZKSI 6 years ago.
Possible fix for this behavior
bcfg2-yum-verify-check-configuration.patch (2.8 KB) - added by https://www.google.com/accounts/o8/id?id=AItOawn0s7RDAZJqmy8qhaS8PFtgoke6VxxZKSI 6 years ago.
A better patch that covers a more limited set of changes

Change History

Changed 6 years ago by https://www.google.com/accounts/o8/id?id=AItOawn0s7RDAZJqmy8qhaS8PFtgoke6VxxZKSI

Possible fix for this behavior

Changed 6 years ago by https://www.google.com/accounts/o8/id?id=AItOawn0s7RDAZJqmy8qhaS8PFtgoke6VxxZKSI

A better patch that covers a more limited set of changes

comment:1 follow-up: ↓ 2 Changed 6 years ago by https://www.google.com/accounts/o8/id?id=AItOawn0s7RDAZJqmy8qhaS8PFtgoke6VxxZKSI

I submitted a patch that only ignores changed config files if their size, checksum, mode or ownership differ. I'm testing it now.

comment:2 in reply to: ↑ 1 Changed 5 years ago by solj

  • Owner changed from desai to solj
  • Status changed from new to accepted

Replying to https://www.google.com/accounts/o8/id?id=AItOawn0s7RDAZJqmy8qhaS8PFtgoke6VxxZKSI:

I submitted a patch that only ignores changed config files if their size, checksum, mode or ownership differ. I'm testing it now.

Were you ever able to verify whether or not this works?

comment:3 Changed 5 years ago by https://www.google.com/accounts/o8/id?id=AItOawn0s7RDAZJqmy8qhaS8PFtgoke6VxxZKSI

I had the package in testing, but I've since upgraded to newer releases without the patch. I'll rebase and build a new package for testing.

WARNING! You need to establish a session before you can create or edit tickets. Otherwise the ticket will get treated as spam.

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as accepted
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.