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

RPM packaging for glusterfs-server for CentOS Storage SIG has a missing dependency on glusterfs-selinux #2780

Closed
sandrobonazzola opened this issue Sep 13, 2021 · 7 comments

Comments

@sandrobonazzola
Copy link

sandrobonazzola commented Sep 13, 2021

Description of problem:
RPM packaging for glusterfs-server for CentOS Storage SIG has a missing dependency on glusterfs-selinux
The issue has been reported against oVirt project here: https://bugzilla.redhat.com/show_bug.cgi?id=2002178

@sandrobonazzola
Copy link
Author

I see that the spec file in the devel repo has it: https://github.com/gluster/glusterfs/blob/devel/glusterfs.spec.in#L725
while the one on release 8 is missing it: https://github.com/gluster/glusterfs/blob/release-8/glusterfs.spec.in#L689

amarts pushed a commit that referenced this issue Sep 15, 2021
Fixes: #2780
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1761088

* Address late comments on glusterfs-selinux fix

Address comments arrived on https://review.gluster.org/#/c/glusterfs/+/24876/8/glusterfs.spec.in
after that patch was merged.
@sandrobonazzola
Copy link
Author

@nixpanic the patch has been merged can you rebuild for CentOS Storage SIG?

@nixpanic
Copy link
Member

nixpanic commented Oct 5, 2021

@mohit84 isn't glusterfs-selinux only required from GlusterFS 9? This problem is reported for GlusterFS 8, does that need glusterfs-selinux now too?

@sandrobonazzola
Copy link
Author

@nixpanic it seems so.

@nixpanic
Copy link
Member

https://lists.gluster.org/pipermail/packaging/2021-October/000975.html

glusterfs-8.6-2.el8 and glusterfs-8.6-2.el8s are available in the
testing repositories. This addresses a dependency issue that recently
was reported in relation to oVirt and SELinux:

Updating should now pull in the glusterfs-selinux package too, assuming
glusterfs-server-8.6.1 or older is installed.

# dnf --enablerepo=centos-gluster8-test update glusterfs-server

Please report back if this addresses the problem, or if any other
changes are required.

@sandrobonazzola
Copy link
Author

Thanks @nixpanic , this works for me, waiting for it to be promoted to release

@nixpanic
Copy link
Member

glusterfs-8.6-2.el8 and glusterfs-8.6-2.el8s are now available on the CentOS mirrors.

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

No branches or pull requests

2 participants