)]}'
{"playbooks/roles/install-apt-repo/tasks/main.yaml":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"8dc6f4e68be11c3a072dc2d913277f5263bce39e","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  become: yes"},{"line_number":3,"context_line":"  apt_key:"},{"line_number":4,"context_line":"    data: \"{{ repo_key }}\""},{"line_number":5,"context_line":"    keyring: \"/etc/apt/trusted.gpg.d/{{ repo_name }}.gpg\""},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"- name: Add apt repo"},{"line_number":8,"context_line":"  become: yes"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"ff570b3c_fde35f03","line":5,"updated":"2020-05-20 18:04:35.000000000","message":"I\u0027m surprised this is working, I thought apt expected the .gpg extension on GnuPG\u0027s native binary format exports and .asc for ASCII-armored copies. Local testing on my Debian/Sid workstation indicates that when I move my personal ASCII-armored repo key from .asc to .gpg then `apt update` it complains \"The key(s) in the keyring /etc/apt/trusted.gpg.d/fungi-yuggoth-org.gpg are ignored as the file has an unsupported filetype.\"","commit_id":"01c01f2b6308501f51d1bbc84c8d2572cd84b5c2"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"0c2355382dea84f5bb2eae7bb9251f8603f085aa","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  become: yes"},{"line_number":3,"context_line":"  apt_key:"},{"line_number":4,"context_line":"    data: \"{{ repo_key }}\""},{"line_number":5,"context_line":"    keyring: \"/etc/apt/trusted.gpg.d/{{ repo_name }}.gpg\""},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"- name: Add apt repo"},{"line_number":8,"context_line":"  become: yes"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"ff570b3c_7dbd8fb5","line":5,"updated":"2020-05-20 18:11:20.000000000","message":"In fact, it\u0027s required to work, using .asc here fails (see earlier PS).  Apparently apt-key will take AA input and write out a binary .gpg file.","commit_id":"01c01f2b6308501f51d1bbc84c8d2572cd84b5c2"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"3925502337aeed26ed4c9ee5c479c68eab89527d","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  become: yes"},{"line_number":3,"context_line":"  apt_key:"},{"line_number":4,"context_line":"    data: \"{{ repo_key }}\""},{"line_number":5,"context_line":"    keyring: \"/etc/apt/trusted.gpg.d/{{ repo_name }}.gpg\""},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"- name: Add apt repo"},{"line_number":8,"context_line":"  become: yes"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"ff570b3c_dd5adb80","line":5,"updated":"2020-05-20 18:15:52.000000000","message":"Oh! I missed this was going through the apt-key command instead of installing the file directly. We could probably just install the file directly? Or is this the workaround for Xenial lacking .asc file support in /etc/apt/trusted.gpg.d/ ? Anyway, yes this makes sense. Thanks for clarifying.","commit_id":"01c01f2b6308501f51d1bbc84c8d2572cd84b5c2"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"8d770d3a91029f10a68ee519c51fd002527efec0","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  become: yes"},{"line_number":3,"context_line":"  apt_key:"},{"line_number":4,"context_line":"    data: \"{{ repo_key }}\""},{"line_number":5,"context_line":"    keyring: \"/etc/apt/trusted.gpg.d/{{ repo_name }}.gpg\""},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"- name: Add apt repo"},{"line_number":8,"context_line":"  become: yes"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"ff570b3c_486727b8","line":5,"updated":"2020-05-20 18:46:37.000000000","message":"Yep, this should work in all distros.  Later we can simplify.","commit_id":"01c01f2b6308501f51d1bbc84c8d2572cd84b5c2"}],"playbooks/roles/zuul-executor/tasks/main.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"0245e4ab14c15048540749b96a5716d4ad65ebad","unresolved":false,"context_lines":[{"line_number":12,"context_line":"  vars:"},{"line_number":13,"context_line":"    repo_name: \"{{ item.name }}\""},{"line_number":14,"context_line":"    repo_key: \"{{ item.key }}\""},{"line_number":15,"context_line":"    repo_content: \" {{item.content }}\""},{"line_number":16,"context_line":"  loop: \u0027{{ zuul_executor_ppas }}\u0027"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"- name: Atomic for focal"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"ff570b3c_0b7a46b8","line":15,"range":{"start_line":15,"start_character":19,"end_line":15,"end_character":20},"updated":"2020-05-20 23:20:51.000000000","message":"Is this extra space intentionally added?","commit_id":"b173fcb1d9c33b0a42c3b5371961c83760b10d8b"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"c3b7a4f3920e13fd7640b9e5c444992641bf4363","unresolved":false,"context_lines":[{"line_number":12,"context_line":"  vars:"},{"line_number":13,"context_line":"    repo_name: \"{{ item.name }}\""},{"line_number":14,"context_line":"    repo_key: \"{{ item.key }}\""},{"line_number":15,"context_line":"    repo_content: \" {{item.content }}\""},{"line_number":16,"context_line":"  loop: \u0027{{ zuul_executor_ppas }}\u0027"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"- name: Atomic for focal"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"ff570b3c_4b1f1e6b","line":15,"updated":"2020-05-20 23:22:13.000000000","message":"No.  But this is definitely tested so seems okay.","commit_id":"b173fcb1d9c33b0a42c3b5371961c83760b10d8b"}],"playbooks/roles/zuul-executor/vars/main.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"0245e4ab14c15048540749b96a5716d4ad65ebad","unresolved":false,"context_lines":[{"line_number":1,"context_line":"openstack_ci_core_ppa_key: |"},{"line_number":2,"context_line":"  -----BEGIN PGP PUBLIC KEY BLOCK-----"},{"line_number":3,"context_line":"  Version: GnuPG v1"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"ff570b3c_2b2d8aa7","line":1,"updated":"2020-05-20 23:20:51.000000000","message":"We should consider putting these keys into the all group as their content will be valid anywhere we want to use them.","commit_id":"b173fcb1d9c33b0a42c3b5371961c83760b10d8b"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"c3b7a4f3920e13fd7640b9e5c444992641bf4363","unresolved":false,"context_lines":[{"line_number":1,"context_line":"openstack_ci_core_ppa_key: |"},{"line_number":2,"context_line":"  -----BEGIN PGP PUBLIC KEY BLOCK-----"},{"line_number":3,"context_line":"  Version: GnuPG v1"},{"line_number":4,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"ff570b3c_ab123a60","line":1,"updated":"2020-05-20 23:22:13.000000000","message":"Yep.","commit_id":"b173fcb1d9c33b0a42c3b5371961c83760b10d8b"}]}