Skip to content
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.

Sign up for GitHub

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

Jump to bottom

sensu-0.29.0-11.el7.x86_64.rpm conflicting with OS packages #219

Closed
portertech opened this issue Jun 23, 2017 · 8 comments
Closed

sensu-0.29.0-11.el7.x86_64.rpm conflicting with OS packages #219

portertech opened this issue Jun 23, 2017 · 8 comments
Assignees
@amdprophet

Comments

@portertech
Copy link
Contributor

This issue is a migrated/copy of sensu/sensu#1695

looks like that new sensu package is conflicting with other OS packages... since some directory are "owned" by the sensu packages, and it shouldn't be the case... Doing a
rpm -qlp sensu-0.29.0-11.el7.x86_64.rpm

indicates that, for instance,
/usr
/usr/bin
/usr/bin/sensu-install
/var

are owned by the sensu package... older versions didnt have that issue. Looks to me like the new RPM as been created using a "/*" in the %files section of the RPM spec...
Centos6.7 and older versions won't complain about it, but starting from 6.8 and 7... the RPM DB is smart enough to detect it's causing conflicts...

/tmp/sesu (root@mtl-lnx-055) $ rpm -qf /usr/bin
filesystem-3.2-21.el7.x86_64
@portertech portertech mentioned this issue Jun 23, 2017
Closed
@portertech
Copy link
Contributor Author

This issue was originally submitted by @patrickatomic

@amdprophet
Copy link
Member

We have fixed this in the past on our fork of Omnibus. I wonder if the latest package wasn't built with our fork. The issue is described in more detail in Chef's Omnibus issues chef/omnibus#663.

@amdprophet
Copy link
Member

https://github.com/sensu/omnibus/pull/4/files is the fix we had implemented before which should be working for /usr/bin. I'll try to reproduce the issue and see if appending the additional paths works.

@amdprophet
Copy link
Member

@patrickatomic can you please see if https://s3.amazonaws.com/sensu-omnibus-artifacts/el/7/x86_64/sensu-0.29.0-4000.el7.x86_64.rpm/sensu-0.29.0-4000.el7.x86_64.rpm resolves the issue for you?

@portertech portertech added this to the 1.1 milestone Jul 7, 2017
@portertech
Copy link
Contributor Author

@patrickatomic ping

@portertech
Copy link
Contributor Author

Seems related to #224

@patrickatomic
Copy link

patrickatomic commented Jul 24, 2017 via email

@cwjohnston cwjohnston mentioned this issue Jul 24, 2017
Merged
@amdprophet
Copy link
Member

amdprophet commented Jul 24, 2017 •
edited
Loading

The fix has been implemented in PR sensu/omnibus#9 and will be in the next release of Sensu (1.0.1).

@cwjohnston cwjohnston removed this from the 1.1 milestone Dec 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@amdprophet amdprophet

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@amdprophet @cwjohnston @portertech @patrickatomic