)]}'
{"jenkins/jobs/macros.yaml":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"27e3ff3363c633420061cf0fb5b18c62479d418c","unresolved":false,"context_lines":[{"line_number":967,"context_line":"      - postbuildscript:"},{"line_number":968,"context_line":"          builders:"},{"line_number":969,"context_line":"            - shell: |"},{"line_number":970,"context_line":"                #!/bin/bash -x"},{"line_number":971,"context_line":"                /usr/local/jenkins/slave_scripts/copy_puppet_logs.sh"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"3a50d1a3_0796f54b","line":970,"updated":"2015-08-03 04:17:02.000000000","message":"nit: we don\u0027t need the wrapper here, see entry above.  scripts runs with -x \u0026 -e","commit_id":"547eec60d4a56c3c5ec558ce3750cf0dc8480a44"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"ac6b961c9603cccd5974d477c14e2b30da111d97","unresolved":false,"context_lines":[{"line_number":967,"context_line":"      - postbuildscript:"},{"line_number":968,"context_line":"          builders:"},{"line_number":969,"context_line":"            - shell: |"},{"line_number":970,"context_line":"                #!/bin/bash -x"},{"line_number":971,"context_line":"                /usr/local/jenkins/slave_scripts/copy_puppet_logs.sh"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"3a50d1a3_14528f35","line":970,"in_reply_to":"3a50d1a3_0796f54b","updated":"2015-08-03 12:46:59.000000000","message":"ack - I\u0027ll drop it in a next iteration, when we will enable the publisher for other jobs.\n\nThanks","commit_id":"547eec60d4a56c3c5ec558ce3750cf0dc8480a44"}],"jenkins/jobs/puppet-module-jobs.yaml":[{"author":{"_account_id":4162,"name":"Paul Belanger","email":"pabelanger@redhat.com","username":"pabelanger"},"change_message_id":"82fe6716dc5c660093f63a442cf59bb091a542c3","unresolved":false,"context_lines":[{"line_number":44,"context_line":"          export BEAKER_debug\u003dyes"},{"line_number":45,"context_line":"          export BEAKER_color\u003dno"},{"line_number":46,"context_line":"          bundle exec rspec spec/acceptance"},{"line_number":47,"context_line":"          bundle exec rake -T copy_logs"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    wrappers:"},{"line_number":50,"context_line":"      - build-timeout:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3a50d1a3_4e8dc67a","line":47,"updated":"2015-07-28 20:50:07.000000000","message":"I think it would be better to move this into its own shell function, that way if anything in the job fails, the logs will still get copied.\n\nthoughts?","commit_id":"709062d891037cfb7e55e0831dae318f7fc92196"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"4f692c2693ac6db4314a5d1b9eb2cfd608917364","unresolved":false,"context_lines":[{"line_number":44,"context_line":"          export BEAKER_debug\u003dyes"},{"line_number":45,"context_line":"          export BEAKER_color\u003dno"},{"line_number":46,"context_line":"          bundle exec rspec spec/acceptance"},{"line_number":47,"context_line":"          bundle exec rake -T copy_logs"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    wrappers:"},{"line_number":50,"context_line":"      - build-timeout:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3a50d1a3_e4987277","line":47,"in_reply_to":"3a50d1a3_4e8dc67a","updated":"2015-07-28 21:22:20.000000000","message":"I\u0027m not sure to understand the comment, but it\u0027s already in verbose mode if you look at my patch in puppet-keystone.\n\nLet me know your idea if I\u0027m wrong.","commit_id":"709062d891037cfb7e55e0831dae318f7fc92196"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"fc5a297b7a23411549fdcf09e4cc6ea0b99833af","unresolved":false,"context_lines":[{"line_number":44,"context_line":"          export BEAKER_debug\u003dyes"},{"line_number":45,"context_line":"          export BEAKER_color\u003dno"},{"line_number":46,"context_line":"          bundle exec rspec spec/acceptance"},{"line_number":47,"context_line":"          bundle exec rake -T copy_logs"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    wrappers:"},{"line_number":50,"context_line":"      - build-timeout:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3a50d1a3_4af12f00","line":47,"in_reply_to":"3a50d1a3_e4987277","updated":"2015-07-28 21:58:01.000000000","message":"oh I just get what you mean, and you\u0027re right. Next patchset will address it.","commit_id":"709062d891037cfb7e55e0831dae318f7fc92196"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"ee4177e45736254114ee61123ecf77ee073304ba","unresolved":false,"context_lines":[{"line_number":44,"context_line":"          export BEAKER_debug\u003dyes"},{"line_number":45,"context_line":"          export BEAKER_color\u003dno"},{"line_number":46,"context_line":"          bundle exec rspec spec/acceptance"},{"line_number":47,"context_line":"          ret\u003d?$"},{"line_number":48,"context_line":"          bundle exec rake -T copy_logs"},{"line_number":49,"context_line":"          exit $ret"},{"line_number":50,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3a50d1a3_5fa1e492","line":47,"updated":"2015-07-29 03:57:28.000000000","message":"you mean $? ?","commit_id":"79efe3a9ecac545f7f87abd0d90049774ee831d5"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"8305b2dd5dac211580e6ba9a7eba4de73fbcbadb","unresolved":false,"context_lines":[{"line_number":65,"context_line":"      - allow-local-ssh-root"},{"line_number":66,"context_line":"      - shell: |"},{"line_number":67,"context_line":"          sudo gem install bundler --no-rdoc --no-ri --verbose"},{"line_number":68,"context_line":"      - revoke-sudo"},{"line_number":69,"context_line":"      - shell: |"},{"line_number":70,"context_line":"          #!/bin/bash -xe"},{"line_number":71,"context_line":"          # checkout the current branch"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3a50d1a3_94711c7a","line":68,"updated":"2015-07-31 05:29:38.000000000","message":"We revoke sudo here, therefore your publisher cannot use it AFAIK (please double check)","commit_id":"f17a26b9ee39017192b76756683b826c6c14984b"},{"author":{"_account_id":8482,"name":"Colleen Murphy","email":"colleen@gazlene.net","username":"krinkle"},"change_message_id":"ffa1a28c12da3cec402e8093528a8ed90d690fe2","unresolved":false,"context_lines":[{"line_number":109,"context_line":"      - timestamps"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"    publishers:"},{"line_number":112,"context_line":"      - copy-puppet-logs"},{"line_number":113,"context_line":"      - devstack-logs"},{"line_number":114,"context_line":"      - console-log"},{"line_number":115,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":14,"id":"1a4dcd0f_da601738","line":112,"updated":"2015-08-04 20:23:24.000000000","message":"Why is this done only for the upgrade job? Shouldn\u0027t we do it for the regular beaker job too?","commit_id":"547eec60d4a56c3c5ec558ce3750cf0dc8480a44"}],"jenkins/scripts/cleanup_host.sh":[{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"8305b2dd5dac211580e6ba9a7eba4de73fbcbadb","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":"# Copyright (C) 2011-2013 OpenStack Foundation"},{"line_number":3,"context_line":"# Copyright (C) 2015 Red Hat Inc."},{"line_number":4,"context_line":"#"}],"source_content_type":"text/x-sh","patch_set":10,"id":"3a50d1a3_944a7c26","line":1,"updated":"2015-07-31 05:29:38.000000000","message":"use -xe so that it fails on errors and we have debug output","commit_id":"f17a26b9ee39017192b76756683b826c6c14984b"},{"author":{"_account_id":6547,"name":"Andreas Jaeger","email":"jaegerandi@gmail.com","username":"jaegerandi"},"change_message_id":"8305b2dd5dac211580e6ba9a7eba4de73fbcbadb","unresolved":false,"context_lines":[{"line_number":91,"context_line":"if uses_debs; then"},{"line_number":92,"context_line":"    sudo cp $SVC_LOG_DIR/syslog $WS_LOG_DIR/syslog.txt"},{"line_number":93,"context_line":"    sudo cp $SVC_LOG_DIR/kern.log $WS_LOG_DIR/kern_log.txt"},{"line_number":94,"context_line":"elif is_fedora; then"},{"line_number":95,"context_line":"    # the journal gives us syslog() and kernel output, so is like"},{"line_number":96,"context_line":"    # a concatenation of the above."},{"line_number":97,"context_line":"    sudo journalctl --no-pager \\"}],"source_content_type":"text/x-sh","patch_set":10,"id":"3a50d1a3_3407e8ca","line":94,"updated":"2015-07-31 05:29:38.000000000","message":"I suggest o check for \"uses systemd\" and not directly for Fedora/CentOS.","commit_id":"f17a26b9ee39017192b76756683b826c6c14984b"}],"jenkins/scripts/copy_puppet_logs.sh":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"0e59c9cd167bdc786bc6b0a5bebabe712ea6924c","unresolved":false,"context_lines":[{"line_number":32,"context_line":"for project in /etc/puppet/modules/*; do"},{"line_number":33,"context_line":"    # find Puppet OpenStack modules"},{"line_number":34,"context_line":"    if [ -f $project/metadata.json ]; then"},{"line_number":35,"context_line":"        if grep -q \u0027github.com/openstack/puppet\u0027 $project/metadata.json; then"},{"line_number":36,"context_line":"            PROJECTS+\u003d\"$(basename $project) \""},{"line_number":37,"context_line":"        fi"},{"line_number":38,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":13,"id":"3a50d1a3_3bff9367","line":35,"updated":"2015-08-02 19:20:13.000000000","message":"this is actually listing our Puppet modules :)","commit_id":"506783def912d5cd2a4a74e4a6c6f99ba5b4301d"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"0e59c9cd167bdc786bc6b0a5bebabe712ea6924c","unresolved":false,"context_lines":[{"line_number":50,"context_line":"}"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"# Archive the project config \u0026 logs"},{"line_number":53,"context_line":"mkdir $LOG_DIR/etc/"},{"line_number":54,"context_line":"for p in $PROJECTS; do"},{"line_number":55,"context_line":"    if [ -d /etc/$p ]; then"},{"line_number":56,"context_line":"        sudo cp -r /etc/$p $LOG_DIR/etc/"}],"source_content_type":"text/x-sh","patch_set":13,"id":"3a50d1a3_7b099b94","line":53,"updated":"2015-08-02 19:20:13.000000000","message":"do not use sudo otherwise it leads to permissions errors later.","commit_id":"506783def912d5cd2a4a74e4a6c6f99ba5b4301d"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"0e59c9cd167bdc786bc6b0a5bebabe712ea6924c","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    if [ -d /etc/$p ]; then"},{"line_number":56,"context_line":"        sudo cp -r /etc/$p $LOG_DIR/etc/"},{"line_number":57,"context_line":"    fi"},{"line_number":58,"context_line":"    if [ -d $/var/log/$p ]; then"},{"line_number":59,"context_line":"        sudo cp -r /var/log/$p $LOG_DIR"},{"line_number":60,"context_line":"    fi"},{"line_number":61,"context_line":"done"}],"source_content_type":"text/x-sh","patch_set":13,"id":"3a50d1a3_1bfad777","line":58,"updated":"2015-08-02 19:20:13.000000000","message":"it was a typo.","commit_id":"506783def912d5cd2a4a74e4a6c6f99ba5b4301d"}]}