Hi Konstantin, Thank you for the diff. I will review it and merge it today. About the missing directories, I think it may be doable to add them to the 'ctdb' rpm. As I'm not using ctdb, what should the ownership/permissions be for those directories? Regards, Vincent On Thu, 11 Jul 2019, Konstantin Shalygin wrote:> On 7/10/19 9:49 PM, vincent at cojot.name wrote: > > > Attached diff for SPEC. I'm not a rpm (many small packages) lover, so this is > may be a dirty solution and should be refactored. > Also, when I was setup ctdb this catalogs should be exists [1] > > ``` > mkdir /var/lib/ctdb/volatile > mkdir /var/lib/ctdb/state > mkdir /var/lib/ctdb/persistent > mkdir /etc/ctdb/events/legacy > ``` > > > > > [1] https://bugzilla.redhat.com/show_bug.cgi?id=1656777 > > k > >
On 7/11/19 8:59 PM, vincent at cojot.name wrote:> Thank you for the diff. I will review it and merge it today. > About the missing directories, I think it may be doable to add them to > the 'ctdb' rpm. As I'm not using ctdb, what should the > ownership/permissions be for those directories?ctdb works from root, so just root ownership.> I've merged your patch and added the missing directories to the ctdb > package. However, I'm not sure I'll be building binary rpms with > cephfs enabled as this makes ctdb dependent on ceph (see below). I'm > probably going to include it as a non-default option and people will > have to rebuild the rpms with '--with vfs_cephfs'This good, thanks. k
Hi Konstantin, On Fri, 12 Jul 2019, Konstantin Shalygin wrote:> On 7/11/19 8:59 PM, vincent at cojot.name wrote: >> Thank you for the diff. I will review it and merge it today. >> About the missing directories, I think it may be doable to add them to the >> 'ctdb' rpm. As I'm not using ctdb, what should the ownership/permissions >> be for those directories? > > ctdb works from root, so just root ownership.Ok, thanks. It's also what I figured by looking at the fedora spec file..>> I've merged your patch and added the missing directories to the ctdb >> package. However, I'm not sure I'll be building binary rpms with cephfs >> enabled as this makes ctdb dependent on ceph (see below). I'm probably >> going to include it as a non-default option and people will have to >> rebuild the rpms with '--with vfs_cephfs' > This good, thanks.The updated packages (version with '3.1') are located at: http://nova.polymtl.ca/~coyote/dist/samba/samba-4.10.6/RHEL7 The binary rpms won't work for you (you're using cephfs) but you should be able to rebuild the samba src.rpm with "rpmbuild -ba <specfile> --with cephfs". Please let me know if that works for you. Regards, Vincent