)]}'
{"Puppetfile_centos8":[{"author":{"_account_id":3153,"name":"Emilien Macchi","email":"emilien@redhat.com","username":"emilienm"},"change_message_id":"306426e44dab521be12a61b1160ec3f84292483a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Auto-generated Puppetfile for Puppet OpenStack project"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"## OpenStack modules"},{"line_number":4,"context_line":"mod \u0027aodh\u0027,"}],"source_content_type":"application/octet-stream","patch_set":11,"id":"3fa7e38b_e41fe86e","line":1,"updated":"2019-12-19 16:45:53.000000000","message":"I don\u0027t like this thing because the file won\u0027t get updated by the bot.","commit_id":"e874c67f197f8917dd1de14ef24d320fb0076a4d"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"3e8b63ef42c27f40634ec99110ec0ac5f341af1a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Auto-generated Puppetfile for Puppet OpenStack project"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"## OpenStack modules"},{"line_number":4,"context_line":"mod \u0027aodh\u0027,"}],"source_content_type":"application/octet-stream","patch_set":11,"id":"3fa7e38b_5b97b23a","line":1,"in_reply_to":"3fa7e38b_728df627","updated":"2019-12-20 04:46:16.000000000","message":"I think this should work considering removal of CentOS7 jobs soon in Ussuri, but we also need to think for stable/train where we would be running centos7 as per current plan atleast, @Emilien,@tobias wdyt?\nOr to move forward wihout messing with bot generated Puppetfile, we can try method[1] with which we just override required module for CentOS8 and once we have updated mariadb, we remove the hack? \n\n[1]\nKeep a seperate Puppetfile-centos8 with override modules, At runtime remove the module from Puppetfile we want to override, and append Puppetfile-centos8 contents to Puppetfile.","commit_id":"e874c67f197f8917dd1de14ef24d320fb0076a4d"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"126d93b34081408e3d69a2754a2fcef670672372","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Auto-generated Puppetfile for Puppet OpenStack project"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"## OpenStack modules"},{"line_number":4,"context_line":"mod \u0027aodh\u0027,"}],"source_content_type":"application/octet-stream","patch_set":11,"id":"3fa7e38b_728df627","line":1,"in_reply_to":"3fa7e38b_e41fe86e","updated":"2019-12-19 18:10:18.000000000","message":"we are trying to update mariadb on centos7 but it does not depends directly on us. \n\nThe plan in RDO is to stop Trunk builder for ussuri on CentOS7 once we get a promotion in CentOS8. By then, i think we can remove centos7 jobs from p-o-i (anyway ussuri will not be supported in CentOS7). What would you think on using the regular Puppetfile (the one updated by the bot) for CentOS8 (that will allow us to unblock puppet-mysql for ubuntu and CentOS8) and keep a Puppetfile-centos7 frozen with current versions that will live only until we finally remove CentOS7 (hopefully Jan-Feb 2020?).","commit_id":"e874c67f197f8917dd1de14ef24d320fb0076a4d"}],"fixtures/scenario001.pp":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"c59b8fedcb546b6d7855e2383f6325f9ab142655","unresolved":false,"context_lines":[{"line_number":107,"context_line":"  notification_topics \u003d\u003e $notification_topics,"},{"line_number":108,"context_line":"}"},{"line_number":109,"context_line":"class { \u0027openstack_integration::cinder\u0027:"},{"line_number":110,"context_line":"  backend \u003d\u003e $backend,"},{"line_number":111,"context_line":"}"},{"line_number":112,"context_line":"include openstack_integration::ceilometer"},{"line_number":113,"context_line":"class { \u0027openstack_integration::aodh\u0027:"}],"source_content_type":"text/x-puppet","patch_set":9,"id":"3fa7e38b_131853cb","line":110,"range":{"start_line":110,"start_character":14,"end_line":110,"end_character":21},"updated":"2019-12-19 05:28:49.000000000","message":"for cinder, it needs to be iscsi, will fix in next PS.","commit_id":"2414a4201d18652193981fa91a3b6f8119307113"}],"manifests/neutron.pp":[{"author":{"_account_id":16137,"name":"Tobias Urdin","email":"tobias.urdin@binero.com","username":"tobasco"},"change_message_id":"991487a0cdc1ac063597c57c6e737331a3982513","unresolved":false,"context_lines":[{"line_number":49,"context_line":"  case $driver {"},{"line_number":50,"context_line":"    \u0027openvswitch\u0027: {"},{"line_number":51,"context_line":"      include vswitch::ovs"},{"line_number":52,"context_line":"      # In CentOS8 puppet-vswitch requires network-scripts package until it\u0027s ported to NM."},{"line_number":53,"context_line":"      if ($::operatingsystem \u003d\u003d \u0027CentOS\u0027) and (versioncmp($::operatingsystemmajrelease, \u00278\u0027) \u003d\u003d 0) {"},{"line_number":54,"context_line":"        package { \u0027network-scripts-openvswitch\u0027:"},{"line_number":55,"context_line":"          ensure \u003d\u003e \u0027latest\u0027"},{"line_number":56,"context_line":"        }"},{"line_number":57,"context_line":"      }"},{"line_number":58,"context_line":"      # Functional test for Open-vSwitch:"},{"line_number":59,"context_line":"      # create dummy loopback interface to exercise adding a port to a bridge"},{"line_number":60,"context_line":"      vs_bridge { \u0027br-ex\u0027:"}],"source_content_type":"text/x-puppet","patch_set":10,"id":"3fa7e38b_2795093e","line":57,"range":{"start_line":52,"start_character":0,"end_line":57,"end_character":7},"updated":"2019-12-19 11:09:31.000000000","message":"Is this a temporary fix or should this actually go into puppet-vswitch to support CentOS 8?","commit_id":"5387d04e28f60403e36fa723c9bb2679932b39ef"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"8e9336c88f76ea0b0a1f29ef748c310962d0b20d","unresolved":false,"context_lines":[{"line_number":49,"context_line":"  case $driver {"},{"line_number":50,"context_line":"    \u0027openvswitch\u0027: {"},{"line_number":51,"context_line":"      include vswitch::ovs"},{"line_number":52,"context_line":"      # In CentOS8 puppet-vswitch requires network-scripts package until it\u0027s ported to NM."},{"line_number":53,"context_line":"      if ($::operatingsystem \u003d\u003d \u0027CentOS\u0027) and (versioncmp($::operatingsystemmajrelease, \u00278\u0027) \u003d\u003d 0) {"},{"line_number":54,"context_line":"        package { \u0027network-scripts-openvswitch\u0027:"},{"line_number":55,"context_line":"          ensure \u003d\u003e \u0027latest\u0027"},{"line_number":56,"context_line":"        }"},{"line_number":57,"context_line":"      }"},{"line_number":58,"context_line":"      # Functional test for Open-vSwitch:"},{"line_number":59,"context_line":"      # create dummy loopback interface to exercise adding a port to a bridge"},{"line_number":60,"context_line":"      vs_bridge { \u0027br-ex\u0027:"}],"source_content_type":"text/x-puppet","patch_set":10,"id":"3fa7e38b_98269a67","line":57,"range":{"start_line":52,"start_character":0,"end_line":57,"end_character":7},"in_reply_to":"3fa7e38b_2795093e","updated":"2019-12-19 13:21:21.000000000","message":"It\u0027s temporary fix, as it\u0027s recommended to move out of network-scripts in RHEL8. I\u0027m not sure, what\u0027s will be the best approach for puppet-vswitch, using nmcli or nmstate which will be in centos 8.1 so i prefer to do the fix here.","commit_id":"5387d04e28f60403e36fa723c9bb2679932b39ef"}],"zuul.d/integration.yaml":[{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"42da8607e5cf08b7a8879117a714c8a3ec3c6a37","unresolved":false,"context_lines":[{"line_number":115,"context_line":"    nodeset: centos-7"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"- job:"},{"line_number":118,"context_line":"    name: puppet-openstack-integration-5-scenario001-tempest-centos-8-luminous"},{"line_number":119,"context_line":"    parent: puppet-openstack-integration-5-scenario001"},{"line_number":120,"context_line":"    nodeset:"},{"line_number":121,"context_line":"      nodes:"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3fa7e38b_ccdbfc1d","line":118,"range":{"start_line":118,"start_character":70,"end_line":118,"end_character":78},"updated":"2019-12-19 09:57:28.000000000","message":"We probably should remove -luminous from this job as there is actually not ceph. And, anyway we will not have luminous for centos8.","commit_id":"5387d04e28f60403e36fa723c9bb2679932b39ef"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"4946e2d083f6cb4beec15f8f13dabfb09b730aa3","unresolved":false,"context_lines":[{"line_number":115,"context_line":"    nodeset: centos-7"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"- job:"},{"line_number":118,"context_line":"    name: puppet-openstack-integration-5-scenario001-tempest-centos-8-luminous"},{"line_number":119,"context_line":"    parent: puppet-openstack-integration-5-scenario001"},{"line_number":120,"context_line":"    nodeset:"},{"line_number":121,"context_line":"      nodes:"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3fa7e38b_a78259fe","line":118,"range":{"start_line":118,"start_character":70,"end_line":118,"end_character":78},"in_reply_to":"3fa7e38b_ccdbfc1d","updated":"2019-12-19 14:20:15.000000000","message":"ack will update","commit_id":"5387d04e28f60403e36fa723c9bb2679932b39ef"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"42da8607e5cf08b7a8879117a714c8a3ec3c6a37","unresolved":false,"context_lines":[{"line_number":221,"context_line":"      ceph: nautilus"},{"line_number":222,"context_line":""},{"line_number":223,"context_line":"- job:"},{"line_number":224,"context_line":"    name: puppet-openstack-integration-5-scenario004-tempest-centos-8-nautilus"},{"line_number":225,"context_line":"    parent: puppet-openstack-integration-5-scenario004"},{"line_number":226,"context_line":"    nodeset:"},{"line_number":227,"context_line":"      nodes:"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3fa7e38b_0cd114f9","line":224,"range":{"start_line":224,"start_character":70,"end_line":224,"end_character":78},"updated":"2019-12-19 09:57:28.000000000","message":"ditto","commit_id":"5387d04e28f60403e36fa723c9bb2679932b39ef"}]}