)]}'
{"modules/openstack_integration/manifests/testing/scenario001.pp":[{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"5de3766f438f9323ff393d726b3d86e0975494e5","unresolved":false,"context_lines":[{"line_number":69,"context_line":"  }"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"  # Deploy Tempest"},{"line_number":72,"context_line":"  class { \u0027::tempest\u0027:"},{"line_number":73,"context_line":"    debug              \u003d\u003e true,"},{"line_number":74,"context_line":"    use_stderr         \u003d\u003e false,"},{"line_number":75,"context_line":"    log_file           \u003d\u003e \u0027tempest.log\u0027,"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"3a50d1a3_c1a024a8","line":72,"updated":"2015-07-31 17:44:24.000000000","message":"looks like tempest is getting installed into /var/lib.  Can we change this to /opt or /tmp.  Something that doesn\u0027t require root access?","commit_id":"dd72b9ea6fe5296b4d3ed225971de37bfea9d66c"}],"run_tests.sh":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"1bef22d3fb429e3f1b97afd7b79c1cb8d1366f64","unresolved":false,"context_lines":[{"line_number":57,"context_line":"    exit 1"},{"line_number":58,"context_line":"fi"},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"# TODO(emilien): - find a flexible way to run tempest without hardcoding \u0027identity\u0027"},{"line_number":61,"context_line":"#                - write output to /var/lib/tempest/test_run.out but puppet-keystone Rakefile will require adjustments"},{"line_number":62,"context_line":"cd /var/lib/tempest; tox -eall -- identity"}],"source_content_type":"text/x-sh","patch_set":3,"id":"3a50d1a3_86f055fe","line":60,"updated":"2015-07-29 22:17:00.000000000","message":"So this is a straight passthrough to testr you\u0027re basically calling:\n\ntestr run --parallel --subunit identity | subunit-trace\n\nwith the tox command. (so venv isolating it) Which is the best way I know how to run this command and only run the keystone tests. What specifically are you looking for with a more flexible method?\n\nI\u0027m just trying to gather requirements for this, because this cycle we\u0027re working on adding a tempest run command as part of better workflow for using and interacting with tempest. (see: https://review.openstack.org/198878 which is the doc patch at the top of the current stack) The plan is to have an service flag option to specify which services to run instead of relying on a regex. (although it\u0027ll likely just be calling testr like this under the covers) But, I want to make sure we have what you need in that work.","commit_id":"de68421753bdc29a5a4ca3c1d0d4e88e482c9aeb"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"1bef22d3fb429e3f1b97afd7b79c1cb8d1366f64","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"# TODO(emilien): - find a flexible way to run tempest without hardcoding \u0027identity\u0027"},{"line_number":61,"context_line":"#                - write output to /var/lib/tempest/test_run.out but puppet-keystone Rakefile will require adjustments"},{"line_number":62,"context_line":"cd /var/lib/tempest; tox -eall -- identity"}],"source_content_type":"text/x-sh","patch_set":3,"id":"3a50d1a3_0601a5ae","line":62,"updated":"2015-07-29 22:17:00.000000000","message":"So we recently landed the pieces to do tempest init, so if you wanted to you could do:\n\ntempest init $path \u0026\u0026 cd $path $$ modify etc/tempest.conf \u0026\u0026 testr run --parallel --subunit identity | subunit trace (or use ostestr --regex identity)\n\n(the last bit will eventually be replaced with tempest run)\nThis may or may not be better then expecting the code to be checked out in a specific location. Although this will probably require changes to the tempest module to be used.","commit_id":"de68421753bdc29a5a4ca3c1d0d4e88e482c9aeb"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"9d6580f210e4b735aac69614b7ff5e0864a01fb3","unresolved":false,"context_lines":[{"line_number":49,"context_line":"    elif [ $first_run_rc -eq 2 ]; then"},{"line_number":50,"context_line":"        echo \"First Puppet apply ran successfully with changes.\""},{"line_number":51,"context_line":"        exit $first_run_rc"},{"line_number":52,"context_line":"    fi"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    if [ $second_run_rc -ne 0 ]; then"},{"line_number":55,"context_line":"        if [ $second_run_rc -eq 2 ]; then"}],"source_content_type":"text/x-sh","patch_set":14,"id":"1a4dcd0f_05a38cb2","line":52,"updated":"2015-08-15 09:15:35.000000000","message":"It seems this place might never be reached because of the \"exit\" cases above?..","commit_id":"4fd2f0e64147adb57cc2e5d8d96db190311e7a32"},{"author":{"_account_id":6926,"name":"Bogdan Dobrelya","email":"bdobreli@redhat.com","username":"bogdando"},"change_message_id":"5b8f80f16d56c704a412fae7ddeee4248aa040a1","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"PUPPET_ARGS\u003d\"--detailed-exitcodes --verbose --color\u003dfalse --debug\""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"function run_puppet() {"},{"line_number":50,"context_line":"    local manifest\u003d$1"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"    sudo puppet apply $PUPPET_ARGS fixtures/${manifest}.pp"}],"source_content_type":"text/x-sh","patch_set":21,"id":"fa1b9901_a3b281e3","line":49,"updated":"2015-08-20 09:06:53.000000000","message":"now this LGTM, thanks","commit_id":"a3b8e778b7555955dc2cdc8e0f94b28fbf1cd08c"}]}