* Update ansible-role-container-registry from branch 'master'
to d68e447c4b1d769fb1efb332fde4f026df48c06d
- Retire Tripleo: remove repo content
TripleO project is retiring
- https://review.opendev.org/c/openstack/governance/+/905145
this commit remove the content of this project repo
Change-Id: I29fb152050adab2912b2528f304c97ea4534fda4
* Update ansible-role-container-registry from branch 'master'
to a091b9cd216c3ecf48a8e0718e57b0d256e9a725
- Remove python-dev from bindep
It is no longer supported by jammy and lead us to the following errors with the announce-release job.
```
No package matching 'python-dev' is available
```
Change-Id: I262efa933975966d6e078eff4e5778bc00331a92
* Update ansible-role-container-registry from branch 'master'
to 57da8455f1710ecd68214bcc2a86d1b4b9338e41
- Remove CentOS 7 based job
https://review.opendev.org/q/topic:ooo_c7_teardown is dropping
the c7 based jobs.
It also removes job definitions.
The last run https://zuul.openstack.org/builds?job_name=tripleo-ci-centos-7-undercloud-containers
was from 1 year ago.
We are removing these jobs as a part of ooo_c7_teardown[1].
Added py_modules=[] in setup.py to disable auto discovery
and fix linters job.
It also fixes tox molecule issues with no-changed-when
warning errors.
[1]. https://review.opendev.org/q/topic:ooo_c7_teardown
Signed-off-by: Chandan Kumar (raukadah) <chkumar@redhat.com>
Change-Id: I94d290321213ae0e097a9ac7d472878940edbafa
* Update ansible-role-container-registry from branch 'master'
to c17c316f23f45892665cd6cc958e54f5a7ebec39
- Merge "setup.cfg: Replace dashes with underscores"
- setup.cfg: Replace dashes with underscores
Setuptools v54.1.0 introduces a warning that the use of dash-separated
options in 'setup.cfg' will not be supported in a future version [1].
Get ahead of the issue by replacing the dashes with underscores. Without
this, we see 'UserWarning' messages like the following on new enough
versions of setuptools:
UserWarning: Usage of dash-separated 'description-file' will not be
supported in future versions. Please use the underscore name
'description_file' instead
[1] https://github.com/pypa/setuptools/commit/a2e9ae4cb
Change-Id: I1f6e855b0c9cddaf714bf5750f3edd167ede1527
* Update ansible-role-container-registry from branch 'master'
to e745554d51a759fa355efcf76fb97bc2062ce3af
- Use ansible-core 2.11 for tests
Change-Id: Id0f5ed1b2ccd061bedd231c8cb50ecfe7db55d97
* Update ansible-role-container-registry from branch 'master'
to a784f1a1916e848e000adc1d9dd84e1487064603
- Correct the tox option for skipping sdist generation
The tox option to skip source distribution building is skipsdist,
but this seems to be often misspelled skipdist instead, which gets
silently ignored and so does not take effect. Correct it
everywhere, in hopes that new projects will finally stop copying
this mistake around.
See https://tox.readthedocs.io/en/latest/config.html#conf-skipsdist
and https://github.com/tox-dev/tox/issues/1388 for details.
Change-Id: Iac474f1d458c3b803eb28a33bd05002db31c4689
* Update ansible-role-container-registry from branch 'master'
to 91f652a9329544250bd74abed9367b6e1d3dee01
- Fixing ci
* adding task names to satisfy lint
* fixing requirements for test setup
Change-Id: Ic7f019c5f58b456cb36c62283bf72a195d82c299
* Update ansible-role-container-registry from branch 'master'
to 41c93a403dd720e505536836865342131193c6eb
- Use ansible_facts
ansible_facts should be used instead of the injected vars
Change-Id: I9e6e0a257d8abf844063b257b3d9026ec8c9ce6c
* Update ansible-role-container-registry from branch 'master'
to 063f15ec2a848f549f73c84a8dc5814732230559
- Fix CI
Update work with newer ansible-lint and address some linting issues.
Change-Id: I69bd204407bd53210492a67bb8f90504a480db11
* Update ansible-role-container-registry from branch 'master'
to dc3a577652b2ecc39866bfba5f123a0510cc2502
- Stop using handlers
With the switch to tripleo_free in stable/train, we can no longer
reliably use handlers. That being said, handlers don't make sense when
you are forcing them to run with a flush handlers anyway. So this change
takes the code we used in the handler and makes them explicit tasks in
the required locations where we used to do a flush handlers. This
change should also improve the scope of some tasks to only occur when
necessary.
Change-Id: I1e31af24a4c1b9356a5971c6c1cb717d39d1e9b2
Closes-Bug: #1909750
* Update ansible-role-container-registry from branch 'master'
- Replace deprecated UPPER_CONSTRAINTS_FILE variable
UPPER_CONSTRAINTS_FILE is deprecated and TOX_CONSTRAINTS_FILE is
the new environment variable name that replaces it [1].
This allows to use upper-constraints file as more
readable way instead of UPPER_CONSTRAINTS_FILE=<lower-constraints file>.
[1] https://zuul-ci.org/docs/zuul-jobs/python-roles.html#rolevar-tox.tox_constraints_file
[2] https://review.opendev.org/#/c/722814/
Change-Id: I56b20e563262e22969ef86df21f5c62fb1355193
* Update ansible-role-container-registry from branch 'master'
- Fix all the ci
1) Bump pyflakes
We need a python 3.9 compatible version.
Related PyCQA/pyflakes #367
2) Fix missing permissions on files
3) Ignore ansible-lint 106 for role name
4) Fix setup-tools install LP#1902986
5) ensure virtualenv is installed
6) fix missing htpasswd
Change-Id: I8302bc6c9250eba78437ccfec35e4de91348333e
* Update ansible-role-container-registry from branch 'master'
- Don't log login
We only will log when the debug option is used.
Change-Id: I6cda4c2f4866cf38d8b561ae4c9b4638d676dd2e
* Update ansible-role-container-registry from branch 'master'
- Migrated to molecule 3.0
Updated configuration files to use newer version of molecule.
Change-Id: I617330a0b8b64028e43fc5d8aa1b7d97d0375ea5
* Update ansible-role-container-registry from branch 'master'
- Don't stop containers before docker package update
Back in Openstack Pike release we hit issue with docker having
changes applied via puppet causing restart of docker service
without --live-restore option. This approach was buggous from the
begining and it's definetly obsolete here. We do only minor updates of
OS during release so we do only patch level updates of docker.
Change-Id: Iff4823d72c4d50838b6997398b23e76cec50ca56
RefBug: #1747851
Closes-Bug: #1851872
* Update ansible-role-container-registry from branch 'master'
- Fix race on slow environment
Ansible triggers notifies at the same time. It created a race as "wait
for registry" is dependant on "reload docker-distribution". This patch
moves notifies to dependant service to fix race condition.
Change-Id: I550d5538238ba6100714dd1ca8fcda25292a6773
* Update ansible-role-container-registry from branch 'master'
- expand login subtasks so it can be used in all rdo jobs
With the upcoming activation of RHEL8 pipeline, we decided with infra it
was better to switch rdo registry to restricted access.
This means that all the job in rdo must login to registry to pull
containers.
All the Centos7 jobs must now call this role to login to the registry,
so we are expanding the login part to satisfy the workflow
RHEL8 job with podman will use a internal role instead.
Change-Id: I6e55bdcf493d04bfc88ae22154124a7888563147
* Update ansible-role-container-registry from branch 'master'
- Specify a default for container_registry_logins
When we run this, if we try to evaulate this without the variable being
set it errors. We assume empty if the variable is unset.
Change-Id: Ic6eea050cd627d9d09745bf194c2dd36b015e6ff
Closes-Bug: #1835657
* Update ansible-role-container-registry from branch 'master'
- Add molecule testing
This change adds molecule testing using a simple base job and pre|run playbooks.
The test will be executed via a native zuul job and will ensure we're exercising
all of the available code path's as provide by this role.
Two molecule scenarios will be executed whenever any change is made to this role
* default - runs through the typical main code path
* login - tests a secure docker registry ensuring our login capabilities are
never broken.
Documentation in the readme has been added to show how local testing can be run.
A bindep.txt file has been added to ensure zuul knows how to install our
required base packages.
Closes-Bug: #1835657
Related-Bug: #1833584
Change-Id: I48f74b69c5d29dce4a576fa96e79563a4b484469
Signed-off-by: Kevin Carter <kecarter@redhat.com>
* Update ansible-role-container-registry from branch 'master'
- Covert lookup to query
This change updates our loop so that it will expect a list.
More on the query lookup can be seen here[0]
[0] https://docs.ansible.com/ansible/2.6/plugins/lookup.html#invoking-lookup-plugins-with-query
Change-Id: Id8bfea751a7239fd9be6e9dbbb5a0a700e29b64e
Closes-Bug: #1835657
Related-Bug: #1833584
Signed-off-by: Kevin Carter <kecarter@redhat.com>
* Update ansible-role-container-registry from branch 'master'
- Merge "Add docker login support"
- Add docker login support
We need to be able to perform a login prior to pulling containers from
registries that require authentication.
Change-Id: Ic3b720ba35db8e3f3f866cd31d9171e91b04a86c
Related-Bug: #1833584
* Update ansible-role-container-registry from branch 'master'
- Merge "Reduce the number of tripleo jobs"
- Reduce the number of tripleo jobs
We only really need an undercloud job for testing at the moment. The
undercloud is the primary consumer of this role anyway.
Change-Id: If2a4be04dd710ea8174bbd0b77bd57c7baf0a6b9
* Update ansible-role-container-registry from branch 'master'
- Fix docker network option condition
The option is being ignored. This is useful to adjust settings
like in bug https://bugs.launchpad.net/kolla-ansible/+bug/1654395.
Change-Id: I9c8f267a92198a06acb3611754bd16278a699df1
Related-Bug: #1654395
* Update ansible-role-container-registry from branch 'master'
- Update to opendev
Change-Id: Iad852417fd90aa04f9c675659c5cec7668d14da8
- OpenDev Migration Patch
This commit was bulk generated and pushed by the OpenDev sysadmins
as a part of the Git hosting and code review systems migration
detailed in these mailing list posts:
http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.htmlhttp://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html
Attempts have been made to correct repository namespaces and
hostnames based on simple pattern matching, but it's possible some
were updated incorrectly or missed entirely. Please reach out to us
via the contact information listed at https://opendev.org/ with any
questions you may have.
* Update ansible-role-container-registry from branch 'master'
- Merge "Improve condition checking about docker registry config"
- Improve condition checking about docker registry config
Avoids use of ignore_errors on a task that is only reading the status.
We only use the result code (rc) of the check so the task iself
should not fail regardless if the registry was managed or not.
Improves output of ansible console and confusing false errors.
Change-Id: I0be5f333a7ac6705279820a1a34667b4ec32f760
* Update ansible-role-container-registry from branch 'master'
- Merge "Update min tox version to 2.0"
- Update min tox version to 2.0
The commands used by constraints need at least tox 2.0. Update to
reflect reality, which should help with local running of constraints
targets.
Change-Id: I98aa14c59474f019bbbbc45957617e7fc6f31ad1
* Update ansible-role-container-registry from branch 'master'
- Merge "Fix become for tasks and handlers"
- Fix become for tasks and handlers
Some of our tasks and handlers need to be run as root but were
previously missing the become: true option. This change addresses this
to ensure the appropriate tasks and handlers are run as an admin user.
Change-Id: I9cf00809e8725d9220bc993c8fd6602e19ed7aa3
* Update ansible-role-container-registry from branch 'master'
- docker: add reset_connection
reset ssh connection to pick up docker group after the task that add the
deployment user to docker group.
Change-Id: Ifb9a9ad6c467b8f244665696ed02cdbdc7f9a9de
* Update ansible-role-container-registry from branch 'master'
- Fix linting issues
Newer versions of ansible lint are stricter. This change fixes up the
issues and ignores rule 405 which requires retries on package items.
We're skipping the retries because yum does that natively.
Change-Id: I9280a736cb1bdeeb1bdcea11ec4ca8f5af9d1e44
Ghanshyam Mann