)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"83b8b4932e5da1aa47010431742c294dbaf48d93","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"fc4d67e8_5c5eb445","updated":"2025-03-05 16:42:54.000000000","message":"just some comment-related comments for a follow-up","commit_id":"d616ec9d9ae2e2fb7f5d53f0f3f14917f0028b0d"}],"zuul.d/infra-prod.yaml":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"fcb973dc35dcb5438051908210bffc6c6a451470","unresolved":true,"context_lines":[{"line_number":78,"context_line":"      - name: infra-prod-bootstrap-bridge"},{"line_number":79,"context_line":"        # This is a hard dependency because if we fail to bootstrap bridge"},{"line_number":80,"context_line":"        # we may not have the correct system-config state on bridge and"},{"line_number":81,"context_line":"        # should not run."},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"- job:"},{"line_number":84,"context_line":"    name: infra-prod-base"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"cf169c9c_34e749a2","line":81,"updated":"2025-02-25 22:03:52.000000000","message":"Slightly misleading comment in that the difference between a hard and soft dependency is that we require bootstrap to be in the job graph in the hard case, but it\u0027s optional in soft.  Both types of dependencies skip their dependent jobs if they fail though.\n\nWe do want it to be in the job graph because we\u0027re using it as a mutex for the whole system, in addition to its role in prepping for the other jobs.  So it\u0027s correct for it to be a hard dependency.  That\u0027s just not the direct reason that we\u0027re certain the playbook jobs are ready to go.\n\nMaybe rephrase as \"This is a hard dependency because we require the bootstrap job to have run before we start any playbook jobs, otherwise our buildset would not hold the bridge semaphore and we may not have the correct system-config state on bridge.\"","commit_id":"0e4d6baf051dddf2023d801fa26ff125ae6652f7"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"83625466464484af1f22e6a85fde8fdceef83f47","unresolved":false,"context_lines":[{"line_number":78,"context_line":"      - name: infra-prod-bootstrap-bridge"},{"line_number":79,"context_line":"        # This is a hard dependency because if we fail to bootstrap bridge"},{"line_number":80,"context_line":"        # we may not have the correct system-config state on bridge and"},{"line_number":81,"context_line":"        # should not run."},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"- job:"},{"line_number":84,"context_line":"    name: infra-prod-base"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"b900b4b4_450c64ee","line":81,"in_reply_to":"cf169c9c_34e749a2","updated":"2025-03-05 16:31:06.000000000","message":"Done","commit_id":"0e4d6baf051dddf2023d801fa26ff125ae6652f7"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"83b8b4932e5da1aa47010431742c294dbaf48d93","unresolved":true,"context_lines":[{"line_number":3,"context_line":"# what\u0027s happening in there."},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"# Make sure only one run happens at a time.  The deploy pipeline"},{"line_number":6,"context_line":"# should keep things in order, but this is to stop perodic jobs"},{"line_number":7,"context_line":"# jumping in."},{"line_number":8,"context_line":"- semaphore:"},{"line_number":9,"context_line":"    name: infra-prod-deployment"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"2522a33d_6a399e52","line":6,"range":{"start_line":6,"start_character":51,"end_line":6,"end_character":58},"updated":"2025-03-05 16:42:54.000000000","message":"periodic","commit_id":"d616ec9d9ae2e2fb7f5d53f0f3f14917f0028b0d"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"83b8b4932e5da1aa47010431742c294dbaf48d93","unresolved":true,"context_lines":[{"line_number":30,"context_line":"        Configure the bastion host (bridge)"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"        This job does minimal configuration on the bastion host"},{"line_number":33,"context_line":"        (bridge.openstack.org) to allow it to run system-config"},{"line_number":34,"context_line":"        playbooks against our production hosts.  It sets up Ansible"},{"line_number":35,"context_line":"        and root keys on the host.  It also synchronizes the"},{"line_number":36,"context_line":"        system-config repo from the executor to the bastion."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1f822e3e_904d5b72","line":33,"range":{"start_line":33,"start_character":16,"end_line":33,"end_character":25},"updated":"2025-03-05 16:42:54.000000000","message":"nit: this is bridge.opendev.org now","commit_id":"d616ec9d9ae2e2fb7f5d53f0f3f14917f0028b0d"}],"zuul.d/project.yaml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"83b8b4932e5da1aa47010431742c294dbaf48d93","unresolved":true,"context_lines":[{"line_number":350,"context_line":"        # run against zuul\u0027s checkout of system-config."},{"line_number":351,"context_line":"        - infra-prod-bootstrap-bridge"},{"line_number":352,"context_line":""},{"line_number":353,"context_line":"        # From now on, all jobs should depend on base"},{"line_number":354,"context_line":"        - infra-prod-base"},{"line_number":355,"context_line":""},{"line_number":356,"context_line":"        # Legacy puppet hosts"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"d5f42452_7a649d53","line":353,"updated":"2025-03-05 16:42:54.000000000","message":"This comment should be moved after infra-prod-base now?","commit_id":"d616ec9d9ae2e2fb7f5d53f0f3f14917f0028b0d"}]}