Review Request 897: [1/1] Account: Add DeleteGroup method to be consistent with user's deletion
by Stephen Gallagher
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/897/
-----------------------------------------------------------
Review request for OpenLMI Developers.
Repository: openlmi-providers
Description
-------
Account: Add DeleteGroup method to be consistent with user's deletion
User deletetion is done by DeleteUser method. Added similar method
to delete a group.
Also do the check if the group isn't a primary group of a user.
Changed documentation accordingly.
Signed-off-by: Roman Rakus <rrakus(a)redhat.com>
Diffs
-----
mof/60_LMI_Account.mof 153222eb477a1e782cb3031507f629e5b5e90d98
src/account/LMI_GroupProvider.c 6e64c2165b5653d00a0ef83312be22658da73514
src/account/doc/source/usage.rst ee88d4045b4a3ad041028ef292fd21479311819d
Diff: http://reviewboard-openlmi.rhcloud.com/r/897/diff/
Testing
-------
Thanks,
Roman Rakus
10 years, 7 months
Review Request 893: lmishell, quicker associators' traversal
by Stephen Gallagher
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard-openlmi.rhcloud.com/r/893/
-----------------------------------------------------------
Review request for OpenLMI Developers.
Repository: openlmi-tools
Description
-------
This patch introduces lmi_associators() function, which should speed up associations traversal by pulling all the instances referenced by association class.
Diffs
-----
cli/lmi/shell/LMIConsole.py bebcce353c9547a9d7ed1c5f0031319e1876b29a
cli/lmi/shell/LMIUtil.py 94f38b4ffedc286e3ac407085b66ad278ed1bcda
Diff: http://reviewboard-openlmi.rhcloud.com/r/893/diff/
Testing
-------
Example of usage:
c = connect(...)
assoc_class_list = [
c.root.cimv2.CIM_BasedOn,
c.root.cimv2.LMI_PhysicalMemoryContainer,
c.root.cimv2.LMI_SystemStorageDevice,
]
for tpl in lmi_associators(assoc_class_list):
print " <-> ".join([get_instance_id(x) for x in tpl])
Output:
/dev/disk/by-uuid/fc6b795d-8722-4222-9252-3cfb3b704dd3 <-> /dev/vda
/dev/disk/by-uuid/09331a46-7876-46de-b9f9-496fc3ff5870 <-> /dev/vda
/dev/disk/by-id/dm-name-volgroup1-lvol0 <-> /dev/disk/by-id/ata-QEMU_HARDDISK_QM00005
/dev/disk/by-id/dm-name-volgroup1-lvol0 <-> /dev/disk/by-id/ata-QEMU_HARDDISK_QM00007
LMI:LMI_MemoryPhysicalPackage:0 <-> LMI:LMI_PhysicalMemory:0
rawhide <-> /dev/disk/by-id/ata-QEMU_HARDDISK_QM00005
rawhide <-> /dev/disk/by-id/ata-QEMU_HARDDISK_QM00007
rawhide <-> /dev/vda
rawhide <-> /dev/disk/by-uuid/fc6b795d-8722-4222-9252-3cfb3b704dd3
rawhide <-> /dev/disk/by-uuid/09331a46-7876-46de-b9f9-496fc3ff5870
rawhide <-> /dev/disk/by-id/dm-name-volgroup1-lvol0
NOTE: Function get_instance_id() is local helper function, which returns key property of a class.
Thanks,
Peter Hatina
10 years, 7 months