- 04 Jun, 2021 1 commit
-
-
Guillaume Abrioux authored
When using grafana behind https `cookie_secure` should be set to `true`. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1966880 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
- 26 May, 2021 1 commit
-
-
Guillaume Abrioux authored
0990ae41 changed the filter in selectattr() from 'match' to 'equalto' but due to an incompatibility with the Jinja2 version for python 2.7 on el7 we must stick to using 'match' filter. Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
- 25 May, 2021 4 commits
-
-
Guillaume Abrioux authored
using 'match' filter in that task will lead to bad behavior if I have the following node names for instance: - node1 - node11 - node111 with `selectattr('name', 'match', inventory_hostname)` it will match 'node1' along with 'node11' and 'node111'. using 'equalto' filter will make sure we only match the target node. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1963066 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
When osd nodes are collocated in the clients group (HCI context for instance), the current logic will exclude osd nodes since they are present in the client group. The best fix would be to exclude clients node only when they are not member of another group but for now, as a workaround, we can enforce the addition of osd nodes to fix this specific case. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1947695 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
This commit rewrites the deprecated syntax used in vagrant_up.sh Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
temporary work around vagrant cloud issue which seems broken at the time of pushing this commit. Let's pull images from cloud.centos.org for now since vagrant cloud hosted images return a 403 error. Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
- 22 May, 2021 1 commit
-
-
Guillaume Abrioux authored
There's no benefit to gather facts again on each play in rolling_update.yml Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
- 07 May, 2021 1 commit
-
-
Guillaume Abrioux authored
Since we need to revert 33bfb10a , this is an alternative to initial approach. We can avoid maintaining this file since it is present in container image. The idea is to simply get it from the image container and write it to the host. Fixes: #6501 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
- 06 May, 2021 1 commit
-
-
Dimitri Savineau authored
The pg_autoscale_mode for rgw pools introduced in 9f03a527 was wrong and was missing a `value` keyword because `rgw_create_pools` is a dict. Fixes: #6516 Signed-off-by:
Dimitri Savineau <dsavinea@redhat.com>
-
- 04 May, 2021 1 commit
-
-
Guillaume Abrioux authored
This is a workaround for an issue in ansible. When trying to stop/mask/disable this service in one task, the stop didn't actually happen, the task doesn't fail but for some reason the container is still present and running. Then the task starting the service in the role ceph-crash fails because it can't start the container since it's already running with the same name. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1955393 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
- 29 Apr, 2021 1 commit
-
-
Guillaume Abrioux authored
We need to filter with the OS architecture in order to fetch the right dev repository in shaman Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
- 28 Apr, 2021 2 commits
-
-
Seena Fallah authored
TCMALLOC_MAX_TOTAL_THREAD_CACHE_BYTES is for both bluestore and filestore Signed-off-by:
Seena Fallah <seenafallah@gmail.com>
-
Guillaume Abrioux authored
ceph-ansible leaves a ceph-crash container in containerized deployment. It means we end up with 2 ceph-crash containers running after the migration playbook is complete. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1954614 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
- 27 Apr, 2021 2 commits
-
-
Guillaume Abrioux authored
Due to a recent breaking change in ceph, this command must be modified to add the <svc_id> parameter. Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
When migrating from a cluster with no MDS nodes deployed, `{{ cephfs_data_pool.name }}` doesn't exist so we need to create a pool for storing nfs export objects. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1950403 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
- 15 Apr, 2021 7 commits
-
-
Dimitri Savineau authored
It looks like the generate_group_vars_sample.sh script wasn't executed during previous PRs that were modifying the default values. Signed-off-by:
Dimitri Savineau <dsavinea@redhat.com>
-
Dimitri Savineau authored
The multisite README file has been merged into a single file. Closes: #6411 Signed-off-by:
Dimitri Savineau <dsavinea@redhat.com>
-
Dimitri Savineau authored
Since 839fac8f we now use ansible 2.10 on the master branch. Signed-off-by:
Dimitri Savineau <dsavinea@redhat.com>
-
Francesco Pantano authored
When dashboard_frontend_vip is provided, all the services should be configured using the related VIP. A new VIP variable is added for both prometheus and alertmanager: we're already able to properly config the grafana vip using dashboard_frontend_vip variable. This change adds the same variable for both prometheus and alertmanager. Signed-off-by:
Francesco Pantano <fpantano@redhat.com>
-
Guillaume Abrioux authored
There's no need to run this playbook on container jobs. Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
We should consider bumping ansible version for future releases, so let's start testing against ansible 2.10 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Benoît Knecht authored
The `set_fact rgw_ports` task was failing due to a templating error, because `hostvars[item].rgw_instances` is a list, but it was treated as if it was a dictionary. Another issue was the fact that the `unique` filter only applied to the list being appended to `rgw_ports` instead of the entire list, which means it was possible to have duplicate items. Lastly, `rgw_ports` would have been a list of integers, but the `seport` module expects a list of strings. This commit fixes all of the issues above, allowing the `ceph-rgw-loadbalancer` role to work on systems with SELinux enabled. Signed-off-by:
Benoît Knecht <bknecht@protonmail.ch>
-
- 14 Apr, 2021 4 commits
-
-
Guillaume Abrioux authored
When collocating daemons, if we chown all files under `/var/lib/ceph` it can cause issues for the collocated daemons that wouldn't have been migrated yet. This commit makes the playbook chown only the files corresponding to the daemon being migrated. Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
This adds a `ExecStartPre=-/usr/bin/mkdir -p /var/log/ceph` in all systemd service templates for all ceph daemon. This is specific to RHCS after a Leapp upgrade is done. Indeed, the `/var/log/ceph` seems to be removed after the upgrade. In order to work around this issue let's ensure the directory is present before trying to start the containers with podman. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1949489 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
This removes the fact `skipped_nodes` which is useless when we run with `--limit` since it gets reset when a new iteration is made. Instead, let's print within a final play which node has been skipped reusing the `skip_this_node` fact. Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
`configure_mirroring.yml` is called right after the daemon is started. Sometimes, it can happen the first task in `configure_mirroring.yml` is run while the daemon isn't yet ready, adding a retries/until on that task should help to avoid causing the playbook to fail. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1944996 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
- 12 Apr, 2021 6 commits
-
-
Guillaume Abrioux authored
This commit adds the nfs-ganesha adoption support in the `cephadm-adopt.yml` playbook. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1944504 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
This fact is never used, let's remove the task. Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
set the name of those tasks accordingly with the fact name being set. Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
the adoption playbook should use `radosgw_num_instances` in order to determine how much rgw instance it should set recreate. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1943170 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
This play doesn't nothing else than stopping/removing rgw daemons. Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
this adds a small documentation in the header of the playbook in order to explain what is the goal of this playbook. Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
- 09 Apr, 2021 2 commits
-
-
Guillaume Abrioux authored
when running docker-to-podman playbook, there's no need to call `ceph-config` and `ceph-rgw` from the role `ceph-handler`. It can even have side effects when coming from a baremetal cluster that was previously migrated using the switch-to-containers playbook. Indeed it might complain about missing .target systemd unit since they are removed during that migration. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1944999 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
This adds the iscsigws migration to containers. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=<bz-number > Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
- 02 Apr, 2021 3 commits
-
-
Guillaume Abrioux authored
This moves some task from the `ceph-nfs` role in `ceph-common` since some of them are needed in `ceph-rgwloadbalancer` role. This avoids duplicated tasks. Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
This adds all rgw ports to the http_port_t selinux type so it allows haproxy to connect to those ports in order to avoid AVC. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1923890 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
kalebskeithley authored
haproxy gets an AVC when configured to connect to port 8081 This commit adds a snippet regarding haproxy in a selinux environment Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1923890 Signed-off-by:
Kaleb S KEITHLEY <kkeithle@redhat.com>
-
- 01 Apr, 2021 1 commit
-
-
Dimitri Savineau authored
Pass the password variable via stdin for the registry login authentication. This allows to remove the no_log statement and see the task output without displaying the password value. Signed-off-by:
Dimitri Savineau <dsavinea@redhat.com>
-
- 31 Mar, 2021 2 commits
-
-
Guillaume Abrioux authored
Currently NFS Ganesha (ceph-nfs) consumes /etc/idmapd.conf, which controls mapping of user/owner identities under NFSv4+. With containerized service deployment, this file is an immutable part of the container image and cannot be modified. Here we provide group variables, and a taskk and templates for the ceph-nfs role, to set the path of the idmap configuration file and to make the most common adjustment to the contents of that file -- namely to set the 'Domain'. We default the path to /etc/ganesha/idmap.conf so that we will not conflict with /etc/idmapd.conf on the controller nodes where ganesha runs. NFSv4 clients, as used for example by the Cinder NFS driver, consume /etc/idmapd.conf and may require different settings than what is wanted for NFS Ganesha. Additionally, because we already bind /etc/ganesha from the host into the ceph-nfs container, the file NFS Ganesha consumes will no longer be an immutable part of the container. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1925646 Signed-off-by: Tom Barron tpb@dyncloud.net Co-Authored-by:
Guillaume Abrioux <gabrioux@redhat.com>
-
Guillaume Abrioux authored
This commit adds the parameter `--storage.tsdb.retention.time` to the prometheus systemd unit template. Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1928000 Signed-off-by:
Guillaume Abrioux <gabrioux@redhat.com>
-