Monday, 2019-01-07

« Sunday, 2019-01-06 Index Tuesday, 2019-01-08 »
*** rascasoft has quit IRC00:02
*** rascasoft has joined #puppet-openstack00:53
*** rascasoft has quit IRC00:57
*** rascasoft has joined #puppet-openstack01:18
*** rascasoft has quit IRC01:22
*** rascasoft has joined #puppet-openstack01:44
*** rascasoft has quit IRC01:50
*** rascasoft has joined #puppet-openstack02:06
*** rascasoft has quit IRC02:13
*** rascasoft has joined #puppet-openstack02:36
*** rascasoft has quit IRC02:41
*** skramaja has joined #puppet-openstack03:38
*** skramaja_ has joined #puppet-openstack03:43
*** skramaja has quit IRC03:43
*** rascasoft has joined #puppet-openstack03:43
*** rascasoft has quit IRC03:50
*** skramaja_ is now known as skramaja03:59
*** rascasoft has joined #puppet-openstack04:10
*** rascasoft has quit IRC04:18
*** rascasoft has joined #puppet-openstack04:54
*** rascasoft has quit IRC05:01
*** rascasoft has joined #puppet-openstack05:20
*** rascasoft has quit IRC05:27
*** ykarel has joined #puppet-openstack05:49
openstackgerritMerged openstack/puppet-qdr master: Use 'dnf' provider for Fedora and RedHat > 7  https://review.openstack.org/62612905:58
*** rascasoft has joined #puppet-openstack06:19
*** rascasoft has quit IRC06:27
openstackgerrityatin proposed openstack/puppet-ceph master: Do not enable Ceph el7 repos for Fedora  https://review.openstack.org/62225206:36
openstackgerrityatin proposed openstack/puppet-ceph master: Do not check for ldap secret with Fedora and RedHat > 7  https://review.openstack.org/62737806:36
*** agurenko has joined #puppet-openstack06:50
*** jtomasek has joined #puppet-openstack06:53
*** rcernin has quit IRC06:56
*** mschuppert has joined #puppet-openstack07:08
*** rascasoft has joined #puppet-openstack07:09
*** dpawlik has joined #puppet-openstack07:29
*** ykarel is now known as ykarel|lunch07:29
*** rpittau has joined #puppet-openstack07:38
*** tosky has joined #puppet-openstack07:45
*** iurygregory has joined #puppet-openstack07:47
tobias-urdino/08:04
tobias-urdinback from pto now08:04
iurygregorywelcome back o/08:07
*** ykarel|lunch is now known as ykarel08:23
lyarwoodwelcome back \o08:33
mschupperto/08:33
tobias-urdinthanks everyone :)08:34
*** skramaja_ has joined #puppet-openstack08:49
*** skramaja has quit IRC08:49
*** ykarel has quit IRC09:06
*** ykarel has joined #puppet-openstack09:06
ykareltobias-urdin, welcome back09:08
tobias-urdinykarel: thanks o/09:09
ykareltobias-urdin, fyi got py3 working with SSL with lyarwood patch in nova:- https://review.openstack.org/#/c/620561/09:09
ykarelthough still some issue, but it's better09:09
ykarellyarwood, i commented on ur DNM patch https://review.openstack.org/#/c/626952/, can u check again09:10
lyarwoodykarel: yeah I was going to rework that today and remove the DNM09:15
ykarellyarwood, cool Thanks, i got it working after removing monkey_patch from wsgi09:15
*** skramaja has joined #puppet-openstack09:15
ykarelplease check that09:15
*** skramaja_ has quit IRC09:16
lyarwoodykarel: utils.monkey_patch() or the eventlet.monkey_patch(os=False) line?09:16
ykarellyarwood, https://review.openstack.org/#/c/620561/15/run_tests.sh@22209:16
ykarelyour patch + ^^09:16
lyarwoodykarel: right but why remove the eventlet.monkey_patch(os=False) call from the wsgi app?09:18
ykarellyarwood, with eventlet.monkey_patch, nova api got stuck into recursion09:19
ykarellyarwood, what i saw is eventlet should not be used with wsgi09:19
ykarelseen https://review.openstack.org/#/c/154642/2/specs/eventlet-best-practices.rst@5409:20
*** gfidente has joined #puppet-openstack09:27
*** shardy has joined #puppet-openstack09:31
*** owalsh_ is now known as owalsh09:35
*** derekh has joined #puppet-openstack09:36
*** chem has joined #puppet-openstack09:38
openstackgerritMerged openstack/puppet-ceph master: Do not enable Ceph el7 repos for Fedora  https://review.openstack.org/62225209:40
tobias-urdinykarel: ack, nice09:51
*** jaosorior has joined #puppet-openstack10:03
*** dtantsur|afk is now known as dtantsur10:08
*** gfidente has quit IRC10:27
*** iurygregory has quit IRC10:28
lyarwoodhttps://review.openstack.org/#/c/628673/ - A pretty basic nit/cleanup in puppet-placement if anyone has time this morning10:47
tobias-urdinlyarwood: ack, password should be required though so imo the current code is correct adding that would make puppet pass (without a password) but placement not working (when using keystone auth)10:58
tobias-urdinor maybe i'm missing some usecase10:58
lyarwoodtobias-urdin: right11:01
lyarwoodI missed the following in puppet-nova11:01
lyarwood  if is_service_default($password) {11:01
lyarwood    fail('Please set password for nova service user')11:01
lyarwood  }11:01
*** gfidente has joined #puppet-openstack11:02
tobias-urdinlooks like somebody likes some extra lines :)11:04
tobias-urdinseems to be left overs when it supported nova::api::admin_password 2-3 years ago11:06
lyarwoodyeah indeed, I'll drop this then and pass a password in my puppet-tripleo changes11:08
lyarwoodtobias-urdin: btw my puppet-openstack-integration changes introducing placement are pretty much blocked until we land stein / py3 support, I had a look over the break but it looks like there's an awkward ordering issue here with enabling py3 for Ubuntu.11:10
lyarwoodhttps://review.openstack.org/#/c/628737/ ah nvm11:11
lyarwood:)11:11
tobias-urdinthere is some annoying parts, unfortunately we placed pyvers in all <module>::params and not in openstacklib::defaults which would only require a single commit to change python version11:12
tobias-urdinand the second part is this https://review.openstack.org/#/c/624944/ which i'm unsure if we should require the enduser to fix or override in the puppet openstack project11:12
tobias-urdini can probably work some on the weekend for ubuntu support, will probably get some ubuntu package issues as well11:13
tobias-urdinthe last part is probably the upgrade process since we don't test the ubuntu bionic py2 -> py3 swap for rocky->stein but that will probably never get tested since we don't gurantee such upgrade progress, atleast not officialy with testing11:15
lyarwoodyeah I don't really have the bandwidth to help there tbh11:15
tobias-urdinlyarwood: yeah, i'm stuck with a lot of other stuff but i can probably pull it off hopefully in good time to milestone-311:23
lyarwoodtobias-urdin: would it be possible to land https://review.openstack.org/#/c/628737/ in that case to allow my puppet-placement / puppet-tripleo work to continue?11:24
tobias-urdinhm we could, do you see any way we could workaround it in p-o-i master branch temporarily to have both working?11:25
lyarwoodwell I could hack the ubuntu params to use the nova-placement package in ubuntu11:26
lyarwooduntil we move to the stein UCA that provides the placement package11:26
* lyarwood gives that a go11:26
tobias-urdinwould that hack puppet-(nova|placement) or could we override that from testing11:26
* tobias-urdin thinking if we could leave modules intact but just hack testing11:27
lyarwoodhmmm yeah that would hack puppet-placement11:27
lyarwoodokay11:27
tobias-urdinbut if we have no other choice we should do that, just brain storming if we could just hack testing11:28
*** tobias-urdin is now known as tobias-urdin_afk11:34
openstackgerritSorin Sbarnea proposed openstack/puppet-openstack-integration master: Workaround for installing correct version of ruby bundler  https://review.openstack.org/62839511:36
*** rpittau is now known as rpittau|lunch11:41
*** dpawlik has quit IRC11:58
*** tobias-urdin_afk is now known as tobias-urdin11:59
openstackgerritLee Yarwood proposed openstack/puppet-openstack-integration master: placement: Extract the service from nova.pp  https://review.openstack.org/61556812:02
openstackgerritLee Yarwood proposed openstack/puppet-openstack-integration master: placement: Extract the service from nova.pp  https://review.openstack.org/61556812:24
*** dpawlik has joined #puppet-openstack12:25
*** ykarel is now known as ykarel|afk12:30
*** zigo has quit IRC12:31
*** iurygregory has joined #puppet-openstack12:44
*** rpittau|lunch is now known as rpittau12:56
openstackgerritLee Yarwood proposed openstack/puppet-openstack-integration master: placement: Extract the service from nova.pp  https://review.openstack.org/61556813:10
*** ykarel|afk is now known as ykarel13:15
*** agurenko has quit IRC13:19
*** trown|outtypewww is now known as trown13:27
openstackgerritVictoria Martinez de la Cruz proposed openstack/puppet-manila stable/rocky: Updating nova and neutron configuration  https://review.openstack.org/62896313:30
*** ykarel is now known as ykarel|away13:32
openstackgerritVictoria Martinez de la Cruz proposed openstack/puppet-manila stable/queens: Updating nova and neutron configuration  https://review.openstack.org/62896413:39
*** skramaja has quit IRC13:40
lyarwoodtobias-urdin: https://review.openstack.org/#/c/615568/29/manifests/placement.pp - stupid question, is there anyway to override ::placement::params here before the ::placement import?13:42
lyarwoodmschuppert: ^ also if you have a second13:46
*** ykarel|away has quit IRC13:47
*** morazi has joined #puppet-openstack13:53
openstackgerritLee Yarwood proposed openstack/puppet-openstack-integration master: placement: Extract the service from nova.pp  https://review.openstack.org/61556813:54
*** ykarel|away has joined #puppet-openstack14:12
*** irclogbot_3 has quit IRC14:14
openstackgerritLee Yarwood proposed openstack/puppet-openstack-integration master: placement: Extract the service from nova.pp  https://review.openstack.org/61556814:26
tobias-urdinlyarwood: some of the vars in params.pp is hardcoded and not parameterized in the classes so we'll probably have to hack puppet-placement14:35
tobias-urdinfor example we could do class { '::placement::wsgi::apache': package_name => 'package-name-here' } based on OS but we cannot override for example this:14:36
tobias-urdinhttps://github.com/openstack/puppet-placement/blob/7f10529b38e94de5d516d1e170ceaba13ca3e14a/manifests/init.pp#L2814:36
tobias-urdinwhich is hardcoded14:36
openstackgerritLee Yarwood proposed openstack/puppet-placement master: params: Revert to original Nova packages from Rocky for Debian  https://review.openstack.org/62898514:36
lyarwoodsorry missed that14:36
* lyarwood reads14:36
lyarwoodyeah understood, I've hacked it above for now14:37
*** irclogbot_3 has joined #puppet-openstack14:40
tobias-urdinykarel|away: https://review.openstack.org/#/c/628986/ if it wasn't fixed yet, sorry I missed that14:42
ykarel|awaytobias-urdin, proposing patch14:42
tobias-urdini pushed one ^14:43
ykarel|awayokk u already proposed, i prepared locally14:43
ykarel|awayThanks14:43
*** ykarel|away is now known as ykarel14:45
ykareltobias-urdin, EmilienM mwhahaha review please https://review.openstack.org/#/c/627378/14:46
ykarelit's aginst ceph bug for Fedora14:46
ykarelgfidente, what we discussed ^^14:47
gfidenteykarel yeah sorry but matt didn't come up with changes yet14:48
ykarelgfidente, i saw patch already proposed in ceph master14:48
mwhahahalooks like there's a github pr14:48
tobias-urdinykarel: ack, mwhahaha was super fast so it's approved now14:48
ykarelmwhahaha, yes it's in master14:48
ykarelbut to get in Fedora will take time14:49
ykarelwith luminous14:49
mwhahahawhere are we running puppet-ceph, just in the weirdo jobs?14:49
ykarelmwhahaha, in upstream poi also14:49
gfidenteah sorry guys didn't see the updates in tracker14:50
ykareli mean scenario 001,00414:50
gfidenteykarel ? those scenarios should be using ceph-ansible from queens14:52
ykarelgfidente, these scenarios are in puppet-openstack-integration, different from tripleo ones14:53
gfidenteykarel ah they just happen to have same number, sorry :D14:53
ykarelyes :)14:53
*** irclogbot_3 has quit IRC15:06
*** irclogbot_3 has joined #puppet-openstack15:14
*** dpawlik has quit IRC15:18
*** openstackgerrit has quit IRC15:22
*** openstackgerrit has joined #puppet-openstack15:27
openstackgerritMerged openstack/puppet-ceph master: Supply hrefs for table of contents according to context  https://review.openstack.org/62286015:27
tobias-urdinmwhahaha: https://review.openstack.org/#/c/619257/ if you have time to do a thorough review15:37
tobias-urdinfixed in tripleo with https://review.openstack.org/#/c/627793/ and packstack https://review.openstack.org/#/c/628996/ if it passes15:37
mwhahahak15:37
mwhahahadon't we want https://review.openstack.org/#/c/628996/ first?15:38
tobias-urdinah yeah, must have though i added api_port15:39
tobias-urdinlet me fix it15:39
mwhahahai've got some comments too15:39
mwhahahayou've got some unrelated user/group var changes15:40
mwhahahashould probably not include those in the port change15:40
tobias-urdinsure, renamed the wsgi_script_path stuff that was keystone_ prefixed so did those as well15:41
tobias-urdinbut can move that out to a separate commit if you prefer15:41
tobias-urdinmwhahaha: do you want to revert wsgi_script_path -> keystone_wsgi_script_path also?15:47
mwhahahayea15:47
mwhahahaif not related to the port change15:47
openstackgerritTobias Urdin proposed openstack/puppet-keystone master: Remove port 35357 deployment  https://review.openstack.org/61925715:49
tobias-urdinarghhh done, though there was both keystone::params::keystone_group and keystone::params::group so i changed both.. will revert that as well15:52
openstackgerritTobias Urdin proposed openstack/puppet-keystone master: Remove port 35357 deployment  https://review.openstack.org/61925715:54
tobias-urdinmwhahaha: ^ done15:55
mwhahahatobias-urdin: you left in $group15:56
tobias-urdinmwhahaha: that was there before and used in policy.pp15:57
mwhahahaah ok15:57
tobias-urdinso didn't want to change it, missed that in PS15 and PS16 corrected that15:57
mwhahahatobias-urdin: you dropped the $keystone/wsgi_public_Script_path tho15:57
mwhahahais that gone?15:57
* mwhahaha looks15:57
tobias-urdinthat's hardcoded to "$wsgi_script_source                = '/usr/bin/keystone-wsgi-public',"15:58
tobias-urdinin keystone::wsgi::apache15:58
mwhahahahmm k15:58
*** bnemec is now known as stackymcstackfac16:04
*** stackymcstackfac is now known as bnemec16:05
*** rpittau has quit IRC17:00
*** trown is now known as trown|lunch17:26
*** ykarel has quit IRC17:39
*** dtantsur is now known as dtantsur|afk17:45
*** ykarel has joined #puppet-openstack18:00
*** derekh has quit IRC18:00
*** ykarel has quit IRC18:06
*** trown|lunch is now known as trown18:20
*** tosky has quit IRC19:05
*** morazi has quit IRC19:07
*** shardy has quit IRC19:24
*** shardy has joined #puppet-openstack19:25
*** gfidente has quit IRC19:32
openstackgerritMerged openstack/puppet-manila stable/rocky: Updating nova and neutron configuration  https://review.openstack.org/62896320:31
openstackgerritMerged openstack/puppet-manila stable/queens: Updating nova and neutron configuration  https://review.openstack.org/62896420:31
openstackgerritTobias Urdin proposed openstack/puppet-keystone master: Remove port 35357 deployment  https://review.openstack.org/61925720:38
tobias-urdinmwhahaha: EmilienM just when it was about get merged I remembered a thing that I missed, new PS fixes that20:41
tobias-urdinhttps://review.openstack.org/61925720:41
mwhahahak20:41
EmilienMok20:41
*** bnemec has quit IRC21:58
*** trown is now known as trown|outtypewww22:01
*** bnemec has joined #puppet-openstack22:02
*** ianw_pto is now known as ianw22:26
*** rcernin has joined #puppet-openstack22:45
*** tosky has joined #puppet-openstack22:52
*** rascasoft has quit IRC23:10
*** tosky has quit IRC23:48
-openstackstatus- NOTICE: The Etherpad service at https://etherpad.openstack.org/ has been offline since 23:22 UTC due to a hypervisor issue in our service provider, but should hopefully return to service shortly.23:50
« Sunday, 2019-01-06 Index Tuesday, 2019-01-08 »

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!