)]}'
{"playbooks/zuul/container-promotion/pre.yaml":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"0ae8e38dd6a9354c4602dac2d8366445cd8dfd7e","unresolved":false,"context_lines":[{"line_number":3,"context_line":"    - name: Create public container in quay.io if not already there"},{"line_number":4,"context_line":"      include_role:"},{"line_number":5,"context_line":"        name: ensure-quay-repo"},{"line_number":6,"context_line":"      no_log: true"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b92c0268_77f9b7f2","line":6,"updated":"2023-04-24 22:34:44.000000000","message":"Sure you want to no_log here?  There\u0027s a bunch of stuff in that role that doesn\u0027t need to be suppressed.\n\nAlso, this can just be:\n\n- hosts: localhost\n  roles:\n    - ensure-quay-repo\n\nAnd finally -- should we do this in opendev/base-jobs for the base container jobs instead of here?","commit_id":"0f515e6d96cbf89eaa8fe61bab944e7fd62d1b29"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"1533425060c6c9643780e34dca2ccd27bb332e65","unresolved":false,"context_lines":[{"line_number":3,"context_line":"    - name: Create public container in quay.io if not already there"},{"line_number":4,"context_line":"      include_role:"},{"line_number":5,"context_line":"        name: ensure-quay-repo"},{"line_number":6,"context_line":"      no_log: true"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"0a512518_4520727d","line":6,"in_reply_to":"b92c0268_77f9b7f2","updated":"2023-04-24 22:38:24.000000000","message":"This was a holdover from when I was needing to pass in thing explicitly to map them from various vars to the role vars. Now that I\u0027ve refactored the role I think this no_log could be dropped and we can simplify as suggested.","commit_id":"0f515e6d96cbf89eaa8fe61bab944e7fd62d1b29"}],"zuul.d/docker-images/base.yaml":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"0ae8e38dd6a9354c4602dac2d8366445cd8dfd7e","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    parent: opendev-upload-container-image"},{"line_number":73,"context_line":"    dependencies: opendev-buildset-registry"},{"line_number":74,"context_line":"    abstract: true"},{"line_number":75,"context_line":"    vars: *system_config_image_vars"},{"line_number":76,"context_line":"    secrets:"},{"line_number":77,"context_line":"      name: container_registry_credentials"},{"line_number":78,"context_line":"      secret: system-config-quayio"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"e99bd237_c4af330c","line":75,"updated":"2023-04-24 22:34:44.000000000","message":"Should we pre-run here too?","commit_id":"0f515e6d96cbf89eaa8fe61bab944e7fd62d1b29"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"1533425060c6c9643780e34dca2ccd27bb332e65","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    parent: opendev-upload-container-image"},{"line_number":73,"context_line":"    dependencies: opendev-buildset-registry"},{"line_number":74,"context_line":"    abstract: true"},{"line_number":75,"context_line":"    vars: *system_config_image_vars"},{"line_number":76,"context_line":"    secrets:"},{"line_number":77,"context_line":"      name: container_registry_credentials"},{"line_number":78,"context_line":"      secret: system-config-quayio"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9a38c7ed_1db81e80","line":75,"in_reply_to":"e99bd237_c4af330c","updated":"2023-04-24 22:38:24.000000000","message":"++","commit_id":"0f515e6d96cbf89eaa8fe61bab944e7fd62d1b29"}]}