)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"59d2fe10bf6e530512b96fd3b7a43a8560869789","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This configures the nova-next job to enable the new experimental"},{"line_number":10,"context_line":"unified limits functionality."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Related to blueprint unified-limits-nova"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Depends-On: https://review.opendev.org/c/openstack/devstack/+/789962"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":30,"id":"85196237_d1c52108","line":11,"updated":"2022-02-11 03:18:12.000000000","message":"This also adds basic testing for nova \"global\" limits in keystone in the post-test-hook for the nova-next job. These can\u0027t be tested in tempest because they involve modifying global quota limits that would affect any other server tests running in parallel.\n\nNeed to add ^ to the commit message.","commit_id":"675c6c6e0c2aba5e789a6534310e3437847698c1"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"f2871f7f1684853f9994193c385484451900c8a6","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This configures the nova-next job to enable the new experimental"},{"line_number":10,"context_line":"unified limits functionality."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Related to blueprint unified-limits-nova"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Depends-On: https://review.opendev.org/c/openstack/devstack/+/789962"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":30,"id":"074832b5_1c44b3ab","line":11,"in_reply_to":"85196237_d1c52108","updated":"2022-03-04 03:46:16.000000000","message":"Done","commit_id":"675c6c6e0c2aba5e789a6534310e3437847698c1"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"688349054871d76c68096156e4710fdad0d4cfc5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"c829dfe0_7303428e","updated":"2022-01-21 19:53:21.000000000","message":"recheck bug 1957941","commit_id":"9f7346bca986e6fb054ad2302e81b8b2b0db874d"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"b6c6c42cf73187c1c111a22b12c5d3e25026f208","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":24,"id":"dec81c8b_3c9dc6fb","updated":"2022-01-21 01:08:51.000000000","message":"recheck dep updated","commit_id":"9f7346bca986e6fb054ad2302e81b8b2b0db874d"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"939918e1ec97e7b826697dd1bbb41b0252f5e7b4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":25,"id":"bc732386_44442806","updated":"2022-01-27 21:46:09.000000000","message":"recheck [ERROR] /opt/stack/devstack/lib/neutron_plugins/ovn_agent:178 Socket /var/run/openvswitch/ovnnb_db.sock not found","commit_id":"b607053488e556e4fb574db5555844f79eb13edf"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"e9f0be06bbcfe40f4918487c011352d078840dfc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":30,"id":"622133e2_b1341d87","updated":"2022-02-24 11:55:37.000000000","message":"I have one question about tempest vs CLI based testing. -1 is just for visibility. I\u0027m OK to keep both tests in place for a while.","commit_id":"675c6c6e0c2aba5e789a6534310e3437847698c1"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e3494eb77be968bf3192d39edb81f56d75f28820","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":31,"id":"60acd496_3e17b715","updated":"2022-02-28 12:03:11.000000000","message":"This looks fine too me.\nill hold +w incase you want to actully update the commit otherwise i think we can proceed with this","commit_id":"cd0b03c8447242f15af26fe6379823c56005231d"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"0f5ccb372acfcd5d53a806b5c14d52291ddf4397","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":31,"id":"4da978ae_f3e70e9e","updated":"2022-03-04 03:24:20.000000000","message":"recheck deps updated","commit_id":"cd0b03c8447242f15af26fe6379823c56005231d"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"162e57b13ef67e2b0cb77e2486a0f367edc15aa3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"36b44b01_2a9d199e","updated":"2022-03-05 03:54:27.000000000","message":"recheck deps merged","commit_id":"9ec6afe8939d6f3d72317b1cc8e0f4273bf31e43"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"665b9e93c255f3286450c89aec24655704c8f377","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"4a4f32f2_313bfb96","updated":"2022-03-04 06:29:31.000000000","message":"test is passing in post hook as well as tempest test https://zuul.opendev.org/t/openstack/build/9d54879682164f05a317ea23f92cda11/log/job-output.txt#57353 ","commit_id":"9ec6afe8939d6f3d72317b1cc8e0f4273bf31e43"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"85af412a7adbc5315f6e9ba97e7cf041f59303a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"561c6850_671719af","updated":"2022-03-04 12:28:53.000000000","message":"the tests look good to me so lets keep this moving, thanks melanie","commit_id":"9ec6afe8939d6f3d72317b1cc8e0f4273bf31e43"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"85af412a7adbc5315f6e9ba97e7cf041f59303a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"e75f9534_3cbd3aa3","in_reply_to":"4a4f32f2_313bfb96","updated":"2022-03-04 12:28:53.000000000","message":"the post hook test starts here\nhttps://zuul.opendev.org/t/openstack/build/9d54879682164f05a317ea23f92cda11/log/job-output.txt#57949\n\nthe server create that exceeds the limit fails here\n\nhttps://zuul.opendev.org/t/openstack/build/9d54879682164f05a317ea23f92cda11/log/job-output.txt#58061\n\nopenstack --os-compute-api-version 2.37 server create --image 444ab3b1-611b-4415-ad84-25de017ba38d --flavor 1 --nic none --property cool\u003dtrue --property location\u003dfridge --wait metadata-items-test2\n\n2022-03-04 05:19:01.022773 | controller | Project None is over a limit for [Resource server_metadata_items is over limit of 1 due to current usage 0 and delta 2] (HTTP 403) (Request-ID: req-9afd0dda-6d6b-4a10-a058-159bea68ee8e)\n\nand it succeed when the limit is raised\n\nhttps://zuul.opendev.org/t/openstack/build/9d54879682164f05a317ea23f92cda11/log/job-output.txt#58079","commit_id":"9ec6afe8939d6f3d72317b1cc8e0f4273bf31e43"}],"gate/post_test_hook.sh":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"030b06eaaf880e9a7927e91ecce96bee9f78c333","unresolved":true,"context_lines":[{"line_number":285,"context_line":"# Requires Bash 4."},{"line_number":286,"context_line":"declare -A id_name_map"},{"line_number":287,"context_line":"while read id name"},{"line_number":288,"context_line":"    do id_name_map[\"$id\"]\u003d\"$name\""},{"line_number":289,"context_line":"done \u003c\u003c\u003c \"$reglimit_ids_names\""},{"line_number":290,"context_line":""},{"line_number":291,"context_line":"# Server metadata items"}],"source_content_type":"text/x-sh","patch_set":8,"id":"94cf0c43_af66e49c","line":288,"range":{"start_line":288,"start_character":19,"end_line":288,"end_character":33},"updated":"2021-09-17 06:57:46.000000000","message":"Argh this is backwards","commit_id":"be88f5661a8c0f527ebe525183a19c874182eca0"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"ed4df8bf8f8ea2ee306551518fbf265ca8549f16","unresolved":false,"context_lines":[{"line_number":285,"context_line":"# Requires Bash 4."},{"line_number":286,"context_line":"declare -A id_name_map"},{"line_number":287,"context_line":"while read id name"},{"line_number":288,"context_line":"    do id_name_map[\"$id\"]\u003d\"$name\""},{"line_number":289,"context_line":"done \u003c\u003c\u003c \"$reglimit_ids_names\""},{"line_number":290,"context_line":""},{"line_number":291,"context_line":"# Server metadata items"}],"source_content_type":"text/x-sh","patch_set":8,"id":"650354c7_b769082b","line":288,"range":{"start_line":288,"start_character":19,"end_line":288,"end_character":33},"in_reply_to":"94cf0c43_af66e49c","updated":"2022-02-09 16:39:30.000000000","message":"Done","commit_id":"be88f5661a8c0f527ebe525183a19c874182eca0"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"3778f71a5355ee11bfc276aad273926983b12cde","unresolved":true,"context_lines":[{"line_number":292,"context_line":"#"},{"line_number":293,"context_line":"# Set the quota to 1."},{"line_number":294,"context_line":"metadata_items_id\u003d\"${id_name_map[\"server_metadata_items\"]}\""},{"line_number":295,"context_line":"openstack --os-cloud devstack-system-admin registered limit set \\"},{"line_number":296,"context_line":"    --service nova --default-limit 1 --resource-name server_metadata_items \\"},{"line_number":297,"context_line":"    \"$metadata_items_id\""},{"line_number":298,"context_line":""}],"source_content_type":"text/x-sh","patch_set":10,"id":"d0382736_823d4f20","line":295,"range":{"start_line":295,"start_character":10,"end_line":295,"end_character":42},"updated":"2021-09-17 23:10:07.000000000","message":"This didn\u0027t work ... probably don\u0027t have the clouds.yaml or something?","commit_id":"fb476f8f61c033e6c7aadac1d05e0b8216d32e5b"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"ed4df8bf8f8ea2ee306551518fbf265ca8549f16","unresolved":false,"context_lines":[{"line_number":292,"context_line":"#"},{"line_number":293,"context_line":"# Set the quota to 1."},{"line_number":294,"context_line":"metadata_items_id\u003d\"${id_name_map[\"server_metadata_items\"]}\""},{"line_number":295,"context_line":"openstack --os-cloud devstack-system-admin registered limit set \\"},{"line_number":296,"context_line":"    --service nova --default-limit 1 --resource-name server_metadata_items \\"},{"line_number":297,"context_line":"    \"$metadata_items_id\""},{"line_number":298,"context_line":""}],"source_content_type":"text/x-sh","patch_set":10,"id":"ece0444b_7c2c9921","line":295,"range":{"start_line":295,"start_character":10,"end_line":295,"end_character":42},"in_reply_to":"08b8486a_68f50d2d","updated":"2022-02-09 16:39:30.000000000","message":"Done","commit_id":"fb476f8f61c033e6c7aadac1d05e0b8216d32e5b"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"3740ccb4a693d0f51dafdff984637f012fd553db","unresolved":true,"context_lines":[{"line_number":292,"context_line":"#"},{"line_number":293,"context_line":"# Set the quota to 1."},{"line_number":294,"context_line":"metadata_items_id\u003d\"${id_name_map[\"server_metadata_items\"]}\""},{"line_number":295,"context_line":"openstack --os-cloud devstack-system-admin registered limit set \\"},{"line_number":296,"context_line":"    --service nova --default-limit 1 --resource-name server_metadata_items \\"},{"line_number":297,"context_line":"    \"$metadata_items_id\""},{"line_number":298,"context_line":""}],"source_content_type":"text/x-sh","patch_set":10,"id":"08b8486a_68f50d2d","line":295,"range":{"start_line":295,"start_character":10,"end_line":295,"end_character":42},"in_reply_to":"d0382736_823d4f20","updated":"2021-09-17 23:37:41.000000000","message":"I think it might be because I left out --region","commit_id":"fb476f8f61c033e6c7aadac1d05e0b8216d32e5b"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"e9f0be06bbcfe40f4918487c011352d078840dfc","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":30,"id":"49dc1e04_2c7b7f42","line":337,"updated":"2022-02-24 11:55:37.000000000","message":"Do we actually need this CLI based testing even after we have the tempest tests added in https://review.opendev.org/c/openstack/tempest/+/804311 ?","commit_id":"675c6c6e0c2aba5e789a6534310e3437847698c1"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"339c290ae6ad3d411411f35b4fdf1d2712ce2cb2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":30,"id":"f9369072_032849d7","line":337,"in_reply_to":"2ec56eec_9c99f103","updated":"2022-02-24 16:00:48.000000000","message":"Ahh I missed the fact that it is different coverage. Then I think we should keep both.","commit_id":"675c6c6e0c2aba5e789a6534310e3437847698c1"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"57191b2b64f4bb4f53b2f756d46ed4ceed9647dd","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":30,"id":"2ec56eec_9c99f103","line":337,"in_reply_to":"49dc1e04_2c7b7f42","updated":"2022-02-24 15:58:28.000000000","message":"It could be debated. I put it here in case reviewers might want all (global scoped and project scoped) unified limits explicitly tested. The ones in tempest test project scoped only and this part is global only. I\u0027m happy to remove this part if reviewers think the tempest tests are sufficient and this is not needed.","commit_id":"675c6c6e0c2aba5e789a6534310e3437847698c1"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"b9c2166096225e98066e26ec8e559343c56c4a90","unresolved":true,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"# We need to get the admin credentials to run the OSC CLIs for Placement."},{"line_number":62,"context_line":"set +x"},{"line_number":63,"context_line":"source $BASE/devstack/openrc admin"},{"line_number":64,"context_line":"set -x"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"# Verify whether instances were archived from all cells. Admin credentials are"}],"source_content_type":"text/x-sh","patch_set":31,"id":"1df565be_3b081ca0","line":63,"updated":"2022-03-03 06:51:49.000000000","message":"You may want to switch to using clouds.yaml by instead doing\n\nexport OS_CLOUD\u003ddevstack-admin\n\nhere, then you can get rid of all the \"unset OS_*\" wrappers below. The plan is for devstack to deprecate the openrc file, too, so you could stay ahead of that.","commit_id":"cd0b03c8447242f15af26fe6379823c56005231d"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"f2871f7f1684853f9994193c385484451900c8a6","unresolved":false,"context_lines":[{"line_number":60,"context_line":""},{"line_number":61,"context_line":"# We need to get the admin credentials to run the OSC CLIs for Placement."},{"line_number":62,"context_line":"set +x"},{"line_number":63,"context_line":"source $BASE/devstack/openrc admin"},{"line_number":64,"context_line":"set -x"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"# Verify whether instances were archived from all cells. Admin credentials are"}],"source_content_type":"text/x-sh","patch_set":31,"id":"54a3db25_96c86529","line":63,"in_reply_to":"1df565be_3b081ca0","updated":"2022-03-04 03:46:16.000000000","message":"Done","commit_id":"cd0b03c8447242f15af26fe6379823c56005231d"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"e3494eb77be968bf3192d39edb81f56d75f28820","unresolved":true,"context_lines":[{"line_number":307,"context_line":"set +e"},{"line_number":308,"context_line":"output\u003d$(openstack --os-compute-api-version 2.37 \\"},{"line_number":309,"context_line":"    server create --image ${image_id} --flavor ${flavor_id} --nic none \\"},{"line_number":310,"context_line":"    --property cool\u003dtrue --property location\u003dfridge \\"},{"line_number":311,"context_line":"    --wait metadata-items-test2)"},{"line_number":312,"context_line":"rc\u003d$?"},{"line_number":313,"context_line":"set -e"}],"source_content_type":"text/x-sh","patch_set":31,"id":"b9b65eb0_a239b485","line":310,"range":{"start_line":310,"start_character":36,"end_line":310,"end_character":51},"updated":"2022-02-28 12:03:11.000000000","message":":)","commit_id":"cd0b03c8447242f15af26fe6379823c56005231d"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"b9c2166096225e98066e26ec8e559343c56c4a90","unresolved":true,"context_lines":[{"line_number":307,"context_line":"set +e"},{"line_number":308,"context_line":"output\u003d$(openstack --os-compute-api-version 2.37 \\"},{"line_number":309,"context_line":"    server create --image ${image_id} --flavor ${flavor_id} --nic none \\"},{"line_number":310,"context_line":"    --property cool\u003dtrue --property location\u003dfridge \\"},{"line_number":311,"context_line":"    --wait metadata-items-test2)"},{"line_number":312,"context_line":"rc\u003d$?"},{"line_number":313,"context_line":"set -e"}],"source_content_type":"text/x-sh","patch_set":31,"id":"373be138_d8e3452c","line":310,"range":{"start_line":310,"start_character":36,"end_line":310,"end_character":51},"in_reply_to":"b9b65eb0_a239b485","updated":"2022-03-03 06:51:49.000000000","message":"Certainly makes it easy to find the matching location in the logs :)","commit_id":"cd0b03c8447242f15af26fe6379823c56005231d"}]}