| Age | Commit message (Collapse) | Author | Files | Lines |
|
Change-Id: I5973943fe83bbb00cfc8623c0cd4d4b98b369315
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Previously, openvswitch-base would enable the openvswitch repo, and
child containers (e.g. ovn-base, ovn-controller, etc) would be built
using ovn packages from that repo. Recently [1] builds were changed to
clean up repos at the end of the build, so now dependent containers must
re-enable the repo manually.
Also added an openvswitch repo override to tests to avoid regresssion.
[1] Ie5b9970a4575b6258dc238059d3f33ddfe155789
Closes-Bug: #2163314
Change-Id: I4ea1cc812167142743e3ebcbd532281bfe8a9799
Signed-off-by: Alex Welsh <alex@stackhpc.com>
|
|
Repos marked build_only: true are backed up before handle_repos()
overwrites them and restored at the end of each image build, so CI
mirror URLs are not baked into the final image.
The cleanup RUN was injected into every Dockerfile, not just images
that called handle_repos() for those repos. Child images with no
backup fell through to "rm -f", deleting the inherited repo file
(e.g. debian.sources in bifrost-base, breaking cpio install).
Fix by adding a {repo}.enabled sentinel that handle_repos creates
when it processes a build_only repo, making the cleanup conditional
on that sentinel.
Change-Id: Ie5b9970a4575b6258dc238059d3f33ddfe155789
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Leaving empty CI template_overrides file in case
we need to use it in future.
Change-Id: Ic46f1a594bcc330536bc78d93ece10e60622f6d4
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Change-Id: I37bb0b5056b2e055d0d0c9da5d0beba4b9ba9067
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
There's a possibility that Zuul is marking uuid secret as !unsafe,
let's convert it to a string.
Change-Id: I63a1d49f6ed62d31fdd8bedccb719a6c5f7b5bb5
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Change-Id: I2ef1b33b8d4527e0aea4f00dd1ce0770da2c495e
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Change-Id: I21d486f356d814319fcc4039fa13a7dbbacf6a04
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Earlier task with no_log causes the UUID in the URL to be redacted
Change-Id: I98cd55dfce2bd9328a0805e893f80a3d5fbac73a
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
It fails now and no_log doesn't help in debugging why
Change-Id: I93f49e5c9bb12cb7c15b5d4ac4bc61000fcd8400
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
It has incorrect format for a list variable
switch to periodic for now to validate success
Change-Id: Id9696e77bf2dfb07940607947ada05b433353091
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
|
|
We don't use npm anywhere
Change-Id: Ibc1b213d575775f79fd5b7b6b8c6e21685fdb0b0
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
BaseOS and AppStream for CentOS/Rocky
ubuntu, ubuntu-security for Ubuntu
debian, debian-security for Debian
ubuntu-cloud-archive is not really a base repo - so
removing it from ubuntu.sources
Change-Id: Iab98de963f2e8532389b7563bb572d8aa20c7aa6
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
[tool.setuptools.data-files] does not support recursive directory
installation. The old setup.cfg [files] worked because PBR's
FilesConfig.expand_globs() used os.walk() to expand docker/*, but
this hook is not triggered by the pbr.build PEP-517 backend.
Move docker/ into kolla/ and use [tool.setuptools.package-data]
instead. Update kolla_worker._get_images_dir() to check the kolla
package directory first so both editable and regular installs work.
Change-Id: I4fc452c03740ef22ebbb4e8d0d86c4f4783e2d09
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
|
|
Change-Id: I74a6d05007488b69fa040c40d01a07afae369794
Needed-By: https://review.opendev.org/c/openstack/kolla-ansible/+/989850
Signed-off-by: Michal Arbet <michal.arbet@ultimum.io>
|
|
The removed Python octal notation test is already covered by
test_handle_permissions_accepts_valid_permission_notations, which
includes 0o2775.
Change-Id: Ia25d10a6ff3608e2a38d1a88bf229fe3f96d19ee
Signed-off-by: Michal Arbet <michal.arbet@ultimum.io>
|
|
kolla_set_configs currently fails to parse permissions
such as 2775 because only legacy values like 0755 are
converted to Python octal notation.
Accept standard UNIX permission notation for
permissions with special mode bits such as SGID, SUID,
and sticky bit.
Change-Id: I5937a17b0d70f9c1db4b2617854d0180c37fe2c0
Closes-Bug: #2154269
Needed-By: https://review.opendev.org/c/openstack/kolla-ansible/+/989850
Signed-off-by: Michal Arbet <michal.arbet@ultimum.io>
|
|
Following the PTG agreement, this commit extends the container
configuration engine (set_configs.py) to support YAML format while
maintaining backward compatibility with JSON.
- Integrated 'yaml' library as a universal parser for both JSON
and YAML formats (YAML is a superset of JSON)
- Updated 'load_config' to search for 'config.json', 'config.yaml'
- Added 'python3-yaml' (Debian) and 'python3-pyyaml' (RHEL) as
required base packages in Dockerfile.j2
This change improves configuration readability and allows for
easier maintenance while ensuring no breakage for existing
JSON-based deployments.
Change-Id: Icfb52b8f94e1036d5952b160c5921348d1427fda
Signed-off-by: Piotr Milewski <vurmil@gmail.com>
|
|
Change-Id: Id63c035fafaa68a9004c65161c5a3a3ad5544f02
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
It has not been working and leaks secret
Related-Bug: #2147406
Change-Id: If5c1905a096d776811ce932cb040e20f5dfebf82
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Change-Id: Ib7c2d8419932e049db871053c3b4faad791c3566
Signed-off-by: Bartosz Bezak <bartosz@stackhpc.com>
|
|
Change-Id: I8156e2c2a57d820afdee0b76cd5a050ad682fb38
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Change-Id: Iae913ff1b917104fdbb3257c24386f22f6764a03
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Add the ability to configure repositories in the same
way as we do Debuntu ones - via repos.yaml.
Removes .repo files from base - since these are not
needed there anymore.
gpgcheck = 1 is default in docker/base/dnf.conf
Change-Id: I7df7d452c45c57c434d5b4ad15a87963e4d6d0d4
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Depends-On: https://review.opendev.org/c/openstack/ansible-collection-kolla/+/961237
Change-Id: Idd809f5efd2d308c1565d7635e67af0a431d18f7
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Change-Id: I80070b3c3658948c7157564fb8a79e6dd427f1d1
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
It was created with extra newlines.
Change-Id: I9394e5ecf913385870396fa51401748ca0de5e22
Signed-off-by: Bartosz Bezak <bartosz@stackhpc.com>
|
|
Updating Erlang RPMs for aarch64 in Fedora COPR build system [1] [2].
Using COPR webhooks feature [3].
[1] https://copr.fedorainfracloud.org/coprs/g/openstack-kolla/rabbitmq-erlang-26
[2] https://copr.fedorainfracloud.org/coprs/g/openstack-kolla/rabbitmq-erlang-27
[3] https://docs.pagure.org/copr.copr/user_documentation.html#custom-webhook
Change-Id: Ibd4450d14691aca627d99c321b313420ce6dd6f2
Signed-off-by: Bartosz Bezak <bartosz@stackhpc.com>
|
|
|
|
RDO has staffing problems and has decided to only
support SLURP releases - let's drop reliance on that
repository.
Change-Id: I56f86a1c5262e3ee2bcbb52cae53c3b2655ed87f
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
It should not be required anymore, all these information are dumped
by different means (generic zuul playbooks, etc).
Change-Id: I80569b6fe9e4d779a5448015b8ad9caf09e5abf6
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Depends-On: https://review.opendev.org/c/opendev/system-config/+/956760
Change-Id: I136bb702b20e941611dd90830e81c9d358a2e355
Signed-off-by: Bartosz Bezak <bartosz@stackhpc.com>
|
|
|
|
Known missing packages:
- mod_auth_mellon - missing SAML support in Keystone/Horizon - built in Kolla COPR
- glusterfs-fuse in manila-share - built in Kolla COPR
- collectd/telegraf - no opstools repository
- redis - needs a switch to valkey
- python3-ethtool - not required for neutron-mlnx-agent since 6 years [1]
Building packages in Kolla COPR is a temporary solution until these show up
in EPEL10.
[1]: Id2cae3ac9ff049e9fc8225551f99e1e99a87fc65
Depends-On: https://review.opendev.org/c/openstack/kolla-ansible/+/951751
Change-Id: I7074fabcf95184fcfd2561648ae1e05acfb0bc11
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Change-Id: I9d1f24b98f1d7c8f29b0dba45105b0263f961ca5
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
If a configuration file is tracked in the state file
but no longer appears in config.json, it should either be
restored or removed.
This patch introduces a new exception StateMismatch and
updates execute_config_check() to detect such cases. If
any destination path is present in the defaults state but
missing from config.json, we now raise StateMismatch.
A dedicated unit test has been added to verify
this behavior.
Closes-Bug: #2114173
Signed-off-by: Michal Arbet <michal.arbet@ultimum.io>
Change-Id: I6e0b4aaa5722990e3ac647578023f474db3d4381
|
|
Python 3.13 introduces a behavior change in unittest.mock.mock_open(),
which now calls close() when exiting the context manager
as described in bpo-44185 [1]. This causes additional mock calls to appear,
breaking strict mock_calls assertions.
This patch fix tests and add compatibility for Python 3.13.
From mock documentation [2]:
"bpo-44185: unittest.mock.mock_open() will call the close() method
of the file handle mock when it is exiting from the context manager.
Patch by Samet Yaslan."
[1] https://bugs.python.org/issue44185
[2] https://mock.readthedocs.io/en/latest/changelog.html
Change-Id: Ib8e2a8a20a828c4c69deedd307c9b726f51489f8
|
|
Change-Id: Ic75fdaedfa62816fae0cc0b7924425b574feb1ee
|
|
|
|
Change-Id: I57e05a235359547a4477dc869e92047818dc5c15
|
|
Removing dbus-python deps because now a-c-k handles that.
Depends-On: https://review.opendev.org/c/openstack/ansible-collection-kolla/+/942179
Change-Id: I97a97da73a7ccb27789e979656867e4adfe8a633
|
|
|
|
Commit 79a234239516b94b1b79c7ddafcc4fbe4e3ba155 introduced an ability to
build 'in-house'/'not-built-in' projects with Kolla.
This commits extends this feature with an ability to use configure_user
macro for 'not-built-in' users.
The implementation is done the same way as for 'sources', i.e. engine
after registers new section structure as soon as it finds unknown
'<project>-user' section.
In addition, the documentation and CI are extended to cover
``--docker-dir`` option.
Change-Id: I690d9f3829083f2493bf286a1c45764b9699219b
|
|
Change-Id: If5aa7371d14041d6e0efca866341be2894fd6f87
|
|
|
|
(Dalmatian)""
|
|
This reverts commit 428f37124986019bedc897469f9b1f3470b62082.
Reason for revert: Should be fixed now
Change-Id: Ife393e7f1b00fa37921e7822668d6a40dd467d5b
|
|
This reverts commit db0f87d3c8e1e7eccdeb1ea46216d530711680d3.
Reason for revert: Epoxy cycle is on
Change-Id: Ie2a747223b33039684650bede8863af0c8906d2f
|