This is the frontend for discover_and_enroll_nodes workflow.
Change-Id: I822ec89add3742020262c091e79ff94f92ef92e7
Depends-On: I158f0b8f5251d9d94e7e57b3fe24362316d26599
Implements: blueprint node-discovery-by-range
Previously we did not handle exceptions from ironic_inspector_client, so nodes
that were never introspected (e.g. ones in maintenance mode) caused this
command to fail.
Change-Id: I58756256fe70992ff5618a3f307ce34f95e71efd
Closes-Bug: #1689540
Make debug puppet logs for undercloud deployments to fit
heat JSON limits. Raise the default limit by an 8 times.
Closes-bug: #1690828
Change-Id: I805247adf2cb669e6db035c8f4c4c014bcf513a7
Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
This command is to be used by an operator to run sosreport on
specific set of servers (or all) and retrieve log bundles that can
be used to debug the status of the cluster or troubleshoot issues.
Depends-On: I47c486d14c46a653c61cfd92d9f484efe0407217
Change-Id: I45699dfa6eb3e83d419c7041dbb72cc5d5e4f0ea
Implements-Blueprint: capture-environment-status-and-logs
The result of syncrhonously called Mistral actions wasn't being checked
to see if the action passed or failed. The result is now checked and if
the action has failed, an exception will be raised.
Change-Id: I95ae8c98fec94cf91f3f209b593f6c1815729fd4
Closes-Bug: #1686811
This change replaces all of the DeployOvercloud __init__ method with a
_setup_clients method that is called from take_action.
Hopefully this defers auth long enough for the new osc-lib behaviour,
and the recommendation from dtroyer[1] that command __init__ methods
shouldn't be used for this kind of setup.
[1] https://review.openstack.org/#/c/461069/1/osc_lib/shell.py
Change-Id: If21d97c09d7c5081306f0005a34eef8dbca4c458
Closes-Bug: #1687032
With a recent osc-lib[1] change it is no longer possible to make
auth-requiring calls in the command __init__.
This change defers calls to tripleoclient.object_store so that it is
made when auth is properly setup.
[1] I72ae6154268bdf26be6054c0fef6a4c67c71119c
Closes-Bug: #1687032
Change-Id: If640da00d9b8d49ff935209e3b31dcf0f327ad63
This patch updates the create plan command to use the latest
changes to the workflows in tripleo-common that allow a
deployment plan to be created or updated from a git
repository.
Partially-Implements: blueprint git-deployment-plan
Change-Id: I71c1f5d58abcf0a55598f02fbd21fdffedcf5291
There are a few legacy secrets created via the heat stack which
aren't currently generated via tripleo-common, so we need to save
these values or they get re-generated when you re-run undercloud
deploy.
Note adding the test also required some fixups to make it run on
python3.
Change-Id: Ib7e30ec7ecb3cc35b333e72a8b56b7e3c7d96899
Related-Bug: #1684044
With pbr 2.0 and Sphinx 1.5, the setting for treat sphinx warnings as
errors is setting warning-is-error in build_sphinx section. Enable this.
Change-Id: I981b05275e1a75e3c42ea3ef24812cded0cefc0d
An upcoming release of pbr will require explicitly stating which
builders are requested, rather than defaulting to html and man. Head off
any potential impact this may cause by explicitly setting this
configuration now.
Change-Id: I06850630c9e85811a09f8464060728679c093dbf
Some tests didn't mock this step out, this means that every test run is
creating it. We really need a better solution to this testing mess, but
for not it would be better to stop creating the file and leaving it
around.
Change-Id: Iac150135c4c205f1bd713ec61c44d70878ef37cf
Add a new cli argument, --skip-deploy-identifier. The argument will
eventually be passed to the tripleo-common DeployStackAction action to
disable setting a unique value for the DeployIdentifier parameter, which
means the SoftwareDeployment resources in the templates will only be
triggered if there is an actual change to their configuration. This
argument can be used to avoid always applying configuration, such as
during node scale out.
Removes the setting of DeployIdentifier completely from tripleoclient,
as that is now handled by the tripleo-common action.
Depends-On: Idb901a841846fec33d189b3c95f669b0380498ce
Change-Id: I294eabe84e070367981534030b5927062f00c628
Multiple plan support has little value while unnecessarily complicating
the command UX. Being able to do one export at a time is enough. Operators
are encouraged to script the command if they wish to achieve the effect
of exporting multiple plans.
Partially implements: blueprint plan-export-command
Change-Id: I99cf5dfde82a8b4b9dfb7b3f61f4f4f1d31b58f7
The update abort is not reliable, and could mess up a TripleO stack
beyond repair. Since this has a potential for data loss, I would
suggest this be removed without a deprecation period.
Change-Id: Ieec4f01e38768eafb3df1f06340bdd3e220d30bd
This command receives the same --config-file argument as "overcloud
container image upload" and invokes kolla-build to build the images
specified in that file.
It is also possible to specify --kolla-config-file multiple times to
control the behaviour of kolla-build.
Change-Id: If06a941eaf8d92c29ed84387c4f040866cf07cfa
Depends-On: I061f626fdb3d71613aa23436bf6c53cf4de62213
There is a potential for a race condition if the websocket is not set
up before the call is made to Mistral. The websocket setup can take
sufficient time to set up, and if an error is thrown immediately from
a workflow, it has the potential of returning a message, and having
the websocket consume it before the websocket client is even set up.
Also, I think this has the benefit of keeping the call to Mistral
from even being made if there is something wrong with the websocket
setup.
Change-Id: Ib331037a7f5f4e59862d2b9646a83acdb18313eb
If a user deploys via the GUI, then this file wont automatically be
created. We can now create it via a Mistral action - this command
provides CLI users a way to call that action and save the file.
While adding tests a typo was spotted in utils.py, this is also resolved
by this patch and covered by the tests.
Change-Id: I7808cd2c43a95bbb154ec6c6f77896b97551e354
This updates the workflow wrapper to use the multiple message interface.
NOTE: the test for this patch use the fake websocket that was updated in
a previous patch, so while this code is tested, no test updates were
needed for this change.
Change-Id: I6f6b09589646b6eaf5811fdf23558c949fbbc2b8
Partial-Bug: #1646887
Dmitry Tantsur