diff mbox

libvirt: Document UNDEFINE_NVRAM in UndefineFlags doc

Message ID a29670b4afae436028e414eb4ff8c083c28baa70.1413281039.git.crobinso@redhat.com
State Accepted
Commit fe6aff3b81773477ca1e83de5fb4f3be197f47d3
Headers show

Commit Message

Cole Robinson Oct. 14, 2014, 10:03 a.m. UTC
---
 src/libvirt.c | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Martin Kletzander Oct. 14, 2014, 10:46 a.m. UTC | #1
On Tue, Oct 14, 2014 at 12:03:59PM +0200, Cole Robinson wrote:
>---
> src/libvirt.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
>diff --git a/src/libvirt.c b/src/libvirt.c
>index caacc2e..dfca4f6 100644
>--- a/src/libvirt.c
>+++ b/src/libvirt.c
>@@ -8760,6 +8760,10 @@ virDomainUndefine(virDomainPtr domain)
>  * whether this flag is present.  On hypervisors where snapshots do
>  * not use libvirt metadata, this flag has no effect.
>  *
>+ * If the domain has any nvram specified, then including
>+ * VIR_DOMAIN_UNDEFINE_NVRAM will also remove that file, and omitting the flag
>+ * will cause the undefine process to fail.
>+ *
>  * Returns 0 in case of success, -1 in case of error
>  */
> int

ACK,

Martin
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Cole Robinson Oct. 15, 2014, 8:16 a.m. UTC | #2
On 10/14/2014 12:46 PM, Martin Kletzander wrote:
> On Tue, Oct 14, 2014 at 12:03:59PM +0200, Cole Robinson wrote:
>> ---
>> src/libvirt.c | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/src/libvirt.c b/src/libvirt.c
>> index caacc2e..dfca4f6 100644
>> --- a/src/libvirt.c
>> +++ b/src/libvirt.c
>> @@ -8760,6 +8760,10 @@ virDomainUndefine(virDomainPtr domain)
>>  * whether this flag is present.  On hypervisors where snapshots do
>>  * not use libvirt metadata, this flag has no effect.
>>  *
>> + * If the domain has any nvram specified, then including
>> + * VIR_DOMAIN_UNDEFINE_NVRAM will also remove that file, and omitting the flag
>> + * will cause the undefine process to fail.
>> + *
>>  * Returns 0 in case of success, -1 in case of error
>>  */
>> int
> 
> ACK,
> 
> Martin

Thanks, pushed now

- Cole

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

Patch

diff --git a/src/libvirt.c b/src/libvirt.c
index caacc2e..dfca4f6 100644
--- a/src/libvirt.c
+++ b/src/libvirt.c
@@ -8760,6 +8760,10 @@  virDomainUndefine(virDomainPtr domain)
  * whether this flag is present.  On hypervisors where snapshots do
  * not use libvirt metadata, this flag has no effect.
  *
+ * If the domain has any nvram specified, then including
+ * VIR_DOMAIN_UNDEFINE_NVRAM will also remove that file, and omitting the flag
+ * will cause the undefine process to fail.
+ *
  * Returns 0 in case of success, -1 in case of error
  */
 int