)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"742d9beb8c3f230ec29f7913b05fdffac4547b81","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a9a29e05_e5ae5be2","updated":"2022-07-14 16:29:45.000000000","message":"I\u0027ve never kept up with the intent behind specific scenario tests, but I\u0027m puzzled by scenario010. It deploys ceph but not cinder, which feels like a pretty boring scenario. Sure, it tests ceph deployment, but so do other scenarios. Does 010 provide coverage that would be otherwise missing?","commit_id":"75d952bf8ccdfe7f8dc87b783c8bcffe5f718c59"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"cef797a231092fbd49fe678ed3fbc78c737ad3c7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ddf4b10f_114ccf19","in_reply_to":"0c4cd8d9_df6e6394","updated":"2022-07-14 17:10:30.000000000","message":"I forgot to mention the most important thing but ceph is already covered by 001(with cinder and telemetry) and 004 (manila) and I\u0027m not aware of any additional requirement we achieve by scenario 010.\n\nThe only difference is scenario 010 have a few services disabled but I don\u0027t know that\u0027s worth testing in a separate job.","commit_id":"75d952bf8ccdfe7f8dc87b783c8bcffe5f718c59"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"5008b7012d5ae1326b4943046a1bff5c7c315e49","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"41b85836_71f1a96d","in_reply_to":"9b3b6ae5_6ab3cb20","updated":"2022-07-15 08:50:06.000000000","message":"Thanks Marios. That makes clear sense and I think that is still the valid requirement.\n\nI\u0027ve updated the patch to describe that. Hopefully that helps us avoid the same discussion in the future.","commit_id":"75d952bf8ccdfe7f8dc87b783c8bcffe5f718c59"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"8215b634e74e7ac567cf975a62840c37e48723ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0c4cd8d9_df6e6394","in_reply_to":"a9a29e05_e5ae5be2","updated":"2022-07-14 17:06:32.000000000","message":"That\u0027s a good point, Alan.\n\nSo digging into git log, scneario 010 was added to test Octavia. I don\u0027t know why but it seems we decided to use ceph in this job.\n\nAs ceph is already tested in the other scenario jobs, disabling ceph and using local file storage would be a better option. Actually we already have scenario013 environment which does implement this but is not used actually.\n\nI recently tested disabling ceph in standalone job we run in puppet-octavia and I confirmed disabling ceph reduces job executing time a lot. https://review.opendev.org/c/openstack/puppet-octavia/+/849768\n\nI\u0027d add a few CI folks to hear any concern but I tend to squash scenario 013 to 010 and disables ceph in 010.","commit_id":"75d952bf8ccdfe7f8dc87b783c8bcffe5f718c59"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"a70244694cf40e774ddb7ba593eba2063cca9713","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9b3b6ae5_6ab3cb20","in_reply_to":"ddf4b10f_114ccf19","updated":"2022-07-15 07:06:07.000000000","message":"o/ folks, as Takashi said sc10 is primarily about testing octavia\n\nI don\u0027t have context about why ceph was included here but I did some digging:\n\nCeph was included from the start - the scen10 standalone env file was added there [1] \n\nThat standalone file was copied from the multinode one at [2]. We can trace the addition of ceph to [2] - it was added in the commit at [3] and the commit message there gives the justification. Does that no longer hold true? \n\n[1]  https://opendev.org/openstack/tripleo-heat-templates/commit/55724c127f4033fbcf0d8b6277c55fb15d988ff7 \n[2] https://opendev.org/openstack/tripleo-heat-templates/src/commit/7685bc1684eb3f6d374af42ac8f2114a4f3e07e3/ci/environments/scenario010-multinode-containers.yaml#L2\n[3] https://opendev.org/openstack/tripleo-heat-templates/commit/a33d42abb6f31149769627b6fc316b02d3c51726","commit_id":"75d952bf8ccdfe7f8dc87b783c8bcffe5f718c59"}]}