Commit Graph

6123 Commits (mergify/bp/stable-8.0/pr-7620)
 

Author SHA1 Message Date
Dmitriy Rabotyagov e751b2960a Don't try to set devices fact when osd_auto_discovery was skipped
Right now, under certain OS and Ansible versions, ie Rocky Linux and
ansible-core 2.17, `devices_check` variable is getting defined even if
task was skipped.

That results in set_fact to fail, as resulting variable has no `results`
key in it.

Structure of such variable looks like that:
```
"devices_check": {
    "changed": false,
    "false_condition": "osd_auto_discovery | default(False) | bool",
    "skip_reason": "Conditional result was False",
    "skipped": true
}
```

Checking for task not being skipped solves such issues.

Signed-off-by: Dmitriy Rabotyagov <noonedeadpunk@gmail.com>
(cherry picked from commit 9405558d03)
2024-10-28 22:22:28 +00:00
Teoman ONAY de7300e74b pytest: python-version typo
Fixes python-version

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit c9ffb2e70c)
2024-10-17 09:26:21 +02:00
Teoman ONAY 3500ea5044 cephadm-adopt: Alertmanager placement count missing
Regression from #7576. Alertmanager placement count was missing
after migration to ceph_orch_apply module

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit 31be495061)
2024-10-14 15:04:51 +02:00
Teoman ONAY ff3a41330c cephadm-adopt: custom prometheus port lost after adoption
If a custom Prometheus port was used before adoption, it was not
taken into account and default 9095 was set instead. Now custom
port is re-applied.

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2242346

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit b41b7bf869)
2024-10-14 15:04:51 +02:00
Amir Nikpour 3d0684781e cephadm-adopt: add cephadm ssh config option
Add task to set customized ssh config for cephadm

Signed-off-by: Amir Nikpour <amnik.free@gmail.com>
(cherry picked from commit e4ab45d756)
2024-08-25 15:22:12 +02:00
Teoman ONAY d0c8bfe66e ceph_orch_apply: fix yaml error when multiple rgw deployed
ceph orch ls rgw --format=yaml returns multiple documents
when multiple rgw are installed, this was not handled
correctly.

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit 823700bc1b)
2024-08-23 23:15:52 +02:00
Seena Fallah 3b5e0b2bbc radosgw_user: support caps on user
Supporting adding/removing caps on the user.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit c1f1c1e900)
2024-08-20 10:47:47 +02:00
Seena Fallah acf444f78f ceph-pool: add support updating crush rule of the pool
Support updating crush rule of the pool via crush rule name.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit d6ac0cc120)
2024-08-13 18:11:35 +02:00
Seena Fallah 5f85f320c9 devices: test devices before collecting on auto discovery
In some scenarios with NVMe, a device might be identified by
Ansible but could actually be a multipath device rather than an
actual device. We need to exclude these as Ceph cannot create
OSDs on them.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit be9b458524)
2024-08-13 18:10:44 +02:00
Teoman ONAY f0bcf7f675 cephadm-adopt: fix "Update the placement of radosgw hosts" task
networks was at the wrong level in the spec file. Failed with
"got an unexpected keyword argument 'networks'"

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit 2c88ecc784)
2024-08-13 18:10:14 +02:00
Teoman ONAY 6d3558fdc0 cephadm-adopt: Fixes binding network for alertmanager
Alertmanager was bind to default * network instead of grafana_server_addr
as it was before. Now on if grafana_server_addr is defined, it will be
bind to that network.

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit 0bf3398774)
2024-08-07 13:33:17 +02:00
Teoman ONAY f1b67bd4d9 vagrant-box atomic-host -> stream9
Signed-off-by: Teoman ONAY <tonay@ibm.com>
2024-07-16 16:58:12 +02:00
Teoman ONAY c2407f4c52 cephadm-adopt: fix "Update the placement of radosgw hosts" task
spec file template conditions were incorrect

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit e85060cb67)
2024-07-16 14:16:57 +02:00
Teoman ONAY 4a981a2019 Replace Centos stream 8 by stream 9
In test scenarios

Signed-off-by: Teoman ONAY <tonay@ibm.com>
2024-07-16 11:42:32 +02:00
Seena Fallah 9737947dde ceph-handler: use haproxy maintenance for rgw restarts
RGW currently restarts without waiting for existing connections to
close. By adjusting the HAProxy weight before the restart, we can
ensure that no active connections are disrupted during the restart
process.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit 4fa9057a3e)
2024-06-27 14:41:39 +02:00
Guillaume Abrioux 438da91b32 Revert "nfs-ganesha support removal"
This reverts commit 675667e1d6.

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit 59198f5bcd)
2024-06-20 14:22:40 +02:00
Teoman ONAY 16f32e981d ceph_orch_apply: fix idempotency
As is idempotency does not work as the ceph orch
output does contain more attributes than the expected
spec.

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit 5a44fbeaa6)
2024-05-29 23:25:38 +02:00
Teoman ONAY d42bb71095 ceph_orch_spec: Add ceph orch apply spec feature
Add new module ceph_orch_spec which applies ceph spec files.
This feature was needed to bind extra mount points to the RGW
container (/etc/pki/ca-trust/).

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2262133

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit f6fd034e7e)
2024-05-22 13:42:40 +02:00
Seena Fallah 189ddfd92a ceph-key: check for key existance on absent state
check if the key is exists before removal.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit 52365166a4)
2024-05-21 15:06:43 +02:00
Seena Fallah 2485f9f7d5 ceph-container: use ceph user and group
Use --setuser and --setgroup to ceph to run daemons with ceph user.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit 7016c6df3c)
2024-05-17 13:16:50 +02:00
Seena Fallah 65a9e0fa92 tests: collect mons and mgrs logs
Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit 779a13eb31)
2024-05-17 13:16:50 +02:00
Seena Fallah 5c0b69af98 ceph-container: keep run dir permission consistent
Make it the same as "Create ceph initial directories" task
and make owner and group 167 for containers so they can write
with ceph user.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit 65c84a3583)
2024-05-17 13:16:50 +02:00
Seena Fallah faae48d75b ceph-rgw: introduce rgw zone to the name schema
This is needed by ceph-exporter as it is parsing the socket by the number of dots.
Although the rgw_zone variable is only using for constructing the client name
and has nothing to do with multisiting.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit 1121e6d98a)
2024-05-17 00:42:49 +02:00
Teoman ONAY 89097bec22 Fix cephadm-adopt test scenario
Fixes cephadm-adopt test scenario by configuring rbd application
on test and test2 pools. Otherwise cephadm-adopt failed at task
"Check pools have an application enabled"

Signed-off-by: Teoman ONAY <tonay@ibm.com>
(cherry picked from commit ca191fad2a)
2024-05-16 11:32:37 +02:00
Seena Fallah e6f7ef7f56 radosgw_user: handle non-existence user on check mode
Set return code to 0 on check mode when the user doesn't exists and it will fail the module.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit 878cce5b48)
2024-05-06 12:11:32 +02:00
Guillaume Abrioux bbf055d015 ceph-mon: move the ceph_config call to ceph-mon role
ba7eb62a1b broke deployments where
mgrs are not collocated with mons.

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit 4f6641fe5c)
2024-04-11 19:03:09 +02:00
Guillaume Abrioux d1ad661102 ceph-config: move `ceph_config` module call to ceph-config role
so you don't have to implement a similar play on your own
if not using main (sample) playbooks.

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit ba7eb62a1b)
2024-04-11 16:40:45 +02:00
Seena Fallah 70836dde00 ceph_volume: set changed to false to readonly actions
Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit fb13ee35bf)
2024-04-08 21:27:20 +02:00
Seena Fallah 3ee8744b95 ceph_crush: add support check mode
Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit 326e2e4d6b)
2024-04-08 21:27:20 +02:00
Guillaume Abrioux 6e877afec6 ceph-mon: fix admin keyring creation task
Although custom cluster name support was dropped, it breaks ceph-volume
functional testing as it uses "test" as cluster name.
The plan is to make ceph-volume use "ceph" but for now it's easier to
address the issue in this task.

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit c4b3520730)
2024-04-04 14:23:06 +02:00
Guillaume Abrioux ae24dbc863 common: install python3-packaging on centos el8
ceph-volume has a dependency on `python3-packaging` which is available
in PowerTools repo.

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit 8ebff2124d)
2024-04-04 14:23:06 +02:00
Seena Fallah 312e824f63 ceph-handler: allow 405 as a success restart for rgw
If rgw is configured to only operate admin api - this is the status code it will return for GET on no path.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit 9f9b32e9e6)
2024-04-04 12:21:35 +02:00
Seena Fallah 27f8dfcb0f ceph_pool: add support check mode
Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit a45defd386)
2024-04-03 17:46:37 +02:00
Guillaume Abrioux fecc615d02 handlers: move tmpdir removal
This moves the tmpdir removal tasks to main playbook.
The handlers in Ansible are too restrictive and doesn't work
quite well for our use case.

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit bab034024c)
2024-03-22 17:32:50 +01:00
Guillaume Abrioux ecea562d1c nfs-ganesha support removal
nfs-ganesha support will be implemented in a separate playbook.

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit 675667e1d6)
2024-03-22 17:32:50 +01:00
Guillaume Abrioux b13c1b4324 vagrantfile: drop grafana group
this has never been used for real, let's drop this

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit 1ebddeccb3)
2024-03-22 17:32:50 +01:00
Guillaume Abrioux d465af8d21 remove legacy task
this was for backward compatibility concerns, it's time to drop this
task.

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit d9e2de1d1d)
2024-03-22 17:32:50 +01:00
Guillaume Abrioux 6d772cb969 common: configure epel repository (rockylinux)
this is needed in order to make some libraries available when
running on RockyLinux

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit 867be7c35c)
2024-03-20 13:42:52 +01:00
Guillaume Abrioux 31bbd5f9b8 tests: various changes
some minor changes

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit c42c0c3558)
2024-03-20 13:42:52 +01:00
Guillaume Abrioux 9ccc4cd7a3 rgw_profiles: update rgw-keystone profiles
See issue #7458

Fixes: #7458

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit 0f341cc68b)
2024-03-20 10:57:51 +01:00
Guillaume Abrioux c1c7c62379 container-engine: enforce docker for Ubuntu os family
This enforces docker.io and docker respectively for
`container_package_name` and `container_service_name` by default
for Ubuntu distribution.

Fixes: #7496

Signed-off-by: Guillaume Abrioux <gabrioux@ibm.com>
(cherry picked from commit ef5a09d0e0)
2024-03-20 10:40:07 +01:00
Seena Fallah 127dab7487 ceph_config: compare current vals in low
Check the current value and asked both in low case as the asked one might be in capital as it should be.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit ed4b6e71b3)
2024-03-19 10:01:07 +01:00
Seena Fallah cdc9e6b341 ceph_config: add support config remove
Add support rm action.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit fa2eb2532e)
2024-03-18 20:08:33 +01:00
Seena Fallah ac3081ecbf radosgw_user: add support check mode
Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit a7e881439e)
2024-03-17 01:20:52 +01:00
Seena Fallah 46fe58ec5e radosgw_user: parse system and admin as boolean
The returned payload from rgw has them as a boolean. By having them as a string it would always report a change and try to modify the user.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit a95726c409)
2024-03-17 01:20:52 +01:00
Seena Fallah ed96fa1465 ceph_ec_profile: make int params str
Ceph return them as string so in comparision it would always fail

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit f0b4cc3f0d)
2024-03-17 00:44:45 +01:00
Seena Fallah 1b58c3ebac ceph-osd: introduce ec profile creation
RGW Pools can now use the existing ec profiles and rules created by ceph-osd role.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit bbc1ba5c05)
2024-03-17 00:44:45 +01:00
Seena Fallah c6ec59752c ceph_ec_profile: add support more plugins
Add support more plugins with their respective args.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit 80666a20da)
2024-03-17 00:44:45 +01:00
Seena Fallah 35233e85f0 radosgw_zonegroup: add support check mode
Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit beb046cba9)
2024-03-16 23:46:18 +01:00
Seena Fallah 0f4dcae54b radosgw_zonegroup: parse master as boolean
The returned payload from rgw has is_master as a boolean. By having master as a string it would always report a change and try to modify the zonegroup.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
(cherry picked from commit e8d34b6ee6)
2024-03-16 23:46:18 +01:00