)]}'
{"releasenotes/notes/Remove-options-prefixed-by-admin_-from-tempest.conf-f038f679b40daaea.yaml":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"8e7a0aaaaec4f1d3f705ec79da4ff4866e32dfa5","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The following options are moved from identity section:"},{"line_number":5,"context_line":"      - admin_username"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5f7c97a3_af996e1f","line":2,"updated":"2018-06-12 10:08:42.000000000","message":"rechecking the reno documentation, it seems that this may be fit for the \"upgrade\" section (and a previous change could be fixed too, but not in this patch :)\n\nSee here:\nhttps://docs.openstack.org/reno/latest/user/usage.html#editing-a-release-note\n\nIn the entry for deprecations, it says:\n\"Deprecations should not be used for something that is removed in the release, use upgrade section instead. \"","commit_id":"4912283b0344c9bfe96b9ae47fa2b7b343fcd9b0"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"dc72abdaf185ce52a46743fdaea07dd3eccd04e0","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    The following options are moved from identity section:"},{"line_number":5,"context_line":"      - admin_username"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5f7c97a3_45043b14","line":2,"in_reply_to":"5f7c97a3_af996e1f","updated":"2018-06-12 11:23:27.000000000","message":"I know \u0027deprecations\u0027 section is not appropriate here, therefor I chose \u0027other\u0027 section. I was thinking about \u0027upgrade\u0027 section, but the description wasn\u0027t clear to me. If it really matters so much, I can change it to upgrade.","commit_id":"4912283b0344c9bfe96b9ae47fa2b7b343fcd9b0"}],"roles/generate-tempestconf-file-cloud/tasks/main.yaml":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"8e7a0aaaaec4f1d3f705ec79da4ff4866e32dfa5","unresolved":false,"context_lines":[{"line_number":33,"context_line":"          --out etc/cloud_tempest.conf \\"},{"line_number":34,"context_line":"          --debug \\"},{"line_number":35,"context_line":"          -v \\"},{"line_number":36,"context_line":"          {% if cloud_user \u003d\u003d \"devstack-admin\" %}"},{"line_number":37,"context_line":"          --create \\"},{"line_number":38,"context_line":"          {% else %}"},{"line_number":39,"context_line":"          --non-admin \\"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5f7c97a3_efdc46e2","line":36,"updated":"2018-06-12 10:08:42.000000000","message":"is this change unrelated?","commit_id":"4912283b0344c9bfe96b9ae47fa2b7b343fcd9b0"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"dc72abdaf185ce52a46743fdaea07dd3eccd04e0","unresolved":false,"context_lines":[{"line_number":33,"context_line":"          --out etc/cloud_tempest.conf \\"},{"line_number":34,"context_line":"          --debug \\"},{"line_number":35,"context_line":"          -v \\"},{"line_number":36,"context_line":"          {% if cloud_user \u003d\u003d \"devstack-admin\" %}"},{"line_number":37,"context_line":"          --create \\"},{"line_number":38,"context_line":"          {% else %}"},{"line_number":39,"context_line":"          --non-admin \\"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"5f7c97a3_e5e8e7df","line":36,"in_reply_to":"5f7c97a3_efdc46e2","updated":"2018-06-12 11:23:27.000000000","message":"Yes, it kind of is. We had a bug here. We always used --create option, even with demo credentials. When --non-admin  is NOT used, a request for list projects is sent. For that request admin_project_name in auth section needs to be admin related. We have a default value there [1]. So before, this default value was accidentally used.\nNow, after this change, credentials from clouds.yaml are going straight to auth section (when --non-admin is NOT used), which means the default value was overridden by demo credentials and it led to an exception [2].\n\n[1] https://github.com/openstack/python-tempestconf/blob/283de2057402ebe9a5c2eabef9e550359b40e05f/config_tempest/main.py#L92\n\n[2] http://logs.openstack.org/22/573022/2/check/python-tempestconf-tempest-devstack-demo/8d65918/job-output.txt.gz#_2018-06-07_09_34_36_598439","commit_id":"4912283b0344c9bfe96b9ae47fa2b7b343fcd9b0"}]}