)]}'
{"playbooks/translation/pre.yaml":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"97e0f28f2f8b24925a11c63aff4fee4b167ebb8e","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  roles:"},{"line_number":3,"context_line":"    - role: bindep"},{"line_number":4,"context_line":"      bindep_profile: test"},{"line_number":5,"context_line":"      bindep_dir: \"src/{{ zuul.project.canonical_name }}\""},{"line_number":6,"context_line":"    - ensure-tox"},{"line_number":7,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_ef6db598","line":5,"updated":"2017-10-20 05:02:39.000000000","message":"{{ zuul.project.src_dir }} might be easier?","commit_id":"b0a976ac8ca7aeae9d5f9a66e0fa50534ae15b6f"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"a8de522c2ec13c3dcafd221532fa0443e1b114f0","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  roles:"},{"line_number":3,"context_line":"    - role: bindep"},{"line_number":4,"context_line":"      bindep_profile: test"},{"line_number":5,"context_line":"      bindep_dir: \"src/{{ zuul.project.canonical_name }}\""},{"line_number":6,"context_line":"    - ensure-tox"},{"line_number":7,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f4b6375_0f093130","line":5,"in_reply_to":"3f4b6375_ef6db598","updated":"2017-10-20 06:05:27.000000000","message":"This is what we use everywhere else for bindep_dir.\n\nDo you want to do a followup patch to change all of them?","commit_id":"b0a976ac8ca7aeae9d5f9a66e0fa50534ae15b6f"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"fb7c79f28e973d69c220f29cfaa6f2f52faac825","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  roles:"},{"line_number":3,"context_line":"    - role: bindep"},{"line_number":4,"context_line":"      bindep_profile: test"},{"line_number":5,"context_line":"      bindep_dir: \"{{ zuul.project.src_dir }}\""},{"line_number":6,"context_line":"    - ensure-tox"},{"line_number":7,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f4b6375_73a887da","line":5,"updated":"2017-10-20 10:57:19.000000000","message":"This is no longer needed, we\u0027ve set it to the default in the role.","commit_id":"352d46351b1b2a0d6eaa1fe4c1cc925288801137"}],"zuul.d/jobs.yaml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"b736a2ec630fa86ec95fd62b9b9bd07b47aa1be1","unresolved":false,"context_lines":[{"line_number":661,"context_line":"    name: upstream-translation-update"},{"line_number":662,"context_line":"    parent: propose-updates"},{"line_number":663,"context_line":"    pre-run:"},{"line_number":664,"context_line":"      - playbooks/proposal/pre"},{"line_number":665,"context_line":"      - playbooks/translation/pre"},{"line_number":666,"context_line":"    run: playbooks/translation/upstream-translation-update"},{"line_number":667,"context_line":"    post-run: playbooks/publish/openstack-artifacts"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f4b6375_800f2ee0","line":664,"updated":"2017-10-20 08:05:44.000000000","message":"This is already included in pre-run for the parent job, isn\u0027t that cumulative? i.e. wouldn\u0027t just adding playbooks/translation/pre be enough?","commit_id":"352d46351b1b2a0d6eaa1fe4c1cc925288801137"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"fb7c79f28e973d69c220f29cfaa6f2f52faac825","unresolved":false,"context_lines":[{"line_number":661,"context_line":"    name: upstream-translation-update"},{"line_number":662,"context_line":"    parent: propose-updates"},{"line_number":663,"context_line":"    pre-run:"},{"line_number":664,"context_line":"      - playbooks/proposal/pre"},{"line_number":665,"context_line":"      - playbooks/translation/pre"},{"line_number":666,"context_line":"    run: playbooks/translation/upstream-translation-update"},{"line_number":667,"context_line":"    post-run: playbooks/publish/openstack-artifacts"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f4b6375_53dd8b6e","line":664,"in_reply_to":"3f4b6375_800f2ee0","updated":"2017-10-20 10:57:19.000000000","message":"Right, if this was added, we\u0027d run the pre playbook twice. So, this can be removed","commit_id":"352d46351b1b2a0d6eaa1fe4c1cc925288801137"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"fb7c79f28e973d69c220f29cfaa6f2f52faac825","unresolved":false,"context_lines":[{"line_number":678,"context_line":"    name: propose-translation-update"},{"line_number":679,"context_line":"    parent: propose-updates"},{"line_number":680,"context_line":"    pre-run:"},{"line_number":681,"context_line":"      - playbooks/proposal/pre"},{"line_number":682,"context_line":"      - playbooks/translation/pre"},{"line_number":683,"context_line":"    run: playbooks/translation/propose-translation-update"},{"line_number":684,"context_line":"    required-projects:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f4b6375_33d8cf7d","line":681,"updated":"2017-10-20 10:57:19.000000000","message":"same here","commit_id":"352d46351b1b2a0d6eaa1fe4c1cc925288801137"}]}