)]}'
{"jenkins/jobs/ansible-role-jobs.yaml":[{"author":{"_account_id":7156,"name":"Mathieu Gagné","email":"mgagne@calavera.ca","username":"mgagne"},"change_message_id":"139eb37d5b80513b9c1e48aa00d431d39a0ba998","unresolved":false,"context_lines":[{"line_number":81,"context_line":"          ssh-keyscan 127.0.0.1 \u003e\u003e ~/.ssh/known_hosts"},{"line_number":82,"context_line":"      - shell: |"},{"line_number":83,"context_line":"          # Allow the optional use of extra job parameters"},{"line_number":84,"context_line":"          export ANSIBLE_CLI_OVERRIDE\u003d\"{ansible-cli-override}\""},{"line_number":85,"context_line":"      - tox:"},{"line_number":86,"context_line":"          envlist: functional"},{"line_number":87,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1aa78d24_aa71ace2","line":84,"updated":"2016-07-08 18:52:35.000000000","message":"Won\u0027t the variable be lost? I think environment variables aren\u0027t shared between builders.","commit_id":"8f2ed7bfb64c08a19dea8aa35a1920216fa620c6"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"34b2fd87f78d38d7ca013c164dcf81f2190dc9ea","unresolved":false,"context_lines":[{"line_number":81,"context_line":"          ssh-keyscan 127.0.0.1 \u003e\u003e ~/.ssh/known_hosts"},{"line_number":82,"context_line":"      - shell: |"},{"line_number":83,"context_line":"          # Allow the optional use of extra job parameters"},{"line_number":84,"context_line":"          export ANSIBLE_CLI_OVERRIDE\u003d\"{ansible-cli-override}\""},{"line_number":85,"context_line":"      - tox:"},{"line_number":86,"context_line":"          envlist: functional"},{"line_number":87,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1aa78d24_6d099edd","line":84,"in_reply_to":"1aa78d24_6d035e0e","updated":"2016-07-08 19:07:05.000000000","message":":( Any thoughts on how to achieve the same goal? I need to be able to pass a var from a job into the tox environment.","commit_id":"8f2ed7bfb64c08a19dea8aa35a1920216fa620c6"},{"author":{"_account_id":7156,"name":"Mathieu Gagné","email":"mgagne@calavera.ca","username":"mgagne"},"change_message_id":"18053507a35df0762a2cc9a5b15acdbdafcd2397","unresolved":false,"context_lines":[{"line_number":81,"context_line":"          ssh-keyscan 127.0.0.1 \u003e\u003e ~/.ssh/known_hosts"},{"line_number":82,"context_line":"      - shell: |"},{"line_number":83,"context_line":"          # Allow the optional use of extra job parameters"},{"line_number":84,"context_line":"          export ANSIBLE_CLI_OVERRIDE\u003d\"{ansible-cli-override}\""},{"line_number":85,"context_line":"      - tox:"},{"line_number":86,"context_line":"          envlist: functional"},{"line_number":87,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1aa78d24_c0569905","line":84,"in_reply_to":"1aa78d24_6d099edd","updated":"2016-07-08 19:32:30.000000000","message":"Here is a couple of options I thought about:\n\n1) Introduce a new tox macro with additional arguments\n2) Tweak existing tox macro to read environ variables from a file if it exists and set variables in there.\n3) The envlist argument is passed as-is to tox like so: tox -v -e$venv\nYou could pass any arguments after the env name that could be used by your testenv afterward. Although I\u0027m not sure if run-tox.sh will read it correctly as it\u0027s reading envlist from $1, not $@.\n\n    - tox:\n        envlist: functional -- --opt1 ARG1","commit_id":"8f2ed7bfb64c08a19dea8aa35a1920216fa620c6"},{"author":{"_account_id":7156,"name":"Mathieu Gagné","email":"mgagne@calavera.ca","username":"mgagne"},"change_message_id":"cc9c4c180424ac6735222eb1350a1e91becdae81","unresolved":false,"context_lines":[{"line_number":81,"context_line":"          ssh-keyscan 127.0.0.1 \u003e\u003e ~/.ssh/known_hosts"},{"line_number":82,"context_line":"      - shell: |"},{"line_number":83,"context_line":"          # Allow the optional use of extra job parameters"},{"line_number":84,"context_line":"          export ANSIBLE_CLI_OVERRIDE\u003d\"{ansible-cli-override}\""},{"line_number":85,"context_line":"      - tox:"},{"line_number":86,"context_line":"          envlist: functional"},{"line_number":87,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1aa78d24_6d035e0e","line":84,"in_reply_to":"1aa78d24_aa71ace2","updated":"2016-07-08 18:55:11.000000000","message":"Yea, I tested with Jenkins and value is lost.","commit_id":"8f2ed7bfb64c08a19dea8aa35a1920216fa620c6"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"304f0a1dca3506f502f5292784c1a727c3f69b0a","unresolved":false,"context_lines":[{"line_number":63,"context_line":"    name: \u0027{pipeline}-{name}-ansible-{scenario}-{ostype}{job-suffix}\u0027"},{"line_number":64,"context_line":"    node: \u0027{ostype}\u0027"},{"line_number":65,"context_line":"    scenario: func"},{"line_number":66,"context_line":"    toxenv: functional"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    wrappers:"},{"line_number":69,"context_line":"      - build-timeout:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1aa78d24_1dc88821","line":66,"updated":"2016-07-12 15:41:38.000000000","message":"You shouldn\u0027t try to re-specify your scenario and toxenv parameters here. It\u0027s not necessary/relevant.","commit_id":"01e06be76618d422d549f1ab4a9626d060e4b343"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"24aa6f8df8f3a439be2e486aacfe18e82f87c7ad","unresolved":false,"context_lines":[{"line_number":63,"context_line":"    name: \u0027{pipeline}-{name}-ansible-{scenario}-{ostype}{job-suffix}\u0027"},{"line_number":64,"context_line":"    node: \u0027{ostype}\u0027"},{"line_number":65,"context_line":"    scenario: func"},{"line_number":66,"context_line":"    toxenv: functional"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"    wrappers:"},{"line_number":69,"context_line":"      - build-timeout:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1aa78d24_9373c97f","line":66,"in_reply_to":"1aa78d24_1dc88821","updated":"2016-07-12 15:58:31.000000000","message":"Done","commit_id":"01e06be76618d422d549f1ab4a9626d060e4b343"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"304f0a1dca3506f502f5292784c1a727c3f69b0a","unresolved":false,"context_lines":[{"line_number":99,"context_line":"            - ubuntu-trusty"},{"line_number":100,"context_line":"            - ubuntu-xenial"},{"line_number":101,"context_line":"          pipeline: gate"},{"line_number":102,"context_line":"          scenario: func"},{"line_number":103,"context_line":"      - \u0027{pipeline}-{name}-ansible-{scenario}-{ostype}{job-suffix}\u0027:"},{"line_number":104,"context_line":"          job-suffix: \u0027\u0027"},{"line_number":105,"context_line":"          ostype:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1aa78d24_dd03a00e","line":102,"updated":"2016-07-12 15:41:38.000000000","message":"You also need to specify toxenv here.","commit_id":"01e06be76618d422d549f1ab4a9626d060e4b343"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"24aa6f8df8f3a439be2e486aacfe18e82f87c7ad","unresolved":false,"context_lines":[{"line_number":99,"context_line":"            - ubuntu-trusty"},{"line_number":100,"context_line":"            - ubuntu-xenial"},{"line_number":101,"context_line":"          pipeline: gate"},{"line_number":102,"context_line":"          scenario: func"},{"line_number":103,"context_line":"      - \u0027{pipeline}-{name}-ansible-{scenario}-{ostype}{job-suffix}\u0027:"},{"line_number":104,"context_line":"          job-suffix: \u0027\u0027"},{"line_number":105,"context_line":"          ostype:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1aa78d24_d369d18d","line":102,"in_reply_to":"1aa78d24_dd03a00e","updated":"2016-07-12 15:58:31.000000000","message":"Done","commit_id":"01e06be76618d422d549f1ab4a9626d060e4b343"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"304f0a1dca3506f502f5292784c1a727c3f69b0a","unresolved":false,"context_lines":[{"line_number":107,"context_line":"            - ubuntu-trusty"},{"line_number":108,"context_line":"            - ubuntu-xenial"},{"line_number":109,"context_line":"          pipeline: gate"},{"line_number":110,"context_line":"          scenario: func"},{"line_number":111,"context_line":"      - \u0027{name}-announce-release\u0027"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"- job-group:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1aa78d24_5006771c","line":110,"updated":"2016-07-12 15:41:38.000000000","message":"And here.","commit_id":"01e06be76618d422d549f1ab4a9626d060e4b343"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"24aa6f8df8f3a439be2e486aacfe18e82f87c7ad","unresolved":false,"context_lines":[{"line_number":107,"context_line":"            - ubuntu-trusty"},{"line_number":108,"context_line":"            - ubuntu-xenial"},{"line_number":109,"context_line":"          pipeline: gate"},{"line_number":110,"context_line":"          scenario: func"},{"line_number":111,"context_line":"      - \u0027{name}-announce-release\u0027"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"- job-group:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1aa78d24_1306b9cd","line":110,"in_reply_to":"1aa78d24_5006771c","updated":"2016-07-12 15:58:31.000000000","message":"Done","commit_id":"01e06be76618d422d549f1ab4a9626d060e4b343"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"eafdde411c19aa86323f9c2b33b60110b58aacd8","unresolved":false,"context_lines":[{"line_number":79,"context_line":"          cat ~/.ssh/id_rsa.pub \u003e\u003e ~/.ssh/authorized_keys"},{"line_number":80,"context_line":"          ssh-keyscan localhost \u003e\u003e ~/.ssh/known_hosts"},{"line_number":81,"context_line":"          ssh-keyscan 127.0.0.1 \u003e\u003e ~/.ssh/known_hosts"},{"line_number":82,"context_line":"      - tox:"},{"line_number":83,"context_line":"          envlist: \u0027{toxenv}\u0027"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    publishers:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"1aa78d24_8ea440a6","line":82,"updated":"2016-07-12 16:14:05.000000000","message":"We can replace this will a new builder, maybe a shell which does some conditional logic to check for func, if found run tox -efunctional otherwise tox -e{scenario}","commit_id":"015ea53095472e86b84c4ecfb9e40f0c5933d184"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"d8787ccc872a9882e73182f9d4883ce10bfab0ff","unresolved":false,"context_lines":[{"line_number":80,"context_line":"          ssh-keyscan localhost \u003e\u003e ~/.ssh/known_hosts"},{"line_number":81,"context_line":"          ssh-keyscan 127.0.0.1 \u003e\u003e ~/.ssh/known_hosts"},{"line_number":82,"context_line":"      - tox:"},{"line_number":83,"context_line":"          envlist: \u0027{toxenv}\u0027"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    publishers:"},{"line_number":86,"context_line":"      - console-log"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"1aa78d24_b399a57b","line":83,"updated":"2016-07-12 16:01:45.000000000","message":"this should be scenario","commit_id":"015ea53095472e86b84c4ecfb9e40f0c5933d184"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"d8787ccc872a9882e73182f9d4883ce10bfab0ff","unresolved":false,"context_lines":[{"line_number":98,"context_line":"            - ubuntu-xenial"},{"line_number":99,"context_line":"          pipeline: gate"},{"line_number":100,"context_line":"          scenario: func"},{"line_number":101,"context_line":"          toxenv: functional"},{"line_number":102,"context_line":"      - \u0027{pipeline}-{name}-ansible-{scenario}-{ostype}{job-suffix}\u0027:"},{"line_number":103,"context_line":"          job-suffix: \u0027\u0027"},{"line_number":104,"context_line":"          ostype:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"1aa78d24_13907942","line":101,"updated":"2016-07-12 16:01:45.000000000","message":"this won\u0027t work.  So the right fix, will be to mass update our tox.ini and replace functional with func  Then drop toxenv from here","commit_id":"015ea53095472e86b84c4ecfb9e40f0c5933d184"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"2e0021f8ea627db6850b9a33f53687128e3a45f9","unresolved":false,"context_lines":[{"line_number":98,"context_line":"            - ubuntu-xenial"},{"line_number":99,"context_line":"          pipeline: gate"},{"line_number":100,"context_line":"          scenario: func"},{"line_number":101,"context_line":"          toxenv: functional"},{"line_number":102,"context_line":"      - \u0027{pipeline}-{name}-ansible-{scenario}-{ostype}{job-suffix}\u0027:"},{"line_number":103,"context_line":"          job-suffix: \u0027\u0027"},{"line_number":104,"context_line":"          ostype:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"1aa78d24_2ea70c0c","line":101,"in_reply_to":"1aa78d24_13907942","updated":"2016-07-12 16:07:35.000000000","message":"The point of doing this is to divorce the scenario name (used for the job name uniqueness) from the tox target name. I don\u0027t see how it is that you think it won\u0027t work?","commit_id":"015ea53095472e86b84c4ecfb9e40f0c5933d184"},{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"ef37481a6781f4be04d115cd7f0fd6f061c67f14","unresolved":false,"context_lines":[{"line_number":85,"context_line":"          # of the scenario \u0027func\u0027 to \u0027functional\u0027 so reduce"},{"line_number":86,"context_line":"          # the need for code churn in those repositories"},{"line_number":87,"context_line":"          # immediately."},{"line_number":88,"context_line":"          if [ \"{scenario}\" \u003d \"func\" ]; then"},{"line_number":89,"context_line":"            /usr/local/jenkins/slave_scripts/run-tox.sh functional"},{"line_number":90,"context_line":"          else"},{"line_number":91,"context_line":"            /usr/local/jenkins/slave_scripts/run-tox.sh {scenario}"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"1aa78d24_c41aa96e","line":88,"updated":"2016-07-12 16:41:57.000000000","message":"\u003d\u003d","commit_id":"3f2ddd663971497a0ff09e65d52ec16da5d66317"}],"jenkins/jobs/projects.yaml":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"304f0a1dca3506f502f5292784c1a727c3f69b0a","unresolved":false,"context_lines":[{"line_number":5332,"context_line":"    doc-publisher-site: docs.openstack.org"},{"line_number":5333,"context_line":""},{"line_number":5334,"context_line":"    jobs:"},{"line_number":5335,"context_line":"      - ansible-role-jobs"},{"line_number":5336,"context_line":"      - openstack-publish-jobs"},{"line_number":5337,"context_line":"      - openstack-releasenotes-jobs"},{"line_number":5338,"context_line":"      - \u0027{pipeline}-{name}-ansible-{scenario}-{ostype}{job-suffix}\u0027:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1aa78d24_1027cfb6","line":5335,"updated":"2016-07-12 15:41:38.000000000","message":"This inclusion of the job group is redundant with the lines you\u0027re now adding below.","commit_id":"01e06be76618d422d549f1ab4a9626d060e4b343"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"74106804d1ccc1c24cc1a768bd3146b8017c40eb","unresolved":false,"context_lines":[{"line_number":5332,"context_line":"    doc-publisher-site: docs.openstack.org"},{"line_number":5333,"context_line":""},{"line_number":5334,"context_line":"    jobs:"},{"line_number":5335,"context_line":"      - ansible-role-jobs"},{"line_number":5336,"context_line":"      - openstack-publish-jobs"},{"line_number":5337,"context_line":"      - openstack-releasenotes-jobs"},{"line_number":5338,"context_line":"      - \u0027{pipeline}-{name}-ansible-{scenario}-{ostype}{job-suffix}\u0027:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1aa78d24_533e818b","line":5335,"in_reply_to":"1aa78d24_1027cfb6","updated":"2016-07-12 15:57:21.000000000","message":"Oops, as discussed in IRC I missed that the additional entries below are OVS-specific, so ignore this comment.","commit_id":"01e06be76618d422d549f1ab4a9626d060e4b343"}]}