This patch adds support to provide unprocessed introspection data
to reapply a node. The provided introspection data will be save to
current introspection storage backend.
Change-Id: I969ae9c32f53f89c006a64a006388ddea9542aa5
Story: 1564863
Task: 11344
This is a mechanically generated change to replace openstack.org
git:// URLs with https:// equivalents.
This is in aid of a planned future move of the git hosting
infrastructure to a self-hosted instance of gitea (https://gitea.io),
which does not support the git wire protocol at this stage.
This update should result in no functional change.
For more information see the thread at
http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003825.html
Change-Id: I2b98df417a982d0169be63630cab29607274324c
Add file to the reno documentation build to show release notes for
stable/stein.
Use pbr instruction to increment the minor version number
automatically so that master versions are higher than the versions on
stable/stein.
Change-Id: Ie52cdd78c85cc54187016f71f7c01bfcd6a64abb
Sem-Ver: feature
Remove log messages which tend to fill up the inspector logs when
debug is enabled without providing much value.
Change-Id: Ib848a13f30f5ef0074c9e43eb0a414bec0254637
This patch removes two reserved fields, ``switch_port_descr`` and
``switch_chassis_descr``, from the json request and response
examples in api-ref, that were still present despite having been
removed from the ironic-python-agent code base during
ocata cycle[0].
[0]73f316d804 (diff-664ce09492251bb45bf8a2fb7c1a5f18)
Change-Id: I6446e167b957b94bc9bb7f6737bcf57e94a4433a
Otherwise updating a driver requires listing all interfaces with their
new values. This patch is designed to be backportable, so it has
a fall-back to the previous behavior if the required API version and/or
ironicclient versions are not available.
Story: #2005148
Task: #29854
Depends-On: https://review.openstack.org/#/c/643264/
Change-Id: I115b92b9d7bcdbb1c03f0d259dbba6efd2baec2a
Addressing commments from https://review.openstack.org/#/c/613515
Contents about introspection sub resouces are moved to a new file.
Change-Id: Icd3efb902e6f836dad653011a96edc84a3feea7c
The commit 771f2d396b breaks it because
it tries to use hostname for lookup. Lookup must always use IP addresses,
because that's what `ipmitool lan print` outputs.
Change-Id: I62ff646b77e2d39f70bade11924b2a47e1173cb5
Follow up to https://review.openstack.org/#/c/495752
Addressed some comments from rloo, as well as some other issues.
Change-Id: I3cdcac63b43c6b51f913d230153c1f8650426478
* According to python docs 'gethostbyname() does not support IPv6 name resolution'
Changing it to socket.getaddrinfo.
* get_ipmi_address method returns tuple (<BMC Address>, ipv4, ipv6)
Change-Id: I7d57556ff00cf29aee6f1591dcb924086099c7d5
Story: #2004648
Task: #28601
The optional query parameter 'limit' is not supported by the
request done to get introspection data.
This patch removes it from the api document reference to avoid
confusion.
Change-Id: I90777d426809ef719817198e6baade8442d60041
This patch provides a simple tool to help with migrating introspection
data between two introspection data storages.
Story: 1726713
Task: 11373
Change-Id: I2a930dbad2178e3dde6725e2620d8099e4e21d78
In https://review.openstack.org/#/c/637673, the get introspection
interface is narrowed down to only accept node uuid, which previously
accepts both uuid and name. But the name to uuid conversion is
missing in the reapply api, which causes feature regression, and
this is the fix :(
Story: 1726713
Task: 11373
Change-Id: I6912853deab77f1365f665ca1e52c13063d2cdf1
Since the introspection data could be retrieved from inspector API,
there is no need to keep an extra link in ironic. This also keeps
inspector's behavior consistent by doing so.
Change-Id: Ib667afc4f16551b195641888831df7b681217184
Story: 1726713
Task: 11373
The patch revises driver interface for the introspection data
backends. Previously getting introspection data supports node
uuid/name, while saving takes a node_info object, which is not
consistent and makes migration tool looks weird if implemented
based upon it.
For the get() interface, actually only uuid will be passed in,
so it's narrowed down to accept only uuids, logic names will be
converted from api level if there is a need.
The save() interface is changed to accept node uuid instead of
node_info, which is consistent with the get() interface.
Change-Id: I4702ed7372d0e60ed6252879a7496649a0453b84
Story: 1726713
Task: 11373
In testing discovery or more advanced workflows in CI
that incorporate with inspector, it is generally useful
to enable the post processing power setting to be updated.
Change-Id: Iece696e91edd78f42bde21738442d0b27ea3dafa
When using store_data=database with sqlalchemy, running introspection results
in a DBDuplicateEntry error. This happens because the query when adding an
entry uses both the primary key (uuid) and the processed flag, but the
processed flag is not a key. This change makes it a key so that both
unprocessed and processed data can be stored in the table.
Note - since the previous migration hasn't been released yet this fixes it
without creating a new one.
Change-Id: I052594d529ae363fce50b1726169d86583bb1439
Story: #2004992
Task: #29463
An issue is spotted during implementing [1], we have a solely code
path from api, do_reapply, reapply, to _reapply. Inspector now reads
intrspection data ahead, so there is no need to do so in _reapply,
reading introspection data there is never reached.
The patch removes unecessary code and corresponding tests.
[1] https://storyboard.openstack.org/#!/story/1564863
Change-Id: I5558bce2bc49de3d1c5dba59e203de4824a3addd
Story: 1726713
Task: 11373