Before, the "config" parameter would only support a JSON file, which
would be sent to Paunch via YAML.
Now, we accept a directory path, where all .json files in that directory
will be sent as a single dict into Paunch via YAML.
It's part of the effort to break down the container config into
individual files per step and per container.
Story: 2006732
Task: 37162
Change-Id: Ie8bda685c499ff5ef1dc2799a85287662b5af407
Make sure the item in the list of volumes isn't empty or we can end up
with a user error where the item is '' and podman will crash with:
"host directory can not be empty"
Note that I hit that since we have some conditional volumes in THT,
where sometimes we get '' in the list of volumes. Let's handle this
case.
Change-Id: I8df1fb5d1bb72bd492e9f8e2b29dc6dae09d6b44
Otherwise we run into dependency problems where a kernel module
may not be loaded yet. This results in the sysctl options not
existing yet in the virtual file system /proc/sys/.
Also add br_netfilter to the required modules for TripleO.
It is required for bridge-nf-call-* sysctl options to work.
Change-Id: Ia28f2fdef34e739801c51828c99e9e6598dd2efb
Related-Bug: #1843259
Signed-off-by: ekultails <ekultails@gmail.com>
This change updates our bindep configurations and local test runner
to ensure it works with RHEL/CENT8.
Change-Id: I5585fa8b6e2094132596073e6c441267a18aea90
Signed-off-by: Kevin Carter <kecarter@redhat.com>
This playbook encapsulated the logic that Mistral provides for the
overcloud delete CLI command. Replacing the following two workflows
- tripleo.plan_management.v1.delete_deployment_plan
- tripleo.deployment.v1.undeploy_plan
Related-Bug: #1835242
Related-Bug: #1835246
Related-Bug: #1835232
Co-Authored-By: Kevin Carter <kecarter@redhat.com>
Co-Authored-By: Luke Short <lshort@redhat.com>
Change-Id: I38d9535d9b71fca93eb34d2018355c0f8c8fa8fd
If the command is in this format ['bash', '-c', 'ls'], it'll now work.
Before only 'bash -c ls' would work.
It's for compatibility with how THT generates container config.
Change-Id: Ib8e943f92b85adcde86d814268a97369b4317ec3
The readme files are unnessesary, and unused. All of the role documentation
is generated as part of the tripleo-ansible docs. Because these files are
generally neglected the documentation they provide is at best missing
information and at worst incorrect. This change removes all of the role
readme files and ensures our role generation process no longer creates
the file.
Change-Id: I030d7f9ef54250cef4019d40366febe899f37bd9
Signed-off-by: Kevin Carter <kecarter@redhat.com>
When using tripleo-transfer role, optionally a file will be created at
a path specified by tripleo_tranfer_flag_file variable (if set). The
transfer will be prevented in case the file already exists. This is to
ensure the data transfer happens only once and the data is not
overwritten by accident.
Additionally, the README.md file from the tripleo-transfer role is
removed because tripleo-ansible docs are generated by molecule and
this file is not used there.
Change-Id: I3a08949436dcb634df34622c9aeead552dd38443
This change modifies the argument_spec making it so we load the options
key from the documentation constant. This will reduce the code we have
to maintain and ensure our documentation is always in sync with the
module capabilities.
Change-Id: Ib3e56f94debd6e99d9319902517811cf77ecc874
Signed-off-by: Kevin Carter <kecarter@redhat.com>
The output format for "openstack port show" has changed breaking the
octavia deployment ansible. This patch changes the output format to json
and uses json_query to parse out the results.
Closes-Bug: #1846399
Change-Id: I3b5c989259fd4347c41d392ad79bfd6b9459d605
Also change parameters to what they should be according to podman
specs. Add defaults for booleans in description. Improve module
docs and other stuff.
Change-Id: Id6e85099d6867eb873aa3bcefc1148e081812f68
This patch moves calling_ansible_environment_variables after the preset
vars when constructing the ansible-playbook commands for ceph-ansible.
This makes it so that variables set on the command line take precedence
over what is set by these roles. For example, setting a customer
ANSIBLE_CONFIG on the command line would now have that value passed to
the ansible-playbook commands used for ceph-ansible, when previously it
would have been overridden as {{ playbook_dir }}/ansible.cfg.
Change-Id: I2a91ca1aa91d91f6f525f2fba0b093c7ec216e1d
The value from all_nodes_extra_map_data must be a list so it can be
concatenated with the existing services data.
It appears the extra [] was accidentally dropped in a previous commit:
eb5bfcf187
Change-Id: I0d7778c7907fcb87aa9b1b6d9e33a1c430cbf025
When certificates are neither generated nor provided (e.g. on update)
related variables would be empty or undefined and need to be handled
accordingly.
Closes-Bug: #1845366
Change-Id: I361c90e438e79e180e2a745da82e619347fed13e
A new module to gather facts from podman containers running on a host.
This will allow to know informations about containers running with
podman on a host. The module allows to gather all containers info or for
a specific one, or a list.
Change-Id: I33595d19cebf6508008792bc08c1bf0a192db128
We found that by mistake we are excluding the controllers
path in the Undercloud node.
This patch makes sure we exclude the path only if its
a controller node (the node has galera and mysql).
Change-Id: I331d89ebf07eb56870a84330ff7ad2de629913bf
- Make self.config optional
- Make self.config_id a list
- Remove useless self.module.check_mode condition (we never reach it)
- Add paunch_cleanup(), calling paunch.cleanup directly
Note: the cleanup has no stdout/stderr/rc for now; it's not implemented
on the paunch side.
Example of playbook:
- name: Cleanup containers for step 1 and step 2
paunch:
config_id:
- tripleo_step1
- tripleo_step2
action: cleanup
Change-Id: Ib9ebc64b69ad778fa29132c9079ba04a0adbd2b4
Emilien Macchi