openlmi-devel September 2013

openlmi-devel@lists.stg.fedorahosted.org

15 participants
76 discussions

Review Request 864: openlmi-networking [1/1] Fix crash when unknown connection type is found (like vpn)
by Stephen Gallagher 18 Sep '13

18 Sep '13

----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/864/ ----------------------------------------------------------- Review request for OpenLMI Developers. Repository: openlmi-networking Description ------- Fix crash when unknown connection type is found (like vpn) Diffs ----- src/connection_nm.c 3524e8c3215b901fd04731ac61b5d8e239ec67bb src/setting.c ce237fdef4b4150d080753070b36d8a72266d199 src/setting_nm.c 6fdfc3b89039d48347fb2ff158085337ae5c8777 Diff: http://reviewboard-openlmi.rhcloud.com/r/864/diff/ Testing ------- Thanks, Radek Novacek _______________________________________________ openlmi-reviews mailing list openlmi-reviews(a)lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews

1 2

Review Request 865: openlmi-providers [1/5] Introduce toplevel openlmi config file
by Stephen Gallagher 18 Sep '13

18 Sep '13

----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/865/ ----------------------------------------------------------- Review request for OpenLMI Developers. Repository: openlmi-providers Description ------- Introduce toplevel openlmi config file Toplevel openlmi configuration file (/etc/openlmi/openlmi.conf) now contains common configuration options for all providers. Configuration for each provider could be overriden in provider-specific config (/etc/openlmi/$provider/$provider.conf). This patch also modify config file handling in python providers to include this config file. There is also support for C providers (in libopenlmicommon) for reading these config files and providing default configuration options. Diffs ----- CMakeLists.txt 8bb4dd27f6104f5d792d7a25e53431cf4ab0cac8 openlmi-providers.spec f85faca43f569ecf5a9087103f3f7d95a48a1d96 src/CMakeLists.txt a9dc85d293b82805d4d956f1adbd212c873fd743 src/account/CMakeLists.txt a993094541546f46607d405f3be420a00885bd48 src/openlmi.c 3b46f8d4ec4393675e064fa9bedec875ac1f4032 src/openlmi.conf PRE-CREATION src/openlmi.h 6ab4158fd08677729c91f50285f70abdd12c0dd1 src/python/lmi/base/BaseConfiguration.py 8790acfb7598a17b9a62da697f300e5765d437cb Diff: http://reviewboard-openlmi.rhcloud.com/r/865/diff/ Testing ------- Thanks, Radek Novacek _______________________________________________ openlmi-reviews mailing list openlmi-reviews(a)lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews

1 3

Re: Review Request 851: [1/1] Service: Propagate error messages, check for invalid (NULL) service names
by Stephen Gallagher 17 Sep '13

17 Sep '13

1 0

Re: Review Request 851: [1/1] Service: Propagate error messages, check for invalid (NULL) service names
by Stephen Gallagher 17 Sep '13

17 Sep '13

----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/851/#review1181 ----------------------------------------------------------- Ship it! src/service-dbus/LMI_ServiceProvider.c <http://reviewboard-openlmi.rhcloud.com/r/851/#comment735> This is not readable, use enum for return codes or at least add comment here. - Radek Novacek On Sept. 11, 2013, 4:24 p.m., Vitezslav Crhonek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard-openlmi.rhcloud.com/r/851/ > ----------------------------------------------------------- > > (Updated Sept. 11, 2013, 4:24 p.m.) > > > Review request for OpenLMI Developers. > > > Repository: openlmi-providers > > > Description > ------- > > Service: Propagate error messages, check for invalid (NULL) service names > > > Diffs > ----- > > src/service-dbus/LMI_ServiceProvider.c e4fa3aaf15c26eaf3fe26f08f5b1c74b412532bc > src/service-dbus/util/serviceutil.h d1ed1be194f5b97f953979a97c778731684bb18f > src/service-dbus/util/serviceutil.c af1e7fff749b4d7278a3fc92d72d333a193af2d2 > > Diff: http://reviewboard-openlmi.rhcloud.com/r/851/diff/ > > > Testing > ------- > > > Thanks, > > Vitezslav Crhonek > > _______________________________________________ openlmi-reviews mailing list openlmi-reviews(a)lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews

1 0

Review Request 863: Spec file: add metapackage
by Stephen Gallagher 16 Sep '13

16 Sep '13

----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/863/ ----------------------------------------------------------- Review request for OpenLMI Developers. Repository: openlmi-providers Description ------- As per the openlmi-devel discussion: let's build an metapackage that would drag the essential bits to the managed system. Diffs ----- openlmi-providers.spec 1558895f7a6934b184ba008fc08824925687d676 Diff: http://reviewboard-openlmi.rhcloud.com/r/863/diff/ Testing ------- Thanks, Tomas Smetana _______________________________________________ openlmi-reviews mailing list openlmi-reviews(a)lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews

1 2

Re: Review Request 836: openlmi-mof-register.py: database listing support
by Stephen Gallagher 16 Sep '13

16 Sep '13

1 0

openlmi-reviews Digest, Vol 5, Issue 13
by openlmi-reviews-request＠lists.fedorahosted.org 16 Sep '13

16 Sep '13

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 859: [1/1] logicalfile: ignore some key properties (Michal Minar) 2. Re: Review Request 854: [1/1] Account: Added QA test for account provider. (Robin Hack) 3. Re: Review Request 854: [1/1] Account: Added QA test for account provider. (Robin Hack) 4. Re: Review Request 859: [1/1] logicalfile: ignore some key properties (Jan Synacek) 5. Re: Review Request 854: [1/1] Account: Added QA test for account provider. (Robin Hack) 6. Re: Review Request 854: [1/1] Account: Added QA test for account provider. (Michal Minar) _______________________________________________ openlmi-reviews mailing list openlmi-reviews(a)lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews

1 0

Re: Review Request 854: [1/1] Account: Added QA test for account provider.
by Stephen Gallagher 16 Sep '13

16 Sep '13

1 3

openlmi metapackage
by Tomáš Smetana 16 Sep '13

16 Sep '13

Hi, we discussed the possibility to make the OpenLMI installation easier by providing a metapackage that would install the necessary pieces for the managed system, i.e. a package that would install the CIMOM and the "core" providers. This is somewhat Fedora specific however appropriate for this list I believe. I would introduce a new metapackage named simply "openlmi" depending on the following packages: tog-pegasus sblim-cmpi-base openlmi-providers openlmi-storage openlmi-networking openlmi-hardware openlmi-software openlmi-power openlmi-account openlmi-service Comments and objections welcome. If there would be none, I'll file a review request for the new package (hope that's the correct process...). Regards, -- Tomáš Smetana Platform Engineering, Red Hat

2 3

Re: Review Request 859: [1/1] logicalfile: ignore some key properties
by Stephen Gallagher 16 Sep '13

16 Sep '13

----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/859/ ----------------------------------------------------------- (Updated Sept. 16, 2013, 10:08 a.m.) Status ------ This change has been marked as submitted. Review request for OpenLMI Developers. Repository: openlmi-providers Description ------- logicalfile: ignore some key properties CIM_LogicalFile and CIM_UnixFile have some key properties that are not needed when calling GetInstance(). On Linux systems, all is needed to get information about a file is its path. Type of the filesystem that the file resides on and the file type itself are not needed. In fact, it's not desirable to require those when querying remote systems. Ignore FSCreationClassName, FSName, LFCreationClassName and CreationClassName. This commit reverts most of the functionality of commit 5c3e1a6203e495bec37acd8c3731ca41377beb94. Diffs ----- src/logicalfile/LMI_DirectoryContainsFileProvider.c 753cb00d184a71bbaa3a3d63883110480750d8d9 src/logicalfile/LMI_FileIdentityProvider.c 7ada7873c0b90f88bcbf5b3ba1a44aa704c713e4 src/logicalfile/LMI_RootDirectoryProvider.c 2be709f0d0a0ca50271508dffe67bbfc357cc710 src/logicalfile/LMI_UnixFileProvider.c 5da5bb0a471228653fabc30c5a7793c526ebaff3 src/logicalfile/file.h b7ecb53cf43ee2f290c5bbdf091165c81b86ac2d src/logicalfile/file.c ca0d9f4e88d1b4b78ee7104eb88f676ab30535af src/logicalfile/test/test_basic.py fbcf6d57829d35e134cfdedab9501043fd3c372a Diff: http://reviewboard-openlmi.rhcloud.com/r/859/diff/ Testing ------- Thanks, Jan Synacek _______________________________________________ openlmi-reviews mailing list openlmi-reviews(a)lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/openlmi-reviews

1 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

openlmi-devel September 2013