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.

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

ceph.spec.in: Don't use noarch for mgr module subpackages, fix /usr/lib64/ceph/mgr dir ownership #26398

Merged
merged 2 commits into from Feb 14, 2019

Conversation

tserong
Copy link
Contributor

@tserong tserong commented Feb 13, 2019

A couple of small kinks resulting from #25977:

  1. Even though ceph-mgr modules are notionally non-architecture-specific, the files themselves are installed to an architecture-specific path (/usr/lib64/ceph/mgr/....), which causes the build to fail on SUSE distros with a huge long list of errors like:

ceph-mgr-diskprediction-cloud.noarch: E: suse-filelist-forbidden-noarch (Badness: 10000) /usr/lib64/ceph/mgr/diskprediction_cloud is not allowed in a noarch package
[...]
(none): E: badness 1120287 exceeds threshold 1000, aborting.

  1. The /usr/lib64/ceph/mgr directory isn't owned by any package, which fails the build with "directories not owned by a package" errors for ceph-mgr, ceph-mgr-diskprediction-local, ceph-mgr-diskprediction-cloud and ceph-mgr-rook.

Even though ceph-mgr modules are notionally non-architecture-specific,
the files themselves are installed to an architecture-specific path
(/usr/lib64/ceph/mgr/....), which causes the build to fail on SUSE
distros with a huge long list of errors like:

ceph-mgr-diskprediction-cloud.noarch: E: suse-filelist-forbidden-noarch
(Badness: 10000) /usr/lib64/ceph/mgr/diskprediction_cloud is not
allowed in a noarch package
[...]
(none): E: badness 1120287 exceeds threshold 1000, aborting.

Signed-off-by: Tim Serong <tserong@suse.com>
Without this, no package actually owns the /usr/lib64/ceph/mgr directory
so the build fails with "directories not owned by a package" errors for
ceph-mgr, ceph-mgr-diskprediction-local, ceph-mgr-diskprediction-cloud
and ceph-mgr-rook.

Signed-off-by: Tim Serong <tserong@suse.com>
@tchaikov
Copy link
Contributor

  1. the files themselves are installed to an architecture-specific path (/usr/lib64/ceph/mgr/....)

hmm, we need to fix it by moving them to /usr/share/ceph/mgr. i will try to pull together a fix for this.

@tchaikov tchaikov merged commit ce6347d into ceph:master Feb 14, 2019
@tserong tserong deleted the wip-fix-spec-badness branch February 14, 2019 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants