)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"2b647326e4c238b29c569673b9c9793ca18e3cb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c695fec3_8615fe50","updated":"2021-10-15 03:21:05.000000000","message":"Thanks for thorough review Martin","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"6d5a04c7a236ff67fa85c31753d312f04848a697","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a386d489_ac5f1c15","updated":"2021-10-14 11:24:27.000000000","message":"oh, one more thing, maybe we could also generate a .rst guideline files in this patch:\n./tools/jsonToRst.py --all 2021.11\n","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"2b647326e4c238b29c569673b9c9793ca18e3cb4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c28bb105_04734315","in_reply_to":"a386d489_ac5f1c15","updated":"2021-10-15 03:21:05.000000000","message":"I will try it but it is not part of the patch","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"9da55bea33cb34b61e11fdfaffb15b3d2933e4dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e8f96739_0cb191cd","updated":"2021-10-29 20:03:07.000000000","message":"I am having problems amending this patch after we moved repo.\nSO I will try to create brand new patch under new openinfra repo, put a ling to it here, add all people on this patch to it and then will abandon it.","commit_id":"ca21124440333d287546c377134dc9bab14c83dd"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"b90fdfde9aa2350efeaf1d0c7597cf5fdc9fae77","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"96599009_413e72a0","updated":"2021-10-21 21:45:36.000000000","message":"Thank you very much for your feedback John and Sean.\nResponse and questions inside.","commit_id":"ca21124440333d287546c377134dc9bab14c83dd"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"121b4e8ae86b39c7bde512c7b0d96270940bfe78","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"dfe7ce8f_9fe489b2","updated":"2021-10-29 20:43:15.000000000","message":"This patch is replaced by https://review.opendev.org/c/openinfra/interop/+/815995","commit_id":"ca21124440333d287546c377134dc9bab14c83dd"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5d463dd5520c1a0d6c2f2498e6e8107dfd526c99","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"29b84686_d9800f76","updated":"2021-10-20 16:40:10.000000000","message":"i think we should not require cinder for openstack powered compute\n\ncinder can be included in openstack powered platform. but cinder to me feels like an add on service above the minimum for a compute powered cloud.","commit_id":"ca21124440333d287546c377134dc9bab14c83dd"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"8734c6a890efd79c69ecb6dbec264919e6dd0f0d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"df9fd4c5_d263b7f3","updated":"2021-10-15 20:51:47.000000000","message":"removed new_key files - my mistake to include these","commit_id":"ca21124440333d287546c377134dc9bab14c83dd"}],"add-ons/guidelines/dns.2021.11.json":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"b573e859165b7288829562e0bf587fb9d2c41edb","unresolved":true,"context_lines":[{"line_number":55,"context_line":"      }"},{"line_number":56,"context_line":"    },"},{"line_number":57,"context_line":"    \"os_trademark_approval\": {"},{"line_number":58,"context_line":"      \"target_approval\": \"dns.2021.11\","},{"line_number":59,"context_line":"      \"replaces\": \"dns.2020.11\","},{"line_number":60,"context_line":"      \"releases\": [\"victoria\", \"wallaby\", \"xena\", \"yoga\"],"},{"line_number":61,"context_line":"      \"status\": \"draft\""}],"source_content_type":"application/json","patch_set":2,"id":"3e066c77_97c34b4b","line":58,"range":{"start_line":58,"start_character":26,"end_line":58,"end_character":29},"updated":"2021-10-08 14:52:59.000000000","message":"ditto","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"2b647326e4c238b29c569673b9c9793ca18e3cb4","unresolved":true,"context_lines":[{"line_number":55,"context_line":"      }"},{"line_number":56,"context_line":"    },"},{"line_number":57,"context_line":"    \"os_trademark_approval\": {"},{"line_number":58,"context_line":"      \"target_approval\": \"dns.2021.11\","},{"line_number":59,"context_line":"      \"replaces\": \"dns.2020.11\","},{"line_number":60,"context_line":"      \"releases\": [\"victoria\", \"wallaby\", \"xena\", \"yoga\"],"},{"line_number":61,"context_line":"      \"status\": \"draft\""}],"source_content_type":"application/json","patch_set":2,"id":"2b250f04_21c4945e","line":58,"range":{"start_line":58,"start_character":26,"end_line":58,"end_character":29},"in_reply_to":"3e066c77_97c34b4b","updated":"2021-10-15 03:21:05.000000000","message":"We can change it starting from this release by dropping dns.","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"b573e859165b7288829562e0bf587fb9d2c41edb","unresolved":true,"context_lines":[{"line_number":56,"context_line":"    },"},{"line_number":57,"context_line":"    \"os_trademark_approval\": {"},{"line_number":58,"context_line":"      \"target_approval\": \"dns.2021.11\","},{"line_number":59,"context_line":"      \"replaces\": \"dns.2020.11\","},{"line_number":60,"context_line":"      \"releases\": [\"victoria\", \"wallaby\", \"xena\", \"yoga\"],"},{"line_number":61,"context_line":"      \"status\": \"draft\""},{"line_number":62,"context_line":"    }"}],"source_content_type":"application/json","patch_set":2,"id":"5f17cb9d_1998c34b","line":59,"range":{"start_line":59,"start_character":19,"end_line":59,"end_character":22},"updated":"2021-10-08 14:52:59.000000000","message":"ditto","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"2b647326e4c238b29c569673b9c9793ca18e3cb4","unresolved":true,"context_lines":[{"line_number":56,"context_line":"    },"},{"line_number":57,"context_line":"    \"os_trademark_approval\": {"},{"line_number":58,"context_line":"      \"target_approval\": \"dns.2021.11\","},{"line_number":59,"context_line":"      \"replaces\": \"dns.2020.11\","},{"line_number":60,"context_line":"      \"releases\": [\"victoria\", \"wallaby\", \"xena\", \"yoga\"],"},{"line_number":61,"context_line":"      \"status\": \"draft\""},{"line_number":62,"context_line":"    }"}],"source_content_type":"application/json","patch_set":2,"id":"0ba40101_0ed1bce1","line":59,"range":{"start_line":59,"start_character":19,"end_line":59,"end_character":22},"in_reply_to":"5f17cb9d_1998c34b","updated":"2021-10-15 03:21:05.000000000","message":"That we can not change since previous release used dns in the target.","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"71951892036ad148b9959a7eb3eafa6c12a5119b","unresolved":true,"context_lines":[{"line_number":188,"context_line":"  \"test_repositories\": {"},{"line_number":189,"context_line":"    \"designate-tempest-plugin\": {"},{"line_number":190,"context_line":"      \"repository\": \"http://opendev.org/openstack/designate-tempest-plugin/\","},{"line_number":191,"context_line":"      \"reference\": \"5da3047851aae36f09e0b52f79523426fec6a104\","},{"line_number":192,"context_line":"      \"description\": \"OpenStack Designate Test Plugin Repository\""},{"line_number":193,"context_line":"    }"},{"line_number":194,"context_line":"  }"}],"source_content_type":"application/json","patch_set":2,"id":"c358f6fe_ef901c21","line":191,"range":{"start_line":191,"start_character":20,"end_line":191,"end_character":60},"updated":"2021-10-08 14:50:05.000000000","message":"da27a70ae2b39695ef6f03bbefb55afeacf1cdf3","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"}],"add-ons/guidelines/orchestration.2021.11.json":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"b573e859165b7288829562e0bf587fb9d2c41edb","unresolved":true,"context_lines":[{"line_number":55,"context_line":"      }"},{"line_number":56,"context_line":"    },"},{"line_number":57,"context_line":"    \"os_trademark_approval\": {"},{"line_number":58,"context_line":"      \"target_approval\": \"orchestration.2021.11\","},{"line_number":59,"context_line":"      \"replaces\": \"orchestration.2020.11\","},{"line_number":60,"context_line":"      \"releases\": [\"victoria\", \"wallaby\", \"xena\", \"yoga\"],"},{"line_number":61,"context_line":"      \"status\": \"draft\""}],"source_content_type":"application/json","patch_set":2,"id":"1dc18fe2_aa253aab","line":58,"range":{"start_line":58,"start_character":26,"end_line":58,"end_character":40},"updated":"2021-10-08 14:52:59.000000000","message":"shouldn\u0027t be here only date? f.e. shared_file_system https://review.opendev.org/c/osf/interop/+/811049/2/add-ons/guidelines/shared_file_system.2021.11.json","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"2b647326e4c238b29c569673b9c9793ca18e3cb4","unresolved":true,"context_lines":[{"line_number":55,"context_line":"      }"},{"line_number":56,"context_line":"    },"},{"line_number":57,"context_line":"    \"os_trademark_approval\": {"},{"line_number":58,"context_line":"      \"target_approval\": \"orchestration.2021.11\","},{"line_number":59,"context_line":"      \"replaces\": \"orchestration.2020.11\","},{"line_number":60,"context_line":"      \"releases\": [\"victoria\", \"wallaby\", \"xena\", \"yoga\"],"},{"line_number":61,"context_line":"      \"status\": \"draft\""}],"source_content_type":"application/json","patch_set":2,"id":"ef1b61b7_fa43cdd4","line":58,"range":{"start_line":58,"start_character":26,"end_line":58,"end_character":40},"in_reply_to":"1dc18fe2_aa253aab","updated":"2021-10-15 03:21:05.000000000","message":"in all previous versions we had the name as part of the target.\nSuggest we keep it.\nWe can change it to just the date. I think it will make is simpler","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"b573e859165b7288829562e0bf587fb9d2c41edb","unresolved":true,"context_lines":[{"line_number":56,"context_line":"    },"},{"line_number":57,"context_line":"    \"os_trademark_approval\": {"},{"line_number":58,"context_line":"      \"target_approval\": \"orchestration.2021.11\","},{"line_number":59,"context_line":"      \"replaces\": \"orchestration.2020.11\","},{"line_number":60,"context_line":"      \"releases\": [\"victoria\", \"wallaby\", \"xena\", \"yoga\"],"},{"line_number":61,"context_line":"      \"status\": \"draft\""},{"line_number":62,"context_line":"    }"}],"source_content_type":"application/json","patch_set":2,"id":"f765d286_f666a4d0","line":59,"range":{"start_line":59,"start_character":19,"end_line":59,"end_character":33},"updated":"2021-10-08 14:52:59.000000000","message":"shouldn\u0027t be here only date?","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"2b647326e4c238b29c569673b9c9793ca18e3cb4","unresolved":true,"context_lines":[{"line_number":56,"context_line":"    },"},{"line_number":57,"context_line":"    \"os_trademark_approval\": {"},{"line_number":58,"context_line":"      \"target_approval\": \"orchestration.2021.11\","},{"line_number":59,"context_line":"      \"replaces\": \"orchestration.2020.11\","},{"line_number":60,"context_line":"      \"releases\": [\"victoria\", \"wallaby\", \"xena\", \"yoga\"],"},{"line_number":61,"context_line":"      \"status\": \"draft\""},{"line_number":62,"context_line":"    }"}],"source_content_type":"application/json","patch_set":2,"id":"0baf0664_753266fa","line":59,"range":{"start_line":59,"start_character":19,"end_line":59,"end_character":33},"in_reply_to":"f765d286_f666a4d0","updated":"2021-10-15 03:21:05.000000000","message":"that we can not change. Since it is the name used last time.\nBut we can change for the future. Will make parsing simpler.","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"71951892036ad148b9959a7eb3eafa6c12a5119b","unresolved":true,"context_lines":[{"line_number":735,"context_line":"  \"test_repositories\": {"},{"line_number":736,"context_line":"    \"heat\": {"},{"line_number":737,"context_line":"      \"repository\": \"http://opendev.org/openstack/heat-tempest-plugin/\","},{"line_number":738,"context_line":"      \"reference\": \"fa9f6ccd34c106bcaa50cb2f13caea2c387a6b5f\","},{"line_number":739,"context_line":"      \"description\": \"OpenStack Heat Tempest Test Plugin Repository\""},{"line_number":740,"context_line":"    }"},{"line_number":741,"context_line":"  }"}],"source_content_type":"application/json","patch_set":2,"id":"7ea01cbc_44d5589e","line":738,"range":{"start_line":738,"start_character":20,"end_line":738,"end_character":60},"updated":"2021-10-08 14:50:05.000000000","message":"adb8ba7dea4e93414957b72ea9ff5423ba39cb56","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"}],"add-ons/guidelines/shared_file_system.2021.11.json":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"71951892036ad148b9959a7eb3eafa6c12a5119b","unresolved":true,"context_lines":[{"line_number":393,"context_line":"  \"test_repositories\": {"},{"line_number":394,"context_line":"    \"manila-tempest-plugin\": {"},{"line_number":395,"context_line":"      \"repository\": \"https://opendev.org/openstack/manila-tempest-plugin/\","},{"line_number":396,"context_line":"      \"reference\": \"master\","},{"line_number":397,"context_line":"      \"description\": \"OpenStack Manila Test Plugin Repository\""},{"line_number":398,"context_line":"    }"},{"line_number":399,"context_line":"  }"}],"source_content_type":"application/json","patch_set":2,"id":"3be697c9_8ce0267a","line":396,"range":{"start_line":396,"start_character":20,"end_line":396,"end_character":26},"updated":"2021-10-08 14:50:05.000000000","message":"9c4a97b150e15b814acd4feb4da858a0eeff881e - this is the latest commit at this moment","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"2b647326e4c238b29c569673b9c9793ca18e3cb4","unresolved":false,"context_lines":[{"line_number":393,"context_line":"  \"test_repositories\": {"},{"line_number":394,"context_line":"    \"manila-tempest-plugin\": {"},{"line_number":395,"context_line":"      \"repository\": \"https://opendev.org/openstack/manila-tempest-plugin/\","},{"line_number":396,"context_line":"      \"reference\": \"master\","},{"line_number":397,"context_line":"      \"description\": \"OpenStack Manila Test Plugin Repository\""},{"line_number":398,"context_line":"    }"},{"line_number":399,"context_line":"  }"}],"source_content_type":"application/json","patch_set":2,"id":"e5d12991_f8fa0132","line":396,"range":{"start_line":396,"start_character":20,"end_line":396,"end_character":26},"in_reply_to":"3be697c9_8ce0267a","updated":"2021-10-15 03:21:05.000000000","message":"Done","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"}],"guidelines/2021.11.json":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"71951892036ad148b9959a7eb3eafa6c12a5119b","unresolved":true,"context_lines":[{"line_number":2616,"context_line":"        \"tempest.api.network.test_routers.RoutersTest.test_update_router_admin_state\": {"},{"line_number":2617,"context_line":"          \"idempotent_id\": \"id-a8902683-c788-4246-95c7-ad9c6d63a4d9\""},{"line_number":2618,"context_line":"        },"},{"line_number":2619,"context_line":"        \"tempest.api.network.test_routers.RoutersTest.test_update_router_unset_gateway\": {"},{"line_number":2620,"context_line":"          \"idempotent_id\": \"id-ad81b7ee-4f81-407b-a19c-17e623f763e8\","},{"line_number":2621,"context_line":"          \"flagged\": {"},{"line_number":2622,"context_line":"            \"reason\": \"D400 This test requires admin credentials.\","},{"line_number":2623,"context_line":"            \"action\": \"This test will be refactored or removed from future guidelines.\","},{"line_number":2624,"context_line":"            \"date\": \"2018-03-02\""},{"line_number":2625,"context_line":"          }"},{"line_number":2626,"context_line":"        },"},{"line_number":2627,"context_line":"        \"tempest.api.network.test_routers_negative.RoutersNegativeTest.test_update_non_existent_router_returns_404\": {"},{"line_number":2628,"context_line":"          \"idempotent_id\": \"id-b23d1569-8b0c-4169-8d4b-6abd34fad5c7\""},{"line_number":2629,"context_line":"        },"}],"source_content_type":"application/json","patch_set":2,"id":"45beffb1_afcf6ae2","line":2626,"range":{"start_line":2619,"start_character":8,"end_line":2626,"end_character":10},"updated":"2021-10-08 14:50:05.000000000","message":"the test got moved to tempest/api/network/admin - so it\u0027s not a good candidate for interop as the reason for the flag says .. can we remove that?","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"71951892036ad148b9959a7eb3eafa6c12a5119b","unresolved":true,"context_lines":[{"line_number":2986,"context_line":"             \"tempest.api.volume.test_snapshot_metadata.SnapshotV2MetadataTestJSON.test_crud_snapshot_metadata\""},{"line_number":2987,"context_line":"          ]"},{"line_number":2988,"context_line":"        },"},{"line_number":2989,"context_line":"        \"tempest.api.volume.test_snapshot_metadata.VolumesMetadataTest.test_update_snapshot_metadata\": {"},{"line_number":2990,"context_line":"          \"idempotent_id\": \"id-bd2363bc-de92-48a4-bc98-28943c6e4be1\","},{"line_number":2991,"context_line":"          \"flagged\": {"},{"line_number":2992,"context_line":"            \"date\": \"2018-03-20\","},{"line_number":2993,"context_line":"            \"reason\": \"[D406] This test was removed from Tempest in commit id cd72e905b24b6653c53517ebd016d8f30817bde8\","},{"line_number":2994,"context_line":"            \"action\": \"Remove flag on next refstack release.\""},{"line_number":2995,"context_line":"          },"},{"line_number":2996,"context_line":"          \"aliases\": ["},{"line_number":2997,"context_line":"             \"tempest.api.volume.test_snapshot_metadata.SnapshotV2MetadataTestJSON.test_update_snapshot_metadata\","},{"line_number":2998,"context_line":"             \"tempest.api.volume.test_snapshot_metadata.SnapshotMetadataTestJSON.test_update_snapshot_metadata\""}],"source_content_type":"application/json","patch_set":2,"id":"87d2ea0e_f1136877","line":2995,"range":{"start_line":2989,"start_character":0,"end_line":2995,"end_character":12},"updated":"2021-10-08 14:50:05.000000000","message":"this test doesn\u0027t exit anymore, it\u0027s been flagged (and thus always skipped) for 3 years, I think we can remove it now.","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"2b647326e4c238b29c569673b9c9793ca18e3cb4","unresolved":true,"context_lines":[{"line_number":2986,"context_line":"             \"tempest.api.volume.test_snapshot_metadata.SnapshotV2MetadataTestJSON.test_crud_snapshot_metadata\""},{"line_number":2987,"context_line":"          ]"},{"line_number":2988,"context_line":"        },"},{"line_number":2989,"context_line":"        \"tempest.api.volume.test_snapshot_metadata.VolumesMetadataTest.test_update_snapshot_metadata\": {"},{"line_number":2990,"context_line":"          \"idempotent_id\": \"id-bd2363bc-de92-48a4-bc98-28943c6e4be1\","},{"line_number":2991,"context_line":"          \"flagged\": {"},{"line_number":2992,"context_line":"            \"date\": \"2018-03-20\","},{"line_number":2993,"context_line":"            \"reason\": \"[D406] This test was removed from Tempest in commit id cd72e905b24b6653c53517ebd016d8f30817bde8\","},{"line_number":2994,"context_line":"            \"action\": \"Remove flag on next refstack release.\""},{"line_number":2995,"context_line":"          },"},{"line_number":2996,"context_line":"          \"aliases\": ["},{"line_number":2997,"context_line":"             \"tempest.api.volume.test_snapshot_metadata.SnapshotV2MetadataTestJSON.test_update_snapshot_metadata\","},{"line_number":2998,"context_line":"             \"tempest.api.volume.test_snapshot_metadata.SnapshotMetadataTestJSON.test_update_snapshot_metadata\""}],"source_content_type":"application/json","patch_set":2,"id":"a50d2c31_6befe7f4","line":2995,"range":{"start_line":2989,"start_character":0,"end_line":2995,"end_character":12},"in_reply_to":"87d2ea0e_f1136877","updated":"2021-10-15 03:21:05.000000000","message":"ok. That includes aliases below","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"71951892036ad148b9959a7eb3eafa6c12a5119b","unresolved":true,"context_lines":[{"line_number":3746,"context_line":"  \"test_repositories\": {"},{"line_number":3747,"context_line":"    \"tempest\": {"},{"line_number":3748,"context_line":"      \"repository\": \"https://opendev.org/openstack/tempest/\","},{"line_number":3749,"context_line":"      \"reference\": \"c8d7a01014d4b82dd9960679db5e77dc31f4d87a\","},{"line_number":3750,"context_line":"      \"description\": \"OpenStack Integrated API and Scenario Test Repository\""},{"line_number":3751,"context_line":"    }"},{"line_number":3752,"context_line":"  }"}],"source_content_type":"application/json","patch_set":2,"id":"92a86f49_20160ebb","line":3749,"range":{"start_line":3749,"start_character":20,"end_line":3749,"end_character":60},"updated":"2021-10-08 14:50:05.000000000","message":"1d500e79156ada6bc6fdb628ed1da0efd4121f6a\n\nthat\u0027s ^^ the current tempest commit we currently have in refstack-client - https://opendev.org/osf/refstack-client/src/commit/0ab5f7ac61dac223965c90622ef1be3ab3483d9f/setup_env#L4","commit_id":"30ef1d5521cc1d7da0b6117a274095d20632fdd2"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"d4482b9e6b9fe7acfb5eac50f0d3696c675966eb","unresolved":true,"context_lines":[{"line_number":121,"context_line":"          \"compute-servers-name\","},{"line_number":122,"context_line":"          \"compute-servers-reboot\","},{"line_number":123,"context_line":"          \"compute-servers-rebuild\","},{"line_number":124,"context_line":"          \"compute-servers-resize\","},{"line_number":125,"context_line":"          \"compute-servers-stop\","},{"line_number":126,"context_line":"          \"compute-servers-update\","},{"line_number":127,"context_line":"          \"compute-servers-verify\","}],"source_content_type":"application/json","patch_set":3,"id":"13aed8e5_6ffb8f52","line":124,"updated":"2021-10-20 16:16:15.000000000","message":"This and images-create I guess do mean ironic based clouds fail, so it would be nice to make these advisory.","commit_id":"ca21124440333d287546c377134dc9bab14c83dd"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"b90fdfde9aa2350efeaf1d0c7597cf5fdc9fae77","unresolved":true,"context_lines":[{"line_number":121,"context_line":"          \"compute-servers-name\","},{"line_number":122,"context_line":"          \"compute-servers-reboot\","},{"line_number":123,"context_line":"          \"compute-servers-rebuild\","},{"line_number":124,"context_line":"          \"compute-servers-resize\","},{"line_number":125,"context_line":"          \"compute-servers-stop\","},{"line_number":126,"context_line":"          \"compute-servers-update\","},{"line_number":127,"context_line":"          \"compute-servers-verify\","}],"source_content_type":"application/json","patch_set":3,"id":"31be6adb_5443ca8b","line":124,"in_reply_to":"13aed8e5_6ffb8f52","updated":"2021-10-21 21:45:36.000000000","message":"Ack","commit_id":"ca21124440333d287546c377134dc9bab14c83dd"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"d4482b9e6b9fe7acfb5eac50f0d3696c675966eb","unresolved":true,"context_lines":[{"line_number":129,"context_line":"          \"compute-servers-metadata-get\","},{"line_number":130,"context_line":"          \"compute-servers-metadata-list\","},{"line_number":131,"context_line":"          \"compute-servers-metadata-set\","},{"line_number":132,"context_line":"          \"compute-servers-metadata-update\","},{"line_number":133,"context_line":"          \"compute-volume-attach\","},{"line_number":134,"context_line":"          \"identity-v3-api-discovery\","},{"line_number":135,"context_line":"          \"identity-v3-catalog\","}],"source_content_type":"application/json","patch_set":3,"id":"d497aaf9_58b67d72","line":132,"updated":"2021-10-20 16:16:15.000000000","message":"it would be nice to have compute server tags in here, they have been around a while now, and preferred over metadata by many users. Although its quite hard to not support that API, so its probably not a big deal.","commit_id":"ca21124440333d287546c377134dc9bab14c83dd"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"b90fdfde9aa2350efeaf1d0c7597cf5fdc9fae77","unresolved":true,"context_lines":[{"line_number":129,"context_line":"          \"compute-servers-metadata-get\","},{"line_number":130,"context_line":"          \"compute-servers-metadata-list\","},{"line_number":131,"context_line":"          \"compute-servers-metadata-set\","},{"line_number":132,"context_line":"          \"compute-servers-metadata-update\","},{"line_number":133,"context_line":"          \"compute-volume-attach\","},{"line_number":134,"context_line":"          \"identity-v3-api-discovery\","},{"line_number":135,"context_line":"          \"identity-v3-catalog\","}],"source_content_type":"application/json","patch_set":3,"id":"26826166_4ec53543","line":132,"in_reply_to":"d497aaf9_58b67d72","updated":"2021-10-21 21:45:36.000000000","message":"so the proposal is to add:\ncompute-servers-tags-get\ncompute-servers-tags-put\ncompute-servers-tags-delete\ncompute-servers-tags-tag-get\ncompute-servers-tags-tag-put\ncompute-servers-tags-tag-delete\n?\n\nAnd the tempest tests for these will be:\ntempest.api.compute.servers.test_server_tags.ServerTagsTestJSON.test_check_tag_existence[81279a66-61c3-4759-b830-a2dbe64cbe08]\ntempest.api.compute.servers.test_server_tags.ServerTagsTestJSON.test_create_delete_tag[8d95abe2-c658-4c42-9a44-c0258500306b]\ntempest.api.compute.servers.test_server_tags.ServerTagsTestJSON.test_delete_all_tags[a63b2a74-e918-4b7c-bcab-10c855f3a57e]\ntempest.api.compute.servers.test_server_tags.ServerTagsTestJSON.test_update_all_tags[a2c1af8c-127d-417d-974b-8115f7e3d831]\n\nI do not see matching tempest test for 6 APIs?\n\nNeed help here!","commit_id":"ca21124440333d287546c377134dc9bab14c83dd"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"5d463dd5520c1a0d6c2f2498e6e8107dfd526c99","unresolved":true,"context_lines":[{"line_number":148,"context_line":"          \"networks-list-api-versions\","},{"line_number":149,"context_line":"          \"networks-security-groups-CRUD\","},{"line_number":150,"context_line":"          \"networks-subnet-pools-CRUD\","},{"line_number":151,"context_line":"          \"volumes-list-api-versions\","},{"line_number":152,"context_line":"          \"volumes-v3-create-delete\","},{"line_number":153,"context_line":"          \"volumes-v3-snapshot-create-delete\","},{"line_number":154,"context_line":"          \"volumes-v3-get\","},{"line_number":155,"context_line":"          \"volumes-v3-list\","},{"line_number":156,"context_line":"          \"volumes-v3-update\","},{"line_number":157,"context_line":"          \"volumes-v3-copy-image-to-volume\","},{"line_number":158,"context_line":"          \"volumes-v3-clone\","},{"line_number":159,"context_line":"          \"volumes-v3-availability-zones\","},{"line_number":160,"context_line":"          \"volumes-v3-extensions\","},{"line_number":161,"context_line":"          \"volumes-v3-metadata\","},{"line_number":162,"context_line":"          \"volumes-v3-readonly\","},{"line_number":163,"context_line":"          \"volumes-v3-upload\""},{"line_number":164,"context_line":"        ],"},{"line_number":165,"context_line":"        \"advisory\": ["},{"line_number":166,"context_line":"          \"identity-v3-list-versions\","}],"source_content_type":"application/json","patch_set":3,"id":"0d97ff2f_76d94aed","line":163,"range":{"start_line":151,"start_character":4,"end_line":163,"end_character":29},"updated":"2021-10-20 16:40:10.000000000","message":"i have provided this feedback in the last few ptg but i do not belive its correct to include cinder requirement in openstack porwered compute requirements.\n\nyou do not need to deploy cinder to meet the minium requirement to have a compute.\n\n\ni feel like block device api should be a addon to the openstack powered compute.\n\ncould this be moved to deprecated or advisory but not required.","commit_id":"ca21124440333d287546c377134dc9bab14c83dd"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"b90fdfde9aa2350efeaf1d0c7597cf5fdc9fae77","unresolved":true,"context_lines":[{"line_number":148,"context_line":"          \"networks-list-api-versions\","},{"line_number":149,"context_line":"          \"networks-security-groups-CRUD\","},{"line_number":150,"context_line":"          \"networks-subnet-pools-CRUD\","},{"line_number":151,"context_line":"          \"volumes-list-api-versions\","},{"line_number":152,"context_line":"          \"volumes-v3-create-delete\","},{"line_number":153,"context_line":"          \"volumes-v3-snapshot-create-delete\","},{"line_number":154,"context_line":"          \"volumes-v3-get\","},{"line_number":155,"context_line":"          \"volumes-v3-list\","},{"line_number":156,"context_line":"          \"volumes-v3-update\","},{"line_number":157,"context_line":"          \"volumes-v3-copy-image-to-volume\","},{"line_number":158,"context_line":"          \"volumes-v3-clone\","},{"line_number":159,"context_line":"          \"volumes-v3-availability-zones\","},{"line_number":160,"context_line":"          \"volumes-v3-extensions\","},{"line_number":161,"context_line":"          \"volumes-v3-metadata\","},{"line_number":162,"context_line":"          \"volumes-v3-readonly\","},{"line_number":163,"context_line":"          \"volumes-v3-upload\""},{"line_number":164,"context_line":"        ],"},{"line_number":165,"context_line":"        \"advisory\": ["},{"line_number":166,"context_line":"          \"identity-v3-list-versions\","}],"source_content_type":"application/json","patch_set":3,"id":"e783ec8f_dc2380b4","line":163,"range":{"start_line":151,"start_character":4,"end_line":163,"end_character":29},"in_reply_to":"0d97ff2f_76d94aed","updated":"2021-10-21 21:45:36.000000000","message":"Thanks for feedback.\nThis is not something we can do on our own.\nWe will need to get TC input on it and then bring it to OIF board for approval.\nSo it will not happen this cycle.\n\nAs of now all distros, private and public cloud providers support cinder. so they do not mind having cinder test included. They pass it.\nUser are free to not use cinder APIs if they do not need block storage.","commit_id":"ca21124440333d287546c377134dc9bab14c83dd"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"f440bda8df5f1a9cd2f90c8daa549f0e93c39a82","unresolved":true,"context_lines":[{"line_number":168,"context_line":"          \"identity-v3-users\","},{"line_number":169,"context_line":"          \"identity-v3-domains\","},{"line_number":170,"context_line":"          \"volumes-v3-snapshots-list\","},{"line_number":171,"context_line":"          \"compute-images-metadata-delete\","},{"line_number":172,"context_line":"          \"compute-images-metadata-get\","},{"line_number":173,"context_line":"          \"compute-images-metadata-list\","},{"line_number":174,"context_line":"          \"compute-images-metadata-set\","}],"source_content_type":"application/json","patch_set":3,"id":"7850fc49_d451ddf2","line":171,"updated":"2021-10-20 16:06:18.000000000","message":"Are these the API we have marked as deprecated?\nhttps://docs.openstack.org/api-ref/compute/#list-image-metadata","commit_id":"ca21124440333d287546c377134dc9bab14c83dd"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"b90fdfde9aa2350efeaf1d0c7597cf5fdc9fae77","unresolved":true,"context_lines":[{"line_number":168,"context_line":"          \"identity-v3-users\","},{"line_number":169,"context_line":"          \"identity-v3-domains\","},{"line_number":170,"context_line":"          \"volumes-v3-snapshots-list\","},{"line_number":171,"context_line":"          \"compute-images-metadata-delete\","},{"line_number":172,"context_line":"          \"compute-images-metadata-get\","},{"line_number":173,"context_line":"          \"compute-images-metadata-list\","},{"line_number":174,"context_line":"          \"compute-images-metadata-set\","}],"source_content_type":"application/json","patch_set":3,"id":"05f0f048_69155426","line":171,"in_reply_to":"7850fc49_d451ddf2","updated":"2021-10-21 21:45:36.000000000","message":"We will remove these. They were not required before and not even listed before.\nSO we can drop them without any damage.\nThanks for feedback.","commit_id":"ca21124440333d287546c377134dc9bab14c83dd"}]}