)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"14fe7545db308ab690a25eb07a0fb585eca86d3b","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix zuul settings to use latest kuryr-kubernetes"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The kuryr-kubernetes patch [1] that changes to use kubeadm for"},{"line_number":10,"context_line":"installing Kubernetes on devstack."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"A patch [2] was previously created to address the change, but the fix"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":24,"id":"0fc62db2_04091081","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":4},"updated":"2021-09-06 17:33:00.000000000","message":"no need","commit_id":"9d79925c062f2cc4d9bf8afc7236dcf07cef3548"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"f9467d9608f2ec64fd02672ad43562079e4c7e7c","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix zuul settings to use latest kuryr-kubernetes"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The kuryr-kubernetes patch [1] that changes to use kubeadm for"},{"line_number":10,"context_line":"installing Kubernetes on devstack."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"A patch [2] was previously created to address the change, but the fix"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":24,"id":"c9559518_4d301dcb","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":4},"in_reply_to":"0fc62db2_04091081","updated":"2021-09-07 12:58:39.000000000","message":"I deleted it.","commit_id":"9d79925c062f2cc4d9bf8afc7236dcf07cef3548"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"14fe7545db308ab690a25eb07a0fb585eca86d3b","unresolved":true,"context_lines":[{"line_number":15,"context_line":"This patch fixes to use latest kuryr-kubernetes."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"* Change the \".zuul.yaml\" setting to use the latest kuryr-kubernetes."},{"line_number":18,"context_line":"* Add the process of creating a ServiceAccount to the ansible task"},{"line_number":19,"context_line":"  running in zuul, and change the vim authentication method to"},{"line_number":20,"context_line":"  bearer_token. This is because kuryr-kubernetes does not create an"},{"line_number":21,"context_line":"  admin ServiceAccount by default."},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":24,"id":"08ad4b05_0e2e041e","line":19,"range":{"start_line":18,"start_character":46,"end_line":19,"end_character":18},"updated":"2021-09-06 17:33:00.000000000","message":"It\u0027s apparent so no need to say here.","commit_id":"9d79925c062f2cc4d9bf8afc7236dcf07cef3548"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"f9467d9608f2ec64fd02672ad43562079e4c7e7c","unresolved":true,"context_lines":[{"line_number":15,"context_line":"This patch fixes to use latest kuryr-kubernetes."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"* Change the \".zuul.yaml\" setting to use the latest kuryr-kubernetes."},{"line_number":18,"context_line":"* Add the process of creating a ServiceAccount to the ansible task"},{"line_number":19,"context_line":"  running in zuul, and change the vim authentication method to"},{"line_number":20,"context_line":"  bearer_token. This is because kuryr-kubernetes does not create an"},{"line_number":21,"context_line":"  admin ServiceAccount by default."},{"line_number":22,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":24,"id":"0180f507_918b5180","line":19,"range":{"start_line":18,"start_character":46,"end_line":19,"end_character":18},"in_reply_to":"08ad4b05_0e2e041e","updated":"2021-09-07 12:58:39.000000000","message":"I deleted it.","commit_id":"9d79925c062f2cc4d9bf8afc7236dcf07cef3548"}],"roles/setup-default-vim/tasks/main.yaml":[{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"14fe7545db308ab690a25eb07a0fb585eca86d3b","unresolved":true,"context_lines":[{"line_number":82,"context_line":"    become_user: stack"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"  - name: Get admin token"},{"line_number":85,"context_line":"    shell: kubectl describe secret {{ options }} $(kubectl get secrets {{ options }} | grep admin | cut -f1 -d \u0027 \u0027) | grep -E \u0027^token\u0027 | cut -f2 -d\u0027:\u0027 | tr -d \u0027\\t\u0027 | tr -d \u0027 \u0027"},{"line_number":86,"context_line":"    vars:"},{"line_number":87,"context_line":"      - options: \"--namespace kube-system\""},{"line_number":88,"context_line":"    register: admin_token"}],"source_content_type":"text/x-yaml","patch_set":24,"id":"c7f69b54_88fddc67","line":85,"updated":"2021-09-06 17:33:00.000000000","message":"We can get any param with `kubectl get ... -o\u003djsonpath ...` which is easier than classical way such as here to get a target param. I think it\u0027s better to use `kubectl get` as below instead. Could you confirm that?\n\n```\n$ kubectl get secret -o jsonpath\u003d\u0027{.items[*].data.token}\u0027\n```","commit_id":"9d79925c062f2cc4d9bf8afc7236dcf07cef3548"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"17a20ab6acbe945d98a569c73949d25ad4e3c238","unresolved":true,"context_lines":[{"line_number":82,"context_line":"    become_user: stack"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"  - name: Get admin token"},{"line_number":85,"context_line":"    shell: kubectl describe secret {{ options }} $(kubectl get secrets {{ options }} | grep admin | cut -f1 -d \u0027 \u0027) | grep -E \u0027^token\u0027 | cut -f2 -d\u0027:\u0027 | tr -d \u0027\\t\u0027 | tr -d \u0027 \u0027"},{"line_number":86,"context_line":"    vars:"},{"line_number":87,"context_line":"      - options: \"--namespace kube-system\""},{"line_number":88,"context_line":"    register: admin_token"}],"source_content_type":"text/x-yaml","patch_set":24,"id":"9588ed9d_895c8559","line":85,"in_reply_to":"3d7b4880_b7072607","updated":"2021-09-09 07:58:49.000000000","message":"\u003e The tokens that got by \"kubectl get secrets\" and \"kubectl describe secret\" are different\n\nSorry, I found decoding with the base64 command results in the same string.\nI revised it again in PS26.","commit_id":"9d79925c062f2cc4d9bf8afc7236dcf07cef3548"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"f9467d9608f2ec64fd02672ad43562079e4c7e7c","unresolved":true,"context_lines":[{"line_number":82,"context_line":"    become_user: stack"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"  - name: Get admin token"},{"line_number":85,"context_line":"    shell: kubectl describe secret {{ options }} $(kubectl get secrets {{ options }} | grep admin | cut -f1 -d \u0027 \u0027) | grep -E \u0027^token\u0027 | cut -f2 -d\u0027:\u0027 | tr -d \u0027\\t\u0027 | tr -d \u0027 \u0027"},{"line_number":86,"context_line":"    vars:"},{"line_number":87,"context_line":"      - options: \"--namespace kube-system\""},{"line_number":88,"context_line":"    register: admin_token"}],"source_content_type":"text/x-yaml","patch_set":24,"id":"3d7b4880_b7072607","line":85,"in_reply_to":"c7f69b54_88fddc67","updated":"2021-09-07 12:58:39.000000000","message":"The tokens that got by \"kubectl get secrets\" and \"kubectl describe secret\" are different, and the value set to vim_config should be the latter.\n\nrefer to Docs: https://docs.openstack.org/tacker/latest/install/kubernetes_vim_installation.html\n\nTherefore, It cannot be gotten required token value by the method you proposed.\nHowever, I changed it with PS25 because I think the command is long and difficult to understand.\n(Because \"Kubectl describe\" does not have the \"-o json/jsonpath\" option, fetching values is a shell string operation.)\n\nPlease confirm it.","commit_id":"9d79925c062f2cc4d9bf8afc7236dcf07cef3548"},{"author":{"_account_id":25701,"name":"Yasufumi Ogawa","email":"yasufum.o@gmail.com","username":"yasufum"},"change_message_id":"6271bfb776acec3b345a18b1dd3e50cf5f977dba","unresolved":true,"context_lines":[{"line_number":91,"context_line":""},{"line_number":92,"context_line":"  - name: Get admin token from described secret"},{"line_number":93,"context_line":"    shell: \u003e"},{"line_number":94,"context_line":"      kubectl get {{ admin_secret_name.stdout }} -n kube-system -o jsonpath\u003d\"{.data.token}\""},{"line_number":95,"context_line":"      | base64 -d"},{"line_number":96,"context_line":"    register: admin_token"},{"line_number":97,"context_line":"    become: yes"},{"line_number":98,"context_line":"    become_user: stack"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"0661cdb6_6048bc15","line":95,"range":{"start_line":94,"start_character":0,"end_line":95,"end_character":17},"updated":"2021-09-09 17:43:17.000000000","message":"Thanks for the revising. It looks tricky a little, but simple and smart to use `kubectl get` for the purpose.","commit_id":"388f7f6c08a802229ea6b4951ef0bf3106c0f0a3"}]}