)]}'
{"ci/environments/scenario001-standalone.yaml":[{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"24e8f6c513375a38ec62c2ab08d534d8f06235fd","unresolved":false,"context_lines":[{"line_number":99,"context_line":"  NovaEnableRbdBackend: true"},{"line_number":100,"context_line":"  CinderEnableRbdBackend: true"},{"line_number":101,"context_line":"  CinderBackupBackend: ceph"},{"line_number":102,"context_line":"  GlanceBackend: cinder"},{"line_number":103,"context_line":"  GlanceStoreDescription: \u0027Cinder glance store\u0027"},{"line_number":104,"context_line":"  GlanceMultistoreConfig:"},{"line_number":105,"context_line":"    rbd_store:"},{"line_number":106,"context_line":"      GlanceBackend: rbd"},{"line_number":107,"context_line":"      GlanceStoreDescription: \u0027RBD glance store\u0027"},{"line_number":108,"context_line":"      CephClientUserName: \u0027glance\u0027"},{"line_number":109,"context_line":"  GnocchiBackend: rbd"},{"line_number":110,"context_line":"  CinderEnableIscsiBackend: false"},{"line_number":111,"context_line":"  GnocchiArchivePolicy: \u0027ceilometer-high-rate\u0027"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3fa7e38b_5013a4b7","line":108,"range":{"start_line":102,"start_character":0,"end_line":108,"end_character":34},"updated":"2020-02-12 13:22:41.000000000","message":"As per the results in CI [1], we see the glance-api.conf has two backends configured as requested by the scenario001 update.\n\n[default_backend]\ncinder_store_auth_address\u003dhttp://192.168.24.1:5000/v3\ncinder_store_project_name\u003dservice\ncinder_store_user_name\u003dglance\ncinder_store_password\u003d7Uo8crUFqw0ByndGRLTpkHHW4\nstore_description\u003dCinder glance store\n\n[rbd_store]\nrbd_store_ceph_conf\u003d/etc/ceph/ceph.conf\nrbd_store_user\u003dglance\nrbd_store_pool\u003dimages\nstore_description\u003dRBD glance store\n\n[1] https://64a908f0333f149ec4d5-b854b998feabd4bf6926393c8ea9e138.ssl.cf1.rackcdn.com/704374/3/check/tripleo-ci-centos-7-scenario001-standalone/f7ebea3/logs/undercloud/var/lib/config-data/puppet-generated/glance_api/etc/glance/glance-api.conf","commit_id":"96937674a7524275b52d90d4b3a0f94c0e384937"},{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"de14676ec0f8b3488fa3dad9b8e7e1f147469ebc","unresolved":false,"context_lines":[{"line_number":99,"context_line":"  NovaEnableRbdBackend: true"},{"line_number":100,"context_line":"  CinderEnableRbdBackend: true"},{"line_number":101,"context_line":"  CinderBackupBackend: ceph"},{"line_number":102,"context_line":"  GlanceBackend: cinder"},{"line_number":103,"context_line":"  GlanceStoreDescription: \u0027Cinder glance store\u0027"},{"line_number":104,"context_line":"  GlanceMultistoreConfig:"},{"line_number":105,"context_line":"    rbd_store:"},{"line_number":106,"context_line":"      GlanceBackend: rbd"},{"line_number":107,"context_line":"      GlanceStoreDescription: \u0027RBD glance store\u0027"},{"line_number":108,"context_line":"      CephClientUserName: \u0027glance\u0027"},{"line_number":109,"context_line":"  GnocchiBackend: rbd"},{"line_number":110,"context_line":"  CinderEnableIscsiBackend: false"},{"line_number":111,"context_line":"  GnocchiArchivePolicy: \u0027ceilometer-high-rate\u0027"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3fa7e38b_4237d34d","line":108,"range":{"start_line":102,"start_character":0,"end_line":108,"end_character":34},"in_reply_to":"3fa7e38b_5013a4b7","updated":"2020-02-20 22:03:09.000000000","message":"So we can see the patch putting the desired thing in the config file because of this change to scenario001-standalone. However, the CI is also failing because we\u0027re pinned to an old version of glance at the moment. \n\nIs the following a way forward:\n\n1. comment out this modification on lines 102-108 and add in line 101 a comment \"todo(abishop)\" uncomment when this scenario gets to centos8\". \n\nI then expect we won\u0027t be blocked because we won\u0027t actually be testing the feature. We\u0027ll just be using the CI to prove the change didn\u0027t break the rest of the system. We can then conduct independent tests to confirmt his works and then get the CI testing the new feature when we have the new bits.\n\n[1] https://lists.rdoproject.org/pipermail/users/2020-February/000692.html","commit_id":"96937674a7524275b52d90d4b3a0f94c0e384937"}],"deployment/glance/glance-api-container-puppet.yaml":[{"author":{"_account_id":18002,"name":"John Fulton","email":"fulton@redhat.com","username":"fultonj"},"change_message_id":"24e8f6c513375a38ec62c2ab08d534d8f06235fd","unresolved":false,"context_lines":[{"line_number":567,"context_line":"                  for_each:"},{"line_number":568,"context_line":"                    \u003c%keyring%\u003e:"},{"line_number":569,"context_line":"                      yaql:"},{"line_number":570,"context_line":"                        expression: let(u \u003d\u003e $.data.user) -\u003e $.data.multistore.values().where($.get(\"CephClusterName\")).select(\"{0}.client.{1}.keyring\".format($.CephClusterName, $.get(\"CephClientUserName\", $u)))"},{"line_number":571,"context_line":"                        data:"},{"line_number":572,"context_line":"                          user: {get_param: CephClientUserName}"},{"line_number":573,"context_line":"                          multistore: {get_param: GlanceMultistoreConfig}"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3fa7e38b_30862818","line":570,"updated":"2020-02-12 13:22:41.000000000","message":"nit: it would be nice if this line could be wrapped so it\u0027s not so long.","commit_id":"96937674a7524275b52d90d4b3a0f94c0e384937"}]}