)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"2568790f224fc56bdf47757908a732b28e4d1ece","unresolved":true,"context_lines":[{"line_number":23,"context_line":"With this values the user have 3 options:"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":" - For the first case (conf.federation.dex_idp.enabled: true),"},{"line_number":26,"context_line":"   the wsgi_keystone configuration will be modified to include"},{"line_number":27,"context_line":"   the OIDC settings, and the script defined in"},{"line_number":28,"context_line":"   conf.federation.dex_idp.bootstrap.script will be appended to"},{"line_number":29,"context_line":"   Keystone’s bootstrap script. If desired, the user can modify"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"f93f310b_f3873c2c","line":26,"range":{"start_line":26,"start_character":21,"end_line":26,"end_character":52},"updated":"2025-10-24 13:27:07.000000000","message":"nit: configuration will only be modified later when templates are updated 😄\n\nthis could be rephrased to: \"enables the option for the wsgi_keystone configuration to be modified to include...\"","commit_id":"747fcc8f33cd0f026eafa90f0f22458dcdaba030"},{"author":{"_account_id":37078,"name":"Daniel Marques Caires","display_name":"Daniel Caires","email":"DanielMarques.Caires@windriver.com","username":"dcaires"},"change_message_id":"611f60ba095f26aaa2d846f378de38ace6dfe8a9","unresolved":false,"context_lines":[{"line_number":23,"context_line":"With this values the user have 3 options:"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":" - For the first case (conf.federation.dex_idp.enabled: true),"},{"line_number":26,"context_line":"   the wsgi_keystone configuration will be modified to include"},{"line_number":27,"context_line":"   the OIDC settings, and the script defined in"},{"line_number":28,"context_line":"   conf.federation.dex_idp.bootstrap.script will be appended to"},{"line_number":29,"context_line":"   Keystone’s bootstrap script. If desired, the user can modify"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"783a41e6_fe2e7474","line":26,"range":{"start_line":26,"start_character":21,"end_line":26,"end_character":52},"in_reply_to":"f93f310b_f3873c2c","updated":"2025-10-24 13:55:49.000000000","message":"Done","commit_id":"747fcc8f33cd0f026eafa90f0f22458dcdaba030"},{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"2568790f224fc56bdf47757908a732b28e4d1ece","unresolved":true,"context_lines":[{"line_number":29,"context_line":"   Keystone’s bootstrap script. If desired, the user can modify"},{"line_number":30,"context_line":"   the DEX script commands as they wish."},{"line_number":31,"context_line":" - For the second case (conf.federation.dex_idp.bootstrap.enabled: false),"},{"line_number":32,"context_line":"   the wsgi_keystone configuration will be updated, but nothing"},{"line_number":33,"context_line":"   will be added to the bootstrap — in this situation, the user may"},{"line_number":34,"context_line":"   choose to manually create groups, projects, roles, etc."},{"line_number":35,"context_line":"   after applying the chart."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"25adaf1f_f413cb7d","line":32,"range":{"start_line":32,"start_character":35,"end_line":32,"end_character":50},"updated":"2025-10-24 13:27:07.000000000","message":"ditto","commit_id":"747fcc8f33cd0f026eafa90f0f22458dcdaba030"},{"author":{"_account_id":37078,"name":"Daniel Marques Caires","display_name":"Daniel Caires","email":"DanielMarques.Caires@windriver.com","username":"dcaires"},"change_message_id":"611f60ba095f26aaa2d846f378de38ace6dfe8a9","unresolved":false,"context_lines":[{"line_number":29,"context_line":"   Keystone’s bootstrap script. If desired, the user can modify"},{"line_number":30,"context_line":"   the DEX script commands as they wish."},{"line_number":31,"context_line":" - For the second case (conf.federation.dex_idp.bootstrap.enabled: false),"},{"line_number":32,"context_line":"   the wsgi_keystone configuration will be updated, but nothing"},{"line_number":33,"context_line":"   will be added to the bootstrap — in this situation, the user may"},{"line_number":34,"context_line":"   choose to manually create groups, projects, roles, etc."},{"line_number":35,"context_line":"   after applying the chart."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ae77b779_121b8146","line":32,"range":{"start_line":32,"start_character":35,"end_line":32,"end_character":50},"in_reply_to":"25adaf1f_f413cb7d","updated":"2025-10-24 13:55:49.000000000","message":"Done","commit_id":"747fcc8f33cd0f026eafa90f0f22458dcdaba030"},{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"2568790f224fc56bdf47757908a732b28e4d1ece","unresolved":true,"context_lines":[{"line_number":39,"context_line":"Test Plan:"},{"line_number":40,"context_line":"PASS - Build OSH package and STX-O tarball"},{"line_number":41,"context_line":"PASS - Keystone values contains the new parameters"},{"line_number":42,"context_line":"PASS - Nothing change during app apply (for now)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Story: 2011517"},{"line_number":45,"context_line":"Task: 52957"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"6901cfed_957a4cf7","line":42,"updated":"2025-10-24 13:27:07.000000000","message":"does this testing includes app-upload/apply/remove/delete?","commit_id":"747fcc8f33cd0f026eafa90f0f22458dcdaba030"},{"author":{"_account_id":37078,"name":"Daniel Marques Caires","display_name":"Daniel Caires","email":"DanielMarques.Caires@windriver.com","username":"dcaires"},"change_message_id":"611f60ba095f26aaa2d846f378de38ace6dfe8a9","unresolved":false,"context_lines":[{"line_number":39,"context_line":"Test Plan:"},{"line_number":40,"context_line":"PASS - Build OSH package and STX-O tarball"},{"line_number":41,"context_line":"PASS - Keystone values contains the new parameters"},{"line_number":42,"context_line":"PASS - Nothing change during app apply (for now)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Story: 2011517"},{"line_number":45,"context_line":"Task: 52957"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5ad6f034_c7768d4a","line":42,"in_reply_to":"6901cfed_957a4cf7","updated":"2025-10-24 13:55:49.000000000","message":"Done","commit_id":"747fcc8f33cd0f026eafa90f0f22458dcdaba030"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"2568790f224fc56bdf47757908a732b28e4d1ece","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"25cc5cf0_8c436112","updated":"2025-10-24 13:27:07.000000000","message":"Great stuff, Daniel.\nPlease address my comments and questions when you find some time.","commit_id":"747fcc8f33cd0f026eafa90f0f22458dcdaba030"},{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"ac66a27f58837b884d30841122a3e14a12cbed14","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"e5f8db46_8122ee0f","updated":"2025-10-27 13:18:48.000000000","message":"WF+1 based on multiple CR+1s","commit_id":"96b012e623e13c8ca53360f7afb213cd7bb82cce"}],"openstack-helm/debian/deb_folder/patches/0029-Add-DEX-integration.patch":[{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"2568790f224fc56bdf47757908a732b28e4d1ece","unresolved":true,"context_lines":[{"line_number":12,"context_line":"This patch integrates DEX to StarligX Openstack using a"},{"line_number":13,"context_line":"federated SSO provider and creating the necessary openstack"},{"line_number":14,"context_line":"resources to allow STX-O to use DEX as an identity provider."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I8540dce2dd0f1d1100872cec0ddf3a9e6df9ae81"},{"line_number":17,"context_line":"--"},{"line_number":18,"context_line":" keystone/values.yaml | 30 ++++++++++++++++++++++++++++++"}],"source_content_type":"text/x-diff","patch_set":4,"id":"bdfe67f6_73d813a9","line":15,"updated":"2025-10-24 13:27:07.000000000","message":"missing the author\u0027s sign-off","commit_id":"747fcc8f33cd0f026eafa90f0f22458dcdaba030"},{"author":{"_account_id":37078,"name":"Daniel Marques Caires","display_name":"Daniel Caires","email":"DanielMarques.Caires@windriver.com","username":"dcaires"},"change_message_id":"5c183b879a0cd0ef9c3d83473f0007eaf04c4187","unresolved":false,"context_lines":[{"line_number":12,"context_line":"This patch integrates DEX to StarligX Openstack using a"},{"line_number":13,"context_line":"federated SSO provider and creating the necessary openstack"},{"line_number":14,"context_line":"resources to allow STX-O to use DEX as an identity provider."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I8540dce2dd0f1d1100872cec0ddf3a9e6df9ae81"},{"line_number":17,"context_line":"--"},{"line_number":18,"context_line":" keystone/values.yaml | 30 ++++++++++++++++++++++++++++++"}],"source_content_type":"text/x-diff","patch_set":4,"id":"8e51fc1b_3e3213cb","line":15,"in_reply_to":"bdfe67f6_73d813a9","updated":"2025-10-24 14:00:18.000000000","message":"Done","commit_id":"747fcc8f33cd0f026eafa90f0f22458dcdaba030"},{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"2568790f224fc56bdf47757908a732b28e4d1ece","unresolved":true,"context_lines":[{"line_number":36,"context_line":"+      group_name: \"federated_users\""},{"line_number":37,"context_line":"+      project_name: \"federation\""},{"line_number":38,"context_line":"+      mapping_name: \"dex_mapping\""},{"line_number":39,"context_line":"+      default_mapping: |"},{"line_number":40,"context_line":"+        [{"},{"line_number":41,"context_line":"+          local\": [{"},{"line_number":42,"context_line":"+            \"user\": {\"name\": \"{0}\"},"}],"source_content_type":"text/x-diff","patch_set":4,"id":"57cd23ce_252ce4fa","line":39,"updated":"2025-10-24 13:27:07.000000000","message":"won\u0027t this be a ConfigMap?","commit_id":"747fcc8f33cd0f026eafa90f0f22458dcdaba030"},{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"1a0cbf7b6294ca3fecac450f3fcb252081e3d331","unresolved":false,"context_lines":[{"line_number":36,"context_line":"+      group_name: \"federated_users\""},{"line_number":37,"context_line":"+      project_name: \"federation\""},{"line_number":38,"context_line":"+      mapping_name: \"dex_mapping\""},{"line_number":39,"context_line":"+      default_mapping: |"},{"line_number":40,"context_line":"+        [{"},{"line_number":41,"context_line":"+          local\": [{"},{"line_number":42,"context_line":"+            \"user\": {\"name\": \"{0}\"},"}],"source_content_type":"text/x-diff","patch_set":4,"id":"5d60f78c_49b82000","line":39,"in_reply_to":"57cd23ce_252ce4fa","updated":"2025-10-24 13:35:09.000000000","message":"Ok, sorry. I see it is being done in the next change on this chain: https://review.opendev.org/c/starlingx/openstack-armada-app/+/964770/6/openstack-helm/debian/deb_folder/patches/0029-Add-DEX-integration.patch","commit_id":"747fcc8f33cd0f026eafa90f0f22458dcdaba030"},{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"2568790f224fc56bdf47757908a732b28e4d1ece","unresolved":true,"context_lines":[{"line_number":47,"context_line":"+      # Add DEX extra bootstrap commands to keystone bootstrap"},{"line_number":48,"context_line":"+      bootstrap:"},{"line_number":49,"context_line":"+        enabled: true"},{"line_number":50,"context_line":"+        script: |"},{"line_number":51,"context_line":"+          openstack role create member 2\u003e/dev/null || true"},{"line_number":52,"context_line":"+          openstack group create {{ .Values.conf.federation.dex_idp.group_name }} 2\u003e/dev/null || true"},{"line_number":53,"context_line":"+          openstack project create {{ .Values.conf.federation.dex_idp.project_name }} 2\u003e/dev/null || true"}],"source_content_type":"text/x-diff","patch_set":4,"id":"eecc0522_4f0a7265","line":50,"updated":"2025-10-24 13:27:07.000000000","message":"won\u0027t this be a ConfigMap?","commit_id":"747fcc8f33cd0f026eafa90f0f22458dcdaba030"},{"author":{"_account_id":33594,"name":"Thales Elero Cervi","display_name":"Thales Cervi","email":"thaleselero.cervi@windriver.com","username":"tcervi"},"change_message_id":"1a0cbf7b6294ca3fecac450f3fcb252081e3d331","unresolved":false,"context_lines":[{"line_number":47,"context_line":"+      # Add DEX extra bootstrap commands to keystone bootstrap"},{"line_number":48,"context_line":"+      bootstrap:"},{"line_number":49,"context_line":"+        enabled: true"},{"line_number":50,"context_line":"+        script: |"},{"line_number":51,"context_line":"+          openstack role create member 2\u003e/dev/null || true"},{"line_number":52,"context_line":"+          openstack group create {{ .Values.conf.federation.dex_idp.group_name }} 2\u003e/dev/null || true"},{"line_number":53,"context_line":"+          openstack project create {{ .Values.conf.federation.dex_idp.project_name }} 2\u003e/dev/null || true"}],"source_content_type":"text/x-diff","patch_set":4,"id":"6e86aab2_f47bf54a","line":50,"in_reply_to":"eecc0522_4f0a7265","updated":"2025-10-24 13:35:09.000000000","message":"Ok, sorry. I see it is being done in the next change on this chain: https://review.opendev.org/c/starlingx/openstack-armada-app/+/964770/6/openstack-helm/debian/deb_folder/patches/0029-Add-DEX-integration.patch","commit_id":"747fcc8f33cd0f026eafa90f0f22458dcdaba030"}]}