On 07/08/2014 10:32 AM, Chen Hanxiao wrote:
In lxc, we could not use setmem command
with --config options.
This patch will add support for this.
Signed-off-by: Chen Hanxiao chenhanx...@cn.fujitsu.com
---
v2: use virDomainSetMemoryFlagsEnsureACL
remove redundant domain running check
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce function xenParseXMMem(virConfPtr conf,);
This function parses the xm memory options
signed-off-by: David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 26 --
1 file changed, 16 insertions(+), 10
From: Kiarie Kahurani davidkiar...@gmail.com
I introduced the function
xenFormatXMVif(virConfPtr conf, ..);
which parses network configuration
signed-off-by: David Kiarie davidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 298 -
1 file
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce the function
xenParseXMPCI(.);
which parses PCI config
signed-off-by:David Kiarie davidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 192 +++--
1 file changed, 99 insertions(+), 93
From: Kiarie Kahurani davidkiar...@gmail.com
A couple of miscellaneous fixed and wrap code common code into
xenParseConfigCommon(...).I will drop some of the functions later
though
signed-off-by: David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 134
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce the function
xenParseXMDisk(..);
Parses disk config
signed-off-by: David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 204 -
1 file changed, 108 insertions(+), 96
From: Kiarie Kahurani davidkiar...@gmail.com
refactor code parsing uuid, memory and name options
signed-off-by:David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 40 +++-
1 file changed, 27 insertions(+), 13 deletions(-)
diff --git
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce the functions
xenParseXMTimeOffset(...);
xenParseXMEventActions(..);
These parse the time config and Event actions configs
respectively
signed-off-by:David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 128
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce function
xenFormatXMEventActions(.)
which formats actions following certain events
signed-off-by:David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 60 ++
1 file changed,
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce function
xenFormatXMVfb(...);
which formats Vfb config
signed-off-by: David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 207 +++--
1 file changed, 107 insertions(+), 100
From: Kiarie Kahurani davidkiar...@gmail.com
introduce function
xenParseXMVfb(.);
which parses the graphics config
signed-off-by: David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 247 -
1 file changed, 131 insertions(+), 116
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce xenParseXMCPUFeatures(.);
This function parses config related to CPU and power
management
signed-off-by:David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 116 -
1 file
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce functions
xenParseXMOS(.) and
xenParseXMMisc(.) to parse the OS and
Miscellaneous devices configs
signed-off-by: David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 146 ++---
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce function
xenFormatXMTimeOffset(..)
which formats virtual time config
signed-off-by:David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 149 -
1 file changed, 78 insertions(+),
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce function
xenFormatXMDomainDisks(virConfPtr conf, .);
which formats disk config
signed-off-by: David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 62 +++---
1 file changed, 36
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce function
xenFormatXMCharDev();
which formats char devices config
signed-off-by:David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 155 -
1 file changed, 82 insertions(+), 73
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce function xenParseXMGeneral(virConfPtr conf, ...);
This function parses the xm general options such as uuid and name
signed-off-by: David Kiarie davidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 65
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce the function
xenFormatXMDomainNet(..)
I think this could be done in a cleaner way
signed-of-by: David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 49 +++--
1 file changed, 31
Similarly to the previous commit, boolean variables should not start
with 'no-' prefix.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/conf/domain_conf.c | 12 ++--
src/security/security_dac.c | 8
src/security/security_selinux.c | 10 +-
https://bugzilla.redhat.com/show_bug.cgi?id=1066894
With current code it's possible to have for instance:
virsh dumpxml mydomain | grep seclabel
seclabel type='dynamic' model='selinux' relabel='yes'/
seclabel type='dynamic' model='selinux' relabel='yes'/
seclabel type='dynamic'
diff to v1:
- rework the 3rd patch
- introduce one more bugfix
Michal Privoznik (4):
virSecurityLabelDef: substitute 'norelabel' with 'relabel'
virSecurityDeviceLabelDef: substitute 'norelabel' with 'relabel'
conf: Always format seclabel's model
conf: Don't allow multiple seclabels for
This negation in names of boolean variables is driving me insane. The
code is much more readable if we drop the 'no-' prefix. Well, at least
for me.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/conf/domain_conf.c | 20 ++--
https://bugzilla.redhat.com/show_bug.cgi?id=1113860
We've always done that. Well, until 990e46c45. Point is, if we don't
format model, we may lose a domain on libvirtd restart. If the
seclabel is implicit however, we should skip it's formatting.
Signed-off-by: Michal Privoznik
Add backends for this frontend function so that we can use it in the
snapshot creation code.
---
src/storage/storage_backend_fs.c | 17 +
src/storage/storage_backend_gluster.c | 15 +++
2 files changed, 32 insertions(+)
diff --git
Move the last operation done on local files to the storage driver API.
---
src/qemu/qemu_driver.c | 17 +++--
1 file changed, 7 insertions(+), 10 deletions(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 9b3c695..a0f49eb 100644
--- a/src/qemu/qemu_driver.c
+++
To integrate the security driver with the storage driver we need to
pass a callback for a function that will chown storage volumes.
Introduce and document the callback prototype.
---
src/qemu/qemu_driver.c | 3 ++-
src/security/security_dac.c | 9 +
With my intended use of storage driver assist to chown files on remote
storage we will need a witness that will tell us whether the given
storage volume supports operations needed by the storage driver.
---
src/storage/storage_driver.c | 30 ++
Use the storage driver to chown remote images.
---
src/qemu/qemu_driver.c | 48 +++-
1 file changed, 47 insertions(+), 1 deletion(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index b30c504..9b3c695 100644
--- a/src/qemu/qemu_driver.c
Peter Krempa (8):
storage: Implement storage driver helper to chown disk images
storage: Add witness for checking storage volume use in security
driver
security: DAC: Remove superfluous link resolution
security: DAC: Introduce callback to perform image chown
security: DAC: Plumb
When restoring security labels in the dac driver the code would resolve
the file path and use the resolved one to be chown-ed. The setting code
doesn't do that. Remove the unnecessary code.
---
src/security/security_dac.c | 19 +--
1 file changed, 1 insertion(+), 18 deletions(-)
Use the callback to set disk and storage image labels by modifying the
existing functions and adding wrappers to avoid refactoring a lot of the
code.
---
src/security/security_dac.c | 89 +++--
1 file changed, 69 insertions(+), 20 deletions(-)
diff --git
Gluster storage works on a similar principle to NFS where it takes the
uid and gid of the actual process and uses it to access the storage
volume on the remote server. This introduces a need to chown storage
files on gluster via native API.
---
src/storage/storage_backend.h | 6 ++
Peter Krempa (3):
conf: storage: Add helper to determine whether storage vol is local
storage: wipe: Don't attempt to wipe remote storage
storage: Split out volume upload/download as separate backend function
src/conf/storage_conf.c | 19 +
src/conf/storage_conf.h
Volume wiping uses external tool to wipe the storage. This will not work
with purely remote storage as native gluster or RBD. Add a check to
report better error when wiping a remote volume.
Before:
$ virsh vol-wipe --pool glusterpool qcow3
error: Failed to wipe vol qcow3
error: Failed to open
For non-local storage drivers we can't expect to use the FDStream
backend for up/downloading volumes. Split the code into a separate
backend function so that we can add protocol specific code later.
---
src/storage/storage_backend.c | 31 +++
Add a simple helper that returns true if a storage vol definition points
to a volume accessible as a file in the host.
---
src/conf/storage_conf.c | 19 +++
src/conf/storage_conf.h | 2 ++
src/libvirt_private.syms | 1 +
3 files changed, 22 insertions(+)
diff --git
On 02.07.2014 18:53, Taowei wrote:
Fix some problems in last patch. Includes:
* Re-manage the patches in a better order, move some symbol definitions into
the proper patch.
* Make the API in vboxUniformedAPI hierarchical. So when more apis introduced
into the
uniformed api, it wouldn't go
On 07/10/2014 04:04 PM, Michal Privoznik wrote:
This negation in names of boolean variables is driving me insane. The
code is much more readable if we drop the 'no-' prefix. Well, at least
for me.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/conf/domain_conf.c |
On 07/10/2014 04:04 PM, Michal Privoznik wrote:
Similarly to the previous commit, boolean variables should not start
with 'no-' prefix.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
---
src/conf/domain_conf.c | 12 ++--
src/security/security_dac.c | 8
On 02.07.2014 15:20, Michele Paolino wrote:
vhost-user is a qemu feature. Initial support for lxc,uml and
xenxs added.
Signed-off-by: Michele Paolino m.paol...@virtualopensystems.com
---
src/lxc/lxc_process.c | 1 +
src/uml/uml_conf.c| 5 +
src/xenxs/xen_sxpr.c | 1 +
3 files
On 02.07.2014 15:20, Michele Paolino wrote:
This patch adds the test files and the documentation for vhost-user.
Signed-off-by: Michele Paolino m.paol...@virtualopensystems.com
---
docs/formatdomain.html.in | 34 +++
docs/schemas/domaincommon.rng
On 02.07.2014 15:20, Michele Paolino wrote:
vhost-user is added as a virDomainChrSourceDefPtr variable in the
virtual network interface configuration structure.
This patch adds support to parse vhost-user element. The XML file
looks like:
interface type='vhostuser'
source type='unix'
On 07/10/2014 04:04 PM, Michal Privoznik wrote:
https://bugzilla.redhat.com/show_bug.cgi?id=1113860
We've always done that. Well, until 990e46c45. Point is, if we don't
format model, we may lose a domain on libvirtd restart. If the
seclabel is implicit however, we should skip it's
On 02.07.2014 15:20, Michele Paolino wrote:
Build the qemu-command line for vhost-user.
Signed-off-by: Michele Paolino m.paol...@virtualopensystems.com
---
src/qemu/qemu_command.c | 58 +
1 file changed, 58 insertions(+)
This is the patch
On 07/10/2014 04:04 PM, Michal Privoznik wrote:
https://bugzilla.redhat.com/show_bug.cgi?id=1066894
With current code it's possible to have for instance:
virsh dumpxml mydomain | grep seclabel
seclabel type='dynamic' model='selinux' relabel='yes'/
seclabel type='dynamic'
On 07/10/2014 04:30 PM, Peter Krempa wrote:
Peter Krempa (3):
conf: storage: Add helper to determine whether storage vol is local
storage: wipe: Don't attempt to wipe remote storage
storage: Split out volume upload/download as separate backend function
src/conf/storage_conf.c
On 07/10/2014 07:42 AM, David Kiarie wrote:
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce function xenParseXMGeneral(virConfPtr conf, ...);
This function parses the xm general options such as uuid and name
[meta-comment]:
When posting a series, it's useful to post a cover letter
On 07/10/2014 09:02 AM, Ján Tomko wrote:
On 07/10/2014 04:04 PM, Michal Privoznik wrote:
This negation in names of boolean variables is driving me insane. The
code is much more readable if we drop the 'no-' prefix. Well, at least
for me.
Signed-off-by: Michal Privoznik mpriv...@redhat.com
On 07/10/2014 08:30 AM, Peter Krempa wrote:
Volume wiping uses external tool to wipe the storage. This will not work
with purely remote storage as native gluster or RBD. Add a check to
report better error when wiping a remote volume.
Before:
$ virsh vol-wipe --pool glusterpool qcow3
On 07/10/2014 08:30 AM, Peter Krempa wrote:
For non-local storage drivers we can't expect to use the FDStream
backend for up/downloading volumes. Split the code into a separate
backend function so that we can add protocol specific code later.
---
src/storage/storage_backend.c | 31
On 06/24/2014 03:21 PM, Eric Blake wrote:
On 06/24/2014 08:07 AM, Jason J. Herne wrote:
During a recent managed save operation I received the following error
message:
error: operation failed: domain save job: unexpectedly failed.
It turns out that I had run out of disk space. After a
Add a method to get the model of the CPU from capabilities.
---
libvirt-gconfig/libvirt-gconfig-capabilities-cpu.c | 6 ++
libvirt-gconfig/libvirt-gconfig-capabilities-cpu.h | 2 ++
libvirt-gconfig/libvirt-gconfig.sym| 2 ++
3 files changed, 10 insertions(+)
diff --git
v2:
* Correct hierarchy for GVirConfigDomainCpuModel
* Correct order of new symbols in .sym file
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Add a class to represent 'model' node under domain/cpu.
---
libvirt-gconfig/Makefile.am| 2 +
libvirt-gconfig/libvirt-gconfig-domain-cpu-model.c | 94 ++
libvirt-gconfig/libvirt-gconfig-domain-cpu-model.h | 72 +
Add a method to set model of domain CPU.
---
libvirt-gconfig/libvirt-gconfig-domain-cpu.c | 11 +++
libvirt-gconfig/libvirt-gconfig-domain-cpu.h | 4
libvirt-gconfig/libvirt-gconfig.sym | 2 ++
3 files changed, 17 insertions(+)
diff --git
David Kiarie wrote:
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce function xenParseXMGeneral(virConfPtr conf, ...);
This function parses the xm general options such as uuid and name
signed-off-by: David Kiarie davidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 65
Eric Blake wrote:
On 07/10/2014 07:42 AM, David Kiarie wrote:
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce function xenParseXMGeneral(virConfPtr conf, ...);
This function parses the xm general options such as uuid and name
[meta-comment]:
When posting a series, it's
David Kiarie wrote:
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce function xenParseXMMem(virConfPtr conf,);
This function parses the xm memory options
signed-off-by: David Kiariedavidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 26 --
1 file changed, 16
David Kiarie wrote:
From: Kiarie Kahurani davidkiar...@gmail.com
I introduced the function
xenFormatXMVif(virConfPtr conf, ..);
which parses network configuration
signed-off-by: David Kiarie davidkiar...@gmail.com
---
src/xenxs/xen_xm.c | 298
David Kiarie wrote:
From: Kiarie Kahurani davidkiar...@gmail.com
Introduce the functions
xenParseXMTimeOffset(...);
xenParseXMEventActions(..);
These aren't related and should be split into two patches IMO.
Regards,
Jim
These parse the time config and Event actions
This patch enables the helper program to detect event(s) triggered when there
is a change in lease length or expiry and client-id. This transfers complete
control of leases database to libvirt and suppresses use of the lease database
file (network-name.leases). That file will not be created, read,
-Original Message-
From: Ján Tomko [mailto:jto...@redhat.com]
Sent: Thursday, July 10, 2014 9:40 PM
-if (virDomainSetMemoryEnsureACL(dom-conn, vm-def) 0)
+if (virDomainSetMemoryFlagsEnsureACL(dom-conn, vm-def, flags) 0)
+goto cleanup;
+
+if (!(caps =
Roman Bogorodskiy wrote:
Roman Bogorodskiy wrote:
Try to reconnect to the running domains after libvirtd restart. To
achieve that, do:
ping?
Roman Bogorodskiy
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
63 matches
Mail list logo