)]}'
{".zuul.yaml":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"869f844376b3762890f2cf12f8f64648ea839471","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"- project:"},{"line_number":14,"context_line":"    templates:"},{"line_number":15,"context_line":"      - docs-on-readthedocs"},{"line_number":16,"context_line":"    vars:"},{"line_number":17,"context_line":"      rtd_webhook_id: \u002747687\u0027"},{"line_number":18,"context_line":"      rtd_project_name: \u0027airship-treasuremap\u0027"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_7bfe003a","line":15,"range":{"start_line":15,"start_character":8,"end_line":15,"end_character":27},"updated":"2020-02-07 20:43:32.000000000","message":"this run openstack-tox-docs, so you have both jobs running.\n\nWhat about removing it or just adding the post job from the template?","commit_id":"2cb6662464a6da9c01707ca9601fa74b07b03b16"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"d0ef69717edafdc93e83a1c29d1b664c1e01a7f7","unresolved":false,"context_lines":[{"line_number":18,"context_line":"      rtd_project_name: \u0027airship-treasuremap\u0027"},{"line_number":19,"context_line":"    check:"},{"line_number":20,"context_line":"      jobs:"},{"line_number":21,"context_line":"        - opendev-tox-docs"},{"line_number":22,"context_line":"        - treasuremap-seaworthy-site-lint"},{"line_number":23,"context_line":"        - treasuremap-seaworthy-virt-site-lint"},{"line_number":24,"context_line":"        - treasuremap-airskiff-ubuntu-site-lint"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_12f47087","line":21,"updated":"2020-02-10 22:43:13.000000000","message":"Ahh, yep, I see now the parent of the promote job sets \"download_artifact_job: openstack-tox-docs\" so it explicitly depends on that job anyway and won\u0027t make use of the opendev-tox-docs I\u0027m adding here. I\u0027ll rework it to remove these now.","commit_id":"2cb6662464a6da9c01707ca9601fa74b07b03b16"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"8db47f956737380ff796e3944757078fdafc0de9","unresolved":false,"context_lines":[{"line_number":18,"context_line":"      rtd_project_name: \u0027airship-treasuremap\u0027"},{"line_number":19,"context_line":"    check:"},{"line_number":20,"context_line":"      jobs:"},{"line_number":21,"context_line":"        - opendev-tox-docs"},{"line_number":22,"context_line":"        - treasuremap-seaworthy-site-lint"},{"line_number":23,"context_line":"        - treasuremap-seaworthy-virt-site-lint"},{"line_number":24,"context_line":"        - treasuremap-airskiff-ubuntu-site-lint"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_de94c237","line":21,"updated":"2020-02-07 21:34:05.000000000","message":"This is using opendev-tox-docs instead of openstack-tox-docs. I was concerned the latter might include unnecessary OpenStackisms in the build it generates. I agree not running both would be preferable from a resource consumption perspective, and it may make sense to convert the docs-on-readthedocs project-template to use the more generic OpenDev job which only requires opendev/base-jobs and not openstack/openstack-zuul-jobs.","commit_id":"2cb6662464a6da9c01707ca9601fa74b07b03b16"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"b5bd17bb84f9d2d398fd6587e9eb03e442a8221f","unresolved":false,"context_lines":[{"line_number":18,"context_line":"      rtd_project_name: \u0027airship-treasuremap\u0027"},{"line_number":19,"context_line":"    check:"},{"line_number":20,"context_line":"      jobs:"},{"line_number":21,"context_line":"        - opendev-tox-docs"},{"line_number":22,"context_line":"        - treasuremap-seaworthy-site-lint"},{"line_number":23,"context_line":"        - treasuremap-seaworthy-virt-site-lint"},{"line_number":24,"context_line":"        - treasuremap-airskiff-ubuntu-site-lint"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_3b8968c7","line":21,"range":{"start_line":21,"start_character":10,"end_line":21,"end_character":26},"updated":"2020-02-07 20:45:57.000000000","message":"remove those, not needed. your change in project-config says \"openstack-tox-docs\" as job and that is part of docs-on-readthedocs.","commit_id":"2cb6662464a6da9c01707ca9601fa74b07b03b16"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"a4739df4fc6efc57c599760c8399a5bc5fc92ad4","unresolved":false,"context_lines":[{"line_number":18,"context_line":"      rtd_project_name: \u0027airship-treasuremap\u0027"},{"line_number":19,"context_line":"    check:"},{"line_number":20,"context_line":"      jobs:"},{"line_number":21,"context_line":"        - opendev-tox-docs"},{"line_number":22,"context_line":"        - treasuremap-seaworthy-site-lint"},{"line_number":23,"context_line":"        - treasuremap-seaworthy-virt-site-lint"},{"line_number":24,"context_line":"        - treasuremap-airskiff-ubuntu-site-lint"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3fa7e38b_8f8376aa","line":21,"in_reply_to":"3fa7e38b_de94c237","updated":"2020-02-08 09:48:15.000000000","message":"the only OpenStackisms are:\n* checking out requirements\n* Building PDF if docs-pdf tox.ini target exists\n\nNeither will affect this.\n\nThe promote job you use is setup to use openstack-tox-docs.\n\nSo, this job is not needed for your change as is.","commit_id":"2cb6662464a6da9c01707ca9601fa74b07b03b16"}]}
