)]}'
{"fixtures/scenario001.pp":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"a579401c6aea35908c64f72abfff77c700292553","unresolved":false,"context_lines":[{"line_number":59,"context_line":"include ::openstack_integration::gnocchi"},{"line_number":60,"context_line":"include ::openstack_integration::ceph"},{"line_number":61,"context_line":"include ::openstack_integration::provision"},{"line_number":62,"context_line":"include ::openstack_integration::panko"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"class { \u0027::openstack_integration::tempest\u0027:"},{"line_number":65,"context_line":"  cinder     \u003d\u003e true,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"7a77a97e_9bcc2c63","line":62,"updated":"2016-11-22 13:42:30.000000000","message":"Panko is not packaged yet in Ubuntu I think. You\u0027ll need to put a conditional here, like we do for some services in other scenarios.  Also maybe ping zigo or jamespage on #openstack-dev aboit it, so they can package it in Debian/Ubuntu.","commit_id":"0d4f141f9eafdfba93beb3052c343e7357fb8747"},{"author":{"_account_id":9414,"name":"zhongshengping","email":"chdzsp@163.com","username":"ZhongShengping"},"change_message_id":"66763ecbbd527ad4a70b9ca4608a95754c5f0c08","unresolved":false,"context_lines":[{"line_number":70,"context_line":"  cinder     \u003d\u003e true,"},{"line_number":71,"context_line":"  gnocchi    \u003d\u003e true,"},{"line_number":72,"context_line":"  ceilometer \u003d\u003e true,"},{"line_number":73,"context_line":"  aodh       \u003d\u003e true,"},{"line_number":74,"context_line":"}"}],"source_content_type":"text/x-puppet","patch_set":7,"id":"3a71b18c_76d84f54","line":73,"updated":"2016-12-02 07:02:30.000000000","message":"You should add panko parameter in ::::openstack_integration::tempest, add panko \u003d\u003e true in there.\nI will add panko service in puppet-tempest.","commit_id":"4bad26536777b6b80e2b0974a2851d18dd640d17"},{"author":{"_account_id":6924,"name":"Pradeep Kilambi","email":"pkilambi@redhat.com","username":"pkilambi"},"change_message_id":"a5150df2e6fd9b9884cc2dafaee9838327e78b6b","unresolved":false,"context_lines":[{"line_number":70,"context_line":"  cinder     \u003d\u003e true,"},{"line_number":71,"context_line":"  gnocchi    \u003d\u003e true,"},{"line_number":72,"context_line":"  ceilometer \u003d\u003e true,"},{"line_number":73,"context_line":"  aodh       \u003d\u003e true,"},{"line_number":74,"context_line":"}"}],"source_content_type":"text/x-puppet","patch_set":7,"id":"3a71b18c_d7774c9d","line":73,"in_reply_to":"3a71b18c_76d84f54","updated":"2016-12-02 13:22:40.000000000","message":"panko is not in tempest hence we dont include it here yet. There functional tests are still in ceilometer and have not moved to panko yet. hence we will be verifying this via ceilometer by setting the event dispatcher.","commit_id":"4bad26536777b6b80e2b0974a2851d18dd640d17"}],"manifests/ceilometer.pp":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"47e49878637115c20420a16ec8d40d5399311a4f","unresolved":false,"context_lines":[{"line_number":69,"context_line":"  }"},{"line_number":70,"context_line":"  class { \u0027::ceilometer::collector\u0027:"},{"line_number":71,"context_line":"    collector_workers \u003d\u003e \u00272\u0027,"},{"line_number":72,"context_line":"    event_dispatcher  \u003d\u003e \u0027panko\u0027,"},{"line_number":73,"context_line":"  }"},{"line_number":74,"context_line":"  class { \u0027::ceilometer::expirer\u0027: }"},{"line_number":75,"context_line":"  class { \u0027::ceilometer::agent::notification\u0027:"}],"source_content_type":"text/x-puppet","patch_set":6,"id":"5a74a57a_00ec1a8e","line":72,"updated":"2016-11-24 13:42:08.000000000","message":"doesn\u0027t it break Ubuntu tests? how can it work if we don\u0027t have panko deployed?","commit_id":"03a33e327980a02baaaaebfe6709a6be7f287e87"},{"author":{"_account_id":6924,"name":"Pradeep Kilambi","email":"pkilambi@redhat.com","username":"pkilambi"},"change_message_id":"375a2a8b18568eed000f24adf0928fd751fb1b05","unresolved":false,"context_lines":[{"line_number":69,"context_line":"  }"},{"line_number":70,"context_line":"  class { \u0027::ceilometer::collector\u0027:"},{"line_number":71,"context_line":"    collector_workers \u003d\u003e \u00272\u0027,"},{"line_number":72,"context_line":"    event_dispatcher  \u003d\u003e \u0027panko\u0027,"},{"line_number":73,"context_line":"  }"},{"line_number":74,"context_line":"  class { \u0027::ceilometer::expirer\u0027: }"},{"line_number":75,"context_line":"  class { \u0027::ceilometer::agent::notification\u0027:"}],"source_content_type":"text/x-puppet","patch_set":6,"id":"5a74a57a_631cf313","line":72,"in_reply_to":"5a74a57a_00ec1a8e","updated":"2016-11-24 15:30:38.000000000","message":"we could deploy to multiple dispatchers. If panko doesnt exist it should ignore it. So may be we just set it to [\u0027database\u0027, \u0027panko\u0027] ?","commit_id":"03a33e327980a02baaaaebfe6709a6be7f287e87"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"e3b8542d5b2e8a0c596c864e864aed099f0599dd","unresolved":false,"context_lines":[{"line_number":69,"context_line":"  }"},{"line_number":70,"context_line":"  class { \u0027::ceilometer::collector\u0027:"},{"line_number":71,"context_line":"    collector_workers \u003d\u003e \u00272\u0027,"},{"line_number":72,"context_line":"    event_dispatcher  \u003d\u003e \u0027panko\u0027,"},{"line_number":73,"context_line":"  }"},{"line_number":74,"context_line":"  class { \u0027::ceilometer::expirer\u0027: }"},{"line_number":75,"context_line":"  class { \u0027::ceilometer::agent::notification\u0027:"}],"source_content_type":"text/x-puppet","patch_set":6,"id":"5a74a57a_3b93fba4","line":72,"in_reply_to":"5a74a57a_631cf313","updated":"2016-11-24 20:03:00.000000000","message":"This needs to be addressed, ceilometer-collector breaks with Ubuntu: http://logs.openstack.org/65/400765/6/check/gate-puppet-openstack-integration-4-scenario001-tempest-ubuntu-xenial/6aa15da/logs/ceilometer/ceilometer-collector.txt.gz\n\nCan you make the event_dispatcher conditional with panko if it\u0027s enabled or something ?","commit_id":"03a33e327980a02baaaaebfe6709a6be7f287e87"}],"manifests/panko.pp":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"a579401c6aea35908c64f72abfff77c700292553","unresolved":false,"context_lines":[{"line_number":18,"context_line":"  }"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"  # https://bugs.launchpad.net/panko/+bug/1557154"},{"line_number":21,"context_line":"  Rabbitmq_user_permissions[\u0027panko@/\u0027] -\u003e Service\u003c| tag \u003d\u003d \u0027panko-service\u0027 |\u003e"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"  if $::openstack_integration::config::ssl {"},{"line_number":24,"context_line":"    openstack_integration::ssl_key { \u0027panko\u0027:"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"7a77a97e_dbd6a4f0","line":21,"updated":"2016-11-22 13:42:30.000000000","message":"please remove it, it was an Aodh bug ;-)","commit_id":"0d4f141f9eafdfba93beb3052c343e7357fb8747"}]}