When running playbook, don't connect to localhost, just use
undercloud as usual and run localhost tasks with delegate_to.
Change-Id: Ib8e6408f232f65d1b7addf00d89574af114ec809
Close-Bug: 1911451
Co-Authored-By: Sagi Shnaidman
This introduce needed changes to be able to deploy an additional
cell via oooq. If additional_cell is set to true after the
overcloud got deployed the additional cell gets deployed. Todo
this information gets extracted from the overcloud stack, which
is requried as input to the second heat stack, where the additional
cell is managed with. As a reference [1] is the scenario which
is installed adding a single additional cell.
config/general_config/featureset063.yml is used for the configuration
and config/nodes/1ctlr_1cellctrl_1comp.yml has a nova config
example.
[1] https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/features/deploy_cellv2_basic.html
Depends-On: https://review.opendev.org/691688
Depends-On: https://review.opendev.org/704661
Change-Id: I2de781fcec64e862bde34929547b578d2af0c16c
- run `pre-commit autoupdate` and fixed new issues
- adopted newer pre-commit config for ansible-lint 4.2.0+
- fixed some reported broken rules
- temporary disabled few rules, just to contain the size of of review,
planning to drop/fix them in follow-ups.
Change-Id: I807ba4e919527be56c85ec72d0f4c7148f04e994
- Fixes 206 errors and removed the skip.
- Also replaces revision with latest tag (pushed today)
Bug: https://bugs.launchpad.net/tripleo/+bug/1848512
Change-Id: I69d75ba9de8ab0e91eb45fd4f9febfdb28cdcb3a
Using wget can lead to problems because it's not default.
We should use curl instead
Change-Id: Idc859c57582ed3b83c6177e91e14ba4828311826
Closes-Bug: #1847344
Upgrades yamllint to latest version and adots use of its strict
checking.
Fix all known problems reported by yamllint so we don't have to do
that while touching these files.
Change-Id: I4bdc520d9e2aff086c4b463718bc1e053261a4f5
Story: https://tree.taiga.io/project/tripleo-ci-board/task/381
Makes those files conformant with current linting rules and avoids
linting errors when we need to toch them again.
Previous doing "pre-commit run -a" uncovered these errors, now is no
longer reporting any other errors.
Change-Id: Ie4cf229c8f11c2b55b323eac23c89483b26d3781
Due to this patch [1] baremetal undercloud jobs were failing because of
the fact that we were considering a string instead of an array inside
the pacakage installation task. This patch fixes the problem considering
the variable as a set of items.
[1] https://review.openstack.org/#/c/517444/
Change-Id: Idf3250b61974bc7947b388d327a07b076dcf5809
This patch contains changes to deploy a containerized undercloud by
using: "openstack undercloud install --use-heat" command.
Details of the changes:
- The experimental containerized undercloud deployments driven via
`undercloud deploy` CLI is not supported anymore for Pike.
- overcloud-prep-containers: fix an jinja2 syntax issue with "is defined"
- overcloud-prep-images: "--http-boot=/var/lib/ironic/httpboot" is
added to the "openstack overcloud image upload" command.
- undercloud_rpm_dependencies parameter has been created, default to
undercloud_tripleo_pkg value.
- Remove "undercloud_tripleo_pkg" so we only use
"undercloud_rpm_dependencies". Note that "undercloud_tripleo_pkg"
was only used in this repository.
- Remove "undercloud_update_packages" option which was technical debt.
- Add "undercloud_install_cli_options" option which is used to add extra
options to "openstack undercloud install" command.
- Add "undercloud_container_images_file" which is the environment file
with default parameters for containers to use with undercloud install CLI.
- Add "undercloud_custom_env_files" which provides custom t-h-t env files
for the "undercloud install" `undercloud.conf` interface.
- Generate the undercloud prep containers script. This script will
reproduce what we expect from our users to run with
"openstack overcloud container image prepare" command, for the
undercloud.
- Update undercloud-deploy.sh.j2, remove tech debt and update for new
bits.
- Install heat libraries and other dependencies for containerized UC
from python-tripleoclient-heat-installer. This should go away
once python-tripleoclient installs python-tripleoclient-heat-installer
automatically.
- Update undercloud-install.sh to use undercloud_install_cli_options
which is empty by default but will be used in the container-undercloud
featureset to override with the right parameters.
Partially-Implements: bp containerized-undercloud
Co-Authored-By: Bogdan Dobrelya <bdobreli@redhat.com>
Co-Authored-By: Emilien Macchi <emilien@redhat.com>
Co-Authored-By: Ian Main <imain@redhat.com>
Co-Authored-By: Martin André <m.andre@redhat.com>
Co-Authored-By: Wes Hayutin <weshayutin@gmail.com>
Change-Id: I7d056b208d5ac0639e3277f3f4ae453a9ee639e8
This commit removes all the extra spaces from the README file related to
the baremetal-undercloud role, fixing also its structure, and updating
some sentences related to the usage.
Change-Id: Id4501dfc15d29f2cc6484462f65ef7484722fddb
This commit removes the validate-ha role which is now contained in the
tripleo-quickstart-utils package [1] and modifies the README file
related to the baremetal-undercloud role accordingly.
[1] https://github.com/redhat-openstack/tripleo-quickstart-utils
Change-Id: I6ebb69c8b42387eccfeca92abea1c9016db8a902
We have jobs using this role in ospphase0 that are failing at the
machine provisioning step, and don't have access slave presently. This
change will let us see the output from the machine provisioning step in
the job output.
This role is used internally, and we have jobs failing
Change-Id: Ia0db5a65e64abfb4f2f161cf71c7076a18257449
This commit fixes the default value for undercloud_network_cidr to what
now is the default in oooq, so 192.168.24.0/24.
Change-Id: I08c3be7823177b9c2f828f29fc17f89d625afd38
When using boolean variables in conditional tests, those variables
should be filtered as '|bool'. Otherwise, they are being evaluated as
a string and return True in any case.
Change-Id: I1e3642cb2d33d839e4808ae79ae991175e32dd0f
Signed-off-by: Gael Chamoulaud <gchamoul@redhat.com>
When pipe is added to command, it should still exit with its
result code, so save it by setting pipeline option before each
command.
Close-Bug: #1676156
Change-Id: Ibbe49b4a15a5b7825447a563fe35af85fd48b3ff
Prepend a timestamp with awk to our deployment and test commands. This
allows html linking from the log servers and also better correlation
with the system logs.
We are not using the "ts" command from the moreutils package to avoid
extra dependencies. The drawback is the lack of sub-second timinig.
It is possible to disable the timestamping by setting the
timestamper_cmd to an empty string.
Also add a README file for the extras-common role.
Closes-Bug: #1673046
Change-Id: I8a2a1e4fde2ba805d3361e2e7a2337e873133ef0
We are moving away from hardcoding the stack user and opting to
replace it with `undercloud_user` and `non_root_user` as appropriate
to increase flexibility of OOOQ and OOOQ-Extras.
Change-Id: I7ea43a3fabab2c69fd4f3b5550ea5e4627443406
We need to differentiate local_working_dir from working_dir
as well as decouple the stack user from `ansible_user` var.
Both of these are causing issues as we begin to automate
deployments in more environments.
- Cleanup duplicate variables that are consumed via extras-common
- Note: extras-common depends on the common role in OOOQ
- Cleanup redundant var and superfluous quotes from overcloud-scale
role
- Cleanup redundant comments in <role>/defaults/main.yml
Closes-bug: 1654574
Change-Id: I9c7a3166ed1fc5042c11e420223134ea912b45c5
As more ansible variables are shared or reused across roles it is
important to define these variables in a role that is always
executed. In this case that role is extras-common.
Note: This review is a blocker for https://review.openstack.org/#/c/418998/
Change-Id: I31fd13d7bcb98d73e7f16048c57c027d95faeec5
This change modifies the default NTP server for the Undercloud to a
publicly reachable hostname. The pool.ntp.org domain name will resolve
to a list of publicly available NTP servers.
Change-Id: I86143837d4fdd9a0cfea02a12e2eac74746d9eff
This commit replaces the actual repo configuration and overcloud
images preparation inclusions with the already existing external
dedicated roles (repo-setup and fetch-images).
To configure this the two playbooks are modified and a new "packages"
subrole is created under the main baremetal-undercloud one.
With this in place it is possible to follow this sequence:
- Provide and prepare the undercloud machine (role baremetal-undercloud)
- Inventory the newly created machine (role tripleo-inventory)
- Setup repositories (role repo-setup)
- Install and configure packages on undercloud (baremetal-undercloud's
subrole named "packages"):
- tripleo package
- additional packages (optional)
- ntp configuration (replaces what was done before in the custom
template undercloud-repos-conf.sh.j2)
- Install the undercloud
A new review to update the documentation will be done after this gets
merged.
Change-Id: Iba11c4d61a63bb05016177abd16403168d82331b
We have built up a lot of cruft over the months within our roles.
This review aims to clean up the unused sections. Now that we have
each role within a single repository, ensuring a clean house
is even more important than when they were separate.
Change-Id: Ibb34b376c516a09ba0f7fdb02580e8803497fc50
This commit adds the ability to pass a specific custom ssh private key
to quickstart so it can be able to connect to machines which are
customized inside the local .ssh/config file and to which you don't
connect with the default key of the user executing quickstart.
If the variable virthost_key is defined the initial ssh.config.ansible
is populated with this content:
Host {{ virthost }}
Hostname {{ virthost }}
IdentityFile {{ virthost_key }}
User root
StrictHostKeyChecking no
UserKnownHostsFile=/dev/null
So quickstart is able to connect to virthost, otherwise a connection
error would happen.
It also updated the README with all the explanations around this new
parameter and the fixes about the changes happened in the meantime
within the role.
Change-Id: Id9c79cf1c4aa3ff3b0c8484cebabe2b67faa691a
This commit refers to the baremetal undercloud role:
- Fix baremetal undercloud playbooks to support new extra roles names;
- Define the repos as a list which can be overridden;
- Removes the restart of libvirt before the customization of the
overcloud image. This was initially included because it seemed to be
necessary for virt-customize to work, but further tests confirmed it is
not needed anymore;
Change-Id: I52b1a82f260fa1a14d32216f3300888cb10f0244
https://review.openstack.org/#/c/397312/ has removed some options in
tripleoclient.
Specifically, --neutron-network-type and --neutron-tunnel-types
are being used in tripleo-quickstart .This patch removes them
from extra_args and scale_extra_args.
This parameters are being passed via environment files or using vxlan
by default so i think it doesn't need any other adjustment.
Change-Id: Icb36a45797bdb5e217ce9f33fa67e6697590b23c