{"push_id":2792280315,"size":4,"distinct_size":4,"ref":"refs/heads/master","head":"8b5e3cd999ba4fa36c62e0d50a106a19f1f15067","before":"4c9e24a90fca2271978a066e38dfadead88d8167","commits":[{"sha":"4522dbfc74a538bf7949f14f5c90819b78945d76","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"doc: update ansible supported version matrix.\n\nCloses: #2989\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":true,"url":"https://api.github.com/repos/ceph/ceph-ansible/commits/4522dbfc74a538bf7949f14f5c90819b78945d76"},{"sha":"db29b5b84dbc6f4a940f2da6c5f1e68547a91f4a","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"config: clean template, remove useless conditions\n\nthere is no need to have all these conditions.\n\nfor instance, assuming `mds_group_name` is set to 'mdss':\n\n  - `if groups[mds_group_name] is defined` checks if `'mdss'` is present in `{{ groups }}`\n\n  - `if {{ mds_group_name }} in group_names` checks if the current node is part\n  the group `'mdss'`\n\n  - `if inventory_hostname in groups.get(mds_group_name, [])` checks if\n  the current node is part of the group 'mdss'\n\nThe third condition is enough to cover the need of ensuring we are\nrunning on a mds node.\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":true,"url":"https://api.github.com/repos/ceph/ceph-ansible/commits/db29b5b84dbc6f4a940f2da6c5f1e68547a91f4a"},{"sha":"f422efb1d6b56ce56a7d39a21736a471e4ed357c","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"config: ensure rgw section has the correct name\n\nthe ceph.conf.j2 always assumes the hostname used to register the\nradosgw in the servicemap is equivalent to `{{ ansible_hostname }}`\nwhich returns the shortname form.\n\nWe need to detect which form of the hostname was used in case of already\ndeployed cluster and update the ceph.conf accordingly.\n\nCloses: https://bugzilla.redhat.com/show_bug.cgi?id=1580408\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":true,"url":"https://api.github.com/repos/ceph/ceph-ansible/commits/f422efb1d6b56ce56a7d39a21736a471e4ed357c"},{"sha":"8b5e3cd999ba4fa36c62e0d50a106a19f1f15067","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"validate: fail if fqdn deployment attempted\n\nfqdn configuration possibility caused a lot of trouble, it's adding a\nlot of complexity because of multiple cases and the relation between\nceph-ansible and ceph-container. Moreover, there is no benefit for such\na feature.\n\nCloses: https://bugzilla.redhat.com/show_bug.cgi?id=1613155\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":true,"url":"https://api.github.com/repos/ceph/ceph-ansible/commits/8b5e3cd999ba4fa36c62e0d50a106a19f1f15067"}]}
{"actor":{"display_login":"leseb"}}