| opendevreview | Takashi Kajinami proposed openstack/puppet-neutron master: Add support for the resource_provider_default_hypervisor parameter https://review.opendev.org/c/openstack/puppet-neutron/+/796326 | 00:50 |
|---|---|---|
| opendevreview | zhangboye proposed openstack/puppet-designate master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-designate/+/796330 | 02:08 |
| opendevreview | zhangboye proposed openstack/puppet-ec2api master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-ec2api/+/796331 | 02:11 |
| opendevreview | zhangboye proposed openstack/puppet-gnocchi master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-gnocchi/+/796333 | 02:17 |
| opendevreview | zhangboye proposed openstack/puppet-heat master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-heat/+/796334 | 02:26 |
| opendevreview | zhangboye proposed openstack/puppet-horizon master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-horizon/+/796335 | 02:30 |
| opendevreview | zhangboye proposed openstack/puppet-magnum master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-magnum/+/796336 | 02:36 |
| opendevreview | zhangboye proposed openstack/puppet-manila master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-manila/+/796337 | 02:41 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-neutron master: Add support for the resource_provider_default_hypervisor parameter https://review.opendev.org/c/openstack/puppet-neutron/+/796326 | 03:08 |
| opendevreview | zhangboye proposed openstack/puppet-tempest master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-tempest/+/796354 | 03:17 |
| opendevreview | zhangboye proposed openstack/puppet-zaqar master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-zaqar/+/796356 | 03:20 |
| opendevreview | zhangboye proposed openstack/puppet-sahara master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-sahara/+/796358 | 03:28 |
| opendevreview | Rabi Mishra proposed openstack/puppet-neutron master: Add ironic endpoint_override parameter https://review.opendev.org/c/openstack/puppet-neutron/+/796359 | 03:30 |
| opendevreview | zhangboye proposed openstack/puppet-tacker master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-tacker/+/796360 | 03:30 |
| opendevreview | zhangboye proposed openstack/puppet-ovn master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-ovn/+/796361 | 03:34 |
| *** ykarel|away is now known as ykarel | 04:18 | |
| opendevreview | zhangboye proposed openstack/puppet-placement master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-placement/+/796370 | 05:48 |
| opendevreview | zhangboye proposed openstack/puppet-qdr master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-qdr/+/796371 | 05:51 |
| opendevreview | zhangboye proposed openstack/puppet-openstack_extras master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-openstack_extras/+/796372 | 05:55 |
| opendevreview | zhangboye proposed openstack/puppet-oslo master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-oslo/+/796373 | 05:58 |
| opendevreview | zhangboye proposed openstack/puppet-neutron master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-neutron/+/796374 | 06:02 |
| *** ysandeep|out is now known as ysandeep | 06:09 | |
| opendevreview | zhangboye proposed openstack/puppet-octavia master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-octavia/+/796376 | 06:12 |
| opendevreview | zhangboye proposed openstack/puppet-mistral master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-mistral/+/796377 | 06:17 |
| opendevreview | OpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints https://review.opendev.org/c/openstack/puppet-openstack-integration/+/794399 | 06:18 |
| *** iurygregory_ is now known as iurygregory | 06:19 | |
| opendevreview | zhangboye proposed openstack/puppet-murano master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-murano/+/796379 | 06:20 |
| *** jpena|off is now known as jpena | 06:34 | |
| *** rpittau|afk is now known as rpittau | 07:13 | |
| opendevreview | Wenping Song proposed openstack/puppet-keystone master: Change release note file name https://review.opendev.org/c/openstack/puppet-keystone/+/796385 | 07:21 |
| *** ykarel is now known as ykarel|lunch | 07:38 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-qdr master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-qdr/+/796139 | 07:56 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-oslo master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-oslo/+/796373 | 07:58 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-oslo master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-oslo/+/792569 | 08:00 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-oslo master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-oslo/+/796373 | 08:00 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-neutron master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-neutron/+/792564 | 08:10 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-neutron master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-neutron/+/796374 | 08:11 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-neutron master: Add support for the resource_provider_default_hypervisor parameter https://review.opendev.org/c/openstack/puppet-neutron/+/796326 | 08:21 |
| opendevreview | Ebbex proposed openstack/puppet-cinder master: Allow for '-' in property values https://review.opendev.org/c/openstack/puppet-cinder/+/796409 | 09:14 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-neutron master: WIP: Follow up of the parent change https://review.opendev.org/c/openstack/puppet-neutron/+/796411 | 09:17 |
| *** ykarel|lunch is now known as ykarel | 09:57 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-oslo master: Set fixture paths for unit tests https://review.opendev.org/c/openstack/puppet-oslo/+/796420 | 10:20 |
| opendevreview | Thomas Goirand proposed openstack/puppet-watcher master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-watcher/+/796238 | 10:59 |
| opendevreview | Thomas Goirand proposed openstack/puppet-zaqar master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-zaqar/+/796239 | 11:00 |
| zigo | tkajinam: Hi ! If you have time, can you review https://review.opendev.org/c/openstack/puppet-swift/+/795486 please ? | 11:13 |
| zigo | It solves real troubles in production ... | 11:13 |
| *** ysandeep is now known as ysandeep|afk | 11:24 | |
| *** jpena is now known as jpena|lunch | 11:25 | |
| tkajinam | zigo, please find my comments | 11:31 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-watcher master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-watcher/+/796103 | 11:33 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-zaqar master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-zaqar/+/796104 | 11:34 |
| zigo | tkajinam: What "the resource notification" are we talking about? | 11:36 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-neutron master: WIP: Follow up of the parent change https://review.opendev.org/c/openstack/puppet-neutron/+/796411 | 11:38 |
| tkajinam | zigo, when run_parted_and_mkfs is false then Exec["create_partition_label-${name}"] is never defined | 11:38 |
| zigo | Allright, indeed. | 11:38 |
| zigo | Will fix. | 11:38 |
| tkajinam | then the subsequent resource definition of swift::storage::xfs would fail | 11:38 |
| zigo | Thanks for the review. | 11:38 |
| tkajinam | because it can't subscribe the resource which doesn't exist | 11:38 |
| tkajinam | :-) | 11:38 |
| zigo | tkajinam: What happens in production, is that calling parted + xfs_admin flushes the HDD controller buffer, which pushes the load up to 200 (seen on a real case on a quite busy Swift cluster...). | 11:39 |
| zigo | So that's what I'm trying to avoid. | 11:39 |
| tkajinam | zigo, that's a problem :-( | 11:40 |
| tkajinam | zigo, I think we need to come up with better way to detect existing partition | 11:40 |
| tkajinam | not partition but xfs filesystem | 11:40 |
| tkajinam | but adding these parameters would work as a quick workaround, (I know that's what you propose here) | 11:41 |
| tkajinam | zigo, which one is the actual trigger ? parted ? or xfs_admin ? | 11:41 |
| zigo | tkajinam: I believe the most annoying one is parted. It also triggers udev rediscovery of devices, possibly even a reset of the HDD controller on some hardware. | 11:42 |
| tkajinam | ahh | 11:42 |
| tkajinam | ok | 11:42 |
| zigo | I'm not even sure why we would need to ever call parted -s ${base_dir}/${name} mklabel gpt ${ext_args} | 11:43 |
| zigo | Maybe there is a way to test in a more efficient way than using "parted ${base_dir}/${name} print" ? | 11:44 |
| *** amoralej is now known as amoralej|lunch | 11:50 | |
| opendevreview | Thomas Goirand proposed openstack/puppet-swift master: Make it possible to skip parted+mkfs https://review.opendev.org/c/openstack/puppet-swift/+/795486 | 11:52 |
| zigo | tkajinam: There you go, new version of the patch taking care of your suggestions... | 11:52 |
| opendevreview | Thomas Goirand proposed openstack/puppet-swift master: Make it possible to skip parted+mkfs https://review.opendev.org/c/openstack/puppet-swift/+/795486 | 12:00 |
| *** jpena|lunch is now known as jpena | 12:17 | |
| tkajinam | zigo, could you please check my comments ? | 12:17 |
| zigo | Ok, thanks. | 12:24 |
| *** ysandeep|afk is now known as ysandeep | 12:33 | |
| opendevreview | Thomas Goirand proposed openstack/puppet-swift master: Make it possible to skip parted+mkfs https://review.opendev.org/c/openstack/puppet-swift/+/795486 | 12:44 |
| zigo | Done. | 12:44 |
| *** amoralej|lunch is now known as amoralej | 12:51 | |
| tkajinam | zigo, seems some unit tests are failing | 13:13 |
| tkajinam | zigo, I leave a hint which would solve the current failures | 13:13 |
| * tkajinam is leaving away from his laptop | 13:14 | |
| opendevreview | Thomas Goirand proposed openstack/puppet-swift master: Make it possible to skip parted+mkfs https://review.opendev.org/c/openstack/puppet-swift/+/795486 | 13:17 |
| zigo | In this skip parted+mkfs patch?!? | 13:17 |
| zigo | What patch are you talking about? | 13:17 |
| tkajinam | zigo, that one | 13:17 |
| zigo | Ok. | 13:17 |
| tkajinam | puppet-oslo one needs the fix for spec_helper.rb, just fyi | 13:18 |
| opendevreview | Ebbex proposed openstack/puppet-cinder master: Allow for '-' in property values https://review.opendev.org/c/openstack/puppet-cinder/+/796409 | 14:07 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-neutron master: WIP: Follow up of the parent change https://review.opendev.org/c/openstack/puppet-neutron/+/796411 | 14:08 |
| *** rpittau is now known as rpittau|afk | 14:29 | |
| *** ykarel_ is now known as ykarel | 14:56 | |
| opendevreview | Daniel Alvarez proposed openstack/puppet-neutron master: [ovn metadata] Add agent report option https://review.opendev.org/c/openstack/puppet-neutron/+/796511 | 15:02 |
| *** jpena is now known as jpena|off | 15:18 | |
| *** ysandeep is now known as ysandeep|out | 15:41 | |
| *** amoralej is now known as amoralej|off | 15:53 | |
| opendevreview | Merged openstack/puppet-murano master: Drop Fedora support https://review.opendev.org/c/openstack/puppet-murano/+/796379 | 16:47 |
| opendevreview | Merged openstack/puppet-watcher master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-watcher/+/796238 | 17:57 |
| opendevreview | Thomas Goirand proposed openstack/puppet-swift master: Make it possible to skip parted+mkfs https://review.opendev.org/c/openstack/puppet-swift/+/795486 | 22:57 |
| opendevreview | Merged openstack/puppet-zaqar master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-zaqar/+/796239 | 23:07 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!