{"push_id":3092010841,"size":72,"distinct_size":0,"ref":"refs/heads/master","head":"61fb6972eceb3a36d7d44eaf7807a516835f8540","before":"90f222f6a597c916b59bfb9a248236acc0b27853","commits":[{"sha":"2b245b923eec4297e5648023a06931799bf6deb9","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"docker-common: remove duplicate set_fact (monitor_name)\n\nthis fact is already set in ceph-defaults, there is no need to set it\nagain in ceph-docker-common\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/2b245b923eec4297e5648023a06931799bf6deb9"},{"sha":"2bcaf9088b5d19b815dcde30a0e25adabbd37fd1","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"docker-common: remove duplicate running cluster check\n\nthis is already done in ceph-defaults, there is no need to have this\ncheck in ceph-docker-common.\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/2bcaf9088b5d19b815dcde30a0e25adabbd37fd1"},{"sha":"cd0195c5622ecc2d0001eb7c988d7b6a6fac1d5e","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"docker-common: remove legacy tasks for ntp configuration\n\nThose tasks aren't needed in docker-common since the introduction of\n`ceph-infra` role. They are duplicated tasks.\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/cd0195c5622ecc2d0001eb7c988d7b6a6fac1d5e"},{"sha":"6947f9c3ea04ab0582d3900728cc52819f4d6b5b","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"docker-common: remove dirs_permissions.yml\n\nthis is already done in `ceph-config` role.\nLet's remove this duplicated task.\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/6947f9c3ea04ab0582d3900728cc52819f4d6b5b"},{"sha":"1144df2852c1a51ca72b7b83a5beebd9a9ac4348","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"docker-common: remove check_mandatory_vars.yml\n\nthis is part of `ceph-validate` role.\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/1144df2852c1a51ca72b7b83a5beebd9a9ac4348"},{"sha":"8069c25ba5dc096f39eb4697b6d7fe0dfc5575f5","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"docker-common: remove system_checks.yml\n\nThis check is now part of `ceph-validate`.\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/8069c25ba5dc096f39eb4697b6d7fe0dfc5575f5"},{"sha":"c783bc70da26c9d54b1ac0613944eb562851e111","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"docker-common: rename role\n\nrename `ceph-docker-common` role to `ceph-container-common`\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/c783bc70da26c9d54b1ac0613944eb562851e111"},{"sha":"bbade5ee0a8a5c84339e921168e4c578b13d03dc","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"site-docker: rename to 'site-container.yml.sample'\n\nAdd a symlink for backward compatibility\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/bbade5ee0a8a5c84339e921168e4c578b13d03dc"},{"sha":"3ac6619fb9aa0ea29041ce122b4dfac9a51fc235","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"tests: set pool size to 1 in ceph-override.json\n\nsetting this setting to 1 makes the CI covering the related code in the\nplaybook without breaking the upgrade scenarios.\n\nThose scenarios were broken because there is a check `TASK [waiting for\nclean pgs...]` in rolling_update.yml, since the pool size for\n`cephfs_metadata` and `cephfs_data` are updated to `2` in\n`ceph-override.json` and there is not enough osd to honor this size,\nsome PGs are degraded and make the mentioned check failing.\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/3ac6619fb9aa0ea29041ce122b4dfac9a51fc235"},{"sha":"f7fcc012e9a5b5d37bcffd39f3062adbc2886006","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"osd: commonize start_osd code\n\nsince `ceph-volume` introduction, there is no need to split those tasks.\n\nLet's refact this part of the code so it's clearer.\n\nBy the way, this was breaking rolling_update.yml when `openstack_config:\ntrue` playbook because nothing ensured OSDs were started in ceph-osd role (In\n`openstack_config.yml` there is a check ensuring all OSD are UP which was\nobviously failing) and resulted with OSDs on the last OSD node not started\nanyway.\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/f7fcc012e9a5b5d37bcffd39f3062adbc2886006"},{"sha":"661213a6a899faae9d4625e9f18f6fa0ab8515e4","author":{"name":"Sébastien Han","email":"62d754cc350e84d3b1c32ae79f976f5348e74a40@redhat.com"},"message":"tox: add lvm setup to shrink mon\n\nFix shrink mon scenario by setting lvm so we can configure ceph-volume\nlvm osds.\n\nSigned-off-by: Sébastien Han <seb@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/661213a6a899faae9d4625e9f18f6fa0ab8515e4"},{"sha":"64dee9be0c8e6ce4cfe01bf7c9956efacc195ba4","author":{"name":"Noah Watkins","email":"61094c6528dcd17030ab039810fc41670575221c@redhat.com"},"message":"Remove outdated documentation\n\nFixes BZ\nhttps://bugzilla.redhat.com/show_bug.cgi?id=1640525\n\nSigned-off-by: Noah Watkins <nwatkins@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/64dee9be0c8e6ce4cfe01bf7c9956efacc195ba4"},{"sha":"63b9835cbb0510415a2d0077697a0107e2d6c4f3","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"infra: don't restart firewalld if unit is masked\n\nif firewalld.service systemd unit is masked, the handler will fail when\ntrying to restart it.\n\nCloses: https://bugzilla.redhat.com/show_bug.cgi?id=1650281\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/63b9835cbb0510415a2d0077697a0107e2d6c4f3"},{"sha":"393ab94728cfff9ab2d4846eb39095becf69ad32","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"client: fix a typo in create_users_keys.yml\n\ncd1e4ee024ef400ded25e8c99948648ead3a0892 introduced a typo.\nThis commit fixes it.\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/393ab94728cfff9ab2d4846eb39095becf69ad32"},{"sha":"976b66842f58df06095769172f631d6bf2c34181","author":{"name":"Sébastien Han","email":"62d754cc350e84d3b1c32ae79f976f5348e74a40@redhat.com"},"message":"ceph.ceph-container-common remove symlink\n\nThis error was introduced in the recent refactor of ceph-docker-common\nin https://github.com/ceph/ceph-ansible/pull/3251. However, the Ansible\ngalaxy linter is not happy about it and fails importing the role.\nRemoving this since it's not used anymore.\n\nSigned-off-by: Sébastien Han <seb@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/976b66842f58df06095769172f631d6bf2c34181"},{"sha":"d20e1960ff2a86d079f33a745b1cf176353cff27","author":{"name":"Guillaume Abrioux","email":"30a9df5a0293a6108675377fa1673f7aa713f915@redhat.com"},"message":"doc: update doc to add stable-3.2 information\n\nSince the branch has been created, we must reflect it in the doc.\n\nSigned-off-by: Guillaume Abrioux <gabrioux@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/d20e1960ff2a86d079f33a745b1cf176353cff27"},{"sha":"10538e9a23c60c4e634226aafe0456680c5ccc6d","author":{"name":"Neha Ojha","email":"f20a5d7c8b1d9ffdf642e49d3f642d59d0fd980a@redhat.com"},"message":"osd_memory_target: standardize unit and fix calculation\n\n* The default value of osd_memory_target used by ceph is 4294967296 bytes,\nso use the same as ceph-ansible default.\n\n* Convert ansible_memtotal_mb to bytes to calculate osd_memory_target\n\nSigned-off-by: Neha Ojha <nojha@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/10538e9a23c60c4e634226aafe0456680c5ccc6d"},{"sha":"a2349f05ac744466318688b79107dafbfe7394df","author":{"name":"Dan Mick","email":"b07550071eaa6a9296289c43bbd6c90559196431@redhat.com"},"message":"validate plugin: handle missing exception fields without traceback\n\n\"missing variable\" errors introduced by PR3058 would attempt to\nbe reported, but since the exception contained no \"path\" definition,\nwould cause a second exception in the Invalid exception handler.\nMake the exception handler verify that any field it tries to use\nexists, clean up its message formatting, and reduce the verbose\nlevel to see the literal error from notario in case more goes\nwrong in future.\n\nSigned-off-by: Dan Mick <dan.mick@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/a2349f05ac744466318688b79107dafbfe7394df"},{"sha":"30ce7e84f4d14657d9447081f7f4c05471f8737c","author":{"name":"Valentin Lorentz","email":"8bc4c83e96e48a666ee5eefbe23dac651dd697e2@progval.net"},"message":"Discover rbd facts.\n\nSigned-off-by: Valentin Lorentz <progval+git@progval.net>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/30ce7e84f4d14657d9447081f7f4c05471f8737c"},{"sha":"c2b0cbd6996a666061a99af1d7cda7d4038b9e9f","author":{"name":"Boris Ranto","email":"e6f2f3d5b819fd5bdcf5d18b7dcb052d2fa0a24f@redhat.com"},"message":"start_osds: Use list instead of keys\n\nIf you use python3 based ansible then keys() returns a dict_keys object,\nnot a list of keys. This breaks the installation on such a system. Using\nthe list filter provides a more robust solution that should work on both\npython2 and python3 based ansible. You can find some more information\nabout the issue, here:\n\nhttps://github.com/ansible/ansible/issues/19514\n\nSigned-off-by: Boris Ranto <branto@redhat.com>","distinct":false,"url":"https://api.github.com/repos/berendt/ceph-ansible/commits/c2b0cbd6996a666061a99af1d7cda7d4038b9e9f"}]}
{"actor":{"display_login":"berendt"}}