)]}'
{"README.md":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"2a22f628fb0a91ced6fa41308c0b4873a9afff1e","unresolved":false,"context_lines":[{"line_number":69,"context_line":"| ceph       |      X      |             |             |      X      |              |"},{"line_number":70,"context_line":"| ceph rgw   |             |             |             |      X      |              |"},{"line_number":71,"context_line":"| mongodb    |             |       X     |             |             |              |"},{"line_number":72,"context_line":"| vitrage    |             |       X     |             |             |              |"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"When the Jenkins slave is created, the *run_tests.sh* script will executed."},{"line_number":75,"context_line":"This script will execute *install_modules.sh* that prepare /etc/puppet/modules"}],"source_content_type":"text/x-gfm","patch_set":1,"id":"1a430d35_23fa03be","line":72,"updated":"2017-02-06 13:43:28.000000000","message":"you picked scenario002 in readme and scenario001 in fixtures. Please pick one of both.","commit_id":"2cd589afaaa9264def384910553d9be13a260ae6"}],"manifests/tempest.pp":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"c12d6b536a70ab32b4a836322d332a999684c38e","unresolved":false,"context_lines":[{"line_number":108,"context_line":"  $sahara                  \u003d false,"},{"line_number":109,"context_line":"  $swift                   \u003d false,"},{"line_number":110,"context_line":"  $trove                   \u003d false,"},{"line_number":111,"context_line":"  $watcher                 \u003d false,"},{"line_number":112,"context_line":"  $vitrage                 \u003d false,"},{"line_number":113,"context_line":"  $zaqar                   \u003d false,"},{"line_number":114,"context_line":"  $attach_encrypted_volume \u003d false,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"1a430d35_431c6f13","line":111,"updated":"2017-02-06 13:43:50.000000000","message":"you need to document this parameter, lint will fail.","commit_id":"2cd589afaaa9264def384910553d9be13a260ae6"}],"manifests/vitrage.pp":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"647f2bc6e4ce7c561e6324d664c8c0cb825209bd","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    debug                 \u003d\u003e true,"},{"line_number":39,"context_line":"  } -\u003e"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"  # Make sure tempest can read the configuration files"},{"line_number":42,"context_line":"  # default installation has a 640 premission"},{"line_number":43,"context_line":"  file { \u0027/etc/vitrage\u0027:"},{"line_number":44,"context_line":"    ensure  \u003d\u003e directory,"},{"line_number":45,"context_line":"    recurse \u003d\u003e true,"},{"line_number":46,"context_line":"    mode    \u003d\u003e \u00270644\u0027,"},{"line_number":47,"context_line":"  } -\u003e"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"  # Make sure tempest can write to the log directory"},{"line_number":50,"context_line":"  # default installation has a 755 premission"},{"line_number":51,"context_line":"  file { \u0027/var/log/vitrage\u0027:"},{"line_number":52,"context_line":"    ensure \u003d\u003e directory,"},{"line_number":53,"context_line":"    mode   \u003d\u003e \u00270766\u0027,"},{"line_number":54,"context_line":"  }"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"  class { \u0027::vitrage::keystone::auth\u0027:"},{"line_number":57,"context_line":"    public_url   \u003d\u003e \"${::openstack_integration::config::base_url}:8999\","}],"source_content_type":"text/x-puppet","patch_set":19,"id":"fa31d9ce_80e1fc9c","line":54,"range":{"start_line":41,"start_character":0,"end_line":54,"end_character":3},"updated":"2017-02-12 18:00:26.000000000","message":"why we don\u0027t do that in the packaging?","commit_id":"4d0a45a64fb8e74a28ad37fec1b1be2232d42b0d"},{"author":{"_account_id":19134,"name":"Eyal","email":"eyalb1@gmail.com","username":"eyalb"},"change_message_id":"dadd3da2fe2ac890e5e837413678dac787e0860f","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    debug                 \u003d\u003e true,"},{"line_number":39,"context_line":"  } -\u003e"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"  # Make sure tempest can read the configuration files"},{"line_number":42,"context_line":"  # default installation has a 640 premission"},{"line_number":43,"context_line":"  file { \u0027/etc/vitrage\u0027:"},{"line_number":44,"context_line":"    ensure  \u003d\u003e directory,"},{"line_number":45,"context_line":"    recurse \u003d\u003e true,"},{"line_number":46,"context_line":"    mode    \u003d\u003e \u00270644\u0027,"},{"line_number":47,"context_line":"  } -\u003e"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"  # Make sure tempest can write to the log directory"},{"line_number":50,"context_line":"  # default installation has a 755 premission"},{"line_number":51,"context_line":"  file { \u0027/var/log/vitrage\u0027:"},{"line_number":52,"context_line":"    ensure \u003d\u003e directory,"},{"line_number":53,"context_line":"    mode   \u003d\u003e \u00270766\u0027,"},{"line_number":54,"context_line":"  }"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"  class { \u0027::vitrage::keystone::auth\u0027:"},{"line_number":57,"context_line":"    public_url   \u003d\u003e \"${::openstack_integration::config::base_url}:8999\","}],"source_content_type":"text/x-puppet","patch_set":19,"id":"fa31d9ce_9a7dce70","line":54,"range":{"start_line":41,"start_character":0,"end_line":54,"end_character":3},"in_reply_to":"fa31d9ce_80e1fc9c","updated":"2017-02-13 14:23:22.000000000","message":"I need this only for the purpose of running tempest, tempest is executed here with permission that cannot read the configuration file or write to the log directory of vitrage.\nI don\u0027t want to change the permission in production this. BTW when running the tempest in the gate in the vitrage project there is no problem probably because it is running under a user with the right permissions","commit_id":"4d0a45a64fb8e74a28ad37fec1b1be2232d42b0d"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"411a31f93ffed74148e0e17cbfed642fb0c92f36","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    debug                 \u003d\u003e true,"},{"line_number":39,"context_line":"  } -\u003e"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"  # Make sure tempest can read the configuration files"},{"line_number":42,"context_line":"  # default installation has a 640 premission"},{"line_number":43,"context_line":"  file { \u0027/etc/vitrage\u0027:"},{"line_number":44,"context_line":"    ensure  \u003d\u003e directory,"},{"line_number":45,"context_line":"    recurse \u003d\u003e true,"},{"line_number":46,"context_line":"    mode    \u003d\u003e \u00270644\u0027,"},{"line_number":47,"context_line":"  } -\u003e"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"  # Make sure tempest can write to the log directory"},{"line_number":50,"context_line":"  # default installation has a 755 premission"},{"line_number":51,"context_line":"  file { \u0027/var/log/vitrage\u0027:"},{"line_number":52,"context_line":"    ensure \u003d\u003e directory,"},{"line_number":53,"context_line":"    mode   \u003d\u003e \u00270766\u0027,"},{"line_number":54,"context_line":"  }"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"  class { \u0027::vitrage::keystone::auth\u0027:"},{"line_number":57,"context_line":"    public_url   \u003d\u003e \"${::openstack_integration::config::base_url}:8999\","}],"source_content_type":"text/x-puppet","patch_set":22,"id":"fa31d9ce_9434f74a","line":54,"range":{"start_line":41,"start_character":0,"end_line":54,"end_character":3},"updated":"2017-02-17 12:41:21.000000000","message":"I don\u0027t like that, but we can live with it I think for now.","commit_id":"94fad23968c3c16a60d17110454e5d14a0e19459"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"b5e491519f59bde4ac5c2a936091b559d35182ee","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    debug                 \u003d\u003e true,"},{"line_number":39,"context_line":"  } -\u003e"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"  # Make sure tempest can read the configuration files"},{"line_number":42,"context_line":"  # default installation has a 640 premission"},{"line_number":43,"context_line":"  file { \u0027/etc/vitrage\u0027:"},{"line_number":44,"context_line":"    ensure  \u003d\u003e directory,"},{"line_number":45,"context_line":"    recurse \u003d\u003e true,"},{"line_number":46,"context_line":"    mode    \u003d\u003e \u00270644\u0027,"},{"line_number":47,"context_line":"  } -\u003e"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"  # Make sure tempest can write to the log directory"},{"line_number":50,"context_line":"  # default installation has a 755 premission"},{"line_number":51,"context_line":"  file { \u0027/var/log/vitrage\u0027:"},{"line_number":52,"context_line":"    ensure \u003d\u003e directory,"},{"line_number":53,"context_line":"    mode   \u003d\u003e \u00270766\u0027,"},{"line_number":54,"context_line":"  }"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"  class { \u0027::vitrage::keystone::auth\u0027:"},{"line_number":57,"context_line":"    public_url   \u003d\u003e \"${::openstack_integration::config::base_url}:8999\","}],"source_content_type":"text/x-puppet","patch_set":22,"id":"fa31d9ce_744ea397","line":54,"range":{"start_line":41,"start_character":0,"end_line":54,"end_character":3},"in_reply_to":"fa31d9ce_9434f74a","updated":"2017-02-17 12:51:16.000000000","message":"Yeah, maybe update vitrage-module to ensure this if possible \u003d)","commit_id":"94fad23968c3c16a60d17110454e5d14a0e19459"}],"run_tests.sh":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"647f2bc6e4ce7c561e6324d664c8c0cb825209bd","unresolved":false,"context_lines":[{"line_number":224,"context_line":""},{"line_number":225,"context_line":"# Vitrage"},{"line_number":226,"context_line":"# until I fix the tempest tests (they only work in the gate)"},{"line_number":227,"context_line":"#echo \"TestEvents\" \u003e\u003e /tmp/openstack/tempest/test-whitelist.txt"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"# Ironic"},{"line_number":230,"context_line":"# Note: running all Ironic tests under SSL is not working"}],"source_content_type":"text/x-sh","patch_set":19,"id":"fa31d9ce_00a34c50","line":227,"range":{"start_line":227,"start_character":0,"end_line":227,"end_character":63},"updated":"2017-02-12 18:00:26.000000000","message":"please un-comment it or we won\u0027t test it.","commit_id":"4d0a45a64fb8e74a28ad37fec1b1be2232d42b0d"},{"author":{"_account_id":19134,"name":"Eyal","email":"eyalb1@gmail.com","username":"eyalb"},"change_message_id":"dadd3da2fe2ac890e5e837413678dac787e0860f","unresolved":false,"context_lines":[{"line_number":224,"context_line":""},{"line_number":225,"context_line":"# Vitrage"},{"line_number":226,"context_line":"# until I fix the tempest tests (they only work in the gate)"},{"line_number":227,"context_line":"#echo \"TestEvents\" \u003e\u003e /tmp/openstack/tempest/test-whitelist.txt"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":"# Ironic"},{"line_number":230,"context_line":"# Note: running all Ironic tests under SSL is not working"}],"source_content_type":"text/x-sh","patch_set":19,"id":"fa31d9ce_9ab5ceef","line":227,"range":{"start_line":227,"start_character":0,"end_line":227,"end_character":63},"in_reply_to":"fa31d9ce_00a34c50","updated":"2017-02-13 14:23:22.000000000","message":"I will add depends on I still need to fix the puppet-vitrage","commit_id":"4d0a45a64fb8e74a28ad37fec1b1be2232d42b0d"},{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"7bf6b7297bd16f06389d7bcbe6890c9b8432d874","unresolved":false,"context_lines":[{"line_number":243,"context_line":"echo \"gnocchi.tempest\" \u003e\u003e /tmp/openstack/tempest/test-whitelist.txt"},{"line_number":244,"context_line":""},{"line_number":245,"context_line":"# Vitrage"},{"line_number":246,"context_line":"# until I fix the tempest tests (they only work in the gate)"},{"line_number":247,"context_line":"echo \"TestTopology.test_compare_api_and_cli\" \u003e\u003e /tmp/openstack/tempest/test-whitelist.txt"},{"line_number":248,"context_line":""},{"line_number":249,"context_line":"# Ironic"}],"source_content_type":"text/x-sh","patch_set":22,"id":"fa31d9ce_14f2c79a","line":246,"updated":"2017-02-17 12:40:50.000000000","message":"maybe you can remove this comment. but can be done later.","commit_id":"94fad23968c3c16a60d17110454e5d14a0e19459"}]}