July 2013 - openlmi-devel - Fedora Mailing-Lists

by openlmi-reviews-request＠lists.fedorahosted.org

Send openlmi-reviews mailing list submissions to openlmi-reviews(a)lists.fedorahosted.org To subscribe or unsubscribe via the World Wide Web, visit https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews or, via email, send a message with subject or body 'help' to openlmi-reviews-request(a)lists.fedorahosted.org You can reach the person managing the list at openlmi-reviews-owner(a)lists.fedorahosted.org When replying, please edit your Subject line so it is more specific than "Re: Contents of openlmi-reviews digest..." Today's Topics: 1. Review Request 564: [1/1] LogicalFile: add property checks (Jan Synacek) 2. Re: Review Request 564: [1/1] LogicalFile: add property checks (Jan Safranek) 3. Re: Review Request 557: Tests [1/1] Account: Tests for indications (Jan Synacek) 4. Re: Review Request 546: Tests cleanup [1/2] Account: Supress error messages (Roman Rakus) 5. Review Request 566: Account tests [1/1] Account: Make tests a little bit robust (Roman Rakus) 6. Review Request 567: Hardware: Package the cmpiLMI_Hardware-cimprovagt file (Peter Schiffer) 7. Review Request 568: lmishell, constant values (Peter Hatina) 8. Re: Review Request 566: Account tests [1/1] Account: Make tests a little bit robust (Jan Synacek) 9. Re: Review Request 567: Hardware: Package the cmpiLMI_Hardware-cimprovagt file (Jan Synacek) 10. Re: Review Request 558: Hardware: Update LMI_PCIDevice provider (Jan Synacek) 11. Re: Review Request 559: [openlmi-providers] Profiles: [1/2] Fill in PG_ProviderModule.Version property. (Jan Synacek) 12. Re: Review Request 560: [openlmi-providers] Profiles: [2/2] Added -v option to opelmi-mof-register. (Jan Synacek) 13. Re: Review Request 563: [openlmi-storage] Profiles: [1/1] Register LMI and SNIA profiles. (Jan Synacek) 14. Re: Review Request 564: [1/1] LogicalFile: add property checks (Jan Synacek) 15. Re: Review Request 564: [1/1] LogicalFile: add property checks (Jan Synacek) 16. Re: Review Request 564: [1/1] LogicalFile: add property checks (Jan Safranek) 17. Re: Review Request 564: [1/1] LogicalFile: add property checks (Jan Synacek) 18. Re: Review Request 568: lmishell, constant values (Jan Safranek) 19. Re: Review Request 559: [openlmi-providers] Profiles: [1/2] Fill in PG_ProviderModule.Version property. (Michal Minar) 20. Re: Review Request 563: [openlmi-storage] Profiles: [1/1] Register LMI and SNIA profiles. (Michal Minar) 21. Re: Review Request 564: [1/1] LogicalFile: add property checks (Jan Synacek) 22. Re: Review Request 564: [1/1] LogicalFile: add property checks (Jan Synacek) 23. Re: Review Request 564: [1/1] LogicalFile: add property checks (Jan Safranek) 24. Re: Review Request 564: [1/1] LogicalFile: add property checks (Jan Synacek) 25. Review Request 569: [1/1] LogicalFile: add tests for property checks (Jan Synacek) 26. Re: Review Request 569: [1/1] LogicalFile: add tests for property checks (Jan Safranek) 27. Review Request 570: [1/3] LogicalFile: mention python-pyudev as a test dependency (Jan Synacek) 28. Review Request 571: [2/3] LogicalFile: set LMI_UnixFile.Name (Jan Synacek) 29. Review Request 572: [3/3] LogicalFile: use get_string_property_from_op() where appropriate (Jan Synacek) 30. Re: Review Request 569: [1/1] LogicalFile: add tests for property checks (Jan Synacek) 31. Re: Review Request 557: Tests [1/1] Account: Tests for indications (Roman Rakus) 32. Re: Review Request 566: Account tests [1/1] Account: Make tests a little bit robust (Roman Rakus) 33. Re: Review Request 570: [1/3] LogicalFile: mention python-pyudev as a test dependency (Roman Rakus) 34. Re: Review Request 571: [2/3] LogicalFile: set LMI_UnixFile.Name (Roman Rakus) 35. Re: Review Request 572: [3/3] LogicalFile: use get_string_property_from_op() where appropriate (Roman Rakus) 36. Re: Review Request 572: [3/3] LogicalFile: use get_string_property_from_op() where appropriate (Jan Synacek) 37. Review Request 573: Coverity [1/1] Indmanager: Fix coverity report (Roman Rakus) 38. Re: Review Request 572: [3/3] LogicalFile: use get_string_property_from_op() where appropriate (Jan Synacek) 39. Re: Review Request 572: [3/3] LogicalFile: use get_string_property_from_op() where appropriate (Roman Rakus) 40. Re: Review Request 573: Coverity [1/1] Indmanager: Fix coverity report (Jan Synacek) 41. Re: Review Request 573: Coverity [1/1] Indmanager: Fix coverity report (Roman Rakus) 42. Re: Review Request 573: Coverity [1/1] Indmanager: Fix coverity report (Jan Synacek) 43. Re: Review Request 573: Coverity [1/1] Indmanager: Fix coverity report (Jan Synacek) 44. Re: Review Request 570: [1/3] LogicalFile: mention python-pyudev as a test dependency (Jan Synacek) 45. Re: Review Request 571: [2/3] LogicalFile: set LMI_UnixFile.Name (Jan Synacek) 46. Re: Review Request 572: [3/3] LogicalFile: use get_string_property_from_op() where appropriate (Jan Synacek) 47. Re: Review Request 573: Coverity [1/1] Indmanager: Fix coverity report (Roman Rakus) 48. Re: Review Request 559: [openlmi-providers] Profiles: [1/2/2] Fill in PG_ProviderModule.Version property. (Jan Safranek) 49. Re: Review Request 559: [openlmi-providers] Profiles: [1/2] Fill in PG_ProviderModule.Version property. (Jan Safranek) 50. Re: Review Request 560: [openlmi-providers] Profiles: [2/2] Added -v option to opelmi-mof-register. (Jan Safranek) 51. Re: Review Request 559: [openlmi-providers] Profiles: [1/2] Fill in PG_ProviderModule.Version property. (Jan Synacek) 52. Re: Review Request 560: [openlmi-providers] Profiles: [2/2] Added -v option to opelmi-mof-register. (Jan Synacek) 53. Re: Review Request 559: [openlmi-providers] Profiles: [1/2] Fill in PG_ProviderModule.Version property. (Michal Minar) 54. Re: Review Request 560: [openlmi-providers] Profiles: [2/2] Added -v option to opelmi-mof-register. (Michal Minar) 55. Re: Review Request 559: [openlmi-providers] Profiles: [1/2] Fill in PG_ProviderModule.Version property. (Jan Safranek) 56. Re: Review Request 559: [openlmi-providers] Profiles: [1/2] Fill in PG_ProviderModule.Version property. (Jan Safranek) 57. Re: Review Request 560: [openlmi-providers] Profiles: [2/2] Added -v option to opelmi-mof-register. (Jan Safranek) 58. Re: Review Request 563: [openlmi-storage] Profiles: [1/1] Register LMI and SNIA profiles. (Jan Safranek) 59. Re: Review Request 458: prototype CIM<->PCP bridge (Jan Safranek) 60. Review Request 575: Add license heares to scripts. (Jan Safranek) 61. Re: Review Request 575: Add license heares to scripts. (Radek Novacek) 62. Review Request 576: Fixed changelog date. (Jan Safranek) 63. Re: Review Request 576: Fixed changelog date. (Radek Novacek) 64. Re: Review Request 558: Hardware: Update LMI_PCIDevice provider (Peter Schiffer) 65. Re: Review Request 567: Hardware: Package the cmpiLMI_Hardware-cimprovagt file (Peter Schiffer) 66. Re: Review Request 576: Fixed changelog date. (Jan Safranek) _______________________________________________ openlmi-reviews mailing list openlmi-reviews(a)lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews

10 years, 9 months

1
0
0 / 0

Regarding simple identity management & role based authorisation profile

by George varghese

Hi, I have requirement to implement simple identity and role based authorisation profile. I have noted simple identity profile exists in openlmi provider. I have some questions about it. 1. In simple identity, implementation is for manipulating user accounts in fedora operating system for right now. Right? .. but my requirement is not this. In my board haven't user acct mgmt system. Have user acct mgmnt system for accessing web only. So I shall integrating that into it. Is it okay? 2. Is newly created users coming to cimom for performing any profiles? I hope its not coming in cimom for accessing it. It's only for os users. Role based authorization profile implementation doesn't exist it? If anybody implemented it, could you please provide the details. It means that defining some roles to this newly created users. Right? Please help me, I am waiting for your response. Regards, George

10 years, 9 months

2
1
0 / 0

openlmi-reviews Digest, Vol 3, Issue 5

by openlmi-reviews-request＠lists.fedorahosted.org

Send openlmi-reviews mailing list submissions to openlmi-reviews(a)lists.fedorahosted.org To subscribe or unsubscribe via the World Wide Web, visit https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews or, via email, send a message with subject or body 'help' to openlmi-reviews-request(a)lists.fedorahosted.org You can reach the person managing the list at openlmi-reviews-owner(a)lists.fedorahosted.org When replying, please edit your Subject line so it is more specific than "Re: Contents of openlmi-reviews digest..." Today's Topics: 1. Re: Review Request 506: [2/3] Change openlmi imports (Jan Safranek) 2. Re: Review Request 507: [3/3] Remove superfluous quote. (Jan Safranek) 3. Re: Review Request 541: [1/1] Add mounting tests. (Jan Synacek) 4. Re: Review Request 541: [1/1] Add mounting tests. (Jan Safranek) 5. Re: Review Request 541: [1/1] Add mounting tests. (Jan Synacek) 6. Re: Review Request 541: [1/1] Add mounting tests. (Jan Synacek) 7. Review Request 543: Add assertNocaseDictEqual to compare pywbem.NocaseDict. (Jan Safranek) 8. Review Request 542: Fix RAID10 tests, support for 2 and 3 devices was removed from blivet. (Jan Safranek) 9. Re: Review Request 543: Add assertNocaseDictEqual to compare pywbem.NocaseDict. (Michal Minar) 10. Re: Review Request 542: Fix RAID10 tests, support for 2 and 3 devices was removed from blivet. (Michal Minar) 11. Re: Review Request 543: Add assertNocaseDictEqual to compare pywbem.NocaseDict. (Jan Safranek) 12. Re: Review Request 542: Fix RAID10 tests, support for 2 and 3 devices was removed from blivet. (Jan Safranek) 13. Review Request 544: openlmi-fan [1/1] fan: get rid of nasty hack (Michal Minar) 14. Re: Review Request 544: openlmi-fan [1/1] fan: get rid of nasty hack (Radek Novacek) 15. Review Request 545: [1/1] Do not use blivet's mount(). (Jan Synacek) 16. Re: Review Request 545: [1/1] Do not use blivet's mount(). (Jan Safranek) 17. Re: Review Request 545: [1/1] Do not use blivet's mount(). (Jan Synacek) 18. Review Request 546: Tests cleanup [1/2] Account: Supress error messages (Roman Rakus) 19. Review Request 547: Tests cleanup [2/2] Account: Use Account prefix in test logs (Roman Rakus) 20. Re: Review Request 546: Tests cleanup [1/2] Account: Supress error messages (Radek Novacek) 21. Re: Review Request 547: Tests cleanup [2/2] Account: Use Account prefix in test logs (Radek Novacek) 22. Review Request 548: [1/1] Add LMI_STORAGE_MNT_DIR to test/README. (Jan Synacek) 23. Re: Review Request 547: Tests cleanup [2/2] Account: Use Account prefix in test logs (Jan Safranek) 24. Re: Review Request 546: Tests cleanup [1/2] Account: Supress error messages (Jan Safranek) 25. Review Request 549: lmishell, synchro method calls (Peter Hatina) 26. Re: Review Request 549: lmishell, synchro method calls (Jan Safranek) 27. Review Request 550: [1/2] LogicalFile: add selinux contexts support (Jan Synacek) 28. Review Request 551: [2/2] LogicalFile: add tests for selinux (Jan Synacek) 29. Re: Review Request 550: [1/2] LogicalFile: add selinux contexts support (Jan Safranek) 30. Re: Review Request 551: [2/2] LogicalFile: add tests for selinux (Jan Safranek) 31. Re: Review Request 550: [1/2] LogicalFile: add selinux contexts support (Jan Synacek) 32. Re: Review Request 550: [1/2] LogicalFile: add selinux contexts support (Jan Synacek) 33. Review Request 552: [1/2] LogicalFile: add selinux contexts support (Jan Synacek) 34. Review Request 554: Fix _lmi_debug (Jan Synacek) 35. Re: Review Request 554: Fix _lmi_debug (Jan Safranek) 36. Re: Review Request 552: [1/2] LogicalFile: add selinux contexts support (Jan Safranek) 37. Re: Review Request 552: [1/2] LogicalFile: add selinux contexts support (Jan Synacek) 38. Re: Review Request 554: Fix _lmi_debug (Jan Synacek) 39. Re: Review Request 551: [2/2] LogicalFile: add tests for selinux (Jan Synacek) 40. Re: Review Request 552: [1/2] LogicalFile: add selinux contexts support (Jan Synacek) 41. Review Request 555: [1/1] LogicalFile: make selinux lookups non-fatal (Jan Synacek) 42. Re: Review Request 555: [1/1] LogicalFile: make selinux lookups non-fatal (Jan Safranek) 43. Review Request 556: spec: add libselinux-devel (Jan Synacek) 44. Re: Review Request 555: [1/1] LogicalFile: make selinux lookups non-fatal (Jan Synacek) 45. Re: Review Request 556: spec: add libselinux-devel (Jan Safranek) 46. Re: Review Request 556: spec: add libselinux-devel (Jan Synacek) 47. Re: Review Request 548: [1/1] Add LMI_STORAGE_MNT_DIR to test/README. (Jan Safranek) 48. Re: Review Request 548: [1/1] Add LMI_STORAGE_MNT_DIR to test/README. (Jan Synacek) 49. Re: Review Request 549: lmishell, synchro method calls (Peter Hatina) 50. Re: Review Request 546: Tests cleanup [1/2] Account: Supress error messages (Roman Rakus) 51. Re: Review Request 549: lmishell, synchro method calls (Peter Hatina) 52. Re: Review Request 549: lmishell, synchro method calls (Jan Safranek) 53. Review Request 557: Tests [1/1] Account: Tests for indications (Roman Rakus) 54. Re: Review Request 547: Tests cleanup [2/2] Account: Use Account prefix in test logs (Roman Rakus) 55. Re: Review Request 546: Tests cleanup [1/2] Account: Supress error messages (Roman Rakus) 56. Review Request 558: Hardware: Update LMI_PCIDevice provider (Peter Schiffer) 57. Review Request 559: [openlmi-providers] Profiles: [1/2] Fill in PG_ProviderModule.Version property. (Jan Safranek) 58. Review Request 560: [openlmi-providers] Profiles: [2/2] Added -v option to opelmi-mof-register. (Jan Safranek) 59. Review Request 563: [openlmi-storage] Profiles: [1/1] Register LMI and SNIA profiles. (Jan Safranek) _______________________________________________ openlmi-reviews mailing list openlmi-reviews(a)lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews

10 years, 9 months

1
0
0 / 0

use_exception(True) by default in lmishell

by Jan Synacek

Hello, currently, exceptions in shell are silently ignored when risen, unless 'use_exception(True)' is explicitly called. I strongly suggest we change the default behavior to use the exceptions by default and, if needed, turn them off by 'use_exception(False)'. This will save a lot of gray hair in the future... -- Jan Synacek Software Engineer, Red Hat

10 years, 9 months

5
4
0 / 0

openlmi-networking builds failing on master

by Stephen Gallagher

-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 The buildbot is failing to build openlmi-networking because of a missing file, CIM_Error.h: http://openlmi-rnovacek.rhcloud.com/builders/networking-fedora19/builds/8... -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.13 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iEYEARECAAYFAlHe71QACgkQeiVVYja6o6M38ACfRRE5XrTVkT5o0hknerooeB3+ dv4AniDU5p24Yj4xiNZGGqfT9MYw+SRU =K0Yg -----END PGP SIGNATURE-----

10 years, 9 months

2
1
0 / 0

openlmi-reviews Digest, Vol 3, Issue 4

by openlmi-reviews-request＠lists.fedorahosted.org

Send openlmi-reviews mailing list submissions to openlmi-reviews(a)lists.fedorahosted.org To subscribe or unsubscribe via the World Wide Web, visit https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews or, via email, send a message with subject or body 'help' to openlmi-reviews-request(a)lists.fedorahosted.org You can reach the person managing the list at openlmi-reviews-owner(a)lists.fedorahosted.org When replying, please edit your Subject line so it is more specific than "Re: Contents of openlmi-reviews digest..." Today's Topics: 1. Re: Review Request 517: openlmi-networking <jobs> [3/5] Implement AffectedNetworkJobElement and OwningNetworkJobElement classes (Jan Synacek) 2. Re: Review Request 518: openlmi-networking <jobs> [4/5] Allow job modification and job cleanup (Jan Synacek) 3. Re: Review Request 519: openlmi-networking <jobs> [5/5] test: add test for jobs (Jan Synacek) 4. Re: Review Request 520: [1/1] LogicalFile: Add tests (Jan Safranek) 5. Re: Review Request 520: [1/1] LogicalFile: Add tests (Jan Synacek) 6. Re: Review Request 458: prototype CIM<->PCP bridge (Frank Eigler) 7. Re: Review Request 515: openlmi-networking <jobs> [1/5] Fill more properties of LMI_NetworkJob (Radek Novacek) 8. Re: Review Request 517: openlmi-networking <jobs> [3/5] Implement AffectedNetworkJobElement and OwningNetworkJobElement classes (Radek Novacek) 9. Re: Review Request 516: openlmi-networking <jobs> [2/5] Allow more affected elements in the job (Radek Novacek) 10. Re: Review Request 519: openlmi-networking <jobs> [5/5] test: add test for jobs (Radek Novacek) 11. Re: Review Request 518: openlmi-networking <jobs> [4/5] Allow job modification and job cleanup (Radek Novacek) 12. Re: Review Request 458: prototype CIM<->PCP bridge (Radek Novacek) 13. Re: Review Request 458: prototype CIM<->PCP bridge (Radek Novacek) 14. Re: Review Request 444: [1/1] Add mounting tests. (Jan Safranek) 15. Re: Review Request 513: openlmi-providers(profile-registration) [1/1] added profile registration class (Jan Safranek) 16. Re: Review Request 458: prototype CIM<->PCP bridge (Frank Eigler) 17. Re: Review Request 510: openlmi-software(tests) [1/3] software: don't use some packages for dangerous tests (Michal Minar) 18. Re: Review Request 511: openlmi-software(tests) [2/3] software: fixed test of LMI_InstalledSoftwareIdentity (Michal Minar) 19. Re: Review Request 512: openlmi-software(tests) [3/3] software: do not check symlink's file mode in tests (Michal Minar) 20. Re: Review Request 513: openlmi-providers(profile-registration) [1/1] added profile registration class (Michal Minar) 21. Review Request 521: openlmi-software [1/1] software: fixed shutdown (Michal Minar) 22. Review Request 525: openlmi-software(configuration) [1/5] python: added BaseConfiguration singleton class (Michal Minar) 23. Review Request 526: openlmi-software(configuration) [2/5] software: added software configuration class (Michal Minar) 24. Review Request 527: openlmi-software(configuration) [3/5] software: added utility function for instance name creation (Michal Minar) 25. Review Request 528: openlmi-software(configuration) [4/5] software: using new Configuration and utility function (Michal Minar) 26. Review Request 529: openlmi-software(configuration) [5/5] software: be more tolerant when checking SystemName (Michal Minar) 27. Review Request 530: openlmi-software(logging) [1/4] software: removed log messages for provider init (Michal Minar) 28. Review Request 531: openlmi-software(logging) [2/4] software: added custom cmpi_logging (Michal Minar) 29. Review Request 533: openlmi-software(logging) [4/4] software: disable logging of separate process (Michal Minar) 30. Review Request 532: openlmi-software(logging) [3/4] software: use new cmpi_logging module (Michal Minar) 31. Review Request 534: openlmi-software(configuration, cleanups) [1/7] software: configurable timeout for job completion (Michal Minar) 32. Review Request 535: openlmi-software(configuration, cleanups) [2/7] software: minimum time before job's removal configurable (Michal Minar) 33. Review Request 536: openlmi-software(configuration, cleanups) [3/7] software: more configurability of jobs (Michal Minar) 34. Review Request 537: openlmi-software(configuration, cleanups) [4/7] software: YumWorker timeout parameters configurable (Michal Minar) 35. Review Request 538: openlmi-software(configuration, cleanups) [5/7] software: test fixes (Michal Minar) 36. Review Request 539: openlmi-software(configuration, cleanups) [6/7] software: added doc strings; spaces, be gone (Michal Minar) 37. Review Request 540: openlmi-software(configuration, cleanups) [7/7] software: added sample configuration (Michal Minar) 38. Re: Review Request 525: openlmi-software(configuration) [1/1] python: added BaseConfiguration singleton class (Michal Minar) 39. Re: Review Request 525: openlmi-software(configuration) [1/5] python: added BaseConfiguration singleton class (Michal Minar) 40. Re: Review Request 526: openlmi-software(configuration) [2/5] software: added software configuration class (Michal Minar) 41. Re: Review Request 540: openlmi-software(configuration, cleanups) [1/1] software: added sample configuration (Michal Minar) 42. Re: Review Request 521: openlmi-software [1/1] software: fixed shutdown (Jan Safranek) 43. Re: Review Request 533: openlmi-software(logging) [4/4] software: disable logging of separate process (Jan Safranek) 44. Re: Review Request 525: openlmi-software(configuration) [1/5] python: added BaseConfiguration singleton class (Jan Safranek) 45. Re: Review Request 526: openlmi-software(configuration) [2/5] software: added software configuration class (Jan Safranek) 46. Re: Review Request 527: openlmi-software(configuration) [3/5] software: added utility function for instance name creation (Jan Safranek) 47. Re: Review Request 528: openlmi-software(configuration) [4/5] software: using new Configuration and utility function (Jan Safranek) 48. Re: Review Request 529: openlmi-software(configuration) [5/5] software: be more tolerant when checking SystemName (Jan Safranek) 49. Re: Review Request 534: openlmi-software(configuration, cleanups) [1/7] software: configurable timeout for job completion (Jan Safranek) 50. Re: Review Request 535: openlmi-software(configuration, cleanups) [2/7] software: minimum time before job's removal configurable (Jan Safranek) 51. Re: Review Request 536: openlmi-software(configuration, cleanups) [3/7] software: more configurability of jobs (Jan Safranek) 52. Re: Review Request 537: openlmi-software(configuration, cleanups) [4/7] software: YumWorker timeout parameters configurable (Jan Safranek) 53. Re: Review Request 538: openlmi-software(configuration, cleanups) [5/7] software: test fixes (Jan Safranek) 54. Re: Review Request 539: openlmi-software(configuration, cleanups) [6/7] software: added doc strings; spaces, be gone (Jan Safranek) 55. Re: Review Request 540: openlmi-software(configuration, cleanups) [1/1] software: added sample configuration (Jan Safranek) 56. Re: Review Request 444: [1/1] Add mounting tests. (Jan Synacek) 57. Review Request 541: [1/1] Add mounting tests. (Jan Synacek) 58. Re: Review Request 530: openlmi-software(logging) [1/4] software: removed log messages for provider init (Jan Safranek) 59. Re: Review Request 531: openlmi-software(logging) [2/4] software: added custom cmpi_logging (Jan Safranek) 60. Re: Review Request 532: openlmi-software(logging) [3/4] software: use new cmpi_logging module (Jan Safranek) 61. Re: Review Request 541: [1/1] Add mounting tests. (Jan Safranek) 62. Re: Review Request 505: [1/3] Rename openlmi package to lmi (Jan Safranek) _______________________________________________ openlmi-reviews mailing list openlmi-reviews(a)lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews

10 years, 9 months

1
0
0 / 0

RFC: Script development

by Jan Safranek

Hi, I've been talking to Stephen Gallagher how to proceed with client script development in lmishell. The goal is to provide high-level functionality to manage remote systems without complete knowledge of the CIM API. We agreed that: - we should provide python modules with high-level functions (we were thinking about nice classes, e.g. VolumeGroup with methods to extend/destroy/examine a volume group, but it would end up in duplicating the API we already have. We also assume that our users are not familiar with OOP). - these python functions try to hide the object model - we assume that administrators won't remember association names and won't use e.g. vg.associators(AssocClass="LMI_VGAssociatedComponentExtent") to get list of physical volumes of a vg. We want nice vg_get_pvs(vg) function. We will expose CIM classes and properties though. - these python functions are synchronous, i.e. they do stuff and return once the stuff is finished. They can do stuff in parallel inside (e.g. format multiple devices simultaneously) but from outside perspective, the stuff is completed once the function returns. (we were thinking about python functions just scheduling multiple actions and do stuff in parallel massively, but we quickly got into lot of corner cases) - each high-level function takes a LmiNamespace parameter, which specifies the WBEM connection + the namespace on which it operates -> i.e. applications/other scripts can run our functions on multiple connections -> if the LmiNamespace is not provided by caller, some 'global' one will be used (so users just connect once and this connection is then used for all high-level functions) - we should probably split these high-level function to several modules by functionality, i.e. have lmi.networking and lmi.storage.vg, lmi.storage.lv etc. - it should be easy to build command-line versions for these high-level functions -> it is not clear if we should mimic existing cmdline tools (mdadm, vgcreate, ip, ...) or make some cleanup (so creation of MD raid looks the same like creation of a VG) - we should introduce some 'lmi' metacommand, which would wrap these command line tools, like 'lmi vgcreate mygroup /dev/sda1 /dev/sdb1' and 'lmi ip addr show'. It's quite similar to fedpkg or koji command line utilities. - 'lmi' metacommand could also have a shell: $ lmi shell > vgcreate mygroup /dev/sda1 /dev/sdb1 > ip addr show I tried to create a simple module for volume group management. I ran into several issues with lmishell (see trac tickets), attached you can find first proposal. It's quite crude and misses several important aspects like proper logging and error handling. Please look at it and let us know what you think. It is just a proposal, we can change it in any way. Once we agree on the concept, we must also define strict documentation and logging standards so all functions and scripts are nicely documented and all of them provide the same user experience. Jan P.S.: note that I'm out of office for next week and with sporadic email access this week.

10 years, 9 months

8
21
0 / 0

openlmi-reviews Digest, Vol 3, Issue 3

by openlmi-reviews-request＠lists.fedorahosted.org

Send openlmi-reviews mailing list submissions to openlmi-reviews(a)lists.fedorahosted.org To subscribe or unsubscribe via the World Wide Web, visit https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews or, via email, send a message with subject or body 'help' to openlmi-reviews-request(a)lists.fedorahosted.org You can reach the person managing the list at openlmi-reviews-owner(a)lists.fedorahosted.org When replying, please edit your Subject line so it is more specific than "Re: Contents of openlmi-reviews digest..." Today's Topics: 1. Re: Review Request 498: openlmi-software(test-and-fixes) [2/7] added tests for SoftwareIdentityFileCheck (Radek Novacek) 2. Re: Review Request 501: openlmi-software(test-and-fixes) [5/7] fixed RequestStateChange of LMI_SoftwareIdentityResource (Radek Novacek) 3. Re: Review Request 316: [openlmi-tools] Add configuration file for ReviewBoard (Stephen Gallagher) 4. Re: Review Request 499: openlmi-software(test-and-fixes) [3/7] added tests for SoftwareIdentityChecks (Radek Novacek) 5. Re: Review Request 317: [openlmi-proxy] Add configuration file for ReviewBoard (Stephen Gallagher) 6. Re: Review Request 502: openlmi-software(test-and-fixes) [6/7] small software test improvements (Radek Novacek) 7. Re: Review Request 503: openlmi-software(test-and-fixes) [7/7] added exception logging to reference enumeration (Radek Novacek) 8. Re: Review Request 100: [openlmi-tools] Build System [2/2] Enable building openlmi-tools with setuptools (Stephen Gallagher) 9. Re: Review Request 99: [openlmi-tools] Build System [1/2] Remove autotools build system (Stephen Gallagher) 10. Review Request 504: [1/1] account: don't require indication manager (Radek Novacek) 11. Re: Review Request 504: [1/1] account: don't require indication manager (Roman Rakus) 12. Re: Review Request 504: [1/1] account: don't require indication manager (Radek Novacek) 13. Review Request 505: [1/3] Rename openlmi package to lmi (Jan Safranek) 14. Review Request 507: [3/3] Remove superfluous quote. (Jan Safranek) 15. Review Request 506: [2/3] Change openlmi imports (Jan Safranek) 16. Review Request 500: openlmi-software(test-and-fixes) [1/1] added cimprovagt wrapper script for SELinux (Michal Minar) 17. Re: Review Request 500: openlmi-software(test-and-fixes) [4/7] added cimprovagt wrapper script for SELinux (Michal Minar) 18. Re: Review Request 498: openlmi-software(test-and-fixes) [2/7] added tests for SoftwareIdentityFileCheck (Michal Minar) 19. Re: Review Request 497: openlmi-software(test-and-fixes) [1/7] call destructor of super class for YumDB (Michal Minar) 20. Re: Review Request 499: openlmi-software(test-and-fixes) [3/7] added tests for SoftwareIdentityChecks (Michal Minar) 21. Re: Review Request 498: openlmi-software(test-and-fixes) [2/7] added tests for SoftwareIdentityFileCheck (Michal Minar) 22. Re: Review Request 505: [1/3] Rename openlmi package to lmi (Radek Novacek) 23. Re: Review Request 506: [2/3] Change openlmi imports (Radek Novacek) 24. Re: Review Request 507: [3/3] Remove superfluous quote. (Radek Novacek) 25. Re: Review Request 500: openlmi-software(test-and-fixes) [4/7] added cimprovagt wrapper script for SELinux (Radek Novacek) 26. Re: Review Request 500: openlmi-software(test-and-fixes) [4/7] added cimprovagt wrapper script for SELinux (Jan Synacek) 27. Re: Review Request 500: openlmi-software(test-and-fixes) [4/7] added cimprovagt wrapper script for SELinux (Michal Minar) 28. Re: Review Request 501: openlmi-software(test-and-fixes) [5/7] fixed RequestStateChange of LMI_SoftwareIdentityResource (Michal Minar) 29. Re: Review Request 502: openlmi-software(test-and-fixes) [6/7] small software test improvements (Michal Minar) 30. Re: Review Request 503: openlmi-software(test-and-fixes) [7/7] added exception logging to reference enumeration (Michal Minar) 31. Review Request 508: openlmi-providers(openlmi=>lmi) [1/2] renamed openlmi namespace to lmi (Michal Minar) 32. Review Request 509: openlmi-providers(openlmi=>lmi) [2/2] spec files changes to cope with python namespace change (Michal Minar) 33. Re: Review Request 508: openlmi-providers(openlmi=>lmi) [1/2] renamed openlmi namespace to lmi (Jan Safranek) 34. Re: Review Request 509: openlmi-providers(openlmi=>lmi) [2/2] spec files changes to cope with python namespace change (Jan Safranek) 35. Review Request 510: openlmi-software(tests) [1/3] software: don't use some packages for dangerous tests (Michal Minar) 36. Review Request 511: openlmi-software(tests) [2/3] software: fixed test of LMI_InstalledSoftwareIdentity (Michal Minar) 37. Review Request 512: openlmi-software(tests) [3/3] software: do not check symlink's file mode in tests (Michal Minar) 38. Re: Review Request 505: [1/3] Rename openlmi package to lmi (Michal Minar) 39. Re: Review Request 506: [2/3] Change openlmi imports (Michal Minar) 40. Re: Review Request 508: openlmi-providers(openlmi=>lmi) [1/2] renamed openlmi namespace to lmi (Michal Minar) 41. Re: Review Request 509: openlmi-providers(openlmi=>lmi) [2/2] spec files changes to cope with python namespace change (Michal Minar) 42. Review Request 513: openlmi-providers(profile-registration) [1/1] added profile registration class (Michal Minar) 43. Re: Review Request 513: openlmi-providers(profile-registration) [1/1] added profile registration class (Michal Minar) 44. Review Request 515: openlmi-networking <jobs> [1/5] Fill more properties of LMI_NetworkJob (Radek Novacek) 45. Review Request 516: openlmi-networking <jobs> [2/5] Allow more affected elements in the job (Radek Novacek) 46. Review Request 517: openlmi-networking <jobs> [3/5] Implement AffectedNetworkJobElement and OwningNetworkJobElement classes (Radek Novacek) 47. Review Request 518: openlmi-networking <jobs> [4/5] Allow job modification and job cleanup (Radek Novacek) 48. Review Request 519: openlmi-networking <jobs> [5/5] test: add test for jobs (Radek Novacek) 49. Review Request 520: [1/1] LogicalFile: Add tests (Jan Synacek) 50. Re: Review Request 510: openlmi-software(tests) [1/3] software: don't use some packages for dangerous tests (Jan Synacek) 51. Re: Review Request 511: openlmi-software(tests) [2/3] software: fixed test of LMI_InstalledSoftwareIdentity (Jan Synacek) 52. Re: Review Request 512: openlmi-software(tests) [3/3] software: do not check symlink's file mode in tests (Jan Synacek) 53. Re: Review Request 515: openlmi-networking <jobs> [1/5] Fill more properties of LMI_NetworkJob (Jan Synacek) 54. Re: Review Request 516: openlmi-networking <jobs> [2/5] Allow more affected elements in the job (Jan Synacek) _______________________________________________ openlmi-reviews mailing list openlmi-reviews(a)lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews

10 years, 9 months

1
0
0 / 0

Version in MOF files

by Jan Safranek

Hi, we're reaching a point where we should report a version of the API in MOF file. Version is 3-component, i.e. X.Y.Z. No API changes are allowed during .Z releases, i.e. you may add new functions, add parameters to functions, add properties, add classes, but existing application must run without problems. When Y is changed, API can be changed (property/function renamed/deleted etc.). Still, you do not _need_ to change the API if the release is a major step forward without any API breakers. The version should be bumped in MOF files only if appropriate class is changed. I.e. when you release X.Y.Z+1 version, you do not need to change all MOF files, change the Version qualifier only in the changed classes. Is there any volunteer to write simple API checker, which would compare two set of MOF files and report changes? Jan

10 years, 10 months

4
13
0 / 0

openlmi-reviews Digest, Vol 3, Issue 2

by openlmi-reviews-request＠lists.fedorahosted.org

Send openlmi-reviews mailing list submissions to openlmi-reviews(a)lists.fedorahosted.org To subscribe or unsubscribe via the World Wide Web, visit https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews or, via email, send a message with subject or body 'help' to openlmi-reviews-request(a)lists.fedorahosted.org You can reach the person managing the list at openlmi-reviews-owner(a)lists.fedorahosted.org When replying, please edit your Subject line so it is more specific than "Re: Contents of openlmi-reviews digest..." Today's Topics: 1. Review Request 483: openlmi-networking <jobs> [1/10] Fix for KonkretCMPI EmbeddedInstance change (Radek Novacek) 2. Review Request 486: openlmi-networking <jobs> [4/10] Fix calling of wrong callback when connection changes (Radek Novacek) 3. Review Request 485: openlmi-networking <jobs> [3/10] Add reason of port status change (Radek Novacek) 4. Review Request 484: openlmi-networking <jobs> [2/10] Rework ActiveConnection handling (Radek Novacek) 5. Review Request 487: openlmi-networking <jobs> [5/10] Make network_activate_connection asynchronous (Radek Novacek) 6. Review Request 488: openlmi-networking <jobs> [6/10] Add LMI_NetworkJob class (Radek Novacek) 7. Review Request 492: openlmi-networking <jobs> [10/10] test: Make tests more robust (Radek Novacek) 8. Review Request 490: openlmi-networking <jobs> [8/10] Emit changed indication when active connection changes status (Radek Novacek) 9. Review Request 489: openlmi-networking <jobs> [7/10] Add job added and changed indications (Radek Novacek) 10. Review Request 491: openlmi-networking <jobs> [9/10] test: fix connection activation to use jobs (Radek Novacek) 11. Review Request 493: [1/2] LogicalFile: remove unused variable (Jan Synacek) 12. Review Request 494: [2/2] LogicalFile: use udev to fill info about the filesystem (Jan Synacek) 13. Re: Review Request 493: [1/2] LogicalFile: remove unused variable (Radek Novacek) 14. Re: Review Request 494: [2/2] LogicalFile: use udev to fill info about the filesystem (Radek Novacek) 15. Re: Review Request 493: [1/2] LogicalFile: remove unused variable (Jan Synacek) 16. Re: Review Request 494: [2/2] LogicalFile: use udev to fill info about the filesystem (Jan Synacek) 17. Re: Review Request 494: [2/2] LogicalFile: use udev to fill info about the filesystem (Jan Synacek) 18. Re: Review Request 494: [2/2] LogicalFile: use udev to fill info about the filesystem (Radek Novacek) 19. Re: Review Request 494: [2/2] LogicalFile: use udev to fill info about the filesystem (Jan Synacek) 20. Review Request 495: [1/2] LogicalFile: remove explicit lib dependency from spec (Jan Synacek) 21. Review Request 496: [2/2] Add LogicalFile to README (Jan Synacek) 22. Re: Review Request 495: [1/2] LogicalFile: remove explicit lib dependency from spec (Radek Novacek) 23. Re: Review Request 496: [2/2] Add LogicalFile to README (Radek Novacek) 24. Re: Review Request 483: openlmi-networking <jobs> [1/10] Fix for KonkretCMPI EmbeddedInstance change (Jan Synacek) 25. Re: Review Request 484: openlmi-networking <jobs> [2/10] Rework ActiveConnection handling (Jan Synacek) 26. Re: Review Request 485: openlmi-networking <jobs> [3/10] Add reason of port status change (Jan Synacek) 27. Re: Review Request 486: openlmi-networking <jobs> [4/10] Fix calling of wrong callback when connection changes (Jan Synacek) 28. Re: Review Request 487: openlmi-networking <jobs> [5/10] Make network_activate_connection asynchronous (Jan Synacek) 29. Re: Review Request 488: openlmi-networking <jobs> [6/10] Add LMI_NetworkJob class (Jan Synacek) 30. Re: Review Request 488: openlmi-networking <jobs> [6/10] Add LMI_NetworkJob class (Jan Synacek) 31. Re: Review Request 489: openlmi-networking <jobs> [7/10] Add job added and changed indications (Jan Synacek) 32. Re: Review Request 496: [2/2] Add LogicalFile to README (Jan Synacek) 33. Re: Review Request 495: [1/2] LogicalFile: remove explicit lib dependency from spec (Jan Synacek) 34. Re: Review Request 490: openlmi-networking <jobs> [8/10] Emit changed indication when active connection changes status (Jan Synacek) 35. Re: Review Request 491: openlmi-networking <jobs> [9/10] test: fix connection activation to use jobs (Jan Synacek) 36. Re: Review Request 492: openlmi-networking <jobs> [10/10] test: Make tests more robust (Jan Synacek) 37. Re: Review Request 488: openlmi-networking <jobs> [6/10] Add LMI_NetworkJob class (Radek Novacek) 38. Re: Review Request 483: openlmi-networking <jobs> [1/10] Fix for KonkretCMPI EmbeddedInstance change (Radek Novacek) 39. Re: Review Request 484: openlmi-networking <jobs> [2/10] Rework ActiveConnection handling (Radek Novacek) 40. Re: Review Request 485: openlmi-networking <jobs> [3/10] Add reason of port status change (Radek Novacek) 41. Re: Review Request 486: openlmi-networking <jobs> [4/10] Fix calling of wrong callback when connection changes (Radek Novacek) 42. Re: Review Request 487: openlmi-networking <jobs> [5/10] Make network_activate_connection asynchronous (Radek Novacek) 43. Re: Review Request 488: openlmi-networking <jobs> [6/10] Add LMI_NetworkJob class (Radek Novacek) 44. Re: Review Request 489: openlmi-networking <jobs> [7/10] Add job added and changed indications (Radek Novacek) 45. Re: Review Request 490: openlmi-networking <jobs> [8/10] Emit changed indication when active connection changes status (Radek Novacek) 46. Re: Review Request 491: openlmi-networking <jobs> [9/10] test: fix connection activation to use jobs (Radek Novacek) 47. Re: Review Request 492: openlmi-networking <jobs> [10/10] test: Make tests more robust (Radek Novacek) 48. Re: Review Request 480: Coverity reports [2/4] Account: Don't use uninitialized fields (Roman Rakus) 49. Re: Review Request 479: Coverity reports [1/4] Account: Use strdup instead of malloc-strcpy combo (Roman Rakus) 50. Re: Review Request 482: Coverity reports [4/4] Account: Fix unreachable code (Roman Rakus) 51. Re: Review Request 481: Coverity reports [3/4] Account: Remove dead code (Roman Rakus) 52. Re: Review Request 481: Coverity reports [3/4] Account: Remove dead code (Roman Rakus) 53. Re: Review Request 333: openlmi-software[verification] [4/9] added software verification classes to mof (Michal Minar) 54. Re: Review Request 334: openlmi-software[verification] [5/9] added implementation for verification classes (Michal Minar) 55. Re: Review Request 335: openlmi-software[verification] [6/9] delete YumDB singleton in YumWorker process (Michal Minar) 56. Re: Review Request 336: openlmi-software[verification] [7/9] added another LMI_SoftwareVerificationJob (Michal Minar) 57. Re: Review Request 337: openlmi-software[verification] [8/9] added some comments strings (Michal Minar) 58. Re: Review Request 338: openlmi-software[verification] [9/9] implemented VerifyInstalledIdentity() (Michal Minar) 59. Re: Review Request 481: Coverity reports [3/4] Account: Remove dead code (Stephen Gallagher) 60. Review Request 497: openlmi-software(test-and-fixes) [1/7] call destructor of super class for YumDB (Michal Minar) 61. Review Request 498: openlmi-software(test-and-fixes) [2/7] added tests for SoftwareIdentityFileCheck (Michal Minar) 62. Review Request 499: openlmi-software(test-and-fixes) [3/7] added tests for SoftwareIdentityChecks (Michal Minar) 63. Review Request 501: openlmi-software(test-and-fixes) [5/7] fixed RequestStateChange of LMI_SoftwareIdentityResource (Michal Minar) 64. Review Request 502: openlmi-software(test-and-fixes) [6/7] small software test improvements (Michal Minar) 65. Review Request 503: openlmi-software(test-and-fixes) [7/7] added exception logging to reference enumeration (Michal Minar) 66. Re: Review Request 481: Coverity reports [3/4] Account: Remove dead code (Roman Rakus) 67. Re: Review Request 497: openlmi-software(test-and-fixes) [1/7] call destructor of super class for YumDB (Radek Novacek) 68. Re: Review Request 314: [openlmi-storage] Add .reviewboardrc configuration file (Stephen Gallagher) _______________________________________________ openlmi-reviews mailing list openlmi-reviews(a)lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews

10 years, 10 months

1
0
0 / 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

openlmi-devel July 2013