)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"change_message_id":"54cae7c8c3bc572c757a54d610a9f40d6167707b","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     jinyuanliu \u003cliujinyuan@inspur.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-01-27 07:27:08 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"bug for build helm-toolkit"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"you should to path \u003cHELM_CHART_ROOT_PATH\u003e,when you want to build"},{"line_number":10,"context_line":"helm-toolkit,not here"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ic3ee9209742a74153935faa7de20411461eabef8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"4078e95d_0c593753","line":10,"range":{"start_line":7,"start_character":1,"end_line":10,"end_character":11},"updated":"2021-02-14 22:00:11.000000000","message":"not entirely sure what\u0027s the intent here\n\nmaybe this could be reworded to be a little more verbose?","commit_id":"58d20f64238402e7b80707afb3cd2995fba44639"},{"author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"change_message_id":"0c4a81141e02de58aa97bd40858b073258c5f974","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Bug for build helm-toolkit"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"About helm-toolkit chart , you should to path \u003cHELM_CHART_ROOT_PATH\u003e run the script when build helm-toolkit,not here.Otherwise, the old code will not work here."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ic3ee9209742a74153935faa7de20411461eabef8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"b8215e72_8dd3c0b2","line":9,"range":{"start_line":9,"start_character":54,"end_line":9,"end_character":160},"updated":"2021-03-16 05:00:12.000000000","message":"Punctuation: Sometimes, spaces after commas\n\nreflow","commit_id":"d1b1f25ce1648628f9e5adc491057778d24cd98c"}],"tools/deployment/common/setup-client.sh":[{"author":{"_account_id":24780,"name":"Sangeet Gupta","email":"sg774j@att.com","username":"sgupta"},"change_message_id":"3977a6d36475926f6815d520fc32fb93b37f8414","unresolved":true,"context_lines":[{"line_number":18,"context_line":"  -c${UPPER_CONSTRAINTS_FILE:\u003dhttps://releases.openstack.org/constraints/upper/${OPENSTACK_RELEASE:-stein}} \\"},{"line_number":19,"context_line":"  cmd2 python-openstackclient python-heatclient --ignore-installed"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"export HELM_CHART_ROOT_PATH\u003d\"${HELM_CHART_ROOT_PATH:\u003d\"${OSH_INFRA_PATH:\u003d\"../openstack-helm-infra\"}\"}\""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"sudo -H mkdir -p /etc/openstack"},{"line_number":24,"context_line":"sudo -H chown -R $(id -un): /etc/openstack"}],"source_content_type":"text/x-sh","patch_set":2,"id":"d220516d_8405018b","line":21,"range":{"start_line":21,"start_character":31,"end_line":21,"end_character":51},"updated":"2021-02-23 13:54:52.000000000","message":"Why are you assigning it to this variable when you are exporting it?\n\nWhy was it working till now?","commit_id":"7174dc99fc18a9fe02fe8caa8bc87cc1bb2a7c6b"},{"author":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"change_message_id":"82e6d81c332eab651a9b6ac9404f774ae4fe1ebe","unresolved":true,"context_lines":[{"line_number":18,"context_line":"  -c${UPPER_CONSTRAINTS_FILE:\u003dhttps://releases.openstack.org/constraints/upper/${OPENSTACK_RELEASE:-stein}} \\"},{"line_number":19,"context_line":"  cmd2 python-openstackclient python-heatclient --ignore-installed"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"export HELM_CHART_ROOT_PATH\u003d\"${HELM_CHART_ROOT_PATH:\u003d\"${OSH_INFRA_PATH:\u003d\"../openstack-helm-infra\"}\"}\""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"sudo -H mkdir -p /etc/openstack"},{"line_number":24,"context_line":"sudo -H chown -R $(id -un): /etc/openstack"}],"source_content_type":"text/x-sh","patch_set":2,"id":"902342f8_2053802a","line":21,"range":{"start_line":21,"start_character":31,"end_line":21,"end_character":51},"in_reply_to":"d220516d_8405018b","updated":"2021-02-23 14:07:46.000000000","message":"This variable is the defined helm-toolkit chart path!If the path is not defined, build helm-toolkit will fail","commit_id":"7174dc99fc18a9fe02fe8caa8bc87cc1bb2a7c6b"},{"author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"change_message_id":"0c4a81141e02de58aa97bd40858b073258c5f974","unresolved":true,"context_lines":[{"line_number":18,"context_line":"  -c${UPPER_CONSTRAINTS_FILE:\u003dhttps://releases.openstack.org/constraints/upper/${OPENSTACK_RELEASE:-stein}} \\"},{"line_number":19,"context_line":"  cmd2 python-openstackclient python-heatclient --ignore-installed"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"export HELM_CHART_ROOT_PATH\u003d\"${HELM_CHART_ROOT_PATH:\u003d\"${OSH_INFRA_PATH:\u003d\"../openstack-helm-infra\"}\"}\""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"sudo -H mkdir -p /etc/openstack"},{"line_number":24,"context_line":"sudo -H chown -R $(id -un): /etc/openstack"}],"source_content_type":"text/x-sh","patch_set":4,"id":"42fd91f2_96f7f604","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":101},"updated":"2021-03-16 05:00:12.000000000","message":"this is one of those things i can\u0027t tell how robust it is\n\nhave we tried shellcheck?","commit_id":"d1b1f25ce1648628f9e5adc491057778d24cd98c"},{"author":{"_account_id":30449,"name":"jinyuanliu","email":"liujinyuan@inspur.com","username":"jinyuanliu"},"change_message_id":"d829b19972ffa78866c214ac4c794e04ecc5ef2e","unresolved":true,"context_lines":[{"line_number":18,"context_line":"  -c${UPPER_CONSTRAINTS_FILE:\u003dhttps://releases.openstack.org/constraints/upper/${OPENSTACK_RELEASE:-stein}} \\"},{"line_number":19,"context_line":"  cmd2 python-openstackclient python-heatclient --ignore-installed"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"export HELM_CHART_ROOT_PATH\u003d\"${HELM_CHART_ROOT_PATH:\u003d\"${OSH_INFRA_PATH:\u003d\"../openstack-helm-infra\"}\"}\""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"sudo -H mkdir -p /etc/openstack"},{"line_number":24,"context_line":"sudo -H chown -R $(id -un): /etc/openstack"}],"source_content_type":"text/x-sh","patch_set":4,"id":"72c02d7d_380c9d3e","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":101},"in_reply_to":"42fd91f2_96f7f604","updated":"2021-03-16 06:15:05.000000000","message":"Without this, the script will not report an error, but it has no effect。\nUnable to build helm toolkit to repository.","commit_id":"d1b1f25ce1648628f9e5adc491057778d24cd98c"}]}