)]}'
{"manifests/client.pp":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"b640b67bc2445cb6989c0213953300b2cb34338a","unresolved":true,"context_lines":[{"line_number":11,"context_line":"class placement::client("},{"line_number":12,"context_line":"  $ensure \u003d \u0027present\u0027"},{"line_number":13,"context_line":") {"},{"line_number":14,"context_line":"  include placement::deps"},{"line_number":15,"context_line":"  include placement::params"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"  package { \u0027python-osc-placement\u0027:"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"cdb08b40_68abcc52","line":14,"updated":"2021-02-21 06:33:43.000000000","message":"Because we replaced all absolute name inclusions by relative name inclusions between Train and Ussuri, you should made these include use absolute names starting with \u0027::\u0027 in stable/train.","commit_id":"c99d9885443bbf4e1ec7f79283f51b3ebf3a1f2c"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"530d59133ba673431e3bf382e4b3d7f690fbd0e4","unresolved":false,"context_lines":[{"line_number":11,"context_line":"class placement::client("},{"line_number":12,"context_line":"  $ensure \u003d \u0027present\u0027"},{"line_number":13,"context_line":") {"},{"line_number":14,"context_line":"  include placement::deps"},{"line_number":15,"context_line":"  include placement::params"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"  package { \u0027python-osc-placement\u0027:"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"f8397a4a_8342caea","line":14,"in_reply_to":"cdb08b40_68abcc52","updated":"2021-02-22 15:03:00.000000000","message":"Done","commit_id":"c99d9885443bbf4e1ec7f79283f51b3ebf3a1f2c"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"5afac7865913a6dd10909a9887b42d37e17206cf","unresolved":true,"context_lines":[{"line_number":20,"context_line":"    tag    \u003d\u003e [\u0027openstack\u0027, \u0027placement-support-package\u0027],"},{"line_number":21,"context_line":"  }"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"  include \u0027openstacklib::openstackclient\u0027"},{"line_number":24,"context_line":"}"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"8d7e5365_dcb96551","line":23,"updated":"2021-02-22 14:44:45.000000000","message":"I guess this needs to be ::openstacklib::openstackclient for train","commit_id":"0b64bd279ec6dd580cc2d20b42fb34b3c2115a9a"},{"author":{"_account_id":27419,"name":"David Vallee Delisle","email":"me@dvd.dev","username":"dvd"},"change_message_id":"530d59133ba673431e3bf382e4b3d7f690fbd0e4","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    tag    \u003d\u003e [\u0027openstack\u0027, \u0027placement-support-package\u0027],"},{"line_number":21,"context_line":"  }"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"  include \u0027openstacklib::openstackclient\u0027"},{"line_number":24,"context_line":"}"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"fe9b97db_a2c4344f","line":23,"in_reply_to":"8d7e5365_dcb96551","updated":"2021-02-22 15:03:00.000000000","message":"you\u0027re right","commit_id":"0b64bd279ec6dd580cc2d20b42fb34b3c2115a9a"}],"spec/classes/placement_client_spec.rb":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"b640b67bc2445cb6989c0213953300b2cb34338a","unresolved":true,"context_lines":[{"line_number":37,"context_line":"            if facts[:operatingsystemmajrelease] \u003e \u00277\u0027"},{"line_number":38,"context_line":"              { :client_package_name \u003d\u003e \u0027python3-osc-placement\u0027 }"},{"line_number":39,"context_line":"            else"},{"line_number":40,"context_line":"              { :client_package_name \u003d\u003e \u0027python-osc-placement\u0027 }"},{"line_number":41,"context_line":"            end"},{"line_number":42,"context_line":"          end"},{"line_number":43,"context_line":"        end"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"fdfa1ec9_06651eb8","line":40,"range":{"start_line":40,"start_character":41,"end_line":40,"end_character":61},"updated":"2021-02-21 06:33:43.000000000","message":"Oops. Sorry this should have been python2-osc-placement. Because we removed unit tests on centos7 during Ussuri we didn\u0027t notice this mistake... We should fix this in master and backport this to V, U, then make this backport include that change, to fix thins in a clean way...","commit_id":"c99d9885443bbf4e1ec7f79283f51b3ebf3a1f2c"}]}