| openstackgerrit | Merged openstack/puppet-openstack-integration master: Fix redis config location for centos > 8 https://review.opendev.org/c/openstack/puppet-openstack-integration/+/792325 | 02:37 |
|---|---|---|
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-aodh master: Use oslo::coordination to manage coordination parameters https://review.opendev.org/c/openstack/puppet-aodh/+/792001 | 03:02 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-aodh master: Use oslo::coordination to manage coordination parameters https://review.opendev.org/c/openstack/puppet-aodh/+/792001 | 03:08 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-aodh master: Use oslo::coordination to manage coordination parameters https://review.opendev.org/c/openstack/puppet-aodh/+/792001 | 03:11 |
| *** rcernin has quit IRC | 03:20 | |
| *** rcernin has joined #puppet-openstack | 03:23 | |
| *** ykarel|away has joined #puppet-openstack | 03:43 | |
| *** ade_lee has quit IRC | 04:26 | |
| *** ade_lee has joined #puppet-openstack | 04:26 | |
| *** ysandeep|away is now known as ysandeep|dol | 04:43 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-aodh master: Use oslo::coordination to manage coordination parameters https://review.opendev.org/c/openstack/puppet-aodh/+/792001 | 05:19 |
| *** ykarel|away is now known as ykarel | 05:40 | |
| *** ykarel has quit IRC | 06:09 | |
| openstackgerrit | Merged openstack/puppet-openstacklib stable/wallaby: Replace URI.escape() with ERB::Util.url_encode() https://review.opendev.org/c/openstack/puppet-openstacklib/+/792291 | 06:13 |
| *** benj_ has quit IRC | 06:19 | |
| *** ykarel has joined #puppet-openstack | 06:27 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-aodh master: Use oslo::coordination to manage coordination parameters https://review.opendev.org/c/openstack/puppet-aodh/+/792001 | 06:56 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-aodh master: Use oslo::coordination to manage coordination parameters https://review.opendev.org/c/openstack/puppet-aodh/+/792001 | 06:57 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-ceilometer master: Use oslo::coordination to manage coordination parameters https://review.opendev.org/c/openstack/puppet-ceilometer/+/792509 | 07:04 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-ceilometer master: Use oslo::coordination to manage coordination parameters https://review.opendev.org/c/openstack/puppet-ceilometer/+/792509 | 07:06 |
| *** amoralej|off is now known as amoralej | 07:13 | |
| *** amoralej is now known as amoralej|off | 07:13 | |
| *** ysandeep|dol is now known as ysandeep|brb | 07:14 | |
| *** ysandeep|brb is now known as ysandeep | 07:37 | |
| *** cgoncalves has quit IRC | 07:43 | |
| *** rpittau|afk is now known as rpittau | 07:43 | |
| *** cgoncalves has joined #puppet-openstack | 07:44 | |
| *** gfidente has joined #puppet-openstack | 07:45 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-ceilometer master: Use oslo::coordination to manage coordination parameters https://review.opendev.org/c/openstack/puppet-ceilometer/+/792509 | 07:51 |
| *** jpena|off is now known as jpena | 07:53 | |
| *** tosky has joined #puppet-openstack | 07:59 | |
| *** amoralej|off is now known as amoralej | 07:59 | |
| *** rcernin has quit IRC | 08:04 | |
| *** ysandeep is now known as ysandeep|away | 08:09 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-aodh master: Use oslo::coordination to manage coordination parameters https://review.opendev.org/c/openstack/puppet-aodh/+/792001 | 08:11 |
| *** derekh has joined #puppet-openstack | 08:12 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-oslo master: coordination: disable config management https://review.opendev.org/c/openstack/puppet-oslo/+/792535 | 08:16 |
| *** ykarel is now known as ykarel|lunch | 08:21 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-horizon master: Add support for independent heat-dashboard configurations https://review.opendev.org/c/openstack/puppet-horizon/+/775322 | 08:22 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-horizon master: Add support for independent octavia-dashboard configurations https://review.opendev.org/c/openstack/puppet-horizon/+/789701 | 08:22 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-horizon master: Add support for independent manila-dashboard configurations https://review.opendev.org/c/openstack/puppet-horizon/+/790350 | 08:22 |
| *** benj_ has joined #puppet-openstack | 08:27 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-gnocchi master: Fix a test case with typo https://review.opendev.org/c/openstack/puppet-gnocchi/+/792540 | 08:30 |
| *** gfidente has quit IRC | 08:34 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-gnocchi master: Simplfy the logic to handle coordination_url https://review.opendev.org/c/openstack/puppet-gnocchi/+/792541 | 08:36 |
| *** rcernin has joined #puppet-openstack | 08:37 | |
| *** rcernin has quit IRC | 08:38 | |
| *** rcernin has joined #puppet-openstack | 08:38 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-aodh master: Use oslo::coordination to manage coordination parameters https://review.opendev.org/c/openstack/puppet-aodh/+/792001 | 08:39 |
| zigo | tkajinam: Is it fine if I start the $pyvers removal now? | 08:40 |
| zigo | I mean, accross all modules ... | 08:40 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-aodh master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-aodh/+/792542 | 08:42 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-barbican master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-barbican/+/792543 | 08:43 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-ceilometer master: Use oslo::coordination to manage coordination parameters https://review.opendev.org/c/openstack/puppet-ceilometer/+/792509 | 08:43 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-aodh master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-aodh/+/792542 | 08:46 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-barbican master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-barbican/+/792543 | 08:47 |
| tkajinam | zigo, no | 08:48 |
| tkajinam | zigo, but you should fix unit tests | 08:48 |
| tkajinam | zigo, there are some leftovers for ancient centos7 | 08:48 |
| tkajinam | which expects that pyvers | 08:48 |
| tkajinam | that path is no longer invoked but should be cleared up | 08:48 |
| zigo | tkajinam: I've grepped for "pyvers" and didn't find it anywhere but in params.pp | 08:49 |
| tkajinam | zigo, | 08:49 |
| tkajinam | I'm talking about this https://github.com/openstack/puppet-aodh/blob/master/spec/classes/aodh_client_spec.rb#L31-L44 | 08:49 |
| zigo | Oh, I see it indeed. | 08:49 |
| zigo | tkajinam: So we just drop support for CentOS 7 ? | 08:50 |
| tkajinam | zigo, we already dropped support for CentOS | 08:50 |
| tkajinam | 2 or 3 cycles ago | 08:50 |
| zigo | :) | 08:50 |
| zigo | Ok. | 08:50 |
| tkajinam | but these implementation were left since nobody has done clean up | 08:50 |
| *** gfidente has joined #puppet-openstack | 08:51 | |
| tkajinam | :-P | 08:51 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-aodh master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-aodh/+/792542 | 08:52 |
| zigo | tkajinam: Does the spec/classes/aodh_client_spec.rb file looks correct to you in this patch? (just so I can do the same kind of patch elsewhere...) | 08:52 |
| tkajinam | zigo, I think it's better to remove only logics inside RedHat | 08:53 |
| tkajinam | keep the current switch based on osfamily | 08:53 |
| zigo | Can do ... | 08:54 |
| zigo | Thanks for voicing your opinion. | 08:54 |
| tkajinam | zigo, note that you should replace "" by '' when you remove ${pyvers} | 08:55 |
| tkajinam | otherwise lint reject these lines | 08:55 |
| zigo | Right. | 08:55 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-aodh master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-aodh/+/792542 | 08:57 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-barbican master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-barbican/+/792543 | 08:59 |
| tobias-urdin | please dont remove the openstacklib::defaults include | 09:00 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-cinder master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-cinder/+/792545 | 09:01 |
| tobias-urdin | zigo: ^ before you push more patches | 09:02 |
| tkajinam | yeah | 09:02 |
| zigo | tobias-urdin: Obviously ! :) | 09:02 |
| zigo | Oh... | 09:02 |
| zigo | tobias-urdin: What's the point if we don't use it anymore? | 09:02 |
| zigo | openstacklib::defaults only has the pyvers stuff ... | 09:03 |
| tobias-urdin | it's the only way to add changes across all modules without patching them | 09:06 |
| tobias-urdin | lets say when puppet 6 is eol we add a notify there like we did for puppet 5 | 09:06 |
| tobias-urdin | or any other changes that needs to be across all modules, like before we were setting virtual_packages=>True for Package resource across all modules until Puppet 4 made it default | 09:06 |
| zigo | Ok, I'll keep it then. | 09:06 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-cloudkitty master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-cloudkitty/+/792546 | 09:07 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-aodh master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-aodh/+/792542 | 09:08 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-barbican master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-barbican/+/792543 | 09:08 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-cinder master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-cinder/+/792545 | 09:09 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-designate master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-designate/+/792547 | 09:10 |
| zigo | tobias-urdin: BTW, do you understand what's going on with validate_legacy at https://review.opendev.org/c/openstack/puppet-vswitch/+/792321 ? | 09:11 |
| zigo | I don't get why it fails in Debian... :/ | 09:11 |
| *** ykarel|lunch is now known as ykarel | 09:12 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-glance master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-glance/+/792549 | 09:15 |
| *** rcernin has quit IRC | 09:16 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-gnocchi master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-gnocchi/+/792550 | 09:21 |
| *** rcernin has joined #puppet-openstack | 09:22 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-heat master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-heat/+/792551 | 09:23 |
| tobias-urdin | zigo: what version of stdlib are you using | 09:25 |
| zigo | tobias-urdin: 5.0.0 | 09:25 |
| zigo | Gosh, we even didn't update that package in Bullseye ... :( | 09:25 |
| tobias-urdin | are you using your own modules? verify you have stdlib dependency defines in your metadata.json files | 09:26 |
| zigo | tobias-urdin: I was just trying to get puppet-openstack-integration to work in Bullseye, so not my own module, no... | 09:27 |
| zigo | I tried installing the puppet-module-puppetlabs-stdlib Debian package, and it didn't help. | 09:28 |
| tobias-urdin | does the puppet apply even look in the correct puppet module directory, i have some vague memory that you had a different directory | 09:29 |
| tobias-urdin | other than that, i don't have any good guesses for you | 09:30 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-horizon master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-horizon/+/792552 | 09:31 |
| *** rcernin has quit IRC | 09:36 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-ironic master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-ironic/+/792553 | 09:37 |
| zigo | tobias-urdin: The Debian system is configured by default to use what's in the packaged stdlib, so it should be good. I have the feeling that maybe, what's different is the prototype of the validate_legacy class, or some types not supported ... | 09:38 |
| zigo | Then it ends up with Puppet saying that I don't have the class at all, when really, it could just be that I have it, just not with the same number of params. | 09:39 |
| zigo | Is something like this possible ? | 09:39 |
| tobias-urdin | dont think so, seems more that i looks in the wrong place then | 09:40 |
| tobias-urdin | but i've never done anything using the native debian packages, only ubuntu but that was a while ago | 09:40 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-keystone master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-keystone/+/792554 | 09:43 |
| zigo | Thanks for your input, I'll try to figure it out... | 09:43 |
| *** dtantsur|afk is now known as dtantsur | 09:44 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-magnum master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-magnum/+/792555 | 09:47 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-manila master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-manila/+/792558 | 09:49 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-murano master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-murano/+/792561 | 09:51 |
| *** rcernin has joined #puppet-openstack | 09:54 | |
| *** rcernin has quit IRC | 09:57 | |
| *** rcernin has joined #puppet-openstack | 09:57 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-neutron master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-neutron/+/792564 | 10:05 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-nova master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-nova/+/792566 | 10:08 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-octavia master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-octavia/+/792568 | 10:10 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-oslo master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-oslo/+/792569 | 10:13 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-gnocchi master: Fix a test case with typo https://review.opendev.org/c/openstack/puppet-gnocchi/+/792540 | 10:18 |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-gnocchi master: Simplfy the logic to handle coordination_url https://review.opendev.org/c/openstack/puppet-gnocchi/+/792541 | 10:18 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-placement master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-placement/+/792573 | 10:25 |
| openstackgerrit | Merged openstack/puppet-oslo master: Add the base implementation for coordination parameters https://review.opendev.org/c/openstack/puppet-oslo/+/791628 | 10:39 |
| *** rcernin has quit IRC | 11:16 | |
| *** jpena is now known as jpena|lunch | 11:31 | |
| openstackgerrit | Merged openstack/puppet-keystone stable/train: Prepare the final stable/train release https://review.opendev.org/c/openstack/puppet-keystone/+/791204 | 11:51 |
| *** olasd has left #puppet-openstack | 11:52 | |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-sahara master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-sahara/+/792584 | 12:09 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-swift master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-swift/+/792585 | 12:13 |
| openstackgerrit | Thomas Goirand proposed openstack/puppet-tempest master: Get rid of the $pyvers variable https://review.opendev.org/c/openstack/puppet-tempest/+/792586 | 12:19 |
| *** amoralej is now known as amoralej|lunch | 12:22 | |
| *** jpena|lunch is now known as jpena | 12:29 | |
| openstackgerrit | Alan Bishop proposed openstack/puppet-openstack-integration stable/queens: Rocky and before: Use ubuntu-bionic by default https://review.opendev.org/c/openstack/puppet-openstack-integration/+/792282 | 12:57 |
| *** amoralej|lunch is now known as amoralej | 13:07 | |
| *** ysandeep|away is now known as ysandeep | 14:15 | |
| *** rpittau is now known as rpittau|afk | 14:27 | |
| openstackgerrit | Merged openstack/puppet-glance master: TripleO: Reduce tempest test scope https://review.opendev.org/c/openstack/puppet-glance/+/791347 | 15:03 |
| *** ykarel is now known as ykarel|away | 15:05 | |
| *** ykarel|away has quit IRC | 15:17 | |
| openstackgerrit | Merged openstack/puppet-cinder master: TripleO: Reduce tempest test scope https://review.opendev.org/c/openstack/puppet-cinder/+/791345 | 15:18 |
| *** gfidente is now known as gfidente|afk | 15:23 | |
| openstackgerrit | Merged openstack/puppet-swift master: Add a swift_drive_audit_config puppet type https://review.opendev.org/c/openstack/puppet-swift/+/790657 | 15:26 |
| *** amoralej is now known as amoralej|off | 15:46 | |
| *** ykarel|away has joined #puppet-openstack | 16:09 | |
| *** dtantsur is now known as dtantsur|afk | 16:25 | |
| *** ykarel|away has quit IRC | 16:28 | |
| *** ykarel|away has joined #puppet-openstack | 16:38 | |
| *** ykarel|away has quit IRC | 16:43 | |
| *** derekh has quit IRC | 17:00 | |
| *** jpena is now known as jpena|off | 17:03 | |
| *** ysandeep is now known as ysandeep|away | 18:44 | |
| *** lbragstad has quit IRC | 18:50 | |
| *** gfidente|afk has quit IRC | 20:55 | |
| *** tosky has quit IRC | 21:15 | |
| *** abishop has quit IRC | 21:15 | |
| *** abishop has joined #puppet-openstack | 21:16 | |
| *** tosky has joined #puppet-openstack | 21:16 | |
| openstackgerrit | Merged openstack/puppet-openstack-integration master: Aodh: Set coordination_url to enable multiple evaluator workers https://review.opendev.org/c/openstack/puppet-openstack-integration/+/791960 | 21:19 |
| *** tosky has quit IRC | 23:03 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!