)]}'
{"jenkins/jobs/openstack-ansible-jobs.yaml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"99d23dd5243caa46e13b84d3e6f4b3bd01b3f7fa","unresolved":false,"context_lines":[{"line_number":32,"context_line":"      - print-template-name:"},{"line_number":33,"context_line":"          template-name: \"{template-name}\""},{"line_number":34,"context_line":"      - zuul-git-branch-prep-upper-constraints:"},{"line_number":35,"context_line":"          branch: \u0027stable/{branch-name}\u0027"},{"line_number":36,"context_line":"      - shell: |"},{"line_number":37,"context_line":"          #!/bin/bash -xe"},{"line_number":38,"context_line":"          sudo scripts/gate-check-commit.sh \"{scenario}\" \"{action}\""}],"source_content_type":"text/x-yaml","patch_set":13,"id":"ba2be162_102df394","line":35,"updated":"2017-03-03 17:01:21.000000000","message":"As I understand it, this prepares the host with the appropriate upper constraints for the specified branch. Where I\u0027m stuck is determining how to tell zuul which branch must be checked out in the first place for the git repo.","commit_id":"11080fdb72bba799e859a75c615bd1186a60702f"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"ae2c02897b538826e61087b15de1cdfe4c709c63","unresolved":false,"context_lines":[{"line_number":32,"context_line":"      - print-template-name:"},{"line_number":33,"context_line":"          template-name: \"{template-name}\""},{"line_number":34,"context_line":"      - zuul-git-branch-prep-upper-constraints:"},{"line_number":35,"context_line":"          branch: \u0027stable/{branch-name}\u0027"},{"line_number":36,"context_line":"      - shell: |"},{"line_number":37,"context_line":"          #!/bin/bash -xe"},{"line_number":38,"context_line":"          sudo scripts/gate-check-commit.sh \"{scenario}\" \"{action}\""}],"source_content_type":"text/x-yaml","patch_set":13,"id":"ba2be162_4644151a","line":35,"in_reply_to":"ba2be162_102df394","updated":"2017-03-03 17:39:45.000000000","message":"This checks out your repo *and* the upper-constraints file for the branch you specify.\n\nCheck the implementation in macros.yaml","commit_id":"11080fdb72bba799e859a75c615bd1186a60702f"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"882cf610ca50ccfbe0ede861743ddf1ceebd53a5","unresolved":false,"context_lines":[{"line_number":41,"context_line":"      - devstack-logs"},{"line_number":42,"context_line":"      - console-log"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"- job-group:"},{"line_number":45,"context_line":"    name: openstack-ansible-periodic-upgrade-master"},{"line_number":46,"context_line":"    action: upgrade"},{"line_number":47,"context_line":"    branch: \u0027master\u0027"},{"line_number":48,"context_line":"    branch-name: \u0027master\u0027"},{"line_number":49,"context_line":"    node: ubuntu-xenial"},{"line_number":50,"context_line":"    scenario: aio"},{"line_number":51,"context_line":"    suffix: \u0027\u0027"},{"line_number":52,"context_line":"    jobs:"},{"line_number":53,"context_line":"      - \u0027periodic-openstack-ansible-{action}-{scenario}-{branch-name}-{node}{suffix}\u0027"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"ba2be162_971fe0b5","line":53,"range":{"start_line":44,"start_character":0,"end_line":53,"end_character":85},"updated":"2017-03-06 11:55:44.000000000","message":"This is really not needed for doing this once. I would move this over into projects.yaml.\n\nThere\u0027s no variable in here to be set from projects.yaml, so no need for it.","commit_id":"549c9b1d851e6d63ee093b70c61b31e1ed313e8c"}],"jenkins/jobs/projects.yaml":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"da7558d2e2d71dc992a5f49382e3fbf5c6840762","unresolved":false,"context_lines":[{"line_number":8117,"context_line":"            - swift"},{"line_number":8118,"context_line":"            - upgrade"},{"line_number":8119,"context_line":"      - \u0027periodic-{name}-openstack-ansible-{scenario}-{node}{suffix}\u0027:"},{"line_number":8120,"context_line":"          suffix: \u0027-nv\u0027"},{"line_number":8121,"context_line":"          node:"},{"line_number":8122,"context_line":"            - ubuntu-xenial"},{"line_number":8123,"context_line":"          scenario:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"fa31d9ce_57e36ca3","line":8120,"updated":"2017-02-10 18:46:01.000000000","message":"no need for -nv for periodic job","commit_id":"6de0abf0b9ec62becf06898996d4c95621d4c666"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"da7558d2e2d71dc992a5f49382e3fbf5c6840762","unresolved":false,"context_lines":[{"line_number":8118,"context_line":"            - upgrade"},{"line_number":8119,"context_line":"      - \u0027periodic-{name}-openstack-ansible-{scenario}-{node}{suffix}\u0027:"},{"line_number":8120,"context_line":"          suffix: \u0027-nv\u0027"},{"line_number":8121,"context_line":"          node:"},{"line_number":8122,"context_line":"            - ubuntu-xenial"},{"line_number":8123,"context_line":"          scenario:"},{"line_number":8124,"context_line":"            - upgrade"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"fa31d9ce_77eeb08c","line":8121,"updated":"2017-02-10 18:46:01.000000000","message":"write:\nnode: ubuntu-xenial - also for upgrade\n\nlists with a single item can cause problems...","commit_id":"6de0abf0b9ec62becf06898996d4c95621d4c666"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"e12285063a203cf536c00b104ad3791a7d27fe58","unresolved":false,"context_lines":[{"line_number":8407,"context_line":"    jobs:"},{"line_number":8408,"context_line":"      - ansible-role-jobs"},{"line_number":8409,"context_line":"      - gate-{name}-requirements"},{"line_number":8410,"context_line":"      - openstack-ansible-periodic-upgrade-master"},{"line_number":8411,"context_line":"      - openstack-publish-jobs"},{"line_number":8412,"context_line":"      - openstack-releasenotes-jobs"},{"line_number":8413,"context_line":"      - deploy-guide-jobs:"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"ba2be162_d7eb1811","line":8410,"updated":"2017-03-06 11:46:14.000000000","message":"Use: \n\n- \u0027periodic-openstack-ansible-{action}-{scenario}-{branch-name}-{node}{suffix}\u0027:\n   action: bla\n   scenario: upgrade\n   branch-name: master\n   node: XX\n   suffix: \u0027\u0027","commit_id":"549c9b1d851e6d63ee093b70c61b31e1ed313e8c"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"882cf610ca50ccfbe0ede861743ddf1ceebd53a5","unresolved":false,"context_lines":[{"line_number":8407,"context_line":"    jobs:"},{"line_number":8408,"context_line":"      - ansible-role-jobs"},{"line_number":8409,"context_line":"      - gate-{name}-requirements"},{"line_number":8410,"context_line":"      - openstack-ansible-periodic-upgrade-master"},{"line_number":8411,"context_line":"      - openstack-publish-jobs"},{"line_number":8412,"context_line":"      - openstack-releasenotes-jobs"},{"line_number":8413,"context_line":"      - deploy-guide-jobs:"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"ba2be162_f7cc9c36","line":8410,"in_reply_to":"ba2be162_37847409","updated":"2017-03-06 11:55:44.000000000","message":"But you don\u0027t pass in any parameters to the job-group today...","commit_id":"549c9b1d851e6d63ee093b70c61b31e1ed313e8c"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"021ded743b0dd4c60a853198063dd3ac333675b0","unresolved":false,"context_lines":[{"line_number":8407,"context_line":"    jobs:"},{"line_number":8408,"context_line":"      - ansible-role-jobs"},{"line_number":8409,"context_line":"      - gate-{name}-requirements"},{"line_number":8410,"context_line":"      - openstack-ansible-periodic-upgrade-master"},{"line_number":8411,"context_line":"      - openstack-publish-jobs"},{"line_number":8412,"context_line":"      - openstack-releasenotes-jobs"},{"line_number":8413,"context_line":"      - deploy-guide-jobs:"}],"source_content_type":"text/x-yaml","patch_set":20,"id":"ba2be162_37847409","line":8410,"in_reply_to":"ba2be162_d7eb1811","updated":"2017-03-06 11:52:40.000000000","message":"Do I then remove the \u0027openstack-ansible-periodic-upgrade-master\u0027 job group, or do I leave it in place?\n\nIf it gets removed, then how do I provide the \u0027branch\u0027 parameter to the job?\n\nRight now the \u0027branch-name\u0027 and \u0027branch\u0027 have the same value, but when we implement this job for stable branches it will be different.","commit_id":"549c9b1d851e6d63ee093b70c61b31e1ed313e8c"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"c85b898c9076aec57395dcf5d1149e815fd6ece1","unresolved":false,"context_lines":[{"line_number":8432,"context_line":"            - swift"},{"line_number":8433,"context_line":"      - \u0027gate-openstack-ansible-{action}-{scenario}-{branch-name}-{node}{suffix}\u0027:"},{"line_number":8434,"context_line":"          action: upgrade"},{"line_number":8435,"context_line":"          branch: \u0027master\u0027"},{"line_number":8436,"context_line":"          branch-name: \u0027master\u0027"},{"line_number":8437,"context_line":"          node: ubuntu-xenial"},{"line_number":8438,"context_line":"          scenario: aio"}],"source_content_type":"text/x-yaml","patch_set":21,"id":"ba2be162_b7006403","line":8435,"range":{"start_line":8435,"start_character":0,"end_line":8435,"end_character":26},"updated":"2017-03-06 12:01:40.000000000","message":"Note that this is the parameter that will be crucial once we start using this for stable branch jobs. This is the parameter that\u0027s not in the job name.","commit_id":"5b38167ea5c8882dd03a8a30c79ec7b03eeb8224"}],"zuul/layout.yaml":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"5a9c169fd57f915620dd898774328366ced6c12a","unresolved":false,"context_lines":[{"line_number":12737,"context_line":"    release:"},{"line_number":12738,"context_line":"      - openstack-ansible-announce-release"},{"line_number":12739,"context_line":"    periodic:"},{"line_number":12740,"context_line":"      - gate-openstack-ansible-openstack-ansible-upgrade-ubuntu-xenial-nv"},{"line_number":12741,"context_line":""},{"line_number":12742,"context_line":"  - name: openstack/openstack-ansible-apt_package_pinning"},{"line_number":12743,"context_line":"    template:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7a3c09a3_24d2e65c","line":12740,"range":{"start_line":12740,"start_character":8,"end_line":12740,"end_character":73},"updated":"2017-01-16 19:37:04.000000000","message":"name this periodic instead of gate and remove -nv - for consistency","commit_id":"6e91b13121a13c201441bcdca36dcaa9d8b41e45"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"e59f57e98d88e421b41c10d6be304279bff455a4","unresolved":false,"context_lines":[{"line_number":12737,"context_line":"    release:"},{"line_number":12738,"context_line":"      - openstack-ansible-announce-release"},{"line_number":12739,"context_line":"    periodic:"},{"line_number":12740,"context_line":"      - gate-openstack-ansible-openstack-ansible-upgrade-ubuntu-xenial-nv"},{"line_number":12741,"context_line":""},{"line_number":12742,"context_line":"  - name: openstack/openstack-ansible-apt_package_pinning"},{"line_number":12743,"context_line":"    template:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"7a3c09a3_575db177","line":12740,"range":{"start_line":12740,"start_character":8,"end_line":12740,"end_character":73},"in_reply_to":"7a3c09a3_24d2e65c","updated":"2017-01-17 17:36:31.000000000","message":"This is still valid","commit_id":"6e91b13121a13c201441bcdca36dcaa9d8b41e45"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"da7558d2e2d71dc992a5f49382e3fbf5c6840762","unresolved":false,"context_lines":[{"line_number":3165,"context_line":"  # Skip the upgrade test on Newton and earlier branches."},{"line_number":3166,"context_line":"  - name: ^periodic-openstack-ansible-.*-ansible-upgrade.*$"},{"line_number":3167,"context_line":"    branch: ^(?!(liberty|stable/(mitaka|newton))).*$"},{"line_number":3168,"context_line":"    voting: false"},{"line_number":3169,"context_line":""},{"line_number":3170,"context_line":"  # Skip any OpenStack-Ansible jobs for os_ repositories which run on CentOS7"},{"line_number":3171,"context_line":"  # except os_keystone, os_glance, os_cinder for stable/newton."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"fa31d9ce_97f32473","line":3168,"updated":"2017-02-10 18:46:01.000000000","message":"remove completely - you need to specify when to run this - and periodic jobs are always non-voting","commit_id":"6de0abf0b9ec62becf06898996d4c95621d4c666"}]}