Conversation
double the timeout from 5 to 10 seconds. this should prevent errors like these: https://zuul.opendev.org/t/openstack/build/547464fe844e4e8ba34faac33f615c92/log/primary/logs/ansible/test-core-openstack#1210 Change-Id: I47157973a8152154a264cc1914b6d2b55bc08cd1 Signed-off-by: Sven Kieske <kieske@osism.tech> (cherry picked from commit 221a9b0)
If the container image used by Mariabackup is different than the one used by MariaDB server, it's possible that mariabackup and mariadb are incompatible. This may cause backup operations to fail. This change queries the running MariaDB server container's image and uses it when taking a backup. If MariaDB server isn't running on the host it falls back to the image defined in configuration. The separate mariabackup_image, mariabackup_tag and mariabackup_image_full variables are no longer required and have been removed. Closes-Bug: #2058644 Change-Id: I45f3f90ec1973dae92131ea16a7b248ab7a8ae69 (cherry picked from commit 667d153)
Change-Id: Id25b4407a8170f69e4cd7278e0aff64c609ace7d (cherry picked from commit 71d1a643bbe419c9e413ee799d30720a0985ebd7)
Ansible passes port as a string - therefore matching does not work and we get https://nova_url:443/v2.1 Closes-Bug: #2063434 Change-Id: I76cce7f491c77b6b788d29c8644e87055f5cfff0 (cherry picked from commit 2c0f94a)
Change the skyline nginx config to point to the internal port. Closes-Bug: #2069855 Change-Id: Ia29d89b2594a604c687469850a67f7fe29d0eb5d (cherry picked from commit cb90041)
Also enable these after an upgrade. Partial-Bug: #2058512 Change-Id: Ib9bdae2e25c2b6cce30e4c8024015ab5875bc1ff (cherry picked from commit 223556e)
When using short notations like `1g` or `512m` to define the container dimensions, we are always getting the container to being restarted in each kolla-ansible run, even with no real changes in the container configs. Change-Id: Ic8e2dd42b95a8f5c2141a820c55642a3ed7beabd Closes-Bug: #2070494 (cherry picked from commit a086b78)
Volumes with a mode specifier were causing the container to fail to start with the error: "Wrong format of volume". ironic-dnsmasq was one services affected by this issue, but you could also hit this issue when using the volume customisation variables. Closes-Bug: #2054834 Change-Id: I848bd2838a17756f1c71b07befe1e3d966f5b6c2 (cherry picked from commit 9b3428c)
Fix kolla systemd unit template to prevent restart all kolla services with docker.service restart Change-Id: I70dd1751dea6bfc9bb265aeda04b3392e135324c Closes-Bug: 2065168 (cherry picked from commit c0b0ddd)
…ion" into stable/2023.2
Closes-Bug: #2070051 Depends-On: https://review.opendev.org/c/openstack/ansible-collection-kolla/+/923544 Change-Id: I221eb136e77c61aef39e8646b48b927352d1419d (cherry picked from commit a535e65)
Fixes an deploy opensearch whith enable TLS on the internal VIP Closes-Bug: #2073224 Change-Id: I50ce48c4e3c645e2f3aeee4913a9bc9ee506040a (cherry picked from commit aed500c)
This patch modifies tasks that are delegated to localhost to use local connection. Firstly, this is correct since SSH connection is not used, and secondly, it fixes the issue when kolla-ansible is packaged in a docker container. If the local connection is not used, the tasks will fail because temporary data are stored outside the container, whereas we need it to be stored inside the container so we can read them and set_facts. Closes-Bug: #2073370 Change-Id: I9547d5da78da30bfeea8e97056cfa9308c977098 (cherry picked from commit 84e95d2)
Configure cloudkitty_influxdb_use_ssl automatically based on the value of kolla_enable_tls_internal. Set cloudkitty_elasticsearch_cafile, cloudkitty_influxdb_cafile and cloudkitty_prometheus_cafile to openstack_cacert. Disable certificate validation when bootstrapping the InfluxDB database: the influxdb_database module and the InfluxDB 1.x Python client don't support specifying a CA certificate file. This fixes bootstrap and execution of CloudKitty with internal TLS. Closes-Bug: #1998831 Change-Id: I5524169b9567819d379726099bf70c692c85acc1 (cherry picked from commit 78e7f13)
Change-Id: Ia7bd309479f098bc0b65ecf76dc294d893dadbc4 (cherry picked from commit 5935918)
* Use proper balancing mode when federation is enabled. Closes-Bug: #2058656 Change-Id: Ia81a6efc38ec2bdc1355d058c03568cf740fdac5 (cherry picked from commit 33d03a4)
This fixes an issue where it is not possible to customise the `host`
config option in the Nova Compute Ironic config file without breaking
detection of the service.
This is a backwards compatible fix, which allows a user to set the
`host` config option using Ansible host or group vars.
Other reasons for not using the default host setting of
`{{ ansible_hostname }}-ironic` are covered in [1].
[1] https://specs.openstack.org/openstack/nova-specs/specs/2024.1/approved/ironic-shards.html#migrate-from-peer-list-to-shard-key.
Closes-Bug: #2056571
Change-Id: I9b562f6a5722f21b7dbec2a4d53a46a57c829155
(cherry picked from commit ac7c1e9)
wrong use of a f-string when no variable is templated Signed-off-by: Sven Kieske <kieske@osism.tech> Change-Id: I4ef5147eacef32ed93c21d44bf23b664adf1eb91 (cherry picked from commit 70e873c)
add gpfs
add file_system_name
…e for monitoring metric databases.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…faults Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add dingo_bear_database_shard_root_user, shard_id and shard config for ProxySQL routing support - Fix loadbalancer.yml project_services from string to dict variable - Fix dingo-bear.conf.j2 database name from dingoops to dingo-bear Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…o-bear Use a dedicated subdirectory so dingo-bear and dingo-command containers no longer share the same work_dir root, while still living under the same host mount point. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
20 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.