)]}'
{"manifests/compute/xenserver.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"8ef28e41af010611a158f74c04be29b1b4417044","unresolved":false,"context_lines":[{"line_number":47,"context_line":"  package { \u0027xenapi\u0027:"},{"line_number":48,"context_line":"    ensure   \u003d\u003e present,"},{"line_number":49,"context_line":"    provider \u003d\u003e pip,"},{"line_number":50,"context_line":"    require  \u003d\u003e Package[\u0027python-pip\u0027],"},{"line_number":51,"context_line":"    tag      \u003d\u003e [\u0027openstack\u0027, \u0027nova-support-package\u0027],"},{"line_number":52,"context_line":"  }"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":3,"id":"9a8ffd7b_591eee0c","line":50,"updated":"2015-12-03 12:23:09.000000000","message":"i\u0027m not sure that we need it, take a look on line 54","commit_id":"7cf6c7d1331bfd037912b70a797a05a92061fcc2"},{"author":{"_account_id":10540,"name":"Clayton O\u0027Neill","email":"clayton.oneill@charter.com","username":"coneill"},"change_message_id":"d8dc661afa2fa1b9cc30a9862ccab0070bcd6b25","unresolved":false,"context_lines":[{"line_number":47,"context_line":"  package { \u0027xenapi\u0027:"},{"line_number":48,"context_line":"    ensure   \u003d\u003e present,"},{"line_number":49,"context_line":"    provider \u003d\u003e pip,"},{"line_number":50,"context_line":"    require  \u003d\u003e Package[\u0027python-pip\u0027],"},{"line_number":51,"context_line":"    tag      \u003d\u003e [\u0027openstack\u0027, \u0027nova-support-package\u0027],"},{"line_number":52,"context_line":"  }"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":3,"id":"9a8ffd7b_d9f3be71","line":50,"in_reply_to":"9a8ffd7b_591eee0c","updated":"2015-12-03 12:27:32.000000000","message":"Done","commit_id":"7cf6c7d1331bfd037912b70a797a05a92061fcc2"}],"manifests/db.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"8ef28e41af010611a158f74c04be29b1b4417044","unresolved":false,"context_lines":[{"line_number":110,"context_line":"      package {\u0027nova-backend-package\u0027:"},{"line_number":111,"context_line":"        ensure \u003d\u003e present,"},{"line_number":112,"context_line":"        name   \u003d\u003e $backend_package,"},{"line_number":113,"context_line":"        tag    \u003d\u003e [\u0027openstack\u0027, \u0027nova-package\u0027],"},{"line_number":114,"context_line":"      }"},{"line_number":115,"context_line":"    }"},{"line_number":116,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":3,"id":"9a8ffd7b_59e58e06","line":113,"updated":"2015-12-03 12:23:09.000000000","message":"nova-support-package?","commit_id":"7cf6c7d1331bfd037912b70a797a05a92061fcc2"},{"author":{"_account_id":10540,"name":"Clayton O\u0027Neill","email":"clayton.oneill@charter.com","username":"coneill"},"change_message_id":"d8dc661afa2fa1b9cc30a9862ccab0070bcd6b25","unresolved":false,"context_lines":[{"line_number":110,"context_line":"      package {\u0027nova-backend-package\u0027:"},{"line_number":111,"context_line":"        ensure \u003d\u003e present,"},{"line_number":112,"context_line":"        name   \u003d\u003e $backend_package,"},{"line_number":113,"context_line":"        tag    \u003d\u003e [\u0027openstack\u0027, \u0027nova-package\u0027],"},{"line_number":114,"context_line":"      }"},{"line_number":115,"context_line":"    }"},{"line_number":116,"context_line":""}],"source_content_type":"text/x-puppet","patch_set":3,"id":"9a8ffd7b_99d216d0","line":113,"in_reply_to":"9a8ffd7b_59e58e06","updated":"2015-12-03 12:27:32.000000000","message":"I decided to make nova-package things that would require restarting nova for the change to take effect, and nova-support-package things that would not.  There is a comment to that effect in nova::deps.","commit_id":"7cf6c7d1331bfd037912b70a797a05a92061fcc2"}],"manifests/generic_service.pp":[{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"758286e9bdf1177c3a411fe59868b3f1f4782216","unresolved":false,"context_lines":[{"line_number":68,"context_line":"      ensure    \u003d\u003e $service_ensure,"},{"line_number":69,"context_line":"      name      \u003d\u003e $service_name,"},{"line_number":70,"context_line":"      enable    \u003d\u003e $enabled,"},{"line_number":71,"context_line":"      hasstatus \u003d\u003e true,"},{"line_number":72,"context_line":"      tag       \u003d\u003e \u0027nova-service\u0027,"},{"line_number":73,"context_line":"    }"},{"line_number":74,"context_line":"  }"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"7a740942_8faa63d3","line":71,"updated":"2015-12-04 11:38:12.000000000","message":"Shouldn\u0027t it has tag \u0027nova-service\u0027?","commit_id":"a6e0a4f9340c857bd40c5157e9e470b9636220db"},{"author":{"_account_id":7745,"name":"Denis Egorenko","email":"degorenko@mirantis.com","username":"degorenko"},"change_message_id":"e4ffc2fb849eaf91d4c93883a3ae0ba93a2c538f","unresolved":false,"context_lines":[{"line_number":68,"context_line":"      ensure    \u003d\u003e $service_ensure,"},{"line_number":69,"context_line":"      name      \u003d\u003e $service_name,"},{"line_number":70,"context_line":"      enable    \u003d\u003e $enabled,"},{"line_number":71,"context_line":"      hasstatus \u003d\u003e true,"},{"line_number":72,"context_line":"      tag       \u003d\u003e \u0027nova-service\u0027,"},{"line_number":73,"context_line":"    }"},{"line_number":74,"context_line":"  }"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"7a740942_c7e2442f","line":71,"in_reply_to":"7a740942_8faa63d3","updated":"2015-12-07 12:48:46.000000000","message":"Sorry, misread next line.","commit_id":"a6e0a4f9340c857bd40c5157e9e470b9636220db"}]}