docker_image_tag variable has to be overridden to point to
dlrn_hash_newest instead of dlrn_hash. This has to be done in
order to substitute tag in containers-prepare-parameter.yaml with newer
tag in downstream update job.
Depends-on: https://review.opendev.org/c/openstack/tripleo-ci/+/888933
Change-Id: If79b38bf427f75143582c1db7e524123fb311e81
Some jobs[1] are not working with current awk command to get
a substring from rdopkg output. This patch replaces with ansible
regex_search instead.
This patch also fix the 'host' value extract from zuul vars. The
previous value is invalid as host when cloning code with git module.
This git checkout isn't failing since the previous step, that copies
cloned code from zuul working dir is always executed with success.
[1] https://logserver.rdoproject.org/46/46046/26/check/periodic-tripleo-ci-build-containers-ubi-9-quay-push-wallaby/1cc4376/job-output.txt
Change-Id: I5efc6adf0c8039f6a618b3ae1e1e8184304465d1
This patch will pass only one of the mutually exclusive parameters
'--ntp-heat-env-file' or '--ntp-server' during ceph deploy
Closes-Bug: #2018384
Change-Id: I9c48806f8c0142100aeff79ddb8da0ce3b57cc03
Lately this is making the Wallaby on CS9 line crumble. After some tests,
it seems, at least on CS9, we're able to get rid of this option - and
should, since it's crashing virt-customize.
Change-Id: I4e3cbe4507cbe7d1471f75cb41af99f84725b3ad
Closes-Bug: #2018356
Related-Bug: #1743749
br-tenant is not created as part of standalone
deployments but bridge_mapping was refererring
it. This results into unnecessary Warning[1] in
ovn-controller logs, this patch fixes it.
[1] Bridge 'br-tenant' not found for network 'tenant'
Related-Bug: #1895822
Change-Id: I9b23d6842cd518971b325ffd29b51d171c353b4f
Until now, the regexp was trying to match content from the link text,
not href. This may lead to issue depending on the remote httpd
configuration, since it may truncate the link text. Matching the link
href is therefore safer.
Change-Id: I264bad88b2f7bbc8f0b6cc295ca92498fe6e3d49
This patch looks huge, so let's split things in the following listing:
- rename role to match the naming convention
- role name shouldn't get hyphen, so let's use underscores instead.
Playbooks don't need to follow this rule.
This should make the role more portable in the future.
- modify role
- "disable" the main.yml - we must call specific task file.
- no more condioned deploy based on variables. Makes it easier to
follow and maintain.
- update how the edpm inventory is edited
- use plain YAML
- output plain YAML
- this format is consistent with latest changes in tripleo-ansible[1]
These changes make the overall role easier to understand and maintain,
while ensuring the playbook is also easier to read. It may also make the
deploy a bit faster with the new way to run the various tasks.
- update playbook
- boostrap both nodes in a 2-stages way (check comment as to why).
- deploy compute from the undercloud (mimic what an operator would do)
- validate the compute from the undercloud (no actual change here)
- target specific task_file during the different stages. It's easier
to understand what's going on in there
These changes should allow a better understanding of the overall playbook.
- update edpm deploy script to use an environment file instead of CLI
params
- this makes the script lighter and easier to understand
- adding new parameters in the env file is easier
- reading the env file for debugging is better
[1] https://review.opendev.org/c/openstack/tripleo-ansible/+/869357
Change-Id: I263f424a5a08214b9c46c5ea73b6b428709ad47b
trunk.registry.rdoproject.org going to decomission soon.
This patch will replace trunk.registry to
quay.rdoproject.org.
Signed-off-by: Amol Kahat <amolkahat@gmail.com>
Change-Id: I7eb7b8dac1c17ba0eaba8cac12861a18224fac30
In order to run specific command on the image, we need
to add support for --run-command to the modify image role.
It will be useful to run specific command instead of using
script.
This functionality will be consumed here:
https://review.opendev.org/c/openstack/tripleo-quickstart/+/868864
Related-Bug: #2000226
Signed-off-by: Chandan Kumar <chkumar@redhat.com>
Change-Id: I8575c712467db8fea403459d26731dd7070df131
Image build in component jobs is pulling content from
centos.repo instead of quickstart repos, see [0]
This can cause mismatch of rpm when mirrors.centos.org and
local mirrors are not in sync.
The base image which we use to build overcloud images already have
centos.repo and when proxy mirros are not updated this can cause
an issue.
This is a workaround patch to pass centos.repo(which are disabled
on host) so that same repos in the image will be overridden.
One change in behavior is at the end of image build, dib will delete
the centos.repos in the overcloud image as dib cleans up what it adds
ignoring what was already present.
oooci-build-images already have same fix.
[0] https://logserver.rdoproject.org/openstack-component-baremetal/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-9-ovb-3ctlr_1comp-featureset001-baremetal-wallaby/1cd7430/logs/undercloud/home/zuul/build.log.txt.gz
~~~
2022-12-27 00:54:07.471 | kernel x86_64 5.14.0-214.el9 baseos 2.8 M
~~~
[1] b2ce4b4c10
Related-Bug: #2000226
Change-Id: Iecf36eff8ef27fd2734f77b09180d0b4d8654c52
validate_edpm.sh.j2 adds the script to spin an instance
on external compute in order to validate the external
compute deployment.
Depends-On: https://review.opendev.org/c/openstack/tripleo-ansible/+/863935
Signed-off-by: Chandan Kumar (raukadah) <chkumar@redhat.com>
Change-Id: I89a7897e6a4cbf83149302a0ea856b151623b9ba
Since Ansible was bumped to 2.14, we've observed the container image
build process gets stuck in the middle of ansible tasks to generate
Docker/Buildah files, because of a bug[1] with ansible-runner.
This removes --debug option from the build command to avoid -vvv option
in the ansible command, to workaround the above bug.
[1] https://github.com/ansible/ansible-runner/issues/1164
Related-Bug: #1996612
Change-Id: I53c688077c65da03d8c3cf104862e02cefc2c615
... because the option is deprecated in master[1] and has no effect.
Also, deployed-server-environment.yaml is not required when
the --deployed-server option is used since stable/wallaby[2].
[1] c6cd190a9f63738deafa4c616e0be8e06afedc96
[2] d6fba3f09f8de569b7fcf2bd9519c3112b1cc056
Change-Id: I4411a06026fc5bdf69061a643bc731e91c27640c
Remove auto-reverse from IPA server install as
this option is causing errors in environments
where the zone being created already exists and
is owned by some other dns server.
See related change in:
https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/737058/.
This patch also stops the unbound service if it
is running on the undercloud.
Change-Id: Ie46809379e5c143910560b4d3f7cb0eee6f80ea1
It adds validate_edpm.yml playbook to list
external compute on the controller node.
Signed-off-by: Chandan Kumar (raukadah) <chkumar@redhat.com>
Change-Id: I5e0b16bf34075d663a52756c4996512610927e47
It contains following improvements to EDPM role.
- use tripleo_podman role to configure insecure registry to pull
containers from content provider.
- Added timestamp in deploy_compute logs
- Use VIP as a controle_plane ip so that
compute can communicate with controller.
Signed-off-by: Chandan Kumar (raukadah) <chkumar@redhat.com>
Change-Id: I59cf9170c4f19ecb58b4b378d2b0accb699381fc
This change removes logic for releases older than train, because these
releases are already EOLed. This covers only standalone roles and
playbooks.
Change-Id: I7b374d5bdb8411a2d773eb1c7ba0e4ded8f5cfbc
This change removes logic for releases older than train, because these
releases are already EOLed. This covers only tempest roles and
playbooks.
Change-Id: I0b50cad30107a83b562b8c22b2afd7af7636c615
This change removes logic for releases older than train, because these
releases are already EOLed. This covers only undercloud-* roles and
playbooks.
Note that this change also removes the containerized_* variables
because current active releases(>=Train) no longer supports
non-containerized deployment. The containerized_undercloud_upgrade
variable is still left because the variable is still required by
the tripleo-ci-base-multinode-standard job template.
In addition, TripleO UI support is removed because the component was
retired before Train.
Change-Id: I5132e5bac4ddaa6f76564cab67e486cb5a4e2bbb
This change removes logic for releases older than train, because these
releases are already EOLed. This covers only overcloud-* roles and
playbooks and the other items will be covered by separate changes.
Note that this change also removes the containerized_* variables
because current active releases(>=Train) no longer supports
non-containerized deployment. The containerized_overcloud_upgrade
variable is still left because the variable is still required by
the tripleo-ci-base-multinode-standard job template.
Change-Id: If29ec2c2219a28a1f79db0e552e2c622c0a7bda6
external-data-plane-management role is a new role to deploy
standalone compute node by using tripleo-ansible roles.
This role includes will run on two nodes and does following
things:
- Bootstrap first node and deploy standalone
- Generate 99 standalone vars and copy it to second node
- Bootstrap second node and install packages
- Copy 99 standalone vars to tripleo-inventory
- Deploy compute serices.
It will be used in tripleo-external-compute-deployment-pipeline
job template[1]
[1]. https://review.opendev.org/c/openstack/tripleo-ci/+/843836
Signed-off-by: Chandan Kumar (raukadah) <chkumar@redhat.com>
Change-Id: I476c7299dec209c89db87e2182a35ec6a4924f92
This reverts commit 8226fa9a25.
Reason for revert: rdopkg-1.5.1 has been pushed to pypi where rpmlint is not longer a install mandatory requirement but only optional for rdopkg[lint].
Change-Id: I3f4a7f90295e30bf32fd8bd6196d5135e44c34c1
Latest release of rdopkg (1.5.0) with
error: metadata-generation-failed and is blocking the whole
CI.
We are currently pinning rdopkg to older version till we fix
the rdopkg or install rdopkg from RPM.
Related-Bug: #1993343
Signed-off-by: Chandan Kumar (raukadah) <chkumar@redhat.com>
Change-Id: I3382ead6eafe8a77b9b874ca7a22d4266851e3ac
This redirect does not work or won't work in the future.
Note: This adds ansible-core to test-requirements.txt,
we may need to add requirements.txt and ansible collections
in ansible-collections-requirements.yml in the future.
Change-Id: Iffb41b68283f77050d8073364156cc0aca85559e
Quotation marks should be removed in set statements from
standalone_config jinja template to make the template compatible with
all ansible versions. When quotation marks are used, variable does not
return its value, but is passed as regular string value.
Change-Id: Ia2d664c89a00f139454a4e7b6322eb3fd383e473
Ghanshyam Mann
yatinkarel