)]}'
{"id":"openstack%2Fopenstack-helm-infra~737103","triplet_id":"openstack%2Fopenstack-helm-infra~master~Ifc61ab20f047b662fa60032df6e98f24638f4dc3","project":"openstack/openstack-helm-infra","branch":"master","hashtags":[],"change_id":"Ifc61ab20f047b662fa60032df6e98f24638f4dc3","subject":"fix(lint): relax yamllint","status":"ABANDONED","created":"2020-06-19 22:55:34.000000000","updated":"2020-06-22 00:07:02.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"f387f01221187b24af1148730d31195615a49b93","_number":737103,"virtual_id_number":737103,"owner":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"tag":"autogenerated:zuul:check","value":1,"date":"2020-06-20 05:55:47.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"disliked":{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"value":-1,"date":"2020-06-21 04:19:29.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":-1,"default_value":0,"optional":true},"Workflow":{"all":[{"date":"2020-06-21 03:56:27.000000000","_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-06-20 00:29:09.000000000","updated_by":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"reviewer":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"state":"REVIEWER"},{"updated":"2020-06-20 05:55:47.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2020-06-21 03:56:27.000000000","updated_by":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"reviewer":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"state":"REVIEWER"},{"updated":"2020-06-21 04:19:29.000000000","updated_by":{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},"reviewer":{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},"state":"REVIEWER"}],"messages":[{"id":"165e1c5e7fbf49f0c5c6f85775b090ac67d74811","author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"date":"2020-06-19 22:55:34.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"7617e55281840045d44068c94b183b26530c732c","author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"date":"2020-06-19 22:56:39.000000000","message":"Patch Set 2: Commit message was updated.","accounts_in_message":[],"_revision_number":2},{"id":"28d983625193f39ad857898359977f88eb4f583f","author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"date":"2020-06-19 22:58:20.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"fd3747195df2ea0ba8ff06cbbaf25c0450ef9ccd","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-19 23:57:03.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/c9beab32ea2144a99eee8cf0da6f5000 : SUCCESS in 5m 12s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/4dc4326448bc4777890b36106c074b3f : FAILURE in 3m 00s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/43b66ab7c282422983013cc1a62f5f03 : SUCCESS in 38m 01s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/145e351f78ff4764b214ec37c262b9db : SUCCESS in 39m 55s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/0e7173d4068947419d0a5e123be7c077 : SUCCESS in 55m 27s\n- openstack-helm-infra-airship-divingbell https://zuul.opendev.org/t/openstack/build/76676b04db324e0bb56697040c9572f8 : SUCCESS in 13m 05s (non-voting)","accounts_in_message":[],"_revision_number":2},{"id":"e46d6efc66c568f9b8fc935498eb2aba6f498c0d","author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"date":"2020-06-20 00:04:24.000000000","message":"Patch Set 3: Published edit on patch set 2.","accounts_in_message":[],"_revision_number":3},{"id":"45a089471f9aa1a32d2121c306e83174e3ef5dff","author":{"_account_id":21420,"name":"Gage Hugo","email":"gagehugo@gmail.com","username":"ghugo"},"date":"2020-06-20 00:29:09.000000000","message":"Patch Set 3: Code-Review+2","accounts_in_message":[],"_revision_number":3},{"id":"af45cc9e16ffda1f54060475598f67d5fed9b365","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-20 00:59:52.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/52d7fcbf52314cc888bd5714d70e3239 : SUCCESS in 3m 15s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/6c513a7bc2ba4477808345b2f655139e : SUCCESS in 2m 17s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/7e4c201e76804d2cbccf61ab49c9264c : SUCCESS in 39m 46s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/27adc9e5d8ea458fb883acd300eac634 : SUCCESS in 36m 32s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/cd74f1de793341d98306ef7766058e5e : SUCCESS in 55m 06s\n- openstack-helm-infra-airship-divingbell https://zuul.opendev.org/t/openstack/build/b3f57256577f4fb29117cea98e88b576 : SUCCESS in 13m 35s (non-voting)","accounts_in_message":[],"_revision_number":3},{"id":"874331762e26dd480a9c2702ca104deab4383572","author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"date":"2020-06-20 05:00:15.000000000","message":"Uploaded patch set 4.","accounts_in_message":[],"_revision_number":4},{"id":"6aaa14db04c8a2c61d8bd48f1f1a7ccc30560aee","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-06-20 05:55:47.000000000","message":"Patch Set 4: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/d85a5177646b4c10b24731c043e06b93 : SUCCESS in 3m 33s\n- openstack-helm-lint https://zuul.opendev.org/t/openstack/build/8fcf439d6aa745c79c6ceab3501f10e1 : SUCCESS in 2m 15s\n- openstack-helm-infra-aio-logging https://zuul.opendev.org/t/openstack/build/e8585243409e45228dd5d3bf3e94d5fb : SUCCESS in 39m 37s\n- openstack-helm-infra-aio-monitoring https://zuul.opendev.org/t/openstack/build/7a6053c8090c41ea8df042d265772229 : SUCCESS in 43m 20s\n- openstack-helm-infra-openstack-support https://zuul.opendev.org/t/openstack/build/ebc3e241592241ce99b0689cfe08c8ff : SUCCESS in 54m 38s\n- openstack-helm-infra-airship-divingbell https://zuul.opendev.org/t/openstack/build/3c49ce3c1b2f4d6286838ea71c7e36b4 : SUCCESS in 12m 49s (non-voting)","accounts_in_message":[],"_revision_number":4},{"id":"03e555f8234181f973e8e0772cf620f97334852f","author":{"_account_id":8863,"name":"Andrii Ostapenko","email":"anost1986@gmail.com","username":"aostapenko"},"date":"2020-06-21 04:19:29.000000000","message":"Patch Set 4: Code-Review-1\n\nCurrent octal notation is not supported by yaml 1.2 and may potentially cause compatibility issues. The safe way is to use a representation stable in both 1.1 and 1.2","accounts_in_message":[],"_revision_number":4},{"id":"f387f01221187b24af1148730d31195615a49b93","author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"date":"2020-06-22 00:07:02.000000000","message":"Abandoned\n\nThis may not be needed - I have cleaned up most of this in the TLS patches so they don\u0027t introduce octal. Existing problems will be addressed via other patches.","accounts_in_message":[],"_revision_number":4}],"current_revision_number":4,"current_revision":"a91500a0a7d753fd38a14e573e0e7e4be41c30f5","revisions":{"5457470c76053878307cfef0be3db05d4a3ab036":{"kind":"REWORK","_number":1,"created":"2020-06-19 22:55:34.000000000","uploader":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"ref":"refs/changes/03/737103/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/03/737103/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/1"}}},"commit":{"parents":[{"commit":"26350f37aa36b64e04f15499d0111db8baf08fab","subject":"Add new python roles to playbooks","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/26350f37aa36b64e04f15499d0111db8baf08fab"}]}],"author":{"name":"Tin Lam","email":"tin@irrational.io","date":"2020-06-19 22:52:32.000000000","tz":-300},"committer":{"name":"Tin Lam","email":"tin@irrational.io","date":"2020-06-19 22:52:32.000000000","tz":-300},"subject":"fix(lint): relax yamllint","message":"fix(lint): relax yamllint\n\nThe draconian octal value in yaml enforcement only impacts the\nspecification of file permission - forcing the yaml author to do a\ndecimal conversion, while most Linux user is custom to the octal\nspecification. E.g., chmod 555 fileName. Changing this to a warning\ninstead to avoid folks getting caught in the conversion.\n\nChange-Id: Ifc61ab20f047b662fa60032df6e98f24638f4dc3\nSigned-off-by: Tin Lam \u003ctin@irrational.io\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/5457470c76053878307cfef0be3db05d4a3ab036"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/5457470c76053878307cfef0be3db05d4a3ab036"}]},"branch":"refs/heads/master"},"971862eba217400a30febc9cc94bb52d3c9b2516":{"kind":"NO_CODE_CHANGE","_number":2,"created":"2020-06-19 22:56:39.000000000","uploader":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"ref":"refs/changes/03/737103/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/03/737103/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/2"}}},"commit":{"parents":[{"commit":"26350f37aa36b64e04f15499d0111db8baf08fab","subject":"Add new python roles to playbooks","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/26350f37aa36b64e04f15499d0111db8baf08fab"}]}],"author":{"name":"Tin Lam","email":"tin@irrational.io","date":"2020-06-19 22:52:32.000000000","tz":-300},"committer":{"name":"Tin Lam","email":"tin@irrational.io","date":"2020-06-19 22:56:36.000000000","tz":0},"subject":"fix(lint): relax yamllint","message":"fix(lint): relax yamllint\n\nThe draconian octal value in yaml enforcement impacts the\nspecification of file permission - forcing the yaml author to do a\ndecimal conversion, while most Linux users are custom to the octal\nspecification. E.g., chmod 0555 fileName. Changing this to a warning\ninstead to avoid folks getting caught in the conversion.\n\nChange-Id: Ifc61ab20f047b662fa60032df6e98f24638f4dc3\nSigned-off-by: Tin Lam \u003ctin@irrational.io\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/971862eba217400a30febc9cc94bb52d3c9b2516"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/971862eba217400a30febc9cc94bb52d3c9b2516"}]},"branch":"refs/heads/master"},"581ee009bd6ca0646eaa9f799a72586b1904a3fd":{"kind":"REWORK","_number":3,"created":"2020-06-20 00:04:24.000000000","uploader":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"ref":"refs/changes/03/737103/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/03/737103/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/3"}}},"commit":{"parents":[{"commit":"26350f37aa36b64e04f15499d0111db8baf08fab","subject":"Add new python roles to playbooks","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/26350f37aa36b64e04f15499d0111db8baf08fab"}]}],"author":{"name":"Tin Lam","email":"tin@irrational.io","date":"2020-06-19 22:52:32.000000000","tz":-300},"committer":{"name":"Tin Lam","email":"tin@irrational.io","date":"2020-06-20 00:04:17.000000000","tz":0},"subject":"fix(lint): relax yamllint","message":"fix(lint): relax yamllint\n\nThe draconian octal value in yaml enforcement impacts the\nspecification of file permission - forcing the yaml author to do a\ndecimal conversion, while most Linux users are custom to the octal\nspecification. E.g., chmod 0555 fileName. Changing this to a warning\ninstead to avoid folks getting caught in the conversion.\n\nChange-Id: Ifc61ab20f047b662fa60032df6e98f24638f4dc3\nSigned-off-by: Tin Lam \u003ctin@irrational.io\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/581ee009bd6ca0646eaa9f799a72586b1904a3fd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/581ee009bd6ca0646eaa9f799a72586b1904a3fd"}]},"branch":"refs/heads/master"},"a91500a0a7d753fd38a14e573e0e7e4be41c30f5":{"kind":"REWORK","_number":4,"created":"2020-06-20 05:00:15.000000000","uploader":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"ref":"refs/changes/03/737103/4","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/openstack-helm-infra","ref":"refs/changes/03/737103/4","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/4 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/4 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/4 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/openstack-helm-infra refs/changes/03/737103/4"}}},"commit":{"parents":[{"commit":"26350f37aa36b64e04f15499d0111db8baf08fab","subject":"Add new python roles to playbooks","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/26350f37aa36b64e04f15499d0111db8baf08fab"}]}],"author":{"name":"Tin Lam","email":"tin@irrational.io","date":"2020-06-19 22:52:32.000000000","tz":-300},"committer":{"name":"Tin Lam","email":"tin@irrational.io","date":"2020-06-20 04:57:29.000000000","tz":-300},"subject":"fix(lint): relax yamllint","message":"fix(lint): relax yamllint\n\nDisallowing octal value in yaml file primarily impacts the\nspecification of Linux file permission. Ultimately, forcing yaml\nauthor to perform a decimal conversion. Most Linux users are custom\nto the octal (rwx) specification. E.g., chmod 0555 fileName.\n\nThis patch set disables the check to avoid contributors from needing\nto an additional conversion.\n\nChange-Id: Ifc61ab20f047b662fa60032df6e98f24638f4dc3\nSigned-off-by: Tin Lam \u003ctin@irrational.io\u003e\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/a91500a0a7d753fd38a14e573e0e7e4be41c30f5"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/openstack-helm-infra/commit/a91500a0a7d753fd38a14e573e0e7e4be41c30f5"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}