)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"change_message_id":"99cfb356c80703f9313a89a1b2cae1da3f7e789e","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The following issued existed"},{"line_number":10,"context_line":"1) tempest.auth.admin_domain_scope should be in"},{"line_number":11,"context_line":"tempest.identity.admin_domain_scope"},{"line_number":12,"context_line":"default is false so it is needed"},{"line_number":13,"context_line":"https://github.com/openstack/tempest/blob/master/tempest/config.py#L109"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"2) tempest.auth.min_compute_nodes should be in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_502486b2","line":12,"range":{"start_line":12,"start_character":16,"end_line":12,"end_character":32},"updated":"2018-09-25 14:58:20.000000000","message":"details as to why it is needed would be helpful.\n\nWe are setting admin_domain_scope to true so that we can perform admin operations with the default domain as all users are provisioned on the LDAP configured domain.","commit_id":"cf612253210a554fc64bc12c6da5e34bac1f2a8d"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"f651be3796d4efeab7d2f5688c364d2bf92cf8b5","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The following issued existed"},{"line_number":10,"context_line":"1) tempest.auth.admin_domain_scope should be in"},{"line_number":11,"context_line":"tempest.identity.admin_domain_scope"},{"line_number":12,"context_line":"default is false so it is needed"},{"line_number":13,"context_line":"https://github.com/openstack/tempest/blob/master/tempest/config.py#L109"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"2) tempest.auth.min_compute_nodes should be in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_7038a29e","line":12,"range":{"start_line":12,"start_character":16,"end_line":12,"end_character":32},"in_reply_to":"3f79a3b5_502486b2","updated":"2018-09-25 15:04:51.000000000","message":"Is there any reason this can\u0027t be handled via overrides and needs to be the default?","commit_id":"cf612253210a554fc64bc12c6da5e34bac1f2a8d"},{"author":{"_account_id":26815,"name":"Kaustuv Royburman","email":"kaustuv.royburman@ericsson.com","username":"rbkaustuv"},"change_message_id":"6cbe645a8f1b67db80c2e2356d3de03b6ebf99fa","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The following issued existed"},{"line_number":10,"context_line":"1) tempest.auth.admin_domain_scope should be in"},{"line_number":11,"context_line":"tempest.identity.admin_domain_scope"},{"line_number":12,"context_line":"default is false so it is needed"},{"line_number":13,"context_line":"https://github.com/openstack/tempest/blob/master/tempest/config.py#L109"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"2) tempest.auth.min_compute_nodes should be in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_90ec5ef5","line":12,"range":{"start_line":12,"start_character":16,"end_line":12,"end_character":32},"in_reply_to":"3f79a3b5_7038a29e","updated":"2018-09-25 15:10:03.000000000","message":"admin_domain_scope is part of the section identity and is not part of the section auth.\n\nhttps://docs.openstack.org/tempest/latest/sampleconf.html","commit_id":"cf612253210a554fc64bc12c6da5e34bac1f2a8d"},{"author":{"_account_id":20378,"name":"Trevor McCasland","email":"TM2086@att.com","username":"twm2016"},"change_message_id":"99cfb356c80703f9313a89a1b2cae1da3f7e789e","unresolved":false,"context_lines":[{"line_number":13,"context_line":"https://github.com/openstack/tempest/blob/master/tempest/config.py#L109"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"2) tempest.auth.min_compute_nodes should be in"},{"line_number":16,"context_line":"tempest.compute min_compute_nodes but it is setting the default so"},{"line_number":17,"context_line":"we are dropping it"},{"line_number":18,"context_line":"https://github.com/openstack/tempest/blob/14.0.0/tempest/config.py#L231"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_1005ae10","line":16,"range":{"start_line":16,"start_character":41,"end_line":16,"end_character":63},"updated":"2018-09-25 14:58:20.000000000","message":"++","commit_id":"cf612253210a554fc64bc12c6da5e34bac1f2a8d"}],"tempest/templates/configmap-etc.yaml":[{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"7986fcb6eaf70d42970a5a7ae1ba5f86b2749acd","unresolved":false,"context_lines":[{"line_number":30,"context_line":"{{- $_ :\u003d set .Values.conf.tempest.auth \"admin_domain_name\" .Values.endpoints.identity.auth.admin.user_domain_name -}}"},{"line_number":31,"context_line":"{{- end -}}"},{"line_number":32,"context_line":"{{- if empty .Values.conf.tempest.identity.admin_domain_scope -}}"},{"line_number":33,"context_line":"{{- $_ :\u003d set .Values.conf.tempest.identity \"admin_domain_scope\" .Values.endpoints.identity.auth.admin.user_domain_name -}}"},{"line_number":34,"context_line":"{{- end -}}"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"{{- if empty .Values.conf.tempest.identity.uri_v3 -}}"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3f79a3b5_c758d758","line":33,"range":{"start_line":33,"start_character":103,"end_line":33,"end_character":119},"updated":"2018-09-25 21:01:56.000000000","message":"we should be fixing this to \"user_domain_scope\" instead of keying it off \"admin_domain_name\", but we can fix that in follow up patch.","commit_id":"636cec440a1a014ba4879ebbc4edad37edc4c38c"}],"tempest/values.yaml":[{"author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"change_message_id":"1bba1527960628cb6f347bd781219e5271ed79bd","unresolved":false,"context_lines":[{"line_number":125,"context_line":"      tempest_roles: admin, member"},{"line_number":126,"context_line":"    identity:"},{"line_number":127,"context_line":"      auth_version: v3"},{"line_number":128,"context_line":"      admin_domain_scope: true"},{"line_number":129,"context_line":"    identity-feature-enabled:"},{"line_number":130,"context_line":"      api_v2: false"},{"line_number":131,"context_line":"      api_v3: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_0d103b59","line":128,"updated":"2018-09-25 14:56:21.000000000","message":"Need to update usage here: https://git.openstack.org/cgit/openstack/openstack-helm/tree/tempest/templates/configmap-etc.yaml?h\u003drefs/heads/master#n32","commit_id":"cf612253210a554fc64bc12c6da5e34bac1f2a8d"},{"author":{"_account_id":17887,"name":"Doug Schveninger","email":"ds6901@att.com","username":"Doug.Schveninger"},"change_message_id":"90ac5cc98edeeace6e42f15eb962852fdbfa289a","unresolved":false,"context_lines":[{"line_number":125,"context_line":"      tempest_roles: admin, member"},{"line_number":126,"context_line":"    identity:"},{"line_number":127,"context_line":"      auth_version: v3"},{"line_number":128,"context_line":"      admin_domain_scope: true"},{"line_number":129,"context_line":"    identity-feature-enabled:"},{"line_number":130,"context_line":"      api_v2: false"},{"line_number":131,"context_line":"      api_v3: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_d4c96351","line":128,"in_reply_to":"3f79a3b5_04057010","updated":"2018-09-25 20:13:56.000000000","message":"yes this is the same point from dustin and I will update line 32 and 33.  In a future commit I will get better support for configmap-etc and make the Values.yaml file dryer.  One change at a time :)","commit_id":"cf612253210a554fc64bc12c6da5e34bac1f2a8d"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"679b4b434ab53590b5c5fed00fb32d4019e6d46b","unresolved":false,"context_lines":[{"line_number":125,"context_line":"      tempest_roles: admin, member"},{"line_number":126,"context_line":"    identity:"},{"line_number":127,"context_line":"      auth_version: v3"},{"line_number":128,"context_line":"      admin_domain_scope: true"},{"line_number":129,"context_line":"    identity-feature-enabled:"},{"line_number":130,"context_line":"      api_v2: false"},{"line_number":131,"context_line":"      api_v3: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_04057010","line":128,"in_reply_to":"3f79a3b5_0d103b59","updated":"2018-09-25 17:26:50.000000000","message":"I wonder now if there is a bug here: [0] where the name should be .Values.endpoints.identity.auth.admin.user_domain_scope instead of .Values.endpoints.identity.auth.admin.user_domain_name.\n\n[0] https://github.com/openstack/openstack-helm/blob/8e4ee070e65d54e2e5aa23b03febfac281278bf3/tempest/templates/configmap-etc.yaml#L33","commit_id":"cf612253210a554fc64bc12c6da5e34bac1f2a8d"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"8dae4c0ee8e83f0883117f165dcba0b66728a86b","unresolved":false,"context_lines":[{"line_number":125,"context_line":"      tempest_roles: admin, member"},{"line_number":126,"context_line":"    identity:"},{"line_number":127,"context_line":"      auth_version: v3"},{"line_number":128,"context_line":"      admin_domain_scope: true"},{"line_number":129,"context_line":"    identity-feature-enabled:"},{"line_number":130,"context_line":"      api_v2: false"},{"line_number":131,"context_line":"      api_v3: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_3058ea2f","line":128,"in_reply_to":"3f79a3b5_0d103b59","updated":"2018-09-25 14:59:38.000000000","message":"The configuration values you\u0027re trying to change to here are based off master. Kolla\u0027s 3.0.3 tag (the default image) maps to Tempest 14.0.0. For reference: https://github.com/openstack/tempest/blob/14.0.0/tempest/config.py#L47","commit_id":"cf612253210a554fc64bc12c6da5e34bac1f2a8d"},{"author":{"_account_id":17591,"name":"Steve Wilkerson","email":"wilkers.steve@gmail.com","username":"srwilkers"},"change_message_id":"e77b767d7caedba348f35a1b0ff38621f9af7218","unresolved":false,"context_lines":[{"line_number":125,"context_line":"      tempest_roles: admin, member"},{"line_number":126,"context_line":"    identity:"},{"line_number":127,"context_line":"      auth_version: v3"},{"line_number":128,"context_line":"      admin_domain_scope: true"},{"line_number":129,"context_line":"    identity-feature-enabled:"},{"line_number":130,"context_line":"      api_v2: false"},{"line_number":131,"context_line":"      api_v3: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_30f46af1","line":128,"in_reply_to":"3f79a3b5_3058ea2f","updated":"2018-09-25 15:02:22.000000000","message":"Disregard","commit_id":"cf612253210a554fc64bc12c6da5e34bac1f2a8d"},{"author":{"_account_id":17887,"name":"Doug Schveninger","email":"ds6901@att.com","username":"Doug.Schveninger"},"change_message_id":"90ac5cc98edeeace6e42f15eb962852fdbfa289a","unresolved":false,"context_lines":[{"line_number":125,"context_line":"      tempest_roles: admin, member"},{"line_number":126,"context_line":"    identity:"},{"line_number":127,"context_line":"      auth_version: v3"},{"line_number":128,"context_line":"      admin_domain_scope: true"},{"line_number":129,"context_line":"    identity-feature-enabled:"},{"line_number":130,"context_line":"      api_v2: false"},{"line_number":131,"context_line":"      api_v3: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_502a462f","line":128,"in_reply_to":"3f79a3b5_3058ea2f","updated":"2018-09-25 20:13:56.000000000","message":"Done","commit_id":"cf612253210a554fc64bc12c6da5e34bac1f2a8d"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"3101c8c4036508d886e31264697b71483df34b63","unresolved":false,"context_lines":[{"line_number":124,"context_line":"      use_dynamic_credentials: true"},{"line_number":125,"context_line":"      admin_domain_scope: true"},{"line_number":126,"context_line":"      tempest_roles: admin, member"},{"line_number":127,"context_line":"      min_compute_nodes: 1"},{"line_number":128,"context_line":"    identity:"},{"line_number":129,"context_line":"      auth_version: v3"},{"line_number":130,"context_line":"    identity-feature-enabled:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3f79a3b5_878bbfbe","side":"PARENT","line":127,"range":{"start_line":127,"start_character":6,"end_line":127,"end_character":23},"updated":"2018-09-25 20:59:04.000000000","message":"I think the compute tests are disabled anyway (L189). Doing a grep through Tempest this only appears for compute/scenario tests so I agree this can be removed, as it\u0027s in the wrong spot anyway.","commit_id":"8e4ee070e65d54e2e5aa23b03febfac281278bf3"},{"author":{"_account_id":23186,"name":"Felipe Monteiro","email":"felipe.carneiro.monteiro@gmail.com","username":"felipe.monteiro"},"change_message_id":"3101c8c4036508d886e31264697b71483df34b63","unresolved":false,"context_lines":[{"line_number":125,"context_line":"      tempest_roles: admin, member"},{"line_number":126,"context_line":"    identity:"},{"line_number":127,"context_line":"      auth_version: v3"},{"line_number":128,"context_line":"      admin_domain_scope: true"},{"line_number":129,"context_line":"    identity-feature-enabled:"},{"line_number":130,"context_line":"      api_v2: false"},{"line_number":131,"context_line":"      api_v3: true"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"3f79a3b5_87489fb3","line":128,"range":{"start_line":128,"start_character":6,"end_line":128,"end_character":24},"updated":"2018-09-25 20:59:04.000000000","message":"This is indeed found under [identity] group in tempest.conf.","commit_id":"636cec440a1a014ba4879ebbc4edad37edc4c38c"}]}