php libvirt show active snapshot
Is php libvirt able to list the active snapshot for the vm? I can easily take snapshots and restore snapshots, but I can't figure out how to display the currently active snapshot using php libvirt. -- Todd Lewis Grizzly Star (469) 363-2907 Cell www.grizzlystar.com
On 4/13/20 9:22 PM, Todd Lewis wrote:
Is php libvirt able to list the active snapshot for the vm?
I can easily take snapshots and restore snapshots, but I can't figure out how to display the currently active snapshot using php libvirt.
Yeah, libvirt-php is not very well maintained and certainly not around snapshot area. Can you point which C APIs are you missing? I will try to implement them, or if you want to have an upstream contribution, be my guest :-) Michal
I was looking for a virDomainSnapshotCurrent I wish I had time to learn to properly Code in C, haven't touch it since that late 90's I'm only only coding in php out of necessity. . On Tue, Apr 14, 2020 at 8:02 AM Michal Privoznik <mprivozn@redhat.com> wrote:
On 4/13/20 9:22 PM, Todd Lewis wrote:
Is php libvirt able to list the active snapshot for the vm?
I can easily take snapshots and restore snapshots, but I can't figure out how to display the currently active snapshot using php libvirt.
Yeah, libvirt-php is not very well maintained and certainly not around snapshot area. Can you point which C APIs are you missing? I will try to implement them, or if you want to have an upstream contribution, be my guest :-)
Michal
-- Todd Lewis Grizzly Star (469) 363-2907 Cell www.grizzlystar.com
On 4/14/20 6:49 PM, Todd Lewis wrote:
I was looking for a virDomainSnapshotCurrent I wish I had time to learn to properly Code in C, haven't touch it since that late 90's I'm only only coding in php out of necessity.
I've just pushed the implementation upstream: https://gitlab.com/libvirt/libvirt-php/-/commit/19be5d2b5b2d3b82d719fe310ad5... Michal
Cool. I'll try it out On Thu, Apr 16, 2020 at 8:38 AM Michal Privoznik <mprivozn@redhat.com> wrote:
On 4/14/20 6:49 PM, Todd Lewis wrote:
I was looking for a virDomainSnapshotCurrent I wish I had time to learn to properly Code in C, haven't touch it since that late 90's I'm only only coding in php out of necessity.
I've just pushed the implementation upstream:
https://gitlab.com/libvirt/libvirt-php/-/commit/19be5d2b5b2d3b82d719fe310ad5...
Michal
-- Todd Lewis Grizzly Star (469) 363-2907 Cell www.grizzlystar.com
Works perfectly thank you. I did get an issue when trying to compile an rpm The autobuild did run into an error when trying to create the rpm. Not a big deal. I can make my own spec file. EXTRA_RELEASE="" if [ -x /usr/bin/rpmbuild ] then rpmbuild --nodeps \ --define "extra_release $EXTRA_RELEASE" \ --define "_sourcedir `pwd`" \ -ba --clean libvirt-php.spec fi pwd error: Macro %extra_release has empty body error: Macro %extra_release has empty body error: Macro %extra_release has empty body On Thu, Apr 16, 2020 at 11:50 AM Todd Lewis <todd@grizzlystar.com> wrote:
Cool. I'll try it out
On Thu, Apr 16, 2020 at 8:38 AM Michal Privoznik <mprivozn@redhat.com> wrote:
On 4/14/20 6:49 PM, Todd Lewis wrote:
I was looking for a virDomainSnapshotCurrent I wish I had time to learn to properly Code in C, haven't touch it since that late 90's I'm only only coding in php out of necessity.
I've just pushed the implementation upstream:
https://gitlab.com/libvirt/libvirt-php/-/commit/19be5d2b5b2d3b82d719fe310ad5...
Michal
-- Todd Lewis Grizzly Star (469) 363-2907 Cell www.grizzlystar.com
-- Todd Lewis Grizzly Star (469) 363-2907 Cell www.grizzlystar.com
On 4/21/20 1:36 AM, Todd Lewis wrote:
Works perfectly thank you. I did get an issue when trying to compile an rpm
The autobuild did run into an error when trying to create the rpm. Not a big deal. I can make my own spec file.
EXTRA_RELEASE=""
1: ^^^
if [ -x /usr/bin/rpmbuild ] then rpmbuild --nodeps \ --define "extra_release $EXTRA_RELEASE" \ --define "_sourcedir `pwd`" \ -ba --clean libvirt-php.spec fi pwd error: Macro %extra_release has empty body error: Macro %extra_release has empty body error: Macro %extra_release has empty body
D'oh. The line [1] shouldn't be there. But actually, whole autobuild script is a bit misleading. The way I build RPMs is autogen.sh + make rpm. Would you mind if I remove the autobuild script? Michal
On Tue, 2020-04-21 at 10:10 +0200, Michal Privoznik wrote:
On 4/21/20 1:36 AM, Todd Lewis wrote:
Works perfectly thank you. I did get an issue when trying to compile an rpm
The autobuild did run into an error when trying to create the rpm. Not a big deal. I can make my own spec file.
EXTRA_RELEASE=""
1: ^^^
if [ -x /usr/bin/rpmbuild ] then rpmbuild --nodeps \ --define "extra_release $EXTRA_RELEASE" \ --define "_sourcedir `pwd`" \ -ba --clean libvirt-php.spec fi pwd error: Macro %extra_release has empty body error: Macro %extra_release has empty body error: Macro %extra_release has empty body
D'oh. The line [1] shouldn't be there. But actually, whole autobuild script is a bit misleading. The way I build RPMs is autogen.sh + make rpm. Would you mind if I remove the autobuild script?
FWIW there were autobuild.sh scripts in most repositories and I have made an effort to get rid of them something like a year ago, so if there are any remaining I would certainly welcome removing them. -- Andrea Bolognani / Red Hat / Virtualization
participants (3)
-
Andrea Bolognani -
Michal Privoznik -
Todd Lewis