)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"a23426aef463316d394cc3c587cb336a76b152e5","unresolved":true,"context_lines":[{"line_number":10,"context_line":"present for credentials to access the cloud, but this is"},{"line_number":11,"context_line":"wrong in cases where we\u0027re trying to setup system"},{"line_number":12,"context_line":"scoped services and need to remove the environment variables"},{"line_number":13,"context_line":"that was being used."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Instead, change the plugin to use the os-cloud entry definitions,"},{"line_number":16,"context_line":"and specifically in this case devstack-admin which makes sense"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"e1f8f117_a75a0973","line":13,"updated":"2021-03-13 08:19:57.000000000","message":"It would be interesting to see whether we can indeed get rid of https://opendev.org/openstack/devstack/src/branch/master/stack.sh#L1049-L1074 completely and switch everything after keystone-bootstrap to using cloud config exclusively.\n\nWith the current approach, I\u0027m a bit sceptical as to whether just specifying --os-cloud will be enough or whether the environment variables that are still being set might interfere, in particular once one starts to use anything other than the devstack-admin cloud, which should be 1:1 identical to what is in the environment.","commit_id":"0f23a719e2a40a093089122427062b838377ce0a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"6ab25af93ba45a4011433e60d652a1e43a50cf44","unresolved":true,"context_lines":[{"line_number":10,"context_line":"present for credentials to access the cloud, but this is"},{"line_number":11,"context_line":"wrong in cases where we\u0027re trying to setup system"},{"line_number":12,"context_line":"scoped services and need to remove the environment variables"},{"line_number":13,"context_line":"that was being used."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Instead, change the plugin to use the os-cloud entry definitions,"},{"line_number":16,"context_line":"and specifically in this case devstack-admin which makes sense"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"64ba3b66_cc34d0dd","line":13,"in_reply_to":"e1f8f117_a75a0973","updated":"2021-06-15 18:42:46.000000000","message":"I see that as a really breaking change, but perhaps it is time to head down that path....\n\nI suspect its removal could break any project not actively heading down the path of setting up a integrated RBAC policy job using system scoped authentication.","commit_id":"0f23a719e2a40a093089122427062b838377ce0a"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"82c549ca10a74b44332fe08429f16910a4deb7ce","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Julia Kreger \u003cjuliaashleykreger@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-06-15 11:54:06 -0700"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use specific credentials for tempest"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Tempest expects the classic environment variables to be"},{"line_number":10,"context_line":"present for credentials to access the cloud, but this is"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"83f2aacc_3ce836f1","line":7,"range":{"start_line":7,"start_character":29,"end_line":7,"end_character":36},"updated":"2021-06-16 09:52:13.000000000","message":"pre-tempest setup? tempest plugin?\n\nIt\u0027s not for tempest itself.","commit_id":"a542a9d59d4f30f8a5638f16850893be38c2c310"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"82c549ca10a74b44332fe08429f16910a4deb7ce","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use specific credentials for tempest"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Tempest expects the classic environment variables to be"},{"line_number":10,"context_line":"present for credentials to access the cloud, but this is"},{"line_number":11,"context_line":"wrong in cases where we\u0027re trying to setup system"},{"line_number":12,"context_line":"scoped services and need to remove the environment variables"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"be8aa9e3_eb4f6e39","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":8},"updated":"2021-06-16 09:52:13.000000000","message":"Tempest plugin?","commit_id":"a542a9d59d4f30f8a5638f16850893be38c2c310"}],"lib/tempest":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"310dc3747a23e5e830c3f8929c806b80cfb6e8e0","unresolved":true,"context_lines":[{"line_number":214,"context_line":"    if is_service_enabled nova; then"},{"line_number":215,"context_line":"        # If ``DEFAULT_INSTANCE_TYPE`` is not declared, use the new behavior"},{"line_number":216,"context_line":"        # Tempest creates its own instance types"},{"line_number":217,"context_line":"        available_flavors\u003d$(nova flavor-list)"},{"line_number":218,"context_line":"        if  [[ -z \"$DEFAULT_INSTANCE_TYPE\" ]]; then"},{"line_number":219,"context_line":"            if [[ ! ( $available_flavors \u003d~ \u0027m1.nano\u0027 ) ]]; then"},{"line_number":220,"context_line":"                # Determine the flavor disk size based on the image size."}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"56f4c50a_569dc1a3","line":217,"range":{"start_line":217,"start_character":28,"end_line":217,"end_character":44},"updated":"2021-03-13 14:59:57.000000000","message":"This needs to be converted to OSC in order to be able to use cloud config.","commit_id":"0f23a719e2a40a093089122427062b838377ce0a"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"6b4e9570ca14964ab0ac526a1d4621084e6d1f84","unresolved":true,"context_lines":[{"line_number":214,"context_line":"    if is_service_enabled nova; then"},{"line_number":215,"context_line":"        # If ``DEFAULT_INSTANCE_TYPE`` is not declared, use the new behavior"},{"line_number":216,"context_line":"        # Tempest creates its own instance types"},{"line_number":217,"context_line":"        available_flavors\u003d$(nova flavor-list)"},{"line_number":218,"context_line":"        if  [[ -z \"$DEFAULT_INSTANCE_TYPE\" ]]; then"},{"line_number":219,"context_line":"            if [[ ! ( $available_flavors \u003d~ \u0027m1.nano\u0027 ) ]]; then"},{"line_number":220,"context_line":"                # Determine the flavor disk size based on the image size."}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"3072f6fd_d27053df","line":217,"range":{"start_line":217,"start_character":28,"end_line":217,"end_character":44},"in_reply_to":"56f4c50a_569dc1a3","updated":"2021-03-13 16:32:51.000000000","message":"++ yes. This is actually where my test CI job failed after I went off to cook dinner last night.","commit_id":"0f23a719e2a40a093089122427062b838377ce0a"}]}