)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"3341679795fbf3fe04783b54d9fa51ac929e4f7b","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Similarly, something in openstacksdk 0.99.0 has started resulting in"},{"line_number":15,"context_line":"a network response parsing error from Rackspace, so by pinning to"},{"line_number":16,"context_line":"the version before it we avoid that problem."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"The script itself had a couple of issues once rehomed, the first"},{"line_number":19,"context_line":"being it was looking for Ansible playbooks relative to its former"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"decc1269_9bcf3b57","line":16,"updated":"2022-11-23 19:47:20.000000000","message":"I guess I can drop this sentence now.","commit_id":"be37abf1f7bf4b26abe38b8633c1c57da5a26f12"}],"launch/pyproject.toml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"0d9bd4ff5059ef12f605df5f9e2c3691d4c8b514","unresolved":true,"context_lines":[{"line_number":15,"context_line":"dependencies \u003d ["},{"line_number":16,"context_line":"  \"paramiko\u003e\u003d2.9.1\","},{"line_number":17,"context_line":"  # This is a very specific list which is known to work"},{"line_number":18,"context_line":"  # with RAX storage..."},{"line_number":19,"context_line":"  \"openstackclient\","},{"line_number":20,"context_line":"  \"python-cinderclient\u003c8\","},{"line_number":21,"context_line":"]"}],"source_content_type":"text/x-toml","patch_set":1,"id":"4ec06298_652f3963","line":18,"updated":"2022-11-22 18:29:36.000000000","message":"This isn\u0027t a specific list anymore, maybe reword to just mention the cap is needed for cinder v2?","commit_id":"99a42e86704eb929f55e6231d9c4fcb1295885fb"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"6f4c0d70db959e9751f9754d1d09beb802798611","unresolved":false,"context_lines":[{"line_number":15,"context_line":"dependencies \u003d ["},{"line_number":16,"context_line":"  \"paramiko\u003e\u003d2.9.1\","},{"line_number":17,"context_line":"  # This is a very specific list which is known to work"},{"line_number":18,"context_line":"  # with RAX storage..."},{"line_number":19,"context_line":"  \"openstackclient\","},{"line_number":20,"context_line":"  \"python-cinderclient\u003c8\","},{"line_number":21,"context_line":"]"}],"source_content_type":"text/x-toml","patch_set":1,"id":"b191945d_a73ea5e7","line":18,"in_reply_to":"4ec06298_652f3963","updated":"2022-11-29 23:05:31.000000000","message":"Done","commit_id":"99a42e86704eb929f55e6231d9c4fcb1295885fb"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"0d9bd4ff5059ef12f605df5f9e2c3691d4c8b514","unresolved":true,"context_lines":[{"line_number":16,"context_line":"  \"paramiko\u003e\u003d2.9.1\","},{"line_number":17,"context_line":"  # This is a very specific list which is known to work"},{"line_number":18,"context_line":"  # with RAX storage..."},{"line_number":19,"context_line":"  \"openstackclient\","},{"line_number":20,"context_line":"  \"python-cinderclient\u003c8\","},{"line_number":21,"context_line":"]"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-toml","patch_set":1,"id":"246f104f_8cd9e4a9","line":19,"updated":"2022-11-22 18:29:36.000000000","message":"This should be \"python-openstackclient\"","commit_id":"99a42e86704eb929f55e6231d9c4fcb1295885fb"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"2b1d8c57be914b745c93e2cf06c8005f76d265f2","unresolved":false,"context_lines":[{"line_number":16,"context_line":"  \"paramiko\u003e\u003d2.9.1\","},{"line_number":17,"context_line":"  # This is a very specific list which is known to work"},{"line_number":18,"context_line":"  # with RAX storage..."},{"line_number":19,"context_line":"  \"openstackclient\","},{"line_number":20,"context_line":"  \"python-cinderclient\u003c8\","},{"line_number":21,"context_line":"]"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-toml","patch_set":1,"id":"2aca526b_3b51b6be","line":19,"updated":"2022-11-22 19:42:29.000000000","message":"Well, I was using openstackclient so we\u0027d get the full set of plugins rather than needing to individually specify them. Should we just start including only plugins we think we\u0027ll need instead?","commit_id":"99a42e86704eb929f55e6231d9c4fcb1295885fb"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"ac4debd16a5aa782dd4ce7324897d6d8055f16a3","unresolved":true,"context_lines":[{"line_number":16,"context_line":"  \"paramiko\u003e\u003d2.9.1\","},{"line_number":17,"context_line":"  # This is a very specific list which is known to work"},{"line_number":18,"context_line":"  # with RAX storage..."},{"line_number":19,"context_line":"  \"openstackclient\","},{"line_number":20,"context_line":"  \"python-cinderclient\u003c8\","},{"line_number":21,"context_line":"]"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-toml","patch_set":1,"id":"ad9d4557_5822afed","line":19,"in_reply_to":"246f104f_8cd9e4a9","updated":"2022-11-22 19:10:21.000000000","message":"Both packages exist. One is an alias that pulls in extra client libs. But I agree maybe we should be specific about that rather than using an alias.","commit_id":"99a42e86704eb929f55e6231d9c4fcb1295885fb"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"779efba7fb939c249a3c40e756c9c5429c3079cf","unresolved":false,"context_lines":[{"line_number":16,"context_line":"  \"paramiko\u003e\u003d2.9.1\","},{"line_number":17,"context_line":"  # This is a very specific list which is known to work"},{"line_number":18,"context_line":"  # with RAX storage..."},{"line_number":19,"context_line":"  \"openstackclient\","},{"line_number":20,"context_line":"  \"python-cinderclient\u003c8\","},{"line_number":21,"context_line":"]"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-toml","patch_set":1,"id":"65156848_8edcb89f","line":19,"in_reply_to":"ad9d4557_5822afed","updated":"2022-11-23 18:46:23.000000000","message":"Yes, pulling in all kinds of clients for services we do not use seems unneccessary.","commit_id":"99a42e86704eb929f55e6231d9c4fcb1295885fb"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"779efba7fb939c249a3c40e756c9c5429c3079cf","unresolved":true,"context_lines":[{"line_number":16,"context_line":"  \"paramiko\u003e\u003d2.9.1\","},{"line_number":17,"context_line":"  # This is a very specific list which is known to work"},{"line_number":18,"context_line":"  # with RAX storage..."},{"line_number":19,"context_line":"  \"openstacksdk\u003c0.99\","},{"line_number":20,"context_line":"  \"python-cinderclient\u003c8\","},{"line_number":21,"context_line":"  \"python-openstackclient\","},{"line_number":22,"context_line":"]"}],"source_content_type":"text/x-toml","patch_set":3,"id":"591c9488_626a92ef","line":19,"updated":"2022-11-23 18:46:23.000000000","message":"A new openstacksdk is on its way that will make this cap obsolete, I would suggest to wait for that\nhttps://review.opendev.org/c/openstack/releases/+/865456","commit_id":"42d29622c67e8739190b0efa2cbd259756cb6e46"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"6f4c0d70db959e9751f9754d1d09beb802798611","unresolved":true,"context_lines":[{"line_number":16,"context_line":"  \"paramiko\u003e\u003d2.9.1\","},{"line_number":17,"context_line":"  \"python-openstackclient\","},{"line_number":18,"context_line":"  # Pin to a version of cinderclient known to work with RAX storage..."},{"line_number":19,"context_line":"  \"python-cinderclient\u003c8\","},{"line_number":20,"context_line":"]"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[project.urls]"}],"source_content_type":"text/x-toml","patch_set":5,"id":"f45c386c_83f34dd7","line":19,"updated":"2022-11-29 23:05:31.000000000","message":"Not listing openstacksdk here means we won\u0027t auto update to 0.103 now that it is available as pip should find 0.102 is sufficient. Also we\u0027ll lose the context that we need \u003e\u003d0.103. For historical debugging. Should we be explicit here to help our future selves and avoid needing to manually update the dep in the venv?","commit_id":"0c66276cae65e2ae4e9516518153b816e55f6d81"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"0b61506522077195c3165881cd3fef37ed91317c","unresolved":false,"context_lines":[{"line_number":16,"context_line":"  \"paramiko\u003e\u003d2.9.1\","},{"line_number":17,"context_line":"  \"python-openstackclient\","},{"line_number":18,"context_line":"  # Pin to a version of cinderclient known to work with RAX storage..."},{"line_number":19,"context_line":"  \"python-cinderclient\u003c8\","},{"line_number":20,"context_line":"]"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"[project.urls]"}],"source_content_type":"text/x-toml","patch_set":5,"id":"fd9bdfa6_489de7d6","line":19,"in_reply_to":"f45c386c_83f34dd7","updated":"2022-11-30 17:39:43.000000000","message":"Done","commit_id":"0c66276cae65e2ae4e9516518153b816e55f6d81"}],"launch/src/opendev_launch/launch_node.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"f7f142e962d1cb95008f0b320137752a4e8efa25","unresolved":true,"context_lines":[{"line_number":160,"context_line":"        for inventory_cache in os.listdir(inventory_cache_dir):"},{"line_number":161,"context_line":"            os.unlink(os.path.join(inventory_cache_dir, inventory_cache))"},{"line_number":162,"context_line":"    except FileNotFoundError:"},{"line_number":163,"context_line":"        pass"},{"line_number":164,"context_line":""},{"line_number":165,"context_line":"    with JobDir(keep) as jobdir:"},{"line_number":166,"context_line":"        # Update the generated-groups file globally and incorporate it"}],"source_content_type":"text/x-python","patch_set":2,"id":"e1621fdc_c7f06bf6","line":163,"updated":"2022-11-22 23:29:19.000000000","message":"note I doubt this file will ever exist with modern ansible given it hasn\u0027t been created on the new bridge yet. But we can clean this up later.","commit_id":"391ea0eb92ae59aa13bf2f8202cc64225c5082ec"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"d835db1fd51a04e65b5e485fe359dc3f8050a181","unresolved":true,"context_lines":[{"line_number":215,"context_line":"                \u0027apply-package-updates.yaml\u0027,"},{"line_number":216,"context_line":"        ]:"},{"line_number":217,"context_line":"            run(ansible_cmd + ["},{"line_number":218,"context_line":"                os.path.join(\u0027/home/zuul/src/opendev.org/opendev/system-config/playbooks\u0027, playbook)],"},{"line_number":219,"context_line":"                env\u003djobdir.env)"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"    try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf1ffb5d_f81ae7aa","line":218,"updated":"2022-11-22 22:58:04.000000000","message":"This seems fine, though we could also make a PLAYBOOKS_DIR global if we like","commit_id":"391ea0eb92ae59aa13bf2f8202cc64225c5082ec"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"f7f142e962d1cb95008f0b320137752a4e8efa25","unresolved":true,"context_lines":[{"line_number":215,"context_line":"                \u0027apply-package-updates.yaml\u0027,"},{"line_number":216,"context_line":"        ]:"},{"line_number":217,"context_line":"            run(ansible_cmd + ["},{"line_number":218,"context_line":"                os.path.join(\u0027/home/zuul/src/opendev.org/opendev/system-config/playbooks\u0027, playbook)],"},{"line_number":219,"context_line":"                env\u003djobdir.env)"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"    try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"e591483b_5de9bbd5","line":218,"in_reply_to":"bf1ffb5d_f81ae7aa","updated":"2022-11-22 23:29:19.000000000","message":"ya I think a good followup would be to make a flag we can set on the command to change the path if we are using some locally edited playbooks.","commit_id":"391ea0eb92ae59aa13bf2f8202cc64225c5082ec"}]}