[libvirt] [PATCH 0/2] Add {lockd|logd} admin sockets to install

John Ferlan posted 2 patches 6 years, 2 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20180206185626.7171-1-jferlan@redhat.com
libvirt.spec.in | 20 ++++++++++++++++----
src/Makefile.am | 39 +++++++++++++++++++++++++++++++--------
2 files changed, 47 insertions(+), 12 deletions(-)
[libvirt] [PATCH 0/2] Add {lockd|logd} admin sockets to install
Posted by John Ferlan 6 years, 2 months ago
Hopefully I covered what's necessary... Essentially a bunch of
copying of existing examples...

I haven't been able to actually build an RPM locally for a long
time, but this does work with make install at least (or at least
it worked in the environment I tried it on).

John Ferlan (2):
  logd: Install the admin sockets
  lockd: Install the admin sockets

 libvirt.spec.in | 20 ++++++++++++++++----
 src/Makefile.am | 39 +++++++++++++++++++++++++++++++--------
 2 files changed, 47 insertions(+), 12 deletions(-)

-- 
2.13.6

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/2] Add {lockd|logd} admin sockets to install
Posted by Daniel P. Berrangé 6 years, 2 months ago
On Tue, Feb 06, 2018 at 01:56:24PM -0500, John Ferlan wrote:
> Hopefully I covered what's necessary... Essentially a bunch of
> copying of existing examples...
> 
> I haven't been able to actually build an RPM locally for a long
> time, but this does work with make install at least (or at least
> it worked in the environment I tried it on).

What sort of problem are you having ?  The CI system is testing our
RPM builds on Fedora so should work.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/2] Add {lockd|logd} admin sockets to install
Posted by John Ferlan 6 years, 2 months ago

On 02/08/2018 05:16 AM, Daniel P. Berrangé wrote:
> On Tue, Feb 06, 2018 at 01:56:24PM -0500, John Ferlan wrote:
>> Hopefully I covered what's necessary... Essentially a bunch of
>> copying of existing examples...
>>
>> I haven't been able to actually build an RPM locally for a long
>> time, but this does work with make install at least (or at least
>> it worked in the environment I tried it on).
> 
> What sort of problem are you having ?  The CI system is testing our
> RPM builds on Fedora so should work.
> 

Mostly bad and/or lazy (and/or annoyed) user on keyboard...  Never quite
seem to get the "correct packages" installed. Sometimes I get a list of
packages I'm missing from the make rpm command, while other times it's
just a failure and I'm left guessing and debugging. After a while I just
give up hoping that the CI system will do its thing...

Still for these -admin sockets, I see the CI env is happy. I don't
believe a running environment would be as happy as I found with the
-admin socket missing there were errors in virtlogd (at least) restart
or in the first connection from a new libvirtd to virtlogd (I've
forgotten the details already as they've been nwfiltered out ;-)).

Anyway, as for make rpm on one system I had problems w/ "uniconvertor":

/usr/bin/mv: cannot stat '/tmp/magick-14461O4gf8V6BuvWf.svg': No such
file or directory
convert: delegate failed `'uniconvertor' '%i' '%o.svg'; /usr/bin/mv
'%o.svg' '%o'' @ error/delegate.c/InvokeDelegate/1919.

So /me thinks - let's remove it... That didn't work because "convert"
and "ImageMagick" are there... So remove that and get:

convert -rotate 90 libvirt-daemon-arch.fig libvirt-daemon-arch.png
make[3]: convert: Command not found

<sigh>...  Even re-running autogen.sh didn't help. Perhaps there's some
package that I do have that caused autogen.sh to set something that
causes something else to believe I have the right bits.

FWIW: On another system I don't have ImageMagick or uniconvertor and
that actually builds the rpm after installing a few things along the
way. So WTF knows - it's all black magic and hokum to me.

John

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/2] Add {lockd|logd} admin sockets to install
Posted by Daniel P. Berrangé 6 years, 2 months ago
On Thu, Feb 08, 2018 at 08:18:57AM -0500, John Ferlan wrote:
> 
> 
> On 02/08/2018 05:16 AM, Daniel P. Berrangé wrote:
> > On Tue, Feb 06, 2018 at 01:56:24PM -0500, John Ferlan wrote:
> >> Hopefully I covered what's necessary... Essentially a bunch of
> >> copying of existing examples...
> >>
> >> I haven't been able to actually build an RPM locally for a long
> >> time, but this does work with make install at least (or at least
> >> it worked in the environment I tried it on).
> > 
> > What sort of problem are you having ?  The CI system is testing our
> > RPM builds on Fedora so should work.
> > 
> 
> Mostly bad and/or lazy (and/or annoyed) user on keyboard...  Never quite
> seem to get the "correct packages" installed. Sometimes I get a list of
> packages I'm missing from the make rpm command, while other times it's
> just a failure and I'm left guessing and debugging. After a while I just
> give up hoping that the CI system will do its thing...

You shouldn't have todo anything more then "dnf builddep libvirt" to
get all the RPM pre-requisites installed. Now technically this will
give you build deps suitable for the version of libvirt that's in
Fedora, rather than git master, but we don't add new build-deps often
enough for that to be a problem.


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/2] Add {lockd|logd} admin sockets to install
Posted by John Ferlan 6 years, 2 months ago

On 02/08/2018 08:26 AM, Daniel P. Berrangé wrote:
> On Thu, Feb 08, 2018 at 08:18:57AM -0500, John Ferlan wrote:
>>
>>
>> On 02/08/2018 05:16 AM, Daniel P. Berrangé wrote:
>>> On Tue, Feb 06, 2018 at 01:56:24PM -0500, John Ferlan wrote:
>>>> Hopefully I covered what's necessary... Essentially a bunch of
>>>> copying of existing examples...
>>>>
>>>> I haven't been able to actually build an RPM locally for a long
>>>> time, but this does work with make install at least (or at least
>>>> it worked in the environment I tried it on).
>>>
>>> What sort of problem are you having ?  The CI system is testing our
>>> RPM builds on Fedora so should work.
>>>
>>
>> Mostly bad and/or lazy (and/or annoyed) user on keyboard...  Never quite
>> seem to get the "correct packages" installed. Sometimes I get a list of
>> packages I'm missing from the make rpm command, while other times it's
>> just a failure and I'm left guessing and debugging. After a while I just
>> give up hoping that the CI system will do its thing...
> 
> You shouldn't have todo anything more then "dnf builddep libvirt" to
> get all the RPM pre-requisites installed. Now technically this will
> give you build deps suitable for the version of libvirt that's in
> Fedora, rather than git master, but we don't add new build-deps often
> enough for that to be a problem.
> 
> 

dnf builddep didn't find anything on one system, but still fails the
make rpm with the same uniconvertor issue.  Since it works elsewhere (CI
and my other host), I'm not going to jump into the rabbit hole.

John

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list