While using tripleo deploy command we need sometimes to save
reproducer script for issues investigation. Add option to enable
it '--reprocuce-command'.
Change-Id: Ie5e694ab3b9e5196aea76da51e4e567ec8970fd7
Recent removal of the 'validations_libs.utils.convert_data'
function, implemented by the Ief9459e914ea73fbd59a85c07a55c26078e0126a,
resulted in a change of accepted argument types of
the 'validations_libs.validation_actions' methods.
This patch turns the passed argument from string into a list.
Closes-bug: #1936465
Signed-off-by: Jiri Podivin <jpodivin@redhat.com>
Change-Id: I4e96f5a0e013de5abcde837dad3c5f9d5228dd4c
ANSIBLE_VARS_PLUGIN_STAGE=inventory doesn't work with implicit
localhost in Ansible 2.11, use 'all' instead.
See https://github.com/ansible/ansible/issues/72590
Change-Id: Idc9e36563854b7e6590f74f14774d506c5dbc4c3
Normally, backups are not done interactively, but during nights or
weekends. This patch introduces the capacity of introducing in an easy
way cron programming that will execute backups in predictable intervals.
Closes-Bug: #1935675
Depends-on: I86b3fb2c3d846bcd56859e0d572f1fe4d5e148fa
Change-Id: I28219759922b7407121a3fa4d2f49ee1f4b8a3cd
Because export imports utils and utils imports export
there is a cyclic import so pylint objects PEP8 R0401.
overcloud_deploy and overcloud_export need the function
export_overcloud which depends on export. The function
export_overcloud was added to utils for that import but
this created the cyclic import. To work around this,
redefine the export_overcloud function in export and
have overcloud_deploy and overcloud_export import it
from export, not utils. Then remove the utils import
of export.
Change-Id: I8bab41ee9af9729cd63ed7132928cfbc1770887a
The `role` commands are warning users that they have been deprecated
and informing users that they should use the alternative `roles`
commands. Sadly, we have no such `roles` plugin, so following the
deprecation warning provides false information which results in an
error should the command be used.
This change removes the deprecations warnings for the `role` commands.
If this needs to be refactored later we can; however, given the
information is wrong and there's no alternative plugin to use, making
the UX bad, we're removing the misinformation.
Change-Id: Ie048cdbe52f5b935ddd4b42febb373f9ff96d8d8
Signed-off-by: Kevin Carter <kecarter@redhat.com>
This change updates the exception call when using the new `vip_file`
argument. Presently there's no CommandError within tripleoclient,
which results in an AttributeError, so we move the call to osc_lib which
has what we're expecting.
Change-Id: I94e00ed3000982c2563c54d73e0c1cfd0ac7c6a3
Signed-off-by: Kevin Carter <kecarter@redhat.com>
The argument to enable disable node port management
when provisioning or unprovisioning baremetal nodes can
be enabled by default.
It is needed to have an ansible inventory generated,
which is required for whole-disk-image growpart execution
in change: I085ab9da30e1e1a7d2b9a9f230dd0275bd40480d.
Growpart should always run.
Change-Id: Ia0df205ea568f089c7cb7e5464bde76d294f4c36
This change removes some redundant definitions of default values
according to the following three points.
- The get method returns None by default if the key doesn't exist,
so it is unnecessary to define None explicitly.
- Setting '' as parameter defaults is redundant because we always use
"if CONF.get('<name>'):" to check whether the parameter is set.
- In oslo.config all options default to None unless the default
parameter is explicitly set. Thus defining default=None is redundant.
Change-Id: Ib14b24b9054fe4d0efc3e03f9c6683ae999e3202
The container_cli option should be set to podman because docker is no
longer supported since Ussuri. This change ensures its value is
validated at parser layer.
Change-Id: I9e1177e68d1d6fbbe0a2ca0350ece1c6c656ccaf
Allow setting '--templates', the path tot the THT root
directory. Environment files created by the node and
network provision commands will be created with the user
specified path to the THT root. Defaults to:
/usr/share/openstack-tripleo-heat-templates/
Closes-Bug: #1932079
Depends-On: I9decd8fb9860cd17dc9e69b9970d9f9dabf11428
Change-Id: I5dd81788c08bb0d797190a5fb8bb120b7d8b4f32
Prior to change Ic9db8d852fbfa50f4ac574ebe524c1fb1d85c588
the tripleoclient constants OVERCLOUD_ROLES_FILE and
OVERCLOUD_NETWORKS_FILE was used as the defaults.
This was changed to use the defaults in process-templates.py
This restores the previous behaviour, honoring the tripleoclient
constants.
Closes-Bug: #1934010
Change-Id: Ie133a6718ca50f30fee7a48fc83ff0e89d84d2dc
The templates option should point to a directory instead of a file, but
its current description doesn't clearly explain that.
This change updates its description so that it is clearly explained
that the value should be a directory path and regarded as a home
directory of template files.
The existing descriptions with 't-h-t' are also updated because
the word might be unclear for users.
Change-Id: Ib1857c259108a0a13909ca53e54105f1f6bda710
Rather than use the generated ansisble inventory.yaml or
tripleo-ansible-inventory.yaml, switch to use the native AnsibleRunner
inventory directory instead.
Change-Id: I74b31d1080fd7b19efbe706024267aa871a6029b
In I9f5dc697dd1cd7895723d4c23ce89a00631b310f this was added.
But we don't call the function to create the archive_dir
before copying.
Fails with "Exception archiving deploy artifacts: RuntimeError: sudo
failed: b"cp: cannot create regular file '/var/lib/tripleo/archive"
Change-Id: Ie7445a691f06fce5349d2fc571e65a78671b3098
... because support for Mistral[1] and Zaqar[2] were both deprecated.
[1] 47ab368caf
[2] d00943a58a
Change-Id: Id0286c75eef31dca8e720e36d82d9f3ec10f5f1c
... becuase these parameters are used not only in undercloud deployment
but also in standalone deployment.
Change-Id: I672627f4f97d241abaab47f56dc92b36d778ac97
We currently re-implement what the download_config function does in
tripleo-common. Let's just reuse that function to align better with what
the overcloud does.
Depends-On: https://review.opendev.org/c/openstack/tripleo-common/+/798153
Change-Id: Idc66d711ca7f4ce6369e38526257b887b9bc7dfd
Zuul
Sagi Shnaidman
Jiri Podivin
Juan Larriba
John Fulton
Kevin Carter
Harald Jensås
Takashi Kajinami
Alex Schultz
ramishra
Sandeep Yadav