| *** tosky has quit IRC | 00:22 | |
| *** tobias-urdin has quit IRC | 01:04 | |
| *** rcernin_ has quit IRC | 02:21 | |
| *** rcernin_ has joined #puppet-openstack | 02:48 | |
| *** rcernin_ has quit IRC | 02:53 | |
| *** rcernin has joined #puppet-openstack | 02:54 | |
| *** ykarel has joined #puppet-openstack | 04:07 | |
| *** ykarel_ has joined #puppet-openstack | 06:03 | |
| *** ysandeep|away is now known as ysandeep | 06:03 | |
| *** ykarel has quit IRC | 06:05 | |
| openstackgerrit | Pranali Deore proposed openstack/puppet-glance stable/train: Add parameters for multipath connection in cinder backend https://review.opendev.org/c/openstack/puppet-glance/+/780090 | 06:06 |
|---|---|---|
| openstackgerrit | Pranali Deore proposed openstack/puppet-glance stable/train: Add parameters for multipath connection in cinder backend https://review.opendev.org/c/openstack/puppet-glance/+/780090 | 06:07 |
| openstackgerrit | Merged openstack/puppet-neutron stable/ussuri: Ensure resource_provider_bandwitdhs is unset by default https://review.opendev.org/c/openstack/puppet-neutron/+/780095 | 06:12 |
| *** ykarel_ has quit IRC | 06:20 | |
| *** ykarel has joined #puppet-openstack | 07:05 | |
| *** pleia2 has quit IRC | 07:12 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration master: Collect logs in post when not collected in run phase https://review.opendev.org/c/openstack/puppet-openstack-integration/+/781073 | 07:38 |
| *** rpittau|afk is now known as rpittau | 07:51 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-keystone master: Add support for keystone_uwsgi_config in Debian https://review.opendev.org/c/openstack/puppet-keystone/+/780918 | 07:53 |
| zigo | tkajinam: With this last version of the patch, I'm not really sure a class in openstacklib would make sense. Your thoughts/ | 08:01 |
| zigo | ? | 08:02 |
| *** jpena|off is now known as jpena | 08:02 | |
| *** amoralej|off is now known as amoralej | 08:08 | |
| *** rcernin has quit IRC | 08:37 | |
| *** tosky has joined #puppet-openstack | 08:37 | |
| *** pleia2 has joined #puppet-openstack | 08:40 | |
| *** ykarel is now known as ykarel|away | 08:46 | |
| tkajinam | zigo, if we expect the same format, (almost) same parameters then having a basic class in openstacklib should make sense | 08:49 |
| tkajinam | it requires a bit effort when we update base interfaces (because we need the same fix in each puppet modules) | 08:51 |
| tkajinam | but let us avoid implementing own proviers/types in each modules | 08:51 |
| tkajinam | zigo, if you want I can submit some patches for your reference... hopefully early next wek | 08:52 |
| tkajinam | week | 08:52 |
| tkajinam | sorry. maybe I'm wrong about this "let us avoid implementing own proviers/types in each modules" | 09:03 |
| *** rcernin has joined #puppet-openstack | 09:05 | |
| *** derekh has joined #puppet-openstack | 09:15 | |
| *** rcernin has quit IRC | 09:19 | |
| *** ykarel|away has quit IRC | 09:25 | |
| *** ysandeep is now known as ysandeep|afk | 09:26 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-openstacklib master: Add support for uwsgi configuration https://review.opendev.org/c/openstack/puppet-openstacklib/+/781230 | 09:33 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-openstacklib master: Add support for uwsgi configuration https://review.opendev.org/c/openstack/puppet-openstacklib/+/781230 | 09:37 |
| *** Caterpillar has joined #puppet-openstack | 09:57 | |
| *** rcernin has joined #puppet-openstack | 10:10 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration master: Add syntax job on CentOS 8 Stream https://review.opendev.org/c/openstack/puppet-openstack-integration/+/780856 | 10:29 |
| *** rcernin has quit IRC | 10:40 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-glance stable/train: Add parameters for multipath connection in cinder backend https://review.opendev.org/c/openstack/puppet-glance/+/780090 | 10:55 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-glance stable/ussuri: cinder backend: password should be secret https://review.opendev.org/c/openstack/puppet-glance/+/781202 | 10:57 |
| *** dtantsur|afk is now known as dtantsur | 11:02 | |
| *** ysandeep|afk is now known as ysandeep | 11:19 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-oslo master: Add support for [oslo_policy] enforce_new_defaults https://review.opendev.org/c/openstack/puppet-oslo/+/781428 | 11:52 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-oslo master: Add support for oslo_policy/enforce_new_defaults https://review.opendev.org/c/openstack/puppet-oslo/+/781428 | 11:55 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-keystone master: Add support for oslo_policy/enforce_new_defaults https://review.opendev.org/c/openstack/puppet-keystone/+/781429 | 11:58 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-keystone master: Add support for oslo_policy/enforce_new_defaults https://review.opendev.org/c/openstack/puppet-keystone/+/781429 | 12:00 |
| zigo | tkajinam: You get me confused now ... :) What should I do then? | 12:15 |
| tkajinam | zigo, add unit tests | 12:16 |
| zigo | tkajinam: Oh, thanks for https://review.opendev.org/c/openstack/puppet-openstacklib/+/781230 ! :) | 12:16 |
| tkajinam | migration to openstacklib can be done later. | 12:16 |
| tkajinam | that implementation in openstacklib requires keystone_uwsgi_config | 12:17 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-openstacklib master: Add support for uwsgi configuration https://review.opendev.org/c/openstack/puppet-openstacklib/+/781230 | 12:19 |
| zigo | Indeed. Now I get what you meant. You just wanted to write a generic: | 12:20 |
| zigo | openstacklib::wsgi::uwsgi {'keystone_uwsgi_config': } | 12:20 |
| zigo | which does some kind of factoring (ie: one single place to edit all *_uwsgi_config in a similar way). | 12:20 |
| zigo | I just didn't understood what you suggested before seeing your patch. | 12:20 |
| tkajinam | :-) | 12:22 |
| tkajinam | yeah | 12:22 |
| *** rcernin has joined #puppet-openstack | 12:22 | |
| tkajinam | in the beginning I thought to migrate type/provider as well but notice it doesn't work because we hard code file path in these code | 12:23 |
| tkajinam | so we still need to implement *_uwsgi_config in each modules | 12:23 |
| tkajinam | but can set parameters using that openstacklib::wsgi::uwsgi and *_uswsgi_config | 12:23 |
| zigo | Yeah. | 12:23 |
| zigo | And we need the puppet-keystone as a dependency of the openstacklib patch, so it must be done in 2 stages. | 12:24 |
| *** rcernin has quit IRC | 12:24 | |
| *** gfidente|afk is now known as gfidente | 12:24 | |
| tkajinam | yes | 12:24 |
| *** rcernin has joined #puppet-openstack | 12:24 | |
| tkajinam | I didn't notice that dependency until I wrote that patch to openstacklib | 12:25 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-keystone master: Add support for keystone_uwsgi_config in Debian https://review.opendev.org/c/openstack/puppet-keystone/+/780918 | 12:27 |
| zigo | This one removes the ineritance, the patch stays otherwise the same. | 12:28 |
| *** jpena is now known as jpena|lunch | 12:30 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Replace the remaning usage of gem --no-ri and --no-rdoc https://review.opendev.org/c/openstack/puppet-openstack-integration/+/780797 | 12:36 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Ubuntu: Switch to Focal and Victoria https://review.opendev.org/c/openstack/puppet-openstack-integration/+/759332 | 12:36 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-openstack-integration master: Switch to Ceph Octopus from Nautilus https://review.opendev.org/c/openstack/puppet-openstack-integration/+/773882 | 12:36 |
| *** jpena|lunch is now known as jpena | 13:26 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-ceilometer master: Remove puppetlabs-apache from dependencies https://review.opendev.org/c/openstack/puppet-ceilometer/+/781462 | 13:29 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-cinder master: Bump requirement of puppetlabs-apache https://review.opendev.org/c/openstack/puppet-cinder/+/781463 | 13:30 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-ironic master: Bump requirement of puppetlabs-apache https://review.opendev.org/c/openstack/puppet-ironic/+/781464 | 13:32 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-neutron master: Add puppetlabs-apache to dependencies https://review.opendev.org/c/openstack/puppet-neutron/+/781465 | 13:34 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-keystone master: Bump requirement of puppetlabs-apache https://review.opendev.org/c/openstack/puppet-keystone/+/781466 | 13:35 |
| openstackgerrit | Merged openstack/puppet-nova master: Bump requirement of puppetlabs-apache https://review.opendev.org/c/openstack/puppet-nova/+/780684 | 13:35 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-placement master: Bump requirement of puppetlabs-apache https://review.opendev.org/c/openstack/puppet-placement/+/781467 | 13:36 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-manila master: Bump requirement of puppetlabs-apache https://review.opendev.org/c/openstack/puppet-manila/+/781468 | 13:37 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-zaqar master: Bump requirement of puppetlabs-apache https://review.opendev.org/c/openstack/puppet-zaqar/+/781469 | 13:38 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-mistral master: Bump requirement of puppetlabs-apache https://review.opendev.org/c/openstack/puppet-mistral/+/781471 | 13:38 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-freezer master: Bump requirement of puppetlabs-apache https://review.opendev.org/c/openstack/puppet-freezer/+/781472 | 13:39 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-neutron master: Add puppetlabs-apache to dependencies https://review.opendev.org/c/openstack/puppet-neutron/+/781465 | 13:41 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-aodh master: Add puppetlabs-apache to dependencies https://review.opendev.org/c/openstack/puppet-aodh/+/781476 | 13:44 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-neutron master: Add puppetlabs-apache to dependencies https://review.opendev.org/c/openstack/puppet-neutron/+/781465 | 13:44 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-barbican master: Add puppetlabs-apache to dependencies https://review.opendev.org/c/openstack/puppet-barbican/+/781477 | 13:46 |
| openstackgerrit | Merged openstack/puppet-ceilometer stable/train: Add TLS options for oslo.cache https://review.opendev.org/c/openstack/puppet-ceilometer/+/775649 | 13:46 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-octavia master: Add puppetlabs-apache to dependencies https://review.opendev.org/c/openstack/puppet-octavia/+/781479 | 13:49 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-panko master: Add puppetlabs-apache to dependencies https://review.opendev.org/c/openstack/puppet-panko/+/781482 | 13:52 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-oslo master: Add support for oslo_policy/enforce_new_defaults https://review.opendev.org/c/openstack/puppet-oslo/+/781428 | 14:17 |
| *** rcernin has quit IRC | 14:35 | |
| openstackgerrit | Merged openstack/puppet-swift master: Do not write /var/log/ceilometer https://review.opendev.org/c/openstack/puppet-swift/+/780779 | 14:43 |
| *** ysandeep is now known as ysandeep|away | 15:51 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration stable/ussuri: Do not use mirror host for Ceph Luminous https://review.opendev.org/c/openstack/puppet-openstack-integration/+/773658 | 16:09 |
| openstackgerrit | Merged openstack/puppet-glance master: Add new conf parameter required for distribute image import https://review.opendev.org/c/openstack/puppet-glance/+/780810 | 16:47 |
| *** rpittau is now known as rpittau|afk | 17:11 | |
| *** jpena is now known as jpena|off | 17:16 | |
| *** amoralej is now known as amoralej|off | 17:18 | |
| *** gfidente is now known as gfidente|afk | 17:29 | |
| openstackgerrit | Merged openstack/puppet-neutron stable/ussuri: Add support for the resource_provider_hypervisors parameter https://review.opendev.org/c/openstack/puppet-neutron/+/780467 | 17:35 |
| openstackgerrit | Merged openstack/puppet-neutron stable/train: Add support for the resource_provider_hypervisors parameter https://review.opendev.org/c/openstack/puppet-neutron/+/780758 | 17:35 |
| openstackgerrit | Merged openstack/puppet-zaqar master: Bump requirement of puppetlabs-apache https://review.opendev.org/c/openstack/puppet-zaqar/+/781469 | 17:41 |
| *** dtantsur is now known as dtantsur|afk | 17:55 | |
| *** derekh has quit IRC | 18:00 | |
| *** hjensas has quit IRC | 18:10 | |
| *** hjensas has joined #puppet-openstack | 18:14 | |
| *** openstackgerrit has quit IRC | 19:34 | |
| *** gfidente|afk has quit IRC | 20:45 | |
| *** rcernin has joined #puppet-openstack | 20:52 | |
| *** rcernin has quit IRC | 21:06 | |
| *** rcernin has joined #puppet-openstack | 21:15 | |
| *** rcernin has quit IRC | 21:22 | |
| *** openstackstatus has quit IRC | 21:26 | |
| *** openstack has joined #puppet-openstack | 21:39 | |
| *** ChanServ sets mode: +o openstack | 21:39 | |
| *** openstack has joined #puppet-openstack | 22:04 | |
| *** ChanServ sets mode: +o openstack | 22:04 | |
| *** iurygregory has quit IRC | 22:09 | |
| *** iurygregory has joined #puppet-openstack | 22:09 | |
| *** openstackgerrit has joined #puppet-openstack | 23:05 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-oslo master: Add support for oslo_policy/enforce_new_defaults https://review.opendev.org/c/openstack/puppet-oslo/+/781428 | 23:05 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-swift master: Fix name of ceilometer middleware package https://review.opendev.org/c/openstack/puppet-swift/+/777564 | 23:07 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!