Version 0.2 of python-redfish #13
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add first exception handling to get_manager_info. Maybe I will have to factorise that later...
- Allow client to choose optional debug parameter with loglevel from command line.
- Change ManagersCollection to not report a list by a dict.
The managers_dict will contain index --> manager object, index is the
redfish index as reported inside url
- Notice that Proliant firmware now uses EthernetInterfaces as specified
by Redfish
- Bug to be reported :
/redfish/v1/Managers/1/EthernetInterfaces/1 returns invalid content (not json)
- Revert some changes on " to ' as it can break json data. Example :
return self.set_parameter_json('{"Boot": {"BootSourceOverrideTarget":
"'+target+'"},{"BootSourceOverrideEnabled" : "'+enabled+'"}}')
- Update type.py with new manager functions and manage exceptions. - Retrieve some basic manager data from client getinfo.
- Specify a name for the logger as recommended by the logging documentation. However for strange reason requests logs are not anymore capture in the log file. This needs to be reviewed.
- Change EthernetInterfacesCollection to not report a list but a dict. This change will be the same on *Collection. - Factorise get_name function from Root to Base class as Name parameter seems defined for all kind of Redfish objects. - Report first EthernetInterfaces data in redfish-client as an example --> need to be elaborated
- This is a WIP.
- Add get_mac, get_fqdn, get_ipv4, get_ipv6.
- Add function get_managed_chassis. - Add function get_managed_systems. - Modiy mapping to handle proliant fw 2.40 bugs reporting : - links instead of Links. - href instead of @odata.id. - I'll report this bug to redfish folks. - Various PEP8 cleans.
We're keeping the previous version to be able to still support them
- Use templating system, so user will be able to customize output. - This could be use later to propose output in various format (html, xml, json). - Undefined EthernetInterfacesCollection in case of failure. So it allows to test definition in the template or elsewhere.
- Fix for empty login or password entry. - Remove pprint because it prints stuff to the console even in no debug mode.
- Remove apache pid file. So it allows to restart the mockup after a brutal shutdown of the container.
My modifications for future version 0.3
- Simplify tagging. - Test removal of old container.
- Adds python-redfish.spec for pb from previous version - first rpm made using project-builder.org - Adds a changelog file for pb - Now delivers redfish-client under /usr/bin - Fix dependencies needed to invoke redfish-client - Versions can now be used through pb instead of being hardcoded - Prepare deb package build
Improve docker container management
- Setup repository delivery on mondorescue.org
- Put in place sphinx mechanism to build documentation. Most of the documentation is generated from python docstrings. - redfish-client.py usage is a bit modified to display a decent documentation. - A link rfclient.py to redfish-client.py was created to allow sphinx parsing. The '-' char is not allowed for module and so sphinx autodoc module. - Note : Copyright may need to be reviewed as well as main.py.
Documentation update
- Add dependencies for deb build (NOTE: the mandatory python-tortilla package doesn't exist in Ubuntu or Debian for now) - reduce setup.cfg in deliveries and fix OpenSTack related content
- Change systems_list to systems_dict["<index>"] . The systems_dict will contain system objects, <index> is the redfish <index> as reported inside url.
- Update pb yml file and pbcl - Fix monitor_sleep default initialization - Fix URLs in the doc Change-Id: I5231a8edcbafe4f89f4104c59d9c4fdd06ba72cd
And tag for version 0.4.4 Change-Id: I82516b1d2aac776c154b1e6b2b52a12d4ab7e5fa
Change-Id: I8750766dd7c3674542a1416eb4fee2997c4321fa
Change-Id: I4c3a58f7441cbfefa8582b95697c956cc31fb4ac
bcornec
force-pushed
the
master
branch
2 times, most recently
from
bc7af53 to
3637d17
Compare
Change-Id: Iadc614e87998f4744ca002928e0fb896d9cfb962
change link of dmtf ref in README.rst
few typos on README.md
Modify the test script
Chiara/headers modif
Typo in dmtf/README.rst
change requirements
Update README.rst
updated OS versions for the test suite
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Replacing the 2 previous requests with a more clean version with a proper history and tags.