)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"23d0944db5cc19c16e6d8e5ab8fc0172d21dbb2f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e0edb2e1_60f4f1f7","updated":"2022-05-16 17:19:36.000000000","message":"Will this need to be updated or reverted when  libvirt changes come online?\nhttps://review.opendev.org/c/starlingx/integ/+/824928","commit_id":"eb048c6289d24a6ac81de586edeb0b4aad45c0cc"},{"author":{"_account_id":33750,"name":"Leonardo Fagundes Luz Serrano","display_name":"Leonardo Serrano","email":"Leonardo.FagundesLuzSerrano@windriver.com","username":"lfagunde"},"change_message_id":"e39605c9ba5b281431067cd211f208941831ebb6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"755526e8_c3bca0cc","in_reply_to":"e0edb2e1_60f4f1f7","updated":"2022-05-16 17:39:38.000000000","message":"yes. And integ/debian_pkg_dirs will probably also need to be updated.\nhttps://review.opendev.org/c/starlingx/integ/+/818768","commit_id":"eb048c6289d24a6ac81de586edeb0b4aad45c0cc"}],"puppet-manifests/src/modules/platform/manifests/compute.pp":[{"author":{"_account_id":28464,"name":"Eric MacDonald","display_name":"Eric","email":"eric.macdonald@windriver.com","username":"rocksolidmtce"},"change_message_id":"3bb19f5e57d6d8ff1e49a7b007cb385390e497bb","unresolved":true,"context_lines":[{"line_number":371,"context_line":"  require ::platform::compute::machine"},{"line_number":372,"context_line":"  require ::platform::compute::config"},{"line_number":373,"context_line":""},{"line_number":374,"context_line":"  # Not included in Debian until libvirt gets included"},{"line_number":375,"context_line":"  if $::osfamily \u003d\u003d \u0027RedHat\u0027 {"},{"line_number":376,"context_line":"    require ::platform::compute::kvm_timer_advance"},{"line_number":377,"context_line":"  }"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"768b2d73_4008677b","line":374,"range":{"start_line":374,"start_character":2,"end_line":374,"end_character":54},"updated":"2022-05-12 13:37:33.000000000","message":"OK, I see several other puppet files using this same os based condition.\n\nHowever, since the change does affect CentOS don\u0027t you think there needs to be a CentOS install test case that validates this code path in CentOS.","commit_id":"e0c30680276702df2af37d278e84a7301e1a2c1a"},{"author":{"_account_id":33750,"name":"Leonardo Fagundes Luz Serrano","display_name":"Leonardo Serrano","email":"Leonardo.FagundesLuzSerrano@windriver.com","username":"lfagunde"},"change_message_id":"93ca6099ede122cc8490294a442ce47de35bb9cb","unresolved":true,"context_lines":[{"line_number":371,"context_line":"  require ::platform::compute::machine"},{"line_number":372,"context_line":"  require ::platform::compute::config"},{"line_number":373,"context_line":""},{"line_number":374,"context_line":"  # Not included in Debian until libvirt gets included"},{"line_number":375,"context_line":"  if $::osfamily \u003d\u003d \u0027RedHat\u0027 {"},{"line_number":376,"context_line":"    require ::platform::compute::kvm_timer_advance"},{"line_number":377,"context_line":"  }"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"df964e71_8202ad49","line":374,"range":{"start_line":374,"start_character":2,"end_line":374,"end_character":54},"in_reply_to":"768b2d73_4008677b","updated":"2022-05-12 13:45:59.000000000","message":"I\u0027ll update the test plan","commit_id":"e0c30680276702df2af37d278e84a7301e1a2c1a"},{"author":{"_account_id":33750,"name":"Leonardo Fagundes Luz Serrano","display_name":"Leonardo Serrano","email":"Leonardo.FagundesLuzSerrano@windriver.com","username":"lfagunde"},"change_message_id":"a2ad365195bc8a2b9a6af4cb55439df103dcbec1","unresolved":false,"context_lines":[{"line_number":371,"context_line":"  require ::platform::compute::machine"},{"line_number":372,"context_line":"  require ::platform::compute::config"},{"line_number":373,"context_line":""},{"line_number":374,"context_line":"  # Not included in Debian until libvirt gets included"},{"line_number":375,"context_line":"  if $::osfamily \u003d\u003d \u0027RedHat\u0027 {"},{"line_number":376,"context_line":"    require ::platform::compute::kvm_timer_advance"},{"line_number":377,"context_line":"  }"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"bd5fd052_0d3f5f6a","line":374,"range":{"start_line":374,"start_character":2,"end_line":374,"end_character":54},"in_reply_to":"df964e71_8202ad49","updated":"2022-05-12 19:40:26.000000000","message":"Confirmed behavior is unchanged on centos","commit_id":"e0c30680276702df2af37d278e84a7301e1a2c1a"}]}
