Wednesday, 2020-10-14

« Tuesday, 2020-10-13 Index Thursday, 2020-10-15 »
openstackgerritMerged openstack/puppet-openstack-integration stable/ussuri: [Ussuri Only] Cleanup workaround for CentOS7  https://review.opendev.org/74815201:29
*** rcernin has quit IRC02:48
*** rcernin has joined #puppet-openstack03:05
*** rcernin has quit IRC03:06
*** rcernin has joined #puppet-openstack03:06
*** ykarel|away has joined #puppet-openstack04:44
*** ykarel|away is now known as ykarel04:58
*** ykarel_ has joined #puppet-openstack05:04
*** ykarel has quit IRC05:08
*** ykarel_ is now known as ykarel05:18
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints  https://review.opendev.org/75800506:10
*** ade_lee__ has quit IRC06:39
*** morazi has quit IRC06:40
*** morazi has joined #puppet-openstack06:54
*** ade_lee has joined #puppet-openstack06:59
*** jtomasek has joined #puppet-openstack07:00
*** ysandeep|away is now known as ysandeep07:09
obretkajinam: Ref the fix i posted a couple of days ago: Does it add value for puppet-openstack that we clean up the old -l/--listen flags? If not I can simply ensure the cleaning in our own puppet-profiles.07:28
openstackgerritMerged openstack/puppet-heat stable/victoria: Update .gitreview for stable/victoria  https://review.opendev.org/75676107:28
obretkajinam: And now that https://review.opendev.org/#/c/749603/ is merged; can we merge it into ussuri as well?07:29
tkajinamobre, hi07:30
tkajinamobre, yeah that one will be backported to V and U07:30
obre(also, sorry for late replies, weekend and upgrade-days happened)07:30
tkajinamobre, and we also have this merged into master https://review.opendev.org/#/c/756773/ I think this is also a candidate for backports07:30
tkajinamthis implements that cleaning step07:31
obreNice!07:31
obreThat would be nice to have in Ussuri as well!07:31
tkajinamyeah07:31
obreWe are currently labbing the ussuri upgrade in our dev-environment; what is the timeline for having these backported? Days?07:32
tkajinamobre, maybe in a few days https://review.opendev.org/#/c/752788/07:32
tkajinamif you use any distribution like rdo or ubuntu cloud archive it might take some more, but I'm not familiar with distro process07:33
obreWe fetch our puppet-openstack modules from github; so I guess that should be ~instant?07:34
tkajinamyeah07:34
openstackgerritTakashi Kajinami proposed openstack/puppet-nova stable/victoria: Cleanup listen option from libvirtd service file  https://review.opendev.org/75801807:35
openstackgerritTakashi Kajinami proposed openstack/puppet-nova stable/ussuri: Ubuntu has libvirt 6.0  https://review.opendev.org/75278807:36
openstackgerritTakashi Kajinami proposed openstack/puppet-nova stable/ussuri: Cleanup listen option from libvirtd service file  https://review.opendev.org/75801907:36
tkajinamobre, submitted backport of the cleanup patch ^^^^07:37
openstackgerritTakashi Kajinami proposed openstack/puppet-nova stable/train: Fix ignored unit test cases for nova::migration::libvirt  https://review.opendev.org/75802007:37
obretkajinam: Thanks!07:37
*** tosky has joined #puppet-openstack07:43
*** gfidente has joined #puppet-openstack07:47
openstackgerritTakashi Kajinami proposed openstack/puppet-nova stable/train: Cleanup listen option from libvirtd service file  https://review.opendev.org/75802407:48
openstackgerritTakashi Kajinami proposed openstack/puppet-nova stable/train: Cleanup listen option from libvirtd service file  https://review.opendev.org/75802407:49
*** rcernin has quit IRC07:50
openstackgerritTakashi Kajinami proposed openstack/puppet-nova stable/train: Cleanup listen option from libvirtd service file  https://review.opendev.org/75802407:50
openstackgerritTakashi Kajinami proposed openstack/puppet-nova stable/train: Cleanup listen option from libvirtd service file  https://review.opendev.org/75802407:50
*** rpittau|afk is now known as rpittau07:52
*** jpena|off is now known as jpena07:57
openstackgerritMerged openstack/puppet-heat stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria  https://review.opendev.org/75676208:09
openstackgerritMerged openstack/puppet-horizon stable/victoria: Update .gitreview for stable/victoria  https://review.opendev.org/75676408:12
*** lbragstad_ has joined #puppet-openstack08:18
*** lbragstad has quit IRC08:20
openstackgerritMerged openstack/puppet-glance stable/victoria: Update .gitreview for stable/victoria  https://review.opendev.org/75675108:24
openstackgerritTobias Urdin proposed openstack/puppet-horizon master: Add ssl_verify_client  https://review.opendev.org/75803208:26
*** rcernin has joined #puppet-openstack08:27
tobias-urdintkajinam: EmilienM mwhahaha ^ somewhat "critical" to fix broken functionality around SSL in horizon08:28
tobias-urdinor required to fix it atleast, we could have it default to a value but I want it backported so using default undef08:29
*** derekh has joined #puppet-openstack08:43
tkajinamtobias-urdin, I think that default is the same as the one defined in apache::vhost, right ?08:44
tobias-urdinyes, default is undef, but with undef SSLCACertificate wont get set so it's up to the end user to set that in that case08:44
openstackgerritTakashi Kajinami proposed openstack/puppet-horizon master: Add ssl_verify_client  https://review.opendev.org/75803208:46
tkajinamtobias-urdin, I updated parameter description by my side. hope the change looks good to you... ^^^08:46
tobias-urdinnice, i'm working on a follow-up for master08:47
tobias-urdindo you think we should fail() if $horizon_ca is set and $ssl_verify_client is set i.e horizon_ca will not be used08:47
tobias-urdins/$ssl_verify_client is not set/g08:48
tkajinamtobias-urdin, honestly speaking I don't prefer such strict validation08:50
tkajinamimplemented in puppet-horizon layer.08:50
tobias-urdinack, i'll add a issues entry in the release notes instead08:51
tkajinamyeah08:51
tkajinamtobias-urdin, ahh, sorry. I now got your point about validation08:56
tkajinamtobias-urdin, yeah. maybe we can implement that validation if puppet-apache doesn't emit something useful, but that should be implemented in only master08:56
* tkajinam should have read the release note more carefully :-P08:57
*** higgins has quit IRC08:58
*** rcernin has quit IRC08:59
*** higgins has joined #puppet-openstack09:01
openstackgerritMerged openstack/puppet-openstacklib stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria  https://review.opendev.org/75680709:02
openstackgerritTobias Urdin proposed openstack/puppet-horizon master: Rename horizon_* ssl params to ssl_*  https://review.opendev.org/75804109:03
tobias-urdintkajinam: ^09:04
tkajinamtobias-urdin, looks good09:06
tkajinamtobias-urdin, do we need the same fix for puppet-openstacklib ?09:07
tkajinamI don't know how many users use ssl termination by apache, instead of putting some other layer like haproxy to do make it ssl'ed09:08
tobias-urdintheoretically we want this for everything, but as you say, we terminate infront as well09:08
tobias-urdinjust not for horizon09:08
tkajinamok09:14
tkajinamtobias-urdin, the follow-up looks good for me, but I'll have 2nd look later. should leave for a while09:15
tobias-urdinack, thanks for the quick feedback09:15
openstackgerritTobias Urdin proposed openstack/puppet-openstacklib master: Add ssl_verify_client to wsgi::apache  https://review.opendev.org/75804509:20
*** jtomasek has quit IRC09:37
*** gfidente has quit IRC09:39
openstackgerritBenedikt Trefzer proposed openstack/puppet-cloudkitty master: add parameter to specify wsgi script source and path  https://review.opendev.org/75508409:45
*** zigo has joined #puppet-openstack10:14
*** gfidente has joined #puppet-openstack10:22
*** rcernin has joined #puppet-openstack10:25
openstackgerritShi Yan proposed openstack/puppet-nova master: Add support for more compute parameters  https://review.opendev.org/75808110:42
openstackgerritShi Yan proposed openstack/puppet-nova master: Add support for more migration parameters  https://review.opendev.org/75808210:42
*** jtomasek has joined #puppet-openstack10:58
*** ysandeep is now known as ysandeep|brb11:23
openstackgerritShi Yan proposed openstack/puppet-nova master: Deprecate database options  https://review.opendev.org/75809011:27
*** ysandeep|brb is now known as ysandeep|afk11:32
*** jpena is now known as jpena|lunch11:36
tobias-urdinmwhahaha: please look at https://review.opendev.org/#/c/758032/ :)11:39
openstackgerritTobias Urdin proposed openstack/puppet-horizon master: Rename horizon_* ssl params to ssl_*  https://review.opendev.org/75804111:44
*** yann-kaelig has joined #puppet-openstack11:45
EmilienMtobias-urdin: done11:54
tobias-urdinthanks :)11:55
*** morazi has quit IRC12:01
*** morazi has joined #puppet-openstack12:02
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Set ssl_verify_client to optional for horizon  https://review.opendev.org/75810012:18
*** jpena|lunch is now known as jpena12:35
*** ysandeep|afk is now known as ysandeep12:38
*** lbragstad_ is now known as lbragstad13:02
openstackgerritAlessandro Italiano proposed openstack/puppet-keystone master: Update auth_openidc_module path  https://review.opendev.org/75811513:10
openstackgerritMerged openstack/puppet-horizon master: Add ssl_verify_client  https://review.opendev.org/75803213:26
openstackgerritTobias Urdin proposed openstack/puppet-horizon stable/victoria: Add ssl_verify_client  https://review.opendev.org/75812513:38
openstackgerritTobias Urdin proposed openstack/puppet-horizon stable/ussuri: Add ssl_verify_client  https://review.opendev.org/75812613:38
openstackgerritTobias Urdin proposed openstack/puppet-horizon stable/train: Add ssl_verify_client  https://review.opendev.org/75812713:38
openstackgerritTobias Urdin proposed openstack/puppet-horizon master: Rename horizon_* ssl params to ssl_*  https://review.opendev.org/75804113:47
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Replace horizon_* with ssl_* params  https://review.opendev.org/75812913:50
*** rcernin has quit IRC14:01
openstackgerritAlessandro Italiano proposed openstack/puppet-keystone master: Check memcached_servers  https://review.opendev.org/75813314:11
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Set ssl_verify_client to optional for horizon  https://review.opendev.org/75810015:20
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Replace horizon_* with ssl_* params  https://review.opendev.org/75812915:21
*** ysandeep is now known as ysandeep|away15:25
*** ykarel is now known as ykarel|away15:34
*** ykarel|away has quit IRC15:42
openstackgerritMerged openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints  https://review.opendev.org/75800515:44
*** derekh has quit IRC15:54
openstackgerritTobias Urdin proposed openstack/puppet-horizon master: Rename horizon_* ssl params to ssl_*  https://review.opendev.org/75804115:56
*** rpittau is now known as rpittau|afk16:05
*** tosky has quit IRC16:21
*** iurygregory has quit IRC16:41
*** iurygregory has joined #puppet-openstack16:44
*** gfidente is now known as gfidente|afk16:46
openstackgerritTakashi Kajinami proposed openstack/puppet-nova master: WIP: Disable libvirtd.service before starting sockets  https://review.opendev.org/75821517:08
*** jpena is now known as jpena|off17:22
openstackgerritMerged openstack/puppet-neutron stable/victoria: Add new classes ovs_driver and sriov_driver  https://review.opendev.org/75780317:43
*** ykarel|away has joined #puppet-openstack17:48
*** ykarel|away has quit IRC17:53
*** rcernin has joined #puppet-openstack18:22
*** rcernin has quit IRC18:37
*** yann-kaelig has quit IRC19:03
*** tosky has joined #puppet-openstack19:53
*** jtomasek has quit IRC20:15
*** jtomasek has joined #puppet-openstack21:24
openstackgerritMerged openstack/puppet-octavia master: Use tripleo-puppet-ci-centos-8-standalone as tripleo job base  https://review.opendev.org/75757821:27
*** jtomasek has quit IRC21:46
openstackgerritMerged openstack/puppet-openstack-integration master: Skip unnecessary tests for release cut off  https://review.opendev.org/75761022:21
*** gfidente|afk has quit IRC22:25
openstackgerritMerged openstack/puppet-horizon stable/victoria: Add ssl_verify_client  https://review.opendev.org/75812522:26
openstackgerritMerged openstack/puppet-horizon stable/ussuri: Add ssl_verify_client  https://review.opendev.org/75812622:26
openstackgerritMerged openstack/puppet-horizon stable/train: Add ssl_verify_client  https://review.opendev.org/75812722:26
openstackgerritMerged openstack/puppet-openstack-integration master: Remove all TripleO jobs in CentOS7  https://review.opendev.org/75758022:29
*** rcernin has joined #puppet-openstack22:34
openstackgerritMerged openstack/puppet-openstacklib master: Add ssl_verify_client to wsgi::apache  https://review.opendev.org/75804522:34
*** rcernin has quit IRC22:36
*** rcernin has joined #puppet-openstack22:36
openstackgerritTakashi Kajinami proposed openstack/puppet-octavia master: Enable Ceph in tripleo jobs  https://review.opendev.org/75826423:07
*** tosky has quit IRC23:27
*** rcernin has quit IRC23:45
*** rcernin has joined #puppet-openstack23:50
« Tuesday, 2020-10-13 Index Thursday, 2020-10-15 »

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!