Split up the error message to wrap the download and conversion to the
actual file system linking steps.
Co-Authored-By: Marek Skrobacki <skrobul@skrobul.com>
Change-Id: I644fc0ff7d56de43189f572ce3f901ceffc1ffd5
Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
Signed-off-by: Marek Skrobacki <skrobul@skrobul.com>
Moves configdrive utility functions to the more appropriate configdrive
dedicated utils module to improve code organization and separation of
concern.
Closes-Bug: #2113892
Change-Id: I5851e84fe8f15de05dcddca773b1f28f639dc617
Signed-off-by: David Nwosu <nwosudavid13@gmail.com>
Signed-off-by: Afonne-CID <afonnepaulc@gmail.com>
The json-rpc debug logging can be... very... verbose. And that
verbosity when including all of the data crossing the RPC bus
provides limited value when your just focused on addressing
a performance issue.
The key is much more "when did I send a request" and what
was the ID, and similarly "when did I get a response".
By default, if there is a request ID when we're in debug
logging mode, we will keep the entire result to a brief
result.
Assisted-By: Claude Code - Claude Sonnet 4
Change-Id: Ib6e4db0e8689ed2081f29b1d1d22a7f01a0e1221
Signed-off-by: Julia Kreger <juliaashleykreger@gmail.com>
Bumps to the latest versions of the tools we use in pre-commit except
for codespell which will come in a follow up.
Change-Id: I61f69d914b28bb13a1183315d6181db872cca638
Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
We are running pycodestyle and pyflakes checks via ruff so we can
disable to double run via hacking using flake8. Remove ignores that were
not used or covered files that did not exist.
Change-Id: I342ef72e0ad007fa6f5b72f634ee90ef30137446
Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
... because the project was retired some time ago.
Change-Id: I41d7656f6c87a340afedcdbf67c582d68a08744d
Signed-off-by: Takashi Kajinami <kajinamit@oss.nttdata.com>
Deprecation of ironic-inspector was announced long ago (during 2023.2
cycle) and ironic-inspector was retired this cycle. So it's time to
drop the all remaining code to force migration to the built-in
agent interface.
Change-Id: I14a87599f9f47b167f8f1a84704982301d033381
Signed-off-by: Takashi Kajinami <kajinamit@oss.nttdata.com>
The code did not initialize a client so it resulted in an exception
always occurring. It also used the image attribute on the client
attribute of the service but the client attribute of the service is
already the image attribute. Create a wrapper method to use the API
correctly and prevent similar issues.
Closes-Bug: #2099276
Change-Id: Ib803c066ca28d1c05a345b7a982a0daabbd7d52e
Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
Adds a configuration file class for Trait Based Networking.
The class can read, validate, and parse a YAML config file conforming to
the expected structure of a TBN configuration file.
Parsing renders the configuration to TBN objects.
Change-Id: I69802006274d2373e73ba3d2779c29e365caea85
Signed-off-by: Clif Houck <me@clifhouck.com>
Base models for the majority of the Trait Based Networking feature.
Adds a lark-based parser for filter expressions found in Trait Based
Networking configuration files.
Change-Id: I4414463c70d37a7c6b5a957941a2607b5c15ab9e
Signed-off-by: Clif Houck <me@clifhouck.com>
The combined Ironic service was passing the no_fork parameter
to ServiceLauncher, which was wrongly mapped to ProcessLauncher [1]
Switch to ProcessLauncher which properly supports no_fork since
oslo.service 4.2.0. This ensures VNC signal handling works correctly
and matches the pattern used by other Ironic services.
[1] 0dfdf810ac
Change-Id: Iea150a5c3f147b7e4f8a778510bfc061a14f289a
Signed-off-by: Riccardo Pittau <elfosardo@gmail.com>
I took a quick look and felt like it wasn't clearly demonstrating
current state, so decided to revise the text. It should be more
clear now as to the state of reality.
Change-Id: I1b3c808f6d75e1e7fa532d18df82418a4747071a
Signed-off-by: Julia Kreger <juliaashleykreger@gmail.com>
Add validation check that raises NetworkError if the bound port list
is empty after attempting to bind all ports.
Closes-Bug: #2131962
Change-Id: I1d533b942144680c2622fec63caa092f96e481f5
Signed-off-by: Afonne-CID <afonnepaulc@gmail.com>
Adds support for comma-separated, non-consecutive port ranges
in ``[console]port_range``.
E.g., ``'1000:1100'`` and ``'1000:1100,2000:2500,3000:3100'`` are now
both valid.
Some of the unit tests were generated with AI.
Closes-Bug: #2131055
Assisted-by: Claude Sonnet 4.5
Change-Id: Ie35cfb6f431a58857f50b9ceda0daf601c8a6737
Signed-off-by: Afonne-CID <afonnepaulc@gmail.com>
