)]}'
{"playbooks/group_vars/zuul-executor-opendev.yaml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"d42e480b22eb9af8ce975feb0e4f625af95dc592","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# NOTE: ubuntu focal has new enough docker, and upstream docker"},{"line_number":2,"context_line":"# does not have focal support yet."},{"line_number":3,"context_line":"use_upstream_docker: true"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"1f493fa4_bef64862","line":3,"updated":"2020-04-27 14:05:33.000000000","message":"Wouldn\u0027t that be false instead of true, then?","commit_id":"01bd86dce6b0d54aef495d4a0cbab3b22f1d8a1f"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"5e5c590b6e21b18d7afc5f6c0c5fdf2a2703c20f","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# NOTE: ubuntu focal has new enough docker, and upstream docker"},{"line_number":2,"context_line":"# does not have focal support yet."},{"line_number":3,"context_line":"use_upstream_docker: true"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"1f493fa4_9a570a59","line":3,"in_reply_to":"1f493fa4_bef64862","updated":"2020-04-27 22:46:44.000000000","message":"Yeah - I sure did get that backwards.","commit_id":"01bd86dce6b0d54aef495d4a0cbab3b22f1d8a1f"}],"playbooks/roles/base-server/tasks/Debian.yaml":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"91579ad581a3ef2a3d509e23606d1873c47a580a","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"- name: Remove ntp and run timesyncd"},{"line_number":30,"context_line":"  when: ansible_distribution_version is version(\u002718.04\u0027, \u0027\u003e\u003d\u0027)"},{"line_number":31,"context_line":"  block:"},{"line_number":32,"context_line":"    - name: Remove ntp"},{"line_number":33,"context_line":"      package:"},{"line_number":34,"context_line":"        name: ntp"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"1f493fa4_2462e60a","line":31,"updated":"2020-04-30 00:28:59.000000000","message":"it\u0027s not really dib doing this but project-config elements.  it seems like we should fix it there rather than installing it in our elements just to then uninstall it","commit_id":"5b567eee1c37518783c3a37e75bce12dae7c057c"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"e55b848b0291545f06e887a30454bb59f931cc7e","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"- name: Remove ntp and run timesyncd"},{"line_number":30,"context_line":"  when: ansible_distribution_version is version(\u002718.04\u0027, \u0027\u003e\u003d\u0027)"},{"line_number":31,"context_line":"  block:"},{"line_number":32,"context_line":"    - name: Remove ntp"},{"line_number":33,"context_line":"      package:"},{"line_number":34,"context_line":"        name: ntp"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"1f493fa4_afd52f1e","line":31,"in_reply_to":"1f493fa4_2462e60a","updated":"2020-04-30 01:15:25.000000000","message":"i\u0027d prefer to see us go with something like https://review.opendev.org/#/c/724450/ to drop it completely","commit_id":"5b567eee1c37518783c3a37e75bce12dae7c057c"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"d21df878228d43c56303e3668c9e7bd1b6133fba","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"- name: Remove ntp and run timesyncd"},{"line_number":30,"context_line":"  when: ansible_distribution_version is version(\u002718.04\u0027, \u0027\u003e\u003d\u0027)"},{"line_number":31,"context_line":"  block:"},{"line_number":32,"context_line":"    - name: Remove ntp"},{"line_number":33,"context_line":"      package:"},{"line_number":34,"context_line":"        name: ntp"}],"source_content_type":"text/x-yaml","patch_set":18,"id":"1f493fa4_0acec761","line":31,"in_reply_to":"1f493fa4_afd52f1e","updated":"2020-04-30 12:54:57.000000000","message":"Totally agree.","commit_id":"5b567eee1c37518783c3a37e75bce12dae7c057c"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"2bc189a22645dd1b62659b2e650f3c7a9fe5748f","unresolved":false,"context_lines":[{"line_number":13,"context_line":"# NOTE(ianw) There are ordering issues with this.  Hopefully when"},{"line_number":14,"context_line":"# we\u0027re bionic only we can just remove ntp"},{"line_number":15,"context_line":"- name: Install NTP"},{"line_number":16,"context_line":"  when: ansible_distribution_version is version(\u002718.04\u0027, \u0027\u003c\u003d\u0027)"},{"line_number":17,"context_line":"  block:"},{"line_number":18,"context_line":"    - name: Install ntp"},{"line_number":19,"context_line":"      package:"}],"source_content_type":"text/x-yaml","patch_set":21,"id":"1f493fa4_a6cf0727","line":16,"updated":"2020-04-30 18:28:24.000000000","message":"This will *start* installing ntpd on Bionic now, right? Is that safe? The commit message doesn\u0027t specifically address this point.","commit_id":"038aae44130748ff11c2806189ce32b3c75e2385"}],"playbooks/roles/zuul-executor/tasks/main.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"164249b45d0c7ffe51687e288d0f30102a0ce335","unresolved":false,"context_lines":[{"line_number":16,"context_line":"  when: ansible_distribution_version is version(\u002720.04\u0027, \u0027\u003e\u003d\u0027)"},{"line_number":17,"context_line":"  block:"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"    - name: Add kubic project atomic GPG key"},{"line_number":20,"context_line":"      become: yes"},{"line_number":21,"context_line":"      apt_key:"},{"line_number":22,"context_line":"        data: \"{{ libcontainers_apt_key }}\""}],"source_content_type":"text/x-yaml","patch_set":23,"id":"1f493fa4_8178918f","line":19,"range":{"start_line":19,"start_character":22,"end_line":19,"end_character":37},"updated":"2020-05-06 19:50:14.000000000","message":"Project Atomic is EOL I think this should be \"Add Kubic Libcontainers OBS repo key\"","commit_id":"29fd0755b5a92583fd18d5b4a210af032b134778"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"75578c5bf6e85a1310d7a325ef66112b698023d0","unresolved":false,"context_lines":[{"line_number":16,"context_line":"  when: ansible_distribution_version is version(\u002720.04\u0027, \u0027\u003e\u003d\u0027)"},{"line_number":17,"context_line":"  block:"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"    - name: Add kubic project atomic GPG key"},{"line_number":20,"context_line":"      become: yes"},{"line_number":21,"context_line":"      apt_key:"},{"line_number":22,"context_line":"        data: \"{{ libcontainers_apt_key }}\""}],"source_content_type":"text/x-yaml","patch_set":23,"id":"1f493fa4_e17d156f","line":19,"range":{"start_line":19,"start_character":22,"end_line":19,"end_character":37},"in_reply_to":"1f493fa4_8178918f","updated":"2020-05-06 20:19:24.000000000","message":"Done","commit_id":"29fd0755b5a92583fd18d5b4a210af032b134778"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"164249b45d0c7ffe51687e288d0f30102a0ce335","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        update_cache: yes"},{"line_number":38,"context_line":"      when: projectatomic_repo is changed"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    - name: Remove old projectatomic ppa"},{"line_number":41,"context_line":"      file:"},{"line_number":42,"context_line":"        path: /etc/apt/sources.list.d/projectatomic-ubuntu-ppa-xenial.list"},{"line_number":43,"context_line":"        state: absent"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    - name: Remove old projectatomic key"},{"line_number":46,"context_line":"      apt_key:"},{"line_number":47,"context_line":"        id: 018BA5AD9DF57A4448F0E6CF8BECF1637AD8C79D"},{"line_number":48,"context_line":"        state: absent"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"- name: Install bindep"},{"line_number":51,"context_line":"  pip:"}],"source_content_type":"text/x-yaml","patch_set":23,"id":"1f493fa4_c159f935","line":48,"range":{"start_line":40,"start_character":0,"end_line":48,"end_character":21},"updated":"2020-05-06 19:50:14.000000000","message":"I don\u0027t think this is necessary because we don\u0027t install the project-atomic ppa on Focal according to the vars for Focal.","commit_id":"29fd0755b5a92583fd18d5b4a210af032b134778"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"75578c5bf6e85a1310d7a325ef66112b698023d0","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        update_cache: yes"},{"line_number":38,"context_line":"      when: projectatomic_repo is changed"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    - name: Remove old projectatomic ppa"},{"line_number":41,"context_line":"      file:"},{"line_number":42,"context_line":"        path: /etc/apt/sources.list.d/projectatomic-ubuntu-ppa-xenial.list"},{"line_number":43,"context_line":"        state: absent"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    - name: Remove old projectatomic key"},{"line_number":46,"context_line":"      apt_key:"},{"line_number":47,"context_line":"        id: 018BA5AD9DF57A4448F0E6CF8BECF1637AD8C79D"},{"line_number":48,"context_line":"        state: absent"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"- name: Install bindep"},{"line_number":51,"context_line":"  pip:"}],"source_content_type":"text/x-yaml","patch_set":23,"id":"1f493fa4_6191059a","line":48,"range":{"start_line":40,"start_character":0,"end_line":48,"end_character":21},"in_reply_to":"1f493fa4_c159f935","updated":"2020-05-06 20:19:24.000000000","message":"Done","commit_id":"29fd0755b5a92583fd18d5b4a210af032b134778"}],"roles/openafs-client/tasks/openafs-client/Debian.yaml":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"16d9a6dc54d405aeefc99ced84687dab95530bd6","unresolved":false,"context_lines":[{"line_number":3,"context_line":"- name: Install openstackci openafs PPA"},{"line_number":4,"context_line":"  apt_repository:"},{"line_number":5,"context_line":"    repo: \u0027ppa:openstack-ci-core/openafs\u0027"},{"line_number":6,"context_line":"  when: ansible_distribution_version is version(\u002718.04\u0027, \u0027\u003d\u003d\u0027)"},{"line_number":7,"context_line":"  become: yes"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- name: Install kernel headers dependency"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"1f493fa4_7ed3f260","line":6,"updated":"2020-04-29 02:01:17.000000000","message":"focal appears to have done the same thing as bionic and shipped a ~pre version of openafs [1].  bugs were filed without much response at the time. [2]\n\nthis caused us problems with bionic and openafs pretty much told us \"don\u0027t use that version in production\" :)\n\nluckily for this case openafs team is ahead of us and has 1.8.5 (the real release) [3], so i\u0027ve reused that and uploaded to this ppa to build for focal (they don\u0027t build arm64 for some reason?). this is better than it has been before when we\u0027ve wanted versions they didn\u0027t package yet.  but i think it keeps everything in order.\n\neither way, debugging afs issues like files seeming to be corrupt is a huge PITA so i wouldn\u0027t want to go into production with the ~pre versions from the base distro unfortunately.\n\n[1] https://launchpad.net/ubuntu/focal/+source/openafs\n[2] https://bugs.launchpad.net/ubuntu/+source/openafs/+bug/1832690\n[3] https://launchpad.net/~openafs/+archive/ubuntu/stable","commit_id":"d1ed25429041d6a6de1a09ea37161117e2b2730f"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"379545fac1ea1511c94ac7d2369a0e03cf3aa9c1","unresolved":false,"context_lines":[{"line_number":3,"context_line":"- name: Install openstackci openafs PPA"},{"line_number":4,"context_line":"  apt_repository:"},{"line_number":5,"context_line":"    repo: \u0027ppa:openstack-ci-core/openafs\u0027"},{"line_number":6,"context_line":"  when: ansible_distribution_version is version(\u002718.04\u0027, \u0027\u003d\u003d\u0027)"},{"line_number":7,"context_line":"  become: yes"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- name: Install kernel headers dependency"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"1f493fa4_590bb0d4","line":6,"in_reply_to":"1f493fa4_7ed3f260","updated":"2020-04-29 02:54:00.000000000","message":"tested with https://review.opendev.org/724214","commit_id":"d1ed25429041d6a6de1a09ea37161117e2b2730f"}]}
