| *** tosky has quit IRC | 00:10 | |
| *** lbragstad_ is now known as lbragstad | 01:23 | |
| *** ysandeep|away is now known as ysandeep|bbl | 01:56 | |
| *** rcernin has quit IRC | 03:22 | |
| *** rcernin has joined #puppet-openstack | 03:33 | |
| *** rcernin has quit IRC | 03:41 | |
| *** rcernin has joined #puppet-openstack | 03:42 | |
| *** pleia2 has joined #puppet-openstack | 04:11 | |
| *** ykarel|away has joined #puppet-openstack | 04:17 | |
| *** ykarel|away is now known as ykarel | 04:17 | |
| *** ykarel_ has joined #puppet-openstack | 06:12 | |
| *** ykarel has quit IRC | 06:13 | |
| *** ykarel_ has quit IRC | 06:26 | |
| *** ykarel has joined #puppet-openstack | 06:30 | |
| *** ysandeep|bbl is now known as ysandeep | 06:57 | |
| *** ykarel_ has joined #puppet-openstack | 07:03 | |
| *** ykarel has quit IRC | 07:03 | |
| *** ykarel_ has quit IRC | 07:07 | |
| *** ykarel_ has joined #puppet-openstack | 07:08 | |
| *** ykarel_ has quit IRC | 07:12 | |
| *** gfidente|afk is now known as gfidente | 07:44 | |
| *** amoralej|off is now known as amoralej | 08:08 | |
| *** rcernin has quit IRC | 08:11 | |
| *** rpittau|afk is now known as rpittau | 08:23 | |
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-barbican master: Deprecate support for [p11_crypto_plugin]token_label https://review.opendev.org/c/openstack/puppet-barbican/+/780843 | 08:33 |
|---|---|---|
| openstackgerrit | Takashi Kajinami proposed openstack/puppet-barbican master: Accept a list value for p11_crypto_plugin_token_labels https://review.opendev.org/c/openstack/puppet-barbican/+/780845 | 08:33 |
| *** ysandeep is now known as ysandeep|lunch | 08:46 | |
| *** jpena|off is now known as jpena | 08:57 | |
| *** dtantsur|afk is now known as dtantsur | 09:01 | |
| *** tosky has joined #puppet-openstack | 09:03 | |
| tkajinam | EmilienM, tobias-urdin It would be nice if I can ask your reviews about these small backports https://review.opendev.org/q/topic:%22rp-bw-default%22+(status:open%20OR%20status:merged) | 09:03 |
| zigo | tobias-urdin: I did what you suggested and split my patch into 2: https://review.opendev.org/c/openstack/puppet-swift/+/780779 https://review.opendev.org/c/openstack/puppet-swift/+/777564 | 09:14 |
| *** rcernin has joined #puppet-openstack | 09:17 | |
| zigo | tkajinam: Would you know how I could do, so that each time keystone_uwsgi_config changes something, then keystone would restart ? In this patch ... https://review.opendev.org/c/openstack/puppet-keystone/+/780918 | 09:25 |
| tkajinam | zigo, you can implement dependencies in deps.pp | 09:31 |
| tkajinam | zigo, like https://github.com/openstack/puppet-keystone/blob/master/manifests/deps.pp#L41-L44 | 09:32 |
| tkajinam | zigo, also, I'd suggest you create an independent class like keystone::wsgi::uwsgi | 09:32 |
| tkajinam | instead of adding that parameter to keystone | 09:33 |
| *** openstackgerrit has quit IRC | 09:33 | |
| zigo | tkajinam: I can do that, though it may be overkill, no ? | 09:33 |
| zigo | tkajinam: I'm really not sure how to implement the thing in deps.pp... What would be the middle line between the 2 Anchor ? | 09:36 |
| tkajinam | zigo, it is consistent with current implementation for apache and also helps us implement a new parameters specific to uwsgi in a future | 09:36 |
| zigo | tkajinam: Right, I'll try to do that then. | 09:37 |
| tkajinam | basically I think we should deprecate the workers, port and etc from the base classes | 09:37 |
| zigo | tkajinam: My intention is to add this to *all* projects that are including an API, because currently in Debian, I have no ways to change the number of workers... | 09:37 |
| tkajinam | regarding the current direction to deprecate eventlet workers | 09:37 |
| tkajinam | zigo, I'll leave some comments on the patch later... including the way how you define dependencies | 09:40 |
| *** ykarel has joined #puppet-openstack | 09:44 | |
| *** rcernin_ has joined #puppet-openstack | 09:48 | |
| *** rcernin has quit IRC | 09:55 | |
| *** ysandeep|lunch is now known as ysandeep | 09:56 | |
| zigo | tkajinam: Thanks, I believe it looks good now. :) | 10:34 |
| *** rcernin_ has quit IRC | 11:32 | |
| EmilienM | tkajinam: ack | 12:24 |
| EmilienM | tkajinam: done | 12:26 |
| tkajinam | EmilienM, thx ! | 12:26 |
| *** jpena is now known as jpena|lunch | 12:34 | |
| *** ykarel has quit IRC | 12:59 | |
| *** trozet has quit IRC | 13:09 | |
| *** ykarel has joined #puppet-openstack | 13:12 | |
| *** amoralej is now known as amoralej|lunch | 13:21 | |
| *** jpena|lunch is now known as jpena | 13:32 | |
| tobias-urdin | tkajinam: checking | 13:45 |
| tobias-urdin | also +1 on the "I think we should deprecate the workers, port and etc from the base classes" and sticking to <project>::wsgi::(apache|uwsgi) etc | 13:46 |
| tobias-urdin | move shared logic to openstacklib etc | 13:46 |
| tkajinam | :-) | 13:47 |
| tobias-urdin | zigo: ack | 13:52 |
| *** amoralej|lunch is now known as amoralej | 14:01 | |
| tobias-urdin | tkajinam: what do you think about pulling in puppet 7 when we have c8s stable in X-cycle | 14:01 |
| tobias-urdin | actually seems to run straight up without any changes | 14:02 |
| tkajinam | tobias-urdin, I think it's ok but let me talk with rdo guys | 14:02 |
| tkajinam | tobias-urdin, and I think we need puppet 7 first in ubuntu | 14:03 |
| tkajinam | recently I was trying bumping ceph versions to octopus | 14:03 |
| tkajinam | and noticed that requires bumping ubuntu to focal | 14:03 |
| tkajinam | then after bumping focal we see horrible number of warning now | 14:03 |
| tkajinam | seems that focal has too newer version of ruby for puppet 6. it works but it shows huge warnings | 14:04 |
| tkajinam | https://review.opendev.org/c/openstack/puppet-nova/+/775512 | 14:04 |
| tkajinam | https://zuul.opendev.org/t/openstack/build/1705d36c642c48b1a88aa1b15ff97952 | 14:05 |
| tkajinam | though interestingly ubuntu currently provides puppet 5.11 for focal ... | 14:06 |
| tobias-urdin | that's a lot of warnings :D | 14:07 |
| tobias-urdin | isn't changed in any newer 6.x release than 6.14? | 14:07 |
| tobias-urdin | changed/fixed | 14:07 |
| tkajinam | I haven't tried that . let me try it later | 14:08 |
| tobias-urdin | that's only units though so we should be fine if packaged version works (which it prob does since it contains it's own ruby stack) | 14:08 |
| tkajinam | but if I read this correctly I'm afraid complete work for ruby 2.7 was done in puppet 7 https://tickets.puppetlabs.com/browse/PUP-10247 | 14:09 |
| tkajinam | yeah | 14:09 |
| tobias-urdin | last time i checked on focal i was stuck on magnum, now it's fixed with https://review.opendev.org/c/openstack/magnum/+/777910 | 14:10 |
| tobias-urdin | so requested release with https://review.opendev.org/c/openstack/releases/+/781103 | 14:10 |
| tobias-urdin | to get focal and victoria | 14:10 |
| tkajinam | tobias-urdin, ah, that's nice | 14:10 |
| tkajinam | I noticed the same and was looking for the way to get the fix released in ubuntu | 14:10 |
| tobias-urdin | sorry haven't been paying much attention, mostly downstream last months | 14:10 |
| tkajinam | np. I didn't notice that at all until I created a series of patches some days ago | 14:11 |
| tobias-urdin | gonna spend more time when X cycle starts, great work on all c8s work btw | 14:15 |
| tobias-urdin | i was able to spend some time on that selinux issue and push https://github.com/puppetlabs/puppet/pull/8537 | 14:15 |
| tobias-urdin | hopefully somebody see's that this year :p | 14:15 |
| tobias-urdin | looks like we need to ask infra to mirror a lot of stuff for focal right, puppet, ubuntu cloud-archive and ceph nautilus/octopus | 14:19 |
| tobias-urdin | or maybe just skip that for now and use public mirrors | 14:19 |
| tkajinam | tobias-urdin, I guess they already have mirrors but I haven't check that. anyway we can start with public ones | 14:20 |
| tkajinam | regarding c8s I submitted a patch to replace c8 jobs by c8s jobs https://review.opendev.org/c/openstack/puppet-openstack-integration/+/780857 | 14:20 |
| tkajinam | because rdo wallaby will be release only for c8s. not for c8 | 14:21 |
| tkajinam | I'll replace existing c8 jobs for older releases before CentOS 8 reaches to its EOL | 14:21 |
| tobias-urdin | ah that's good to know, so c8 victoria -> c8s victoria -> c8s wallaby then | 14:22 |
| tobias-urdin | we should maybe add a release note about that, usually did that in openstacklib and a prelude | 14:23 |
| tkajinam | tobias-urdin, yeah | 14:28 |
| *** openstackgerrit has joined #puppet-openstack | 14:30 | |
| openstackgerrit | Douglas Mendizábal proposed openstack/puppet-barbican stable/victoria: Add new options to PKCS#11 backend https://review.opendev.org/c/openstack/puppet-barbican/+/780895 | 14:30 |
| tkajinam | I was wondering how we should update metadata.json but I didn't see any document describing how CentOS Stream should be defined | 14:31 |
| tkajinam | maybe we can just keep using CentOS | 14:31 |
| *** ysandeep is now known as ysandeep|afk | 14:32 | |
| tobias-urdin | tkajinam: yeah, saw some mile long issue on how facter should represent centos stream as well | 14:32 |
| tobias-urdin | looking at your focal patches, and swap to octopus, looks like if magnum is solved it's only the neutron-tempest-plugin issue then it might pass all jobs | 14:33 |
| tkajinam | tobias-urdin, neutron-tempest-plugin job should have been fixed by https://review.opendev.org/c/openstack/puppet-openstack-integration/+/777149 | 14:34 |
| tkajinam | tobias-urdin, I found an error with gnocchi pool creation in ceph but I think it was a kind of random timeout | 14:34 |
| tkajinam | so once the fixed version of magnum is released in cloud-archive victoria then all tests should pass | 14:35 |
| tobias-urdin | great, nice work :) | 14:35 |
| tobias-urdin | one last thing, im thinking about blocking https://review.opendev.org/c/openstack/puppet-openstack-integration/+/778138 | 14:36 |
| tobias-urdin | pretty much only major versions there, since we need to bump every single metadata.json and already cut some RCs | 14:36 |
| tobias-urdin | we should bump that in next cycle | 14:36 |
| tobias-urdin | what do you think? | 14:36 |
| tkajinam | I think it's ok to block it now | 14:41 |
| tkajinam | but I think merging that wouldn't cause a huge issue unless we intentionally implement a change depending on the bumped versions | 14:43 |
| tkajinam | (of cause it assumes that nothing is broken with the version bumps | 14:43 |
| tkajinam | so... either is fine. tbh I don't have strong opinion about that | 14:43 |
| tkajinam | with my downstream hat on... as far as I see in rdo we have older version for most of these packages | 14:43 |
| tkajinam | one exception is puppet-memcached=6.0.0 but this is only used in puppet-openstack-integration | 14:46 |
| tkajinam | so ... it doesn't cause any problem if we freeze these versions until W release | 14:46 |
| tkajinam | tobias-urdin, ^^^ | 14:47 |
| tobias-urdin | i don't really mind either, other than having it consistent with what we test and what modules say would be nice :) | 14:47 |
| tkajinam | maintaining requirement is always tough work ... | 14:47 |
| tkajinam | unless we set very strict constraint as is done in openstack modules | 14:47 |
| tkajinam | but I don't think it really helps to pin version very specifically | 14:48 |
| tkajinam | like x.y.z | 14:48 |
| tobias-urdin | yeah you're correct | 14:48 |
| tkajinam | it would be better to follow at least the major version than leaving requirements unmaintained, though | 14:50 |
| tkajinam | s/would/might/ | 14:50 |
| tkajinam | I'm going to bed soon because it is becoming midnight here in my tz | 14:54 |
| tkajinam | tobias-urdin, I'll give another thought tomorrow and put any comment if I happen to come up with different opinion | 14:54 |
| tkajinam | about that version bump | 14:54 |
| tkajinam | I think I need to review current dependencies | 14:55 |
| *** ysandeep|afk is now known as ysandeep | 14:55 | |
| openstackgerrit | Merged openstack/puppet-neutron stable/victoria: Ensure resource_provider_bandwitdhs is unset by default https://review.opendev.org/c/openstack/puppet-neutron/+/780078 | 14:57 |
| *** ykarel is now known as ykarel|away | 15:06 | |
| tobias-urdin | tkajinam: ack | 15:13 |
| *** ykarel|away has quit IRC | 15:42 | |
| openstackgerrit | Merged openstack/puppet-neutron stable/victoria: Add support for the resource_provider_hypervisors parameter https://review.opendev.org/c/openstack/puppet-neutron/+/780085 | 15:52 |
| zigo | tkajinam: I have no idea how to implement the <service>_uwsgi_config thingy in openstacklib... | 15:54 |
| zigo | tkajinam: Or maybe you thought about doing like manifests/wsgi/apache.pp in openstacklib? | 15:55 |
| *** ysandeep is now known as ysandeep|dinner | 16:26 | |
| *** ysandeep|dinner is now known as ysandeep | 16:57 | |
| *** rpittau is now known as rpittau|afk | 17:08 | |
| *** amoralej is now known as amoralej|off | 17:19 | |
| *** ysandeep is now known as ysandeep|away | 17:41 | |
| *** gfidente is now known as gfidente|afk | 17:43 | |
| openstackgerrit | Merged openstack/puppet-nova stable/ussuri: Expose [workaround]/ensure_libvirt_rbd_instance_dir_cleanup https://review.opendev.org/c/openstack/puppet-nova/+/779947 | 17:53 |
| *** jpena is now known as jpena|off | 18:06 | |
| *** dtantsur is now known as dtantsur|afk | 18:14 | |
| openstackgerrit | Christian Rohmann proposed openstack/puppet-designate master: Add designate::db::postgresql class. https://review.opendev.org/c/openstack/puppet-designate/+/781148 | 19:16 |
| *** rcernin_ has joined #puppet-openstack | 19:26 | |
| *** rcernin_ has quit IRC | 19:32 | |
| *** rcernin_ has joined #puppet-openstack | 19:57 | |
| *** rcernin_ has quit IRC | 20:02 | |
| *** rcernin_ has joined #puppet-openstack | 20:32 | |
| *** rcernin_ has quit IRC | 20:37 | |
| *** rcernin_ has joined #puppet-openstack | 20:50 | |
| *** rcernin_ has quit IRC | 21:04 | |
| *** rcernin_ has joined #puppet-openstack | 21:09 | |
| *** rcernin_ has quit IRC | 21:15 | |
| *** rcernin_ has joined #puppet-openstack | 21:41 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!