)]}'
{"manifests/init.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"cfb7af709cf4ab031f6f0d096aa03bad1e6c176b","unresolved":false,"context_lines":[{"line_number":332,"context_line":"    }"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"    exec { \u0027install-tox\u0027:"},{"line_number":335,"context_line":"      command \u003d\u003e \u0027/usr/bin/pip install -U tox\u0027,"},{"line_number":336,"context_line":"      unless  \u003d\u003e \u0027/usr/bin/which tox\u0027,"},{"line_number":337,"context_line":"      require \u003d\u003e Exec[\u0027install-pip\u0027],"},{"line_number":338,"context_line":"    }"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"bab6814e_8d9356fa","line":335,"updated":"2016-05-18 08:04:40.000000000","message":"why don\u0027t change $::tempest::params value?\n\nalso, looks like need to update tests https://github.com/openstack/puppet-tempest/blob/master/spec/classes/tempest_spec.rb","commit_id":"caf9eccd8a390e26d18f16c1570ca5110c9fe36d"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"eaf554d48de6d56f2ec574a13523e2c70e99b306","unresolved":false,"context_lines":[{"line_number":332,"context_line":"    }"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"    exec { \u0027install-tox\u0027:"},{"line_number":335,"context_line":"      command \u003d\u003e \u0027/usr/bin/pip install -U tox\u0027,"},{"line_number":336,"context_line":"      unless  \u003d\u003e \u0027/usr/bin/which tox\u0027,"},{"line_number":337,"context_line":"      require \u003d\u003e Exec[\u0027install-pip\u0027],"},{"line_number":338,"context_line":"    }"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"bab6814e_fefc91b9","line":335,"updated":"2016-05-21 14:46:11.000000000","message":"why not add a path \u003d\u003e [\u0027/bin\u0027,\u0027/usr/bin\u0027,\u0027/usr/local/bin\u0027]...","commit_id":"caf9eccd8a390e26d18f16c1570ca5110c9fe36d"},{"author":{"_account_id":9500,"name":"Matt Fischer","email":"matt@mattfischer.com","username":"mfisch"},"change_message_id":"7ddcd8525073a5b05917ef2ff8aa5e7247e907dc","unresolved":false,"context_lines":[{"line_number":332,"context_line":"    }"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"    exec { \u0027install-tox\u0027:"},{"line_number":335,"context_line":"      command \u003d\u003e \u0027/usr/bin/pip install -U tox\u0027,"},{"line_number":336,"context_line":"      unless  \u003d\u003e \u0027/usr/bin/which tox\u0027,"},{"line_number":337,"context_line":"      require \u003d\u003e Exec[\u0027install-pip\u0027],"},{"line_number":338,"context_line":"    }"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"bab6814e_09873a62","line":335,"in_reply_to":"bab6814e_8d9356fa","updated":"2016-05-18 15:26:05.000000000","message":"why bother to change it when it\u0027s the same for every OS? deleting is cleaner. Will look at the tests.","commit_id":"caf9eccd8a390e26d18f16c1570ca5110c9fe36d"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"1fa6d0d659ec00859d162da08e39486ee20d67cf","unresolved":false,"context_lines":[{"line_number":332,"context_line":"    }"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"    exec { \u0027install-tox\u0027:"},{"line_number":335,"context_line":"      command \u003d\u003e \u0027/usr/bin/pip install -U tox\u0027,"},{"line_number":336,"context_line":"      path    \u003d\u003e [\u0027/bin\u0027,\u0027/usr/bin\u0027,\u0027/usr/local/bin\u0027],"},{"line_number":337,"context_line":"      unless  \u003d\u003e \u0027/usr/bin/which tox\u0027,"},{"line_number":338,"context_line":"      require \u003d\u003e Exec[\u0027install-pip\u0027],"}],"source_content_type":"text/x-puppet","patch_set":3,"id":"bab6814e_e306311f","line":335,"range":{"start_line":335,"start_character":18,"end_line":335,"end_character":27},"updated":"2016-05-25 02:50:14.000000000","message":"the whole point of the path thing was so we don\u0027t hard code the pip path :)  should remove this","commit_id":"499e2032e8f67ba9721ad2b7f23105e3e36ac565"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"f4a1b034dc266c4474c827f0f6ff5eafa21a4b12","unresolved":false,"context_lines":[{"line_number":332,"context_line":"    }"},{"line_number":333,"context_line":""},{"line_number":334,"context_line":"    exec { \u0027install-tox\u0027:"},{"line_number":335,"context_line":"      command \u003d\u003e \u0027install -U tox\u0027,"},{"line_number":336,"context_line":"      path    \u003d\u003e [\u0027/bin\u0027,\u0027/usr/bin\u0027,\u0027/usr/local/bin\u0027],"},{"line_number":337,"context_line":"      unless  \u003d\u003e \u0027/usr/bin/which tox\u0027,"},{"line_number":338,"context_line":"      require \u003d\u003e Exec[\u0027install-pip\u0027],"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"9abb7d3a_8cff7f00","line":335,"updated":"2016-06-03 15:16:11.000000000","message":"you lost the pip :D","commit_id":"5cc6a1a9d9b677d58246f56bfd7e019f7192afca"}],"spec/classes/tempest_spec.rb":[{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"d6df443ea15dba440fb71c7d3b4257733e9c901a","unresolved":false,"context_lines":[{"line_number":254,"context_line":"          )"},{"line_number":255,"context_line":"        end"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"        it \u0027sets up virtuelnv for tempest\u0027 do"},{"line_number":258,"context_line":"          is_expected.to contain_exec(\u0027setup-venv\u0027).with("},{"line_number":259,"context_line":"              :command \u003d\u003e \u0027virtualenv /var/lib/tempest/.venv \u0026\u0026 /var/lib/tempest/.venv/bin/pip install -U -r requirements.txt\u0027,"},{"line_number":260,"context_line":"              :cwd     \u003d\u003e \u0027/var/lib/tempest\u0027,"}],"source_content_type":"text/x-ruby","patch_set":5,"id":"9abb7d3a_91cdcbb9","line":257,"updated":"2016-06-03 18:38:14.000000000","message":"typo","commit_id":"7b87beb3a2d5bf43914079bf396d267f8d0e7b4e"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"cdb7eeb5ed838330a68719a259a73a7b91dfdd83","unresolved":false,"context_lines":[{"line_number":254,"context_line":"          )"},{"line_number":255,"context_line":"        end"},{"line_number":256,"context_line":""},{"line_number":257,"context_line":"        it \u0027sets up virtuelnv for tempest\u0027 do"},{"line_number":258,"context_line":"          is_expected.to contain_exec(\u0027setup-venv\u0027).with("},{"line_number":259,"context_line":"              :command \u003d\u003e \u0027virtualenv /var/lib/tempest/.venv \u0026\u0026 /var/lib/tempest/.venv/bin/pip install -U -r requirements.txt\u0027,"},{"line_number":260,"context_line":"              :cwd     \u003d\u003e \u0027/var/lib/tempest\u0027,"}],"source_content_type":"text/x-ruby","patch_set":5,"id":"9abb7d3a_f1f9b7c9","line":257,"in_reply_to":"9abb7d3a_91cdcbb9","updated":"2016-06-03 18:40:47.000000000","message":"Done","commit_id":"7b87beb3a2d5bf43914079bf396d267f8d0e7b4e"}]}