)]}'
{"playbooks/setup-infrastructure.yml":[{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"7b43e245987f402f7d3933ec2db0b72b13a0e858","unresolved":true,"context_lines":[{"line_number":22,"context_line":"- import_playbook: rabbitmq-install.yml"},{"line_number":23,"context_line":"- import_playbook: utility-install.yml"},{"line_number":24,"context_line":"- import_playbook: etcd-install.yml"},{"line_number":25,"context_line":"- import_playbook: ceph-install.yml"},{"line_number":26,"context_line":"- import_playbook: ceph-nfs-install.yml"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"# The journald-remote should be disabled by default until"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"81988408_84215040","side":"PARENT","line":25,"updated":"2022-10-26 08:54:22.000000000","message":"didn\u0027t we agree yesterday that it\u0027s not necessary?\n\nour reason behind that was to avoid unexpected ceph upgrade but as it turns out it won\u0027t happen if upgrade_ceph_packages\u003dFalse(default value).\n\nIn that case, I\u0027ll personally prefer to leave it as it is.\nEspecially when we explicitly say in our docs that integrated ceph is not recommended scenario, we shouldn\u0027t care much about these things in my opinion.","commit_id":"bc5428b21dda44ec9cb27528eeebaa4cd9cf895b"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"f7fefed0a31eed960b422f3347540df0c1a914df","unresolved":true,"context_lines":[{"line_number":22,"context_line":"- import_playbook: rabbitmq-install.yml"},{"line_number":23,"context_line":"- import_playbook: utility-install.yml"},{"line_number":24,"context_line":"- import_playbook: etcd-install.yml"},{"line_number":25,"context_line":"- import_playbook: ceph-install.yml"},{"line_number":26,"context_line":"- import_playbook: ceph-nfs-install.yml"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"# The journald-remote should be disabled by default until"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"b1afc45d_6a1e08cd","side":"PARENT","line":25,"in_reply_to":"622da3ec_8a05be56","updated":"2022-10-26 12:21:08.000000000","message":"Ceph is a storage layer. So it can be compared with networking setup but not with galera/rabbitmq/memcached. And we do not provide any default way for networking configuration. So Ceph is not infrastructure from my understanding and has nothing to do with it.","commit_id":"bc5428b21dda44ec9cb27528eeebaa4cd9cf895b"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"52cd4f69e52c264715333bd23b83ff80b158987d","unresolved":true,"context_lines":[{"line_number":22,"context_line":"- import_playbook: rabbitmq-install.yml"},{"line_number":23,"context_line":"- import_playbook: utility-install.yml"},{"line_number":24,"context_line":"- import_playbook: etcd-install.yml"},{"line_number":25,"context_line":"- import_playbook: ceph-install.yml"},{"line_number":26,"context_line":"- import_playbook: ceph-nfs-install.yml"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"# The journald-remote should be disabled by default until"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"b8dd6a4e_a9cc3ed8","side":"PARENT","line":25,"in_reply_to":"81988408_84215040","updated":"2022-10-26 09:10:06.000000000","message":"For me ceph still has nothing to do with infrastructure - there\u0027s no logic in that. It\u0027s quite independent thing, but fully eligible for setup-everything.\n\nAlso for usecase that was mentioned yesterday - folk is running setup-infrastructure periodically to ensure config state. While it make sense to run same for ceph, but I\u0027m not sure about including ceph into same periodic run (as example).\n\nBut ok, I will propose alternative patch with documentation changes only so we can select.","commit_id":"bc5428b21dda44ec9cb27528eeebaa4cd9cf895b"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"54a0e89f67924aa60a5fa0fa9bb4897739beda38","unresolved":true,"context_lines":[{"line_number":22,"context_line":"- import_playbook: rabbitmq-install.yml"},{"line_number":23,"context_line":"- import_playbook: utility-install.yml"},{"line_number":24,"context_line":"- import_playbook: etcd-install.yml"},{"line_number":25,"context_line":"- import_playbook: ceph-install.yml"},{"line_number":26,"context_line":"- import_playbook: ceph-nfs-install.yml"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"# The journald-remote should be disabled by default until"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"da863b93_dc7ae722","side":"PARENT","line":25,"in_reply_to":"b8dd6a4e_a9cc3ed8","updated":"2022-10-26 09:42:21.000000000","message":"thanks Dmitriy\n\none small note according to: \"For me ceph still has nothing to do with infrastructure\".\nI think it depends on a perspective.\nI have ceph integrated with OSA, so for me ceph is definitely a part of my infrastructure, the same as rabbitmq, memcached etc.","commit_id":"bc5428b21dda44ec9cb27528eeebaa4cd9cf895b"},{"author":{"_account_id":34653,"name":"Danila Balagansky","email":"dbalagansky@me.com","username":"dbalagansky"},"change_message_id":"d762af33a07ef83534ab39203fba8cec827c133d","unresolved":true,"context_lines":[{"line_number":22,"context_line":"- import_playbook: rabbitmq-install.yml"},{"line_number":23,"context_line":"- import_playbook: utility-install.yml"},{"line_number":24,"context_line":"- import_playbook: etcd-install.yml"},{"line_number":25,"context_line":"- import_playbook: ceph-install.yml"},{"line_number":26,"context_line":"- import_playbook: ceph-nfs-install.yml"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"# The journald-remote should be disabled by default until"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"622da3ec_8a05be56","side":"PARENT","line":25,"in_reply_to":"da863b93_dc7ae722","updated":"2022-10-26 11:46:08.000000000","message":"I do agree with Damian here: Ceph cluster\u0027s control plane is part of the infrastructure _for_ OpenStack and it should be deployed prior to Glance/Cinder/\u003cother _OpenStack_ services\u003e.","commit_id":"bc5428b21dda44ec9cb27528eeebaa4cd9cf895b"}],"playbooks/setup-openstack-qa.yml":[{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"7b43e245987f402f7d3933ec2db0b72b13a0e858","unresolved":true,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- import_playbook: os-tempest-install.yml"},{"line_number":17,"context_line":"- import_playbook: os-rally-install.yml"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7e4723ef_af2125c6","line":16,"updated":"2022-10-26 08:54:22.000000000","message":"why are we moving it outside setup-openstack.yml?\n\nif someone doesn\u0027t want to run these playbooks during setup-openstack.yml, it can be controlled with ``tempest_install`` and ``rally_install`` variables.\n\nEven if we want to move it anyway, it probably should be a separate change.","commit_id":"43a356442dd8b960b33d948a11dacafa02a6fa0f"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"52cd4f69e52c264715333bd23b83ff80b158987d","unresolved":true,"context_lines":[{"line_number":13,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":14,"context_line":"# limitations under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- import_playbook: os-tempest-install.yml"},{"line_number":17,"context_line":"- import_playbook: os-rally-install.yml"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"ed77a1b0_7db7b398","line":16,"in_reply_to":"7e4723ef_af2125c6","updated":"2022-10-26 09:10:06.000000000","message":"well, it\u0027s more a consequence of moving ceph... Also we can save some time for upgrade jobs this way and fully avoid running these playbooks on N-1 and run just on N.","commit_id":"43a356442dd8b960b33d948a11dacafa02a6fa0f"}],"playbooks/setup-openstack.yml":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"b9778936ecb309cb5a456693debca84de09d9cfe","unresolved":true,"context_lines":[{"line_number":42,"context_line":"- import_playbook: os-mistral-install.yml"},{"line_number":43,"context_line":"- import_playbook: os-murano-install.yml"},{"line_number":44,"context_line":"- import_playbook: os-zun-install.yml"},{"line_number":45,"context_line":"- import_playbook: os-tempest-install.yml"},{"line_number":46,"context_line":"- import_playbook: os-rally-install.yml"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"d4dfeb0d_618f5d57","line":45,"range":{"start_line":45,"start_character":0,"end_line":45,"end_character":41},"updated":"2022-10-24 20:33:58.000000000","message":"won\u0027t tempest now fail for anything to do with object storage as we now run the tempest tests before deploying RGW? Not sure why this is passing for focal/ceph right now, unless we are missing teet coverage.","commit_id":"79fd6b14e06cdbcf2102c07dce780cdc4e56e708"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"321157a3d7742b591de8327591a2740e7d06c8a4","unresolved":true,"context_lines":[{"line_number":42,"context_line":"- import_playbook: os-mistral-install.yml"},{"line_number":43,"context_line":"- import_playbook: os-murano-install.yml"},{"line_number":44,"context_line":"- import_playbook: os-zun-install.yml"},{"line_number":45,"context_line":"- import_playbook: os-tempest-install.yml"},{"line_number":46,"context_line":"- import_playbook: os-rally-install.yml"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"0c0c1060_32cdfb52","line":45,"range":{"start_line":45,"start_character":0,"end_line":45,"end_character":41},"in_reply_to":"2ee42c83_3f982907","updated":"2022-10-26 09:19:11.000000000","message":"Also I\u0027m not sure if newer ceph-ansible always compatible with older ceph versions? As how it behaves when some breaking changes get implemented, like ceph-disk vs ceph-volume or smth like that? Can\u0027t it cause setup-infrastracture failure?","commit_id":"79fd6b14e06cdbcf2102c07dce780cdc4e56e708"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"8f28b5e33201ea39500604bff492aab9da87c40d","unresolved":true,"context_lines":[{"line_number":42,"context_line":"- import_playbook: os-mistral-install.yml"},{"line_number":43,"context_line":"- import_playbook: os-murano-install.yml"},{"line_number":44,"context_line":"- import_playbook: os-zun-install.yml"},{"line_number":45,"context_line":"- import_playbook: os-tempest-install.yml"},{"line_number":46,"context_line":"- import_playbook: os-rally-install.yml"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"2ee42c83_3f982907","line":45,"range":{"start_line":45,"start_character":0,"end_line":45,"end_character":41},"in_reply_to":"d4dfeb0d_618f5d57","updated":"2022-10-25 08:41:29.000000000","message":"yeah, we don\u0027t test object storage at all with tempest now.\n\nI wonder if we should also move out tempest/rally out of setup-openstack? As it\u0027s kind of extra. Or just revert all changes except note in docs.","commit_id":"79fd6b14e06cdbcf2102c07dce780cdc4e56e708"}]}