)]}'
{"gate/post_test_hook.sh":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"42dfd801423f570baa1e6ccaf30187c5c5513766","unresolved":false,"context_lines":[{"line_number":178,"context_line":"allocations\u003d$(openstack resource provider allocation show ${server_id} \\"},{"line_number":179,"context_line":"              -c resources -f value)"},{"line_number":180,"context_line":"if [[ \"$allocations\" \u003d\u003d \"\" ]]; then"},{"line_number":181,"context_line":"    echo \"No allocations found for the server.\""},{"line_number":182,"context_line":"    exit 2"},{"line_number":183,"context_line":"fi"},{"line_number":184,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"3fa7e38b_8b975f9d","line":181,"updated":"2019-11-06 23:22:07.000000000","message":"Yup this is failing for the non-admin user.\n\n+ /opt/stack/nova/gate/post_test_hook.sh:main:164 :   echo \u0027Creating non-admin server for heal_allocations testing\u0027\nCreating non-admin server for heal_allocations testing\n+ /opt/stack/nova/gate/post_test_hook.sh:main:166 :   openstack --os-compute-api-version 2.72 server create --image c055fbd6-7334-4232-a5b7-925c90267c2c --flavor 1 --nic net-id\u003d085dc1c2-c59f-4e99-a1b5-b15f56363bea --nic port-id\u003dport-normal-qos --wait heal-allocations-test\nError creating server: heal-allocations-test\nError creating server\n++ /opt/stack/nova/gate/post_test_hook.sh:main:170 :   openstack server show heal-allocations-test -f value -c id\n+ /opt/stack/nova/gate/post_test_hook.sh:main:170 :   server_id\u003d075a5bc0-1774-423a-b9ae-902eb498ae89\n+ /opt/stack/nova/gate/post_test_hook.sh:main:173 :   set +x\nWARNING: setting legacy OS_TENANT_NAME to support cli tools.\n++ /opt/stack/nova/gate/post_test_hook.sh:main:179 :   openstack resource provider allocation show 075a5bc0-1774-423a-b9ae-902eb498ae89 -c resources -f value\n+ /opt/stack/nova/gate/post_test_hook.sh:main:179 :   allocations\u003d\n+ /opt/stack/nova/gate/post_test_hook.sh:main:180 :   [[ \u0027\u0027 \u003d\u003d \u0027\u0027 ]]\n+ /opt/stack/nova/gate/post_test_hook.sh:main:181 :   echo \u0027No allocations found for the server.\u0027\nNo allocations found for the server.\n+ /opt/stack/nova/gate/post_test_hook.sh:main:182 :   exit 2","commit_id":"50fe21bd9e10c3e76c380aed4bb2f395cc9e61e6"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"9e69610f6c734705c5eb2868e2be0ff334059bb8","unresolved":false,"context_lines":[{"line_number":154,"context_line":""},{"line_number":155,"context_line":"image_id\u003d$(openstack image list -f value -c ID | awk \u0027NR\u003d\u003d1{print $1}\u0027)"},{"line_number":156,"context_line":"flavor_id\u003d$(openstack flavor list -f value -c ID | awk \u0027NR\u003d\u003d1{print $1}\u0027)"},{"line_number":157,"context_line":"network_id\u003d$(openstack network list --no-share -f value -c ID | awk \u0027NR\u003d\u003d1{print $1}\u0027)"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"# Switch to the demo user/project for the port resource request test (bug 1849657)."},{"line_number":160,"context_line":"set +x"},{"line_number":161,"context_line":"source $BASE/devstack/openrc demo demo"},{"line_number":162,"context_line":"set -x"},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"echo \"Creating non-admin server for heal_allocations testing\""},{"line_number":165,"context_line":"# microversion 2.72 introduced the support for bandwidth aware ports"}],"source_content_type":"text/x-sh","patch_set":2,"id":"3fa7e38b_83ca786b","line":162,"range":{"start_line":157,"start_character":0,"end_line":162,"end_character":6},"updated":"2019-12-05 15:32:03.000000000","message":"Yeah so this is a problem. We\u0027re getting the network_id as admin, and then switching to demo and trying to use that network_id. We should just hard-code the network to use to net0, right?","commit_id":"89e7022408813d2bbda397518cee5e5d2df92d35"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"8f141bbc31a0d04a11779ae9f3000f45364a1dbd","unresolved":false,"context_lines":[{"line_number":154,"context_line":""},{"line_number":155,"context_line":"image_id\u003d$(openstack image list -f value -c ID | awk \u0027NR\u003d\u003d1{print $1}\u0027)"},{"line_number":156,"context_line":"flavor_id\u003d$(openstack flavor list -f value -c ID | awk \u0027NR\u003d\u003d1{print $1}\u0027)"},{"line_number":157,"context_line":"network_id\u003d$(openstack network list --no-share -f value -c ID | awk \u0027NR\u003d\u003d1{print $1}\u0027)"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"# Switch to the demo user/project for the port resource request test (bug 1849657)."},{"line_number":160,"context_line":"set +x"},{"line_number":161,"context_line":"source $BASE/devstack/openrc demo demo"},{"line_number":162,"context_line":"set -x"},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"echo \"Creating non-admin server for heal_allocations testing\""},{"line_number":165,"context_line":"# microversion 2.72 introduced the support for bandwidth aware ports"}],"source_content_type":"text/x-sh","patch_set":2,"id":"3fa7e38b_e3ed6c8e","line":162,"range":{"start_line":157,"start_character":0,"end_line":162,"end_character":6},"in_reply_to":"3fa7e38b_83ca786b","updated":"2019-12-05 15:45:58.000000000","message":"im guessing the --no-share flag filters out shared network so\nthe but since its the admin user and porject the net id we get could be a would be a differnt network then the one we just created. \n\nso ya hard coding this to net0 shoudl work\n\nor we should change how we get the network id to use the demo project.\n\nbefore this was all done with the admin porject so it was not an issue.","commit_id":"89e7022408813d2bbda397518cee5e5d2df92d35"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f98c752dcc6443b2c364971cbff28c0ee7268a3c","unresolved":false,"context_lines":[{"line_number":165,"context_line":"# microversion 2.72 introduced the support for bandwidth aware ports"},{"line_number":166,"context_line":"openstack --os-compute-api-version 2.72 \\"},{"line_number":167,"context_line":"server create --image ${image_id} --flavor ${flavor_id} \\"},{"line_number":168,"context_line":"--nic net-id\u003d${network_id} --nic port-id\u003dport-normal-qos \\"},{"line_number":169,"context_line":"--wait heal-allocations-test"},{"line_number":170,"context_line":"server_id\u003d$(openstack server show heal-allocations-test -f value -c id)"},{"line_number":171,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"3fa7e38b_c3cf1026","line":168,"range":{"start_line":168,"start_character":0,"end_line":168,"end_character":57},"updated":"2019-12-05 15:45:37.000000000","message":"I missed that we\u0027re specifying both net-id and port-id here, so I guess the idea was to create a server with a qos port and a non-qos port, with the latter coming from a non-net0 tenant network.","commit_id":"89e7022408813d2bbda397518cee5e5d2df92d35"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"766119212ef0be9ea4242954fd6e560702221bb8","unresolved":false,"context_lines":[{"line_number":182,"context_line":"              -c resources -f value)"},{"line_number":183,"context_line":"if [[ \"$allocations\" \u003d\u003d \"\" ]]; then"},{"line_number":184,"context_line":"    echo \"No allocations found for the server.\""},{"line_number":185,"context_line":"    exit 2"},{"line_number":186,"context_line":"fi"},{"line_number":187,"context_line":""},{"line_number":188,"context_line":"# Make sure that the binding:profile.allocation key is updated"}],"source_content_type":"text/x-sh","patch_set":3,"id":"3fa7e38b_a59d3158","line":185,"updated":"2019-12-06 01:07:05.000000000","message":"Hit this this time:\n\n+ /opt/stack/nova/gate/post_test_hook.sh:main:167 :   echo \u0027Creating non-admin server for heal_allocations testing\u0027\nCreating non-admin server for heal_allocations testing\n+ /opt/stack/nova/gate/post_test_hook.sh:main:169 :   openstack --os-compute-api-version 2.72 server create --image a8c3dd79-5e75-4142-814f-7ec2712427bc --flavor 1 --nic net-id\u003d663ba550-4874-45de-8c34-ebf08baae349 --nic port-id\u003dport-normal-qos --wait heal-allocations-test\nError creating server: heal-allocations-test\nError creating server\n++ /opt/stack/nova/gate/post_test_hook.sh:main:173 :   openstack server show heal-allocations-test -f value -c id\n+ /opt/stack/nova/gate/post_test_hook.sh:main:173 :   server_id\u003d119aeaa8-90c0-4c8f-9470-2e07ad15d677\n+ /opt/stack/nova/gate/post_test_hook.sh:main:176 :   set +x\nWARNING: setting legacy OS_TENANT_NAME to support cli tools.\n++ /opt/stack/nova/gate/post_test_hook.sh:main:182 :   openstack resource provider allocation show 119aeaa8-90c0-4c8f-9470-2e07ad15d677 -c resources -f value\n+ /opt/stack/nova/gate/post_test_hook.sh:main:182 :   allocations\u003d\n+ /opt/stack/nova/gate/post_test_hook.sh:main:183 :   [[ \u0027\u0027 \u003d\u003d \u0027\u0027 ]]\n+ /opt/stack/nova/gate/post_test_hook.sh:main:184 :   echo \u0027No allocations found for the server.\u0027\nNo allocations found for the server.\n+ /opt/stack/nova/gate/post_test_hook.sh:main:185 :   exit 2","commit_id":"94001f001e2e5fbad6ab65589fa4875f33727441"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"7b7f4159a4490870a79d8d19131bf52d8cff3e30","unresolved":false,"context_lines":[{"line_number":122,"context_line":"    --provider-physical-network public \\"},{"line_number":123,"context_line":"    --provider-segment 100 \\"},{"line_number":124,"context_line":"    --project demo \\"},{"line_number":125,"context_line":"    --share"},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"openstack subnet create subnet0 \\"},{"line_number":128,"context_line":"    --network net0 \\"}],"source_content_type":"text/x-sh","patch_set":4,"id":"3fa7e38b_7cbfc4aa","line":125,"range":{"start_line":125,"start_character":4,"end_line":125,"end_character":11},"updated":"2019-12-06 21:07:53.000000000","message":"This will create a network that is shared between projects:\n\nhttps://docs.openstack.org/python-openstackclient/latest/cli/command-objects/network.html#cmdoption-openstack-network-create-share","commit_id":"b537f31faf27f2147c27a7a2b5e2043abf8e3c6e"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"7b7f4159a4490870a79d8d19131bf52d8cff3e30","unresolved":false,"context_lines":[{"line_number":163,"context_line":""},{"line_number":164,"context_line":"# Get the network after switching to the demo project since the networks"},{"line_number":165,"context_line":"# available will be different from the admin project."},{"line_number":166,"context_line":"network_id\u003d$(openstack network list --no-share -f value -c ID | awk \u0027NR\u003d\u003d1{print $1}\u0027)"},{"line_number":167,"context_line":""},{"line_number":168,"context_line":"echo \"Creating non-admin server for heal_allocations testing\""},{"line_number":169,"context_line":"# microversion 2.72 introduced the support for bandwidth aware ports"}],"source_content_type":"text/x-sh","patch_set":4,"id":"3fa7e38b_9cbc00b6","line":166,"range":{"start_line":166,"start_character":36,"end_line":166,"end_character":46},"updated":"2019-12-06 21:07:53.000000000","message":"But this will list only networks that are not shared between projects:\n\nhttps://docs.openstack.org/python-openstackclient/latest/cli/command-objects/network.html#cmdoption-openstack-network-list-no-share","commit_id":"b537f31faf27f2147c27a7a2b5e2043abf8e3c6e"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"7b7f4159a4490870a79d8d19131bf52d8cff3e30","unresolved":false,"context_lines":[{"line_number":169,"context_line":"# microversion 2.72 introduced the support for bandwidth aware ports"},{"line_number":170,"context_line":"openstack --os-compute-api-version 2.72 \\"},{"line_number":171,"context_line":"server create --image ${image_id} --flavor ${flavor_id} \\"},{"line_number":172,"context_line":"--nic net-id\u003d${network_id} --nic port-id\u003dport-normal-qos \\"},{"line_number":173,"context_line":"--wait heal-allocations-test"},{"line_number":174,"context_line":"server_id\u003d$(openstack server show heal-allocations-test -f value -c id)"},{"line_number":175,"context_line":""}],"source_content_type":"text/x-sh","patch_set":4,"id":"3fa7e38b_7ced84b6","line":172,"range":{"start_line":172,"start_character":13,"end_line":172,"end_character":26},"updated":"2019-12-06 21:07:53.000000000","message":"So I don\u0027t understand, how are we getting the correct network_id here for net0? According to the use of --share and --no-share commented above, this should be a network_id other than net0 ... yet the test verification works? I am confused.","commit_id":"b537f31faf27f2147c27a7a2b5e2043abf8e3c6e"}]}