)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4257,"name":"Zane Bitter","email":"zbitter@redhat.com","username":"zaneb"},"change_message_id":"3c050ca48a59f62de82be85cf8f406312f8e64db","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix cfn-init-data path in ignition"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The \u0027local-data\u0027 in path /var/lib/os-collect-config/local-data should"},{"line_number":10,"context_line":"be a directly not a file. This patch fixes it. And now there is an"},{"line_number":11,"context_line":"issue when doing merging configs in os-apply-config, so we need to be"},{"line_number":12,"context_line":"backward compatible by writing file to /var/lib/heat-cfntools/cfn-init-data"},{"line_number":13,"context_line":"and /var/lib/cloud/data/cfn-init-data."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_d261d106","line":11,"range":{"start_line":10,"start_character":47,"end_line":11,"end_character":51},"updated":"2019-10-14 19:12:51.000000000","message":"Can you describe the issue?","commit_id":"1f5163cc2799d44c9ede5c9526c0854fe1ebc75c"},{"author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"change_message_id":"2e2ce9abd396f4e4cfc0217033b62191213d1647","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix cfn-init-data path in ignition"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The \u0027local-data\u0027 in path /var/lib/os-collect-config/local-data should"},{"line_number":10,"context_line":"be a directly not a file. This patch fixes it. And now there is an"},{"line_number":11,"context_line":"issue when doing merging configs in os-apply-config, so we need to be"},{"line_number":12,"context_line":"backward compatible by writing file to /var/lib/heat-cfntools/cfn-init-data"},{"line_number":13,"context_line":"and /var/lib/cloud/data/cfn-init-data."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_e8f6608f","line":11,"range":{"start_line":10,"start_character":47,"end_line":11,"end_character":51},"in_reply_to":"3fa7e38b_d261d106","updated":"2019-10-14 21:00:33.000000000","message":"Sorry, here is the patch I proposed to os-apply-config https://review.opendev.org/#/c/688317/","commit_id":"1f5163cc2799d44c9ede5c9526c0854fe1ebc75c"},{"author":{"_account_id":28022,"name":"Bharat Kunwar","email":"brtknr@bath.edu","username":"brtknr"},"change_message_id":"4e473c4186bb6653323c4bf93f874b186f756f45","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix cfn-init-data path in ignition"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The \u0027local-data\u0027 in path /var/lib/os-collect-config/local-data should"},{"line_number":10,"context_line":"be a directly not a file. And now there is an issue[1] when doing merging"},{"line_number":11,"context_line":"configs in os-apply-config, so at this moment we have to use"},{"line_number":12,"context_line":"/var/lib/heat-cfntools/cfn-init-data and /var/lib/cloud/data/cfn-init-data"},{"line_number":13,"context_line":"until the issue in os-apply-config being fixed."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_7d5c547c","line":10,"range":{"start_line":10,"start_character":5,"end_line":10,"end_character":13},"updated":"2019-10-16 10:00:00.000000000","message":"directory","commit_id":"ae47a18bc812fecad8c61c1dab0f3c33a78929ab"},{"author":{"_account_id":28022,"name":"Bharat Kunwar","email":"brtknr@bath.edu","username":"brtknr"},"change_message_id":"4e473c4186bb6653323c4bf93f874b186f756f45","unresolved":false,"context_lines":[{"line_number":10,"context_line":"be a directly not a file. And now there is an issue[1] when doing merging"},{"line_number":11,"context_line":"configs in os-apply-config, so at this moment we have to use"},{"line_number":12,"context_line":"/var/lib/heat-cfntools/cfn-init-data and /var/lib/cloud/data/cfn-init-data"},{"line_number":13,"context_line":"until the issue in os-apply-config being fixed."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"[1] https://review.opendev.org/688317"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_3d525c68","line":13,"range":{"start_line":13,"start_character":35,"end_line":13,"end_character":40},"updated":"2019-10-16 10:00:00.000000000","message":"is","commit_id":"ae47a18bc812fecad8c61c1dab0f3c33a78929ab"},{"author":{"_account_id":12404,"name":"Rico Lin","email":"ricolin@ricolky.com","username":"rico.lin"},"change_message_id":"c97f0bb961bfb5a2403900833fd2466559d977e0","unresolved":false,"context_lines":[{"line_number":11,"context_line":"configs in os-apply-config, so at this moment we have to use"},{"line_number":12,"context_line":"/var/lib/heat-cfntools/cfn-init-data and /var/lib/cloud/data/cfn-init-data"},{"line_number":13,"context_line":"until the issue in os-apply-config being fixed."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"[1] https://review.opendev.org/688317"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Story: 2006566"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3fa7e38b_6fbfac60","line":14,"updated":"2019-10-17 03:17:34.000000000","message":"It will be more accurate to mention that [1] is no longer a valid patch, so we might migrate to use cfn-init-data directly","commit_id":"ae47a18bc812fecad8c61c1dab0f3c33a78929ab"}],"heat/engine/clients/os/nova.py":[{"author":{"_account_id":4257,"name":"Zane Bitter","email":"zbitter@redhat.com","username":"zaneb"},"change_message_id":"3c050ca48a59f62de82be85cf8f406312f8e64db","unresolved":false,"context_lines":[{"line_number":450,"context_line":"        encoded_metadata \u003d urlparse.quote(jsonutils.dumps(metadata))"},{"line_number":451,"context_line":"        path_list \u003d [\"/var/lib/heat-cfntools/cfn-init-data\","},{"line_number":452,"context_line":"                     \"/var/lib/cloud/data/cfn-init-data\","},{"line_number":453,"context_line":"                     \"/var/lib/os-collect-config/local-data/cfn-init-data\"]"},{"line_number":454,"context_line":"        cfn_init_data \u003d {"},{"line_number":455,"context_line":"            \"filesystem\": \"root\","},{"line_number":456,"context_line":"            \"group\": {\"name\": \"root\"},"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_12b6898b","line":453,"range":{"start_line":453,"start_character":60,"end_line":453,"end_character":73},"updated":"2019-10-14 19:12:51.000000000","message":"Can we use a name like \"heat-metadata\"?\n\nThis is not in any sense cfn-init data. cfn-init uses metadata of the form {\"AWS::CloudFormation::Init\": {...}}. This path in the code can never be reached if the user is in fact passing metadata in this format.","commit_id":"1f5163cc2799d44c9ede5c9526c0854fe1ebc75c"},{"author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"change_message_id":"2e2ce9abd396f4e4cfc0217033b62191213d1647","unresolved":false,"context_lines":[{"line_number":450,"context_line":"        encoded_metadata \u003d urlparse.quote(jsonutils.dumps(metadata))"},{"line_number":451,"context_line":"        path_list \u003d [\"/var/lib/heat-cfntools/cfn-init-data\","},{"line_number":452,"context_line":"                     \"/var/lib/cloud/data/cfn-init-data\","},{"line_number":453,"context_line":"                     \"/var/lib/os-collect-config/local-data/cfn-init-data\"]"},{"line_number":454,"context_line":"        cfn_init_data \u003d {"},{"line_number":455,"context_line":"            \"filesystem\": \"root\","},{"line_number":456,"context_line":"            \"group\": {\"name\": \"root\"},"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_2836b852","line":453,"range":{"start_line":453,"start_character":60,"end_line":453,"end_character":73},"in_reply_to":"3fa7e38b_12b6898b","updated":"2019-10-14 21:00:33.000000000","message":"I will fix it in next ps.","commit_id":"1f5163cc2799d44c9ede5c9526c0854fe1ebc75c"},{"author":{"_account_id":4257,"name":"Zane Bitter","email":"zbitter@redhat.com","username":"zaneb"},"change_message_id":"3c050ca48a59f62de82be85cf8f406312f8e64db","unresolved":false,"context_lines":[{"line_number":451,"context_line":"        path_list \u003d [\"/var/lib/heat-cfntools/cfn-init-data\","},{"line_number":452,"context_line":"                     \"/var/lib/cloud/data/cfn-init-data\","},{"line_number":453,"context_line":"                     \"/var/lib/os-collect-config/local-data/cfn-init-data\"]"},{"line_number":454,"context_line":"        cfn_init_data \u003d {"},{"line_number":455,"context_line":"            \"filesystem\": \"root\","},{"line_number":456,"context_line":"            \"group\": {\"name\": \"root\"},"},{"line_number":457,"context_line":"            \"path\": \"\","}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_f273ade5","line":454,"range":{"start_line":454,"start_character":8,"end_line":454,"end_character":21},"updated":"2019-10-14 19:12:51.000000000","message":"A better name for this would be less confusing too.","commit_id":"1f5163cc2799d44c9ede5c9526c0854fe1ebc75c"},{"author":{"_account_id":6484,"name":"Feilong Wang","email":"hustemb@gmail.com","username":"flwang"},"change_message_id":"2e2ce9abd396f4e4cfc0217033b62191213d1647","unresolved":false,"context_lines":[{"line_number":451,"context_line":"        path_list \u003d [\"/var/lib/heat-cfntools/cfn-init-data\","},{"line_number":452,"context_line":"                     \"/var/lib/cloud/data/cfn-init-data\","},{"line_number":453,"context_line":"                     \"/var/lib/os-collect-config/local-data/cfn-init-data\"]"},{"line_number":454,"context_line":"        cfn_init_data \u003d {"},{"line_number":455,"context_line":"            \"filesystem\": \"root\","},{"line_number":456,"context_line":"            \"group\": {\"name\": \"root\"},"},{"line_number":457,"context_line":"            \"path\": \"\","}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_8846ccbf","line":454,"range":{"start_line":454,"start_character":8,"end_line":454,"end_character":21},"in_reply_to":"3fa7e38b_f273ade5","updated":"2019-10-14 21:00:33.000000000","message":"I will fix it in next ps.","commit_id":"1f5163cc2799d44c9ede5c9526c0854fe1ebc75c"}]}
