)]}'
{"tox.ini":[{"author":{"_account_id":9746,"name":"Doug Shelley","email":"dougshelley66@gmail.com","username":"dougshelley"},"change_message_id":"33a3123475f0f850590035f088e4324a179f238e","unresolved":false,"context_lines":[{"line_number":6,"context_line":"[testenv]"},{"line_number":7,"context_line":"setenv \u003d VIRTUAL_ENV\u003d{envdir}"},{"line_number":8,"context_line":"usedevelop \u003d True"},{"line_number":9,"context_line":"install_command \u003d pip install -chttps://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt -U {opts} {packages}"},{"line_number":10,"context_line":"deps \u003d -r{toxinidir}/requirements.txt"},{"line_number":11,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":12,"context_line":"commands \u003d find ./trove -type f -name \"*.pyc\" -delete"}],"source_content_type":"text/x-properties","patch_set":1,"id":"ba0121b8_23eb1cb2","line":9,"updated":"2016-03-29 17:07:50.000000000","message":"You need to put the {env:UPPER_CONTRAINTS_FILE... back in here.","commit_id":"216793bf545a0c691aeb4f64e88010200d169f87"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"48aa961203b96c8384b104d3f2635a3e28545b6c","unresolved":false,"context_lines":[{"line_number":6,"context_line":"[testenv]"},{"line_number":7,"context_line":"setenv \u003d VIRTUAL_ENV\u003d{envdir}"},{"line_number":8,"context_line":"usedevelop \u003d True"},{"line_number":9,"context_line":"install_command \u003d pip install -chttps://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt -U {opts} {packages}"},{"line_number":10,"context_line":"deps \u003d -r{toxinidir}/requirements.txt"},{"line_number":11,"context_line":"       -r{toxinidir}/test-requirements.txt"},{"line_number":12,"context_line":"commands \u003d find ./trove -type f -name \"*.pyc\" -delete"}],"source_content_type":"text/x-properties","patch_set":1,"id":"ba0121b8_836e682d","line":9,"in_reply_to":"ba0121b8_23eb1cb2","updated":"2016-03-29 17:10:04.000000000","message":"hmm, I was hoping we could consciously not do that ... I guess it comes out to the same thing anyway.","commit_id":"216793bf545a0c691aeb4f64e88010200d169f87"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"4b1f173b03886f67cf318736360995eb3176fc16","unresolved":false,"context_lines":[{"line_number":64,"context_line":"commands \u003d openstack-doc-test --check-build --publish --force"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"[testenv:releasenotes]"},{"line_number":67,"context_line":"install_command \u003d pip install -U {opts} {packages}"},{"line_number":68,"context_line":"commands \u003d sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasenotes/source releasenotes/build/html"}],"source_content_type":"text/x-properties","patch_set":3,"id":"ba0121b8_8fd794ec","line":67,"updated":"2016-03-30 05:48:12.000000000","message":"nit: I\u0027d add a comment here about only needing this *until* infra supports constraints on releasenotes jobs","commit_id":"96451d61d69f788dc2316b8de01f326915363945"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"0ff92bc67180095505a94940adf86f75c53a44f3","unresolved":false,"context_lines":[{"line_number":64,"context_line":"commands \u003d openstack-doc-test --check-build --publish --force"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"[testenv:releasenotes]"},{"line_number":67,"context_line":"install_command \u003d pip install -U {opts} {packages}"},{"line_number":68,"context_line":"commands \u003d sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasenotes/source releasenotes/build/html"}],"source_content_type":"text/x-properties","patch_set":3,"id":"ba0121b8_2369358c","line":67,"in_reply_to":"ba0121b8_8fd794ec","updated":"2016-03-30 10:54:33.000000000","message":"Done","commit_id":"96451d61d69f788dc2316b8de01f326915363945"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"da72c968e0ec21d0b91a2ec34cb6d08f12256adc","unresolved":false,"context_lines":[{"line_number":29,"context_line":"[testenv:debug]"},{"line_number":30,"context_line":"commands \u003d oslo_debug_helper {posargs}"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"[testenv:cover]"},{"line_number":33,"context_line":"basepython \u003d python2.7"},{"line_number":34,"context_line":"commands \u003d"},{"line_number":35,"context_line":"     coverage erase"}],"source_content_type":"text/x-properties","patch_set":4,"id":"ba0121b8_2883c599","line":32,"updated":"2016-03-30 13:09:26.000000000","message":"This is run in post, so cannot use constraints.","commit_id":"8c29292d21f60f97bcfa2ba276d0e61124d0b6dc"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"f95836f51bedfb0280031563343a493ea61807ba","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"[testenv:venv]"},{"line_number":44,"context_line":"commands \u003d {posargs}"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"[flake8]"},{"line_number":47,"context_line":"show-source \u003d True"}],"source_content_type":"text/x-properties","patch_set":4,"id":"ba0121b8_a3ded63c","line":44,"updated":"2016-03-30 13:17:46.000000000","message":"Ah, here\u0027s venv - run also in post queue.","commit_id":"8c29292d21f60f97bcfa2ba276d0e61124d0b6dc"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"da72c968e0ec21d0b91a2ec34cb6d08f12256adc","unresolved":false,"context_lines":[{"line_number":60,"context_line":"  openstack-doc-test --check-niceness --check-syntax --check-deletions {posargs}"},{"line_number":61,"context_line":"  openstack-doc-test --check-build {posargs}"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"[testenv:publishdocs]"},{"line_number":64,"context_line":"commands \u003d openstack-doc-test --check-build --publish --force"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"[testenv:releasenotes]"}],"source_content_type":"text/x-properties","patch_set":4,"id":"ba0121b8_68649d21","line":63,"updated":"2016-03-30 13:09:26.000000000","message":"This is also run in post queue, so cannot use constraints.","commit_id":"8c29292d21f60f97bcfa2ba276d0e61124d0b6dc"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"64c23851643f03222919a4697eea7211f8ff1307","unresolved":false,"context_lines":[{"line_number":68,"context_line":"# is only required because currently infra does not actually"},{"line_number":69,"context_line":"# support constraints files for the release notes job, and while"},{"line_number":70,"context_line":"# the environment variable UPPER_CONSTRAINTS_FILE is set, there\u0027s"},{"line_number":71,"context_line":"# no file there. It can be removed when infra changes this."},{"line_number":72,"context_line":"install_command \u003d pip install -U {opts} {packages}"},{"line_number":73,"context_line":"commands \u003d sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasenotes/source releasenotes/build/html"}],"source_content_type":"text/x-properties","patch_set":4,"id":"ba0121b8_084b61b1","line":71,"updated":"2016-03-30 13:05:33.000000000","message":"I\u0027m not sure this is true anymore, ironic\u0027s releasenotes job works fine with upper-constraints, as far as I can tell.\n\nhttp://logs.openstack.org/16/16bf1763327725ee6e9ab1c9afbc8300df7bd142/post/ironic-releasenotes/9b285c0/console.html#_2016-03-30_11_43_45_687\nhttps://github.com/openstack/ironic/blob/master/tox.ini#L9\nhttps://github.com/openstack/ironic/blob/master/tox.ini#L63","commit_id":"8c29292d21f60f97bcfa2ba276d0e61124d0b6dc"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"f991053f23eaf4354e3f0b398aa8e2d850a72742","unresolved":false,"context_lines":[{"line_number":68,"context_line":"# is only required because currently infra does not actually"},{"line_number":69,"context_line":"# support constraints files for the release notes job, and while"},{"line_number":70,"context_line":"# the environment variable UPPER_CONSTRAINTS_FILE is set, there\u0027s"},{"line_number":71,"context_line":"# no file there. It can be removed when infra changes this."},{"line_number":72,"context_line":"install_command \u003d pip install -U {opts} {packages}"},{"line_number":73,"context_line":"commands \u003d sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasenotes/source releasenotes/build/html"}],"source_content_type":"text/x-properties","patch_set":4,"id":"ba0121b8_c8d7a9bd","line":71,"in_reply_to":"ba0121b8_084b61b1","updated":"2016-03-30 13:06:40.000000000","message":"Jim, the *post* job will not work. And this environment is used for gate and post.","commit_id":"8c29292d21f60f97bcfa2ba276d0e61124d0b6dc"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"5c2e230ae9aa4575650acb26f554e2ea284c1025","unresolved":false,"context_lines":[{"line_number":68,"context_line":"# is only required because currently infra does not actually"},{"line_number":69,"context_line":"# support constraints files for the release notes job, and while"},{"line_number":70,"context_line":"# the environment variable UPPER_CONSTRAINTS_FILE is set, there\u0027s"},{"line_number":71,"context_line":"# no file there. It can be removed when infra changes this."},{"line_number":72,"context_line":"install_command \u003d pip install -U {opts} {packages}"},{"line_number":73,"context_line":"commands \u003d sphinx-build -a -E -W -d releasenotes/build/doctrees -b html releasenotes/source releasenotes/build/html"}],"source_content_type":"text/x-properties","patch_set":4,"id":"ba0121b8_7f59eafd","line":71,"in_reply_to":"ba0121b8_c8d7a9bd","updated":"2016-03-30 14:36:56.000000000","message":"Jim, please see Jenkins logs from patch set 2. It does fail.","commit_id":"8c29292d21f60f97bcfa2ba276d0e61124d0b6dc"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"3f5a4223409018ad5e517b52e5f0301c486a75c3","unresolved":false,"context_lines":[{"line_number":32,"context_line":"[testenv:cover]"},{"line_number":33,"context_line":"# NOTE(amrith) The setting of the install_command in this location"},{"line_number":34,"context_line":"# is only required because currently infra does not actually"},{"line_number":35,"context_line":"# support constraints files for the release notes job, and while"},{"line_number":36,"context_line":"# the environment variable UPPER_CONSTRAINTS_FILE is set, there\u0027s"},{"line_number":37,"context_line":"# no file there. It can be removed when infra changes this."},{"line_number":38,"context_line":"install_command \u003d pip install -U {opts} {packages}"}],"source_content_type":"text/x-properties","patch_set":5,"id":"ba0121b8_d1c3c655","line":35,"range":{"start_line":35,"start_character":36,"end_line":35,"end_character":49},"updated":"2016-03-30 15:47:06.000000000","message":"isn\u0027t this the cover job?","commit_id":"d996468908b93b51988f09947f8d2b22d5fcade6"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"3f5a4223409018ad5e517b52e5f0301c486a75c3","unresolved":false,"context_lines":[{"line_number":49,"context_line":"[testenv:venv]"},{"line_number":50,"context_line":"# NOTE(amrith) The setting of the install_command in this location"},{"line_number":51,"context_line":"# is only required because currently infra does not actually"},{"line_number":52,"context_line":"# support constraints files for the release notes job, and while"},{"line_number":53,"context_line":"# the environment variable UPPER_CONSTRAINTS_FILE is set, there\u0027s"},{"line_number":54,"context_line":"# no file there. It can be removed when infra changes this."},{"line_number":55,"context_line":"install_command \u003d pip install -U {opts} {packages}"}],"source_content_type":"text/x-properties","patch_set":5,"id":"ba0121b8_51ae9698","line":52,"range":{"start_line":52,"start_character":36,"end_line":52,"end_character":49},"updated":"2016-03-30 15:47:06.000000000","message":"venv","commit_id":"d996468908b93b51988f09947f8d2b22d5fcade6"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"3f5a4223409018ad5e517b52e5f0301c486a75c3","unresolved":false,"context_lines":[{"line_number":75,"context_line":"[testenv:publishdocs]"},{"line_number":76,"context_line":"# NOTE(amrith) The setting of the install_command in this location"},{"line_number":77,"context_line":"# is only required because currently infra does not actually"},{"line_number":78,"context_line":"# support constraints files for the release notes job, and while"},{"line_number":79,"context_line":"# the environment variable UPPER_CONSTRAINTS_FILE is set, there\u0027s"},{"line_number":80,"context_line":"# no file there. It can be removed when infra changes this."},{"line_number":81,"context_line":"install_command \u003d pip install -U {opts} {packages}"}],"source_content_type":"text/x-properties","patch_set":5,"id":"ba0121b8_318a021c","line":78,"range":{"start_line":78,"start_character":36,"end_line":78,"end_character":49},"updated":"2016-03-30 15:47:06.000000000","message":"publishdocs","commit_id":"d996468908b93b51988f09947f8d2b22d5fcade6"}]}