[libvirt] [PATCH v2] doc: Add statment about permissions needed to do a core dump

Daniel Veillard veillard at redhat.com
Fri Sep 16 12:41:08 UTC 2011


On Fri, Sep 16, 2011 at 12:04:40PM +0200, Peter Krempa wrote:
> Documentation did not specify, that some permissions are required on
> target path for coredump for the user running the hypervisor.
> 
> Diff to v1:
> - reword statements
> ---
>  src/libvirt.c   |    3 ++-
>  tools/virsh.pod |    3 +++
>  2 files changed, 5 insertions(+), 1 deletions(-)
> 
> diff --git a/src/libvirt.c b/src/libvirt.c
> index c32c7a6..4fb200d 100644
> --- a/src/libvirt.c
> +++ b/src/libvirt.c
> @@ -2777,7 +2777,8 @@ error:
>   *
>   * This method will dump the core of a domain on a given file for analysis.
>   * Note that for remote Xen Daemon the file path will be interpreted in
> - * the remote host.
> + * the remote host. Hypervisors may require  the user to manually ensure
> + * proper permissions on the file named by @to.
>   *
>   * If @flags includes VIR_DUMP_CRASH, then leave the guest shut off with
>   * a crashed state after the dump completes.  If @flags includes
> diff --git a/tools/virsh.pod b/tools/virsh.pod
> index 5114580..02726f3 100644
> --- a/tools/virsh.pod
> +++ b/tools/virsh.pod
> @@ -621,6 +621,9 @@ rather than merely left in a paused state.
>  If I<--bypass-cache> is specified, the save will avoid the file system
>  cache, although this may slow down the operation.
> 
> +NOTE: Some hypervisors may require the user to manually ensure proper
> +permissions on file and path specified by argument I<corefilepath>.
> +
>  =item B<dumpxml> I<domain-id> [I<--inactive>] [I<--security-info>]
>  [I<--update-cpu>]

  ACK, I will push, thanks !

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel at veillard.com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/




More information about the libvir-list mailing list