delta/openstack/ironic-python-agent.git/ironic_python_agent/ironic_api_client.py, branch 8.2.2

Remove runtime dependency on pbr

2021-04-26T07:16:34+00:00

Pbr is a very heavy package to depend on. It requires git-core, which is
16 MiB on my Fedora. We only use it to detect the version, which can be
done without pbr using a much lighter importlib_metadata.

Copied from https://review.opendev.org/c/openstack/osprofiler/+/739379

Change-Id: I5f434e6bfde6f645804941f3a36d5458a28270e7

Correctly decode error messages from ironic API

2020-12-07T17:59:08+00:00

Knowing a status code is simply not enough for debugging.

Change-Id: If1d3f182ab028948ff05aea7e8024d4e7bc3d53c

Allow manual setting of Ironic API Version

2020-10-23T15:38:42+00:00

Typically, the Ironic API client in IPA will autodetect the API version
based on the output of a GET of the root of the API. If for some reason
this API endpoint is restricted, or the operator wishes to limit the
Ironic API version IPA uses, they can now set CONF.ironic_api_version to
avoid autodetection and force a version.

Change-Id: Ib96a1057792f45f2e4554671e32c436140463ee8

Merge "Generate a TLS certificate and send it to ironic"

2020-09-23T12:14:38+00:00

Merge "Refactor API version negotiation code"

2020-09-12T15:24:42+00:00

Generate a TLS certificate and send it to ironic

2020-09-11T15:46:52+00:00

Adds a new flag (on by default) that enables generating a TLS
certificate and sending it to ironic via heartbeat. Whether
ironic supports auto-generated certificates is determined by
checking its API version.

Change-Id: I01f83dd04cfec2adc9e2a6b9c531391773ed36e5
Depends-On: https://review.opendev.org/747136
Depends-On: https://review.opendev.org/749975
Story: #2007214
Task: #40604

Refactor API version negotiation code

2020-09-11T15:45:37+00:00

Makes sure heartbeats can send versions higher than one required for
tokens while also making sure we never send a version we don't know.

Also makes code easier to understand.

Change-Id: Ice1e7d45ea90c9fd8220c4b94e691b6015e23074

Replace oslo's loopingcall with tenacity

2020-09-01T09:53:25+00:00

The latter has a more natural API and does not have a hard requirement
of eventlet. It is already a dependency of ironic-lib.

Change-Id: I68de9e989af137b34c19bbaf9b7c0a5ba6e1d4e3

Clarify connection error on heartbeats

2020-08-20T20:45:47+00:00

Heartbeat connection errors are often a sign of a transitory
network failures which may resolve themselves. But an operator
looking at the screen doesn't necessarilly know that.

They don't understand that there could have been a network
failure, or a misconfiguration that caused the connectivity
failure and soft of kind of default to "well it failed"
without further clarification.

As such, this patch adds explicit catching of the requests
ConnectionError exception and rasies a new internal error
with a more verbose error message in that event to provide
operators with additional clarity.

Change-Id: I4cb2c0d1f577df1c4451308bd86efa8f94390b0c
Story: 2008046
Task: 40709

Add possibility to pass global request ID

2020-08-12T12:21:08+00:00

It can be done via ipa-global-request-id kernel commandline parameter.

Story: 2007681
Task: 39792
Change-Id: I6f544327d310c976a1625cfb411947591867882a