)]}'
{"jenkins/jobs/puppet-module-jobs.yaml":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"f42c9c1d6b274e0541e334330c500ce47cdcb796","unresolved":false,"context_lines":[{"line_number":239,"context_line":"          template-name: \"{template-name}\""},{"line_number":240,"context_line":"      - link-logs"},{"line_number":241,"context_line":"      - zuul-clone:"},{"line_number":242,"context_line":"          project: \"openstack/puppet-{project}\""},{"line_number":243,"context_line":"      - install-distro-packages"},{"line_number":244,"context_line":"      - puppet-prepare-node"},{"line_number":245,"context_line":"      - shell: |"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"dada55a8_db21294f","line":242,"updated":"2016-07-19 19:25:56.000000000","message":"can someone confirm it will work?","commit_id":"d5feda6f35f73cc2adf52f0e909114bd26e9e754"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"61f36a3673171fef74b832c2b3b83d0893ed72f1","unresolved":false,"context_lines":[{"line_number":231,"context_line":"      - console-log"},{"line_number":232,"context_line":""},{"line_number":233,"context_line":"- job-template:"},{"line_number":234,"context_line":"    name: \u0027gate-{name}-lint-{node}\u0027"},{"line_number":235,"context_line":"    node: \u0027{node}\u0027"},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"    builders:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dada55a8_81befc62","line":234,"updated":"2016-07-20 12:52:42.000000000","message":"this is too generic, you should use \u0027gate-{name}-puppet-lint-{node}","commit_id":"c7dd5ffc5d3b5e974ad99133457c475a6bd556fc"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"61f36a3673171fef74b832c2b3b83d0893ed72f1","unresolved":false,"context_lines":[{"line_number":239,"context_line":"          template-name: \"{template-name}\""},{"line_number":240,"context_line":"      - link-logs"},{"line_number":241,"context_line":"      - zuul-clone:"},{"line_number":242,"context_line":"          project: \"openstack/puppet-{project}\""},{"line_number":243,"context_line":"      - install-distro-packages"},{"line_number":244,"context_line":"      - puppet-prepare-node"},{"line_number":245,"context_line":"      - shell: |"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dada55a8_61c308ee","line":242,"updated":"2016-07-20 12:52:42.000000000","message":"you should use ZUUL_PROJECT","commit_id":"c7dd5ffc5d3b5e974ad99133457c475a6bd556fc"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"61f36a3673171fef74b832c2b3b83d0893ed72f1","unresolved":false,"context_lines":[{"line_number":256,"context_line":"      - console-log"},{"line_number":257,"context_line":""},{"line_number":258,"context_line":"- job-template:"},{"line_number":259,"context_line":"    name: \u0027gate-{name}-syntax-{puppet_version}-{node}\u0027"},{"line_number":260,"context_line":"    node: \u0027{node}\u0027"},{"line_number":261,"context_line":""},{"line_number":262,"context_line":"    builders:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dada55a8_c1b47483","line":259,"updated":"2016-07-20 12:52:42.000000000","message":"same comment as above, gate-{name}-puppet-syntax....","commit_id":"c7dd5ffc5d3b5e974ad99133457c475a6bd556fc"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"61f36a3673171fef74b832c2b3b83d0893ed72f1","unresolved":false,"context_lines":[{"line_number":264,"context_line":"          template-name: \"{template-name}\""},{"line_number":265,"context_line":"      - link-logs"},{"line_number":266,"context_line":"      - zuul-clone:"},{"line_number":267,"context_line":"          project: \"openstack/puppet-{project}\""},{"line_number":268,"context_line":"      - install-distro-packages"},{"line_number":269,"context_line":"      - puppet-prepare-node"},{"line_number":270,"context_line":"      - shell: |"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dada55a8_a1b9807b","line":267,"updated":"2016-07-20 12:52:42.000000000","message":"same comment","commit_id":"c7dd5ffc5d3b5e974ad99133457c475a6bd556fc"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"61f36a3673171fef74b832c2b3b83d0893ed72f1","unresolved":false,"context_lines":[{"line_number":269,"context_line":"      - puppet-prepare-node"},{"line_number":270,"context_line":"      - shell: |"},{"line_number":271,"context_line":"          #!/bin/bash -xe"},{"line_number":272,"context_line":"          cd openstack/puppet-{project}"},{"line_number":273,"context_line":"          # FUTURE_PARSER\u003dyes is only supported by Puppet 3.x"},{"line_number":274,"context_line":"          if [ \"{puppet_version}\" -lt \"4\" ]; then"},{"line_number":275,"context_line":"              export FUTURE_PARSER\u003dyes"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dada55a8_01b32c99","line":272,"updated":"2016-07-20 12:52:42.000000000","message":"use ZUUL_PROJECT","commit_id":"c7dd5ffc5d3b5e974ad99133457c475a6bd556fc"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"61f36a3673171fef74b832c2b3b83d0893ed72f1","unresolved":false,"context_lines":[{"line_number":286,"context_line":"      - console-log"},{"line_number":287,"context_line":""},{"line_number":288,"context_line":"- job-template:"},{"line_number":289,"context_line":"    name: \u0027gate-{name}-unit-{node}\u0027"},{"line_number":290,"context_line":"    node: \u0027{node}\u0027"},{"line_number":291,"context_line":""},{"line_number":292,"context_line":"    builders:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dada55a8_e1aff827","line":289,"updated":"2016-07-20 12:52:42.000000000","message":"again, too generic gate-{name}-puppet-unit-{node}","commit_id":"c7dd5ffc5d3b5e974ad99133457c475a6bd556fc"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"61f36a3673171fef74b832c2b3b83d0893ed72f1","unresolved":false,"context_lines":[{"line_number":311,"context_line":"      - console-log"},{"line_number":312,"context_line":""},{"line_number":313,"context_line":"- job-template:"},{"line_number":314,"context_line":"    name: \u0027gate-{name}-beaker-rspec-{node}\u0027"},{"line_number":315,"context_line":"    node: \u0027{node}\u0027"},{"line_number":316,"context_line":""},{"line_number":317,"context_line":"    builders:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dada55a8_41a9a423","line":314,"updated":"2016-07-20 12:52:42.000000000","message":"use puppet-beaker here","commit_id":"c7dd5ffc5d3b5e974ad99133457c475a6bd556fc"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"61f36a3673171fef74b832c2b3b83d0893ed72f1","unresolved":false,"context_lines":[{"line_number":459,"context_line":"    name: \u0027puppet-openstack-libraries-jobs\u0027"},{"line_number":460,"context_line":"    node:"},{"line_number":461,"context_line":"      - centos-7"},{"line_number":462,"context_line":"    project:"},{"line_number":463,"context_line":"      - openstack_spec_helper"},{"line_number":464,"context_line":"      - openstack-cookiecutter"},{"line_number":465,"context_line":"      - modulesync-configs"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dada55a8_21a63053","line":462,"updated":"2016-07-20 12:52:42.000000000","message":"this is not correct, and should be removed.","commit_id":"c7dd5ffc5d3b5e974ad99133457c475a6bd556fc"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"61f36a3673171fef74b832c2b3b83d0893ed72f1","unresolved":false,"context_lines":[{"line_number":467,"context_line":"      - 3"},{"line_number":468,"context_line":"      - 4"},{"line_number":469,"context_line":"    jobs:"},{"line_number":470,"context_line":"      - \u0027gate-puppet-{project}-lint-{node}\u0027"},{"line_number":471,"context_line":"      - \u0027gate-puppet-{project}-syntax-{puppet_version}-{node}\u0027"},{"line_number":472,"context_line":"      - \u0027gate-puppet-{project}-unit-{node}\u0027"},{"line_number":473,"context_line":"      - \u0027gate-puppet-{project}-beaker-rspec-{node}\u0027"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dada55a8_81655c1f","line":470,"updated":"2016-07-20 12:52:42.000000000","message":"switch theses to use {name} for example: \u0027gate-{name}-puppet-lint-{node}\u0027","commit_id":"c7dd5ffc5d3b5e974ad99133457c475a6bd556fc"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"d8e56e8956cbadbdf17aca51b810bdba94596a44","unresolved":false,"context_lines":[{"line_number":264,"context_line":"          template-name: \"{template-name}\""},{"line_number":265,"context_line":"      - link-logs"},{"line_number":266,"context_line":"      - zuul-clone:"},{"line_number":267,"context_line":"          project: $ZUUL_PROJECT"},{"line_number":268,"context_line":"      - install-distro-packages"},{"line_number":269,"context_line":"      - puppet-prepare-node"},{"line_number":270,"context_line":"      - shell: |"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"dada55a8_e0322da6","line":267,"updated":"2016-07-21 03:21:06.000000000","message":"$ZUUL_PROJECT is not expanded here.\n\nI guess you would want to make this something like a template\n\n gate-{name}-puppet-syntax-{puppet_version}-{node}\n\nand use {name} in here\n\nThese names are very long ... that can cause problems with path names (at least with virtualenv\u0027s).  if we can find a way to make them shorter, that would be good","commit_id":"432b2f6c7b94bdc02aaed00e1bdb94d633c2d36f"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"4cd50d3afeb217a2933b480682d7b54ed9b924f3","unresolved":false,"context_lines":[{"line_number":264,"context_line":"          template-name: \"{template-name}\""},{"line_number":265,"context_line":"      - link-logs"},{"line_number":266,"context_line":"      - zuul-clone:"},{"line_number":267,"context_line":"          project: $ZUUL_PROJECT"},{"line_number":268,"context_line":"      - install-distro-packages"},{"line_number":269,"context_line":"      - puppet-prepare-node"},{"line_number":270,"context_line":"      - shell: |"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"dada55a8_a50f422d","line":267,"updated":"2016-07-22 16:16:54.000000000","message":"However, $ZUUL_PROJECT is subsequently expanded in the shell builder for the zuul-clone macro when this ends up substituted for its {project} parameter.\n\nAs for the path length issue impacting virtualenvs comes about because virtualenv edits shebang lines to include the full path to the python interpreter. For jobs that don\u0027t use frameworks reliant on #! line mangling this is probably a non-issue (though long job names are still cosmetically annoying at the very least).","commit_id":"432b2f6c7b94bdc02aaed00e1bdb94d633c2d36f"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"c2fc0646758ba805f98bf10556da7ca52bbcdadf","unresolved":false,"context_lines":[{"line_number":264,"context_line":"          template-name: \"{template-name}\""},{"line_number":265,"context_line":"      - link-logs"},{"line_number":266,"context_line":"      - zuul-clone:"},{"line_number":267,"context_line":"          project: $ZUUL_PROJECT"},{"line_number":268,"context_line":"      - install-distro-packages"},{"line_number":269,"context_line":"      - puppet-prepare-node"},{"line_number":270,"context_line":"      - shell: |"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"dada55a8_44f1726a","line":267,"in_reply_to":"dada55a8_e0322da6","updated":"2016-07-21 13:24:35.000000000","message":"no I don\u0027t want a template, I want the same job running for 3 projects. The name was a suggestion from pabelanger who suggested me to add \"puppet-\" before \"lint, syntax, unit, and beaker\".","commit_id":"432b2f6c7b94bdc02aaed00e1bdb94d633c2d36f"}]}