Thursday, 2017-09-07

*** dixiaoli has joined #puppet-openstack00:57
*** agrebennikov has quit IRC01:05
*** ilbot3 has quit IRC01:46
*** ilbot3 has joined #puppet-openstack01:54
*** markvoelker has quit IRC02:11
*** xarses_ has joined #puppet-openstack02:24
*** andreww has joined #puppet-openstack02:26
*** andreww has quit IRC02:27
*** xarses_ has quit IRC02:28
*** andreww has joined #puppet-openstack02:29
*** andreww has quit IRC02:38
dmsimardFedora definitely not02:50
dmsimardIt's not supported by RDO since... liberty I think ?02:50
dmsimardDebian I would drop if no one steps up to maintain the things02:51
dmsimardIt's already challenging to keep ubuntu and centos working without maintaining a third distro without volounteers (imo)02:51
*** guimaluf has quit IRC03:10
*** iurygregory has quit IRC03:10
*** iurygregory has joined #puppet-openstack03:10
*** guimaluf has joined #puppet-openstack03:11
*** markvoelker has joined #puppet-openstack03:11
*** ykarel has joined #puppet-openstack03:42
*** markvoelker has quit IRC03:46
*** udesale has joined #puppet-openstack03:50
*** ntpttr_ has joined #puppet-openstack04:19
*** jistr|off has quit IRC04:20
*** oyrogerg has quit IRC04:21
*** oyrogerg has joined #puppet-openstack04:21
*** ntpttr has quit IRC04:22
*** jistr has joined #puppet-openstack04:24
*** iurygregory has quit IRC04:30
openstackgerritMerged openstack/puppet-openstack_spec_helper master: Drop Fedora 24 from supported operating systems  https://review.openstack.org/50148104:34
*** iurygregory has joined #puppet-openstack04:36
*** asilenkov has quit IRC04:42
*** markvoelker has joined #puppet-openstack04:43
*** asilenkov has joined #puppet-openstack04:43
*** yprokule has joined #puppet-openstack04:46
*** jtomasek has joined #puppet-openstack04:51
*** skramaja has joined #puppet-openstack05:03
*** andreww has joined #puppet-openstack05:05
*** xarses_ has joined #puppet-openstack05:08
*** andreww has quit IRC05:09
*** markvoelker has quit IRC05:16
*** ratailor has joined #puppet-openstack05:18
*** xarses_ has quit IRC05:29
*** jaosorior has joined #puppet-openstack05:39
*** udesale__ has joined #puppet-openstack05:40
*** udesale has quit IRC05:42
*** udesale has joined #puppet-openstack05:42
*** udesale__ has quit IRC05:45
*** agurenko has joined #puppet-openstack06:04
openstackgerritMerged openstack/puppet-ironic stable/pike: Fixes for Ironic-inspector introspection when client using UEFI  https://review.openstack.org/50144506:04
*** markvoelker has joined #puppet-openstack06:13
*** ccamacho has joined #puppet-openstack06:24
*** xarses_ has joined #puppet-openstack06:30
*** andreww has joined #puppet-openstack06:32
*** jaosorior has quit IRC06:33
*** xarses_ has quit IRC06:34
*** andreww has quit IRC06:38
*** markvoelker has quit IRC06:47
*** simon-AS559 has joined #puppet-openstack06:50
*** indistylo has joined #puppet-openstack06:51
*** rcernin has joined #puppet-openstack06:53
*** yprokule_ has joined #puppet-openstack07:00
*** jaganathan has joined #puppet-openstack07:01
*** yprokule has quit IRC07:03
*** yprokule_ is now known as yprokule07:03
*** agurenko has quit IRC07:04
*** agurenko has joined #puppet-openstack07:06
*** tesseract has joined #puppet-openstack07:13
*** shardy has joined #puppet-openstack07:22
*** jpena|off is now known as jpena07:38
*** markvoelker has joined #puppet-openstack07:44
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/50161307:49
*** simon-AS559 has quit IRC08:00
*** indistylo has quit IRC08:05
*** shardy is now known as shardy_afk08:06
*** markvoelker has quit IRC08:17
*** indistylo has joined #puppet-openstack08:17
*** jaosorior has joined #puppet-openstack08:17
*** vnogin has joined #puppet-openstack08:18
*** mwarad has joined #puppet-openstack08:23
*** rmart04 has joined #puppet-openstack08:31
*** yprokule_ has joined #puppet-openstack08:32
*** indistylo has quit IRC08:32
*** yprokule has quit IRC08:34
*** yprokule_ is now known as yprokule08:34
*** vnogin has quit IRC08:44
*** indistylo has joined #puppet-openstack08:45
*** derekh has joined #puppet-openstack08:45
*** jaosorior has quit IRC08:50
*** simon-AS559 has joined #puppet-openstack09:10
*** vnogin has joined #puppet-openstack09:12
*** markvoelker has joined #puppet-openstack09:14
*** mwarad has quit IRC09:17
*** jaosorior has joined #puppet-openstack09:47
*** markvoelker has quit IRC09:48
*** dtantsur|afk is now known as dtantsur10:05
*** shardy_afk is now known as shardy10:05
*** vnogin has quit IRC10:17
openstackgerritMerged openstack/puppet-nova master: Ensure novajoin's log directory ownership is correct  https://review.openstack.org/50069010:22
*** vnogin has joined #puppet-openstack10:25
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-nova stable/pike: Ensure novajoin's log directory ownership is correct  https://review.openstack.org/50166510:31
openstackgerritJavier Peña proposed openstack/puppet-nova master: Sanitize nova_manage output in provider  https://review.openstack.org/50166910:42
*** markvoelker has joined #puppet-openstack10:45
*** jpena is now known as jpena|lunch10:57
*** udesale has quit IRC11:04
*** ccamacho has quit IRC11:06
*** markvoelker has quit IRC11:17
*** vnogin has quit IRC11:21
*** jaganathan has quit IRC11:32
*** dixiaoli has quit IRC11:45
*** enemarke has quit IRC11:45
*** vnogin has joined #puppet-openstack11:57
*** ansmith has quit IRC12:04
*** jpena|lunch is now known as jpena12:05
*** jaganathan has joined #puppet-openstack12:07
*** vnogin has quit IRC12:12
*** markvoelker has joined #puppet-openstack12:15
*** jaosorior has quit IRC12:15
*** vnogin has joined #puppet-openstack12:18
*** markvoelker has quit IRC12:24
*** markvoelker has joined #puppet-openstack12:24
*** vnogin has quit IRC12:28
*** trown|outtypewww is now known as trown12:34
*** ratailor has quit IRC12:34
*** ccamacho has joined #puppet-openstack12:39
mnasermwhahaha thanks :)12:39
mwhahahanp12:40
mnaserjpena is it possible to get stdout only when we run commands so we don't have that warning issue12:42
mnaserassuming that these warnings go to stderr12:42
*** jaosorior has joined #puppet-openstack12:44
mnasermwhahaha are there projects with some sort of special treatment?  i cant give a +2 for this puppet-cinder change https://review.openstack.org/#/c/500578/12:48
mwhahahano12:48
mwhahahayou may need to logout/login12:48
*** jtomasek has quit IRC12:48
*** jtomasek has joined #puppet-openstack12:49
*** vnogin has joined #puppet-openstack12:49
mnasernope, still nothing, that's weird (because i can see that i can do it for other changes for example)12:50
mnaserill ask infra12:50
mwhahahayou may try removing your +1 first12:50
mwhahahaand refreshing12:50
mnasermwhahaha that was it.. the "turn it off and on" of gerrit :>12:50
mwhahaha:D12:51
mnaserhttps://review.openstack.org/#/c/501711/ passed ci.. is it okay to merge or you have some other prereq to keep in mind (given it is related to a version bump?)12:51
jpenamnaser: I'm not sure, let me have a look at it. My gut feeling is no, because we have similar output sanitizing functions in other puppet modules12:53
mnaserjpena: im not opposed to it if it's what we've always done, just feel like your life will be easier (cause it doesn't seem like it's the first cli to do this) and it'll be cleaner overall12:53
*** dprince has joined #puppet-openstack12:54
mwhahahamnaser: yea we can merge that, we need to cut a release for it12:58
mwhahahamnaser: that's all we need to do as a prereq for releasing the modules12:59
mnasermwhahaha done.  would it be okay if i tried my hand at the release commit.. just to have a first hand experience13:00
mwhahahamnaser: eventually we can drop the reno config bump as there's a way to not require that13:00
mwhahahamnaser: yea there's a command in the releases repo to do it13:00
mnasermwhahaha thats what i thought, i guess it was EmilienM with EmilienM big bulk of patches13:00
mwhahahayea13:00
jpenamnaser: I've found https://ask.puppet.com/question/6299/combine-false-for-provider-commands/ which hints that stdout/stderr are always merged if we use the provider command13:01
mwhahahamnaser: i think the command is: tox -e venv -- new-release --stable-branch independent puppet-ceph bug13:01
jpenaI've found a module that tries to work around that: https://github.com/duritong/puppet-sysctl/blob/master/lib/puppet/provider/sysctl_runtime/sysctl_runtime.rb#L11-L2113:01
mwhahahabbl13:02
mnaserjpena ouch13:02
mnaserhttps://github.com/puppetlabs/puppet/blob/master/lib/puppet/provider.rb#L259 looks like they still merge it13:02
mnaserjpena: i guess the nice thing is we don't support puppet 3 anymore so we dont need that little ternary operation13:03
mnaserPuppet::Util::Execution seems like a public API as well13:03
mnaserhttps://github.com/puppetlabs/puppet/blob/master/lib/puppet/util/execution.rb#L14-L1613:03
*** ansmith has joined #puppet-openstack13:04
mnaserthe only concern i would have is.. do we ever rely on the stderr output in case of failure (because i assume if it fails, it will give the failure message to stderr)13:04
mnaseri think not, because command provider would fail if it doesn't exit with anything other than 013:04
mnaserjpena: id propose adding a nova_manage function into https://github.com/openstack/puppet-nova/blob/master/lib/puppet/provider/nova.rb which uses the puppet executor as shown in sysctl module, then just refactor the existing one to use it and i think we should be good to go13:06
*** udesale has joined #puppet-openstack13:07
mnaser(assuming these warnings are going to stderr lol)13:07
jpenait seems to go to stderr, fortunately13:12
mnaser\o/13:13
*** skramaja has quit IRC13:13
*** shardy has quit IRC13:15
*** shardy has joined #puppet-openstack13:19
openstackgerritMohammed Naser proposed openstack/puppet-openstack_spec_helper stable/pike: Drop Fedora 24 from supported operating systems  https://review.openstack.org/50173513:41
*** udesale has quit IRC13:41
openstackgerritMohammed Naser proposed openstack/puppet-openstack_spec_helper stable/ocata: Drop Fedora 24 from supported operating systems  https://review.openstack.org/50173613:41
openstackgerritMohammed Naser proposed openstack/puppet-openstack_spec_helper stable/newton: Drop Fedora 24 from supported operating systems  https://review.openstack.org/50174013:44
*** thegreenhundred has joined #puppet-openstack13:50
*** ansmith has quit IRC13:57
*** trown is now known as trown|brb14:04
*** evidex has joined #puppet-openstack14:06
*** evidex has quit IRC14:07
*** trown|brb is now known as trown14:09
*** ansmith has joined #puppet-openstack14:10
*** agrebennikov has joined #puppet-openstack14:10
*** ccamacho has quit IRC14:11
*** ykarel has quit IRC14:11
*** agurenko has quit IRC14:14
openstackgerritMohammed Naser proposed openstack/puppet-openstack-integration master: Add puppet-archive to external modules  https://review.openstack.org/50175214:18
*** mwarad has joined #puppet-openstack14:29
openstackgerritBob Fournier proposed openstack/puppet-ironic stable/ocata: Fixes for Ironic-inspector introspection when client using UEFI  https://review.openstack.org/50175614:31
*** vnogin has quit IRC14:41
*** vnogin has joined #puppet-openstack14:45
openstackgerritMoshe Levi proposed openstack/puppet-neutron master: ovs agent: add support for ovs hw-offload option  https://review.openstack.org/50176014:48
*** xarses_ has joined #puppet-openstack14:56
*** simon-AS559 has quit IRC14:59
*** simon-AS559 has joined #puppet-openstack15:01
*** ansmith has quit IRC15:04
*** mwarad has quit IRC15:09
openstackgerritBob Fournier proposed openstack/puppet-ironic stable/ocata: Fixes for Ironic-inspector introspection when client using UEFI  https://review.openstack.org/50175615:12
openstackgerritMerged openstack/puppet-cinder master: add parameter to overwrite/add wsgi process options  https://review.openstack.org/50057815:15
*** rmart04 has quit IRC15:15
*** yprokule has quit IRC15:19
*** dprince has quit IRC15:28
*** simon-AS5591 has joined #puppet-openstack15:28
*** simon-AS559 has quit IRC15:28
mnasermwhahaha EmilienM https://review.openstack.org/#/c/501774/1 if you have a few minutes15:29
*** vnogin has quit IRC15:32
*** vnogin has joined #puppet-openstack15:33
*** simon-AS5591 has quit IRC15:46
*** owalsh is now known as owalsh_afk15:56
*** shardy is now known as shardy_afk15:57
*** rmart04 has joined #puppet-openstack16:02
*** mwynne has joined #puppet-openstack16:06
*** ansmith has joined #puppet-openstack16:08
*** simon-AS559 has joined #puppet-openstack16:10
*** vnogin has quit IRC16:15
*** jaosorior has quit IRC16:30
*** rmart04 has quit IRC16:33
*** rmart04 has joined #puppet-openstack16:34
openstackgerritJavier Peña proposed openstack/puppet-nova master: Sanitize nova_manage output in provider  https://review.openstack.org/50166916:43
*** rcernin has quit IRC16:45
*** rmart04_ has joined #puppet-openstack16:46
*** rmart04 has quit IRC16:46
*** rmart04_ is now known as rmart0416:46
*** dtantsur is now known as dtantsur|afk16:52
*** dprince has joined #puppet-openstack17:04
*** dprince has quit IRC17:16
*** trown is now known as trown|lunch17:16
*** jpena is now known as jpena|off17:17
*** tesseract has quit IRC17:23
*** jaganathan has quit IRC17:56
*** rmart04 has quit IRC18:01
EmilienMmnaser: replied18:01
*** derekh has quit IRC18:05
*** trown|lunch is now known as trown18:13
*** vnogin has joined #puppet-openstack18:19
*** vnogin has quit IRC18:28
*** dprince has joined #puppet-openstack18:53
mnaserEmilienM mwhahaha thank you, hope i got it right this time hah19:16
*** derekh has joined #puppet-openstack19:22
*** vnogin has joined #puppet-openstack19:36
*** derekh has quit IRC19:45
*** derekh has joined #puppet-openstack19:48
*** vnogin has quit IRC20:07
*** owalsh_afk is now known as owalsh20:12
*** gfidente has quit IRC20:29
*** indistylo has quit IRC20:35
*** ansmith has quit IRC20:38
openstackgerritMerged openstack/puppet-magnum stable/pike: Fix magnum cluster TLS cert generation  https://review.openstack.org/50114120:38
openstackgerritMerged openstack/puppet-magnum stable/ocata: Fix magnum cluster TLS cert generation  https://review.openstack.org/50114220:38
openstackgerritMerged openstack/puppet-magnum stable/newton: Fix magnum cluster TLS cert generation  https://review.openstack.org/50114320:38
*** trown is now known as trown|outtypewww20:41
*** mwarad has joined #puppet-openstack21:05
openstackgerritMerged openstack/puppet-nova stable/pike: Ensure novajoin's log directory ownership is correct  https://review.openstack.org/50166521:06
*** alee has quit IRC21:06
*** simon-AS5591 has joined #puppet-openstack21:07
*** vnogin has joined #puppet-openstack21:07
*** simon-AS559 has quit IRC21:08
*** mwarad has quit IRC21:09
*** dprince has quit IRC21:21
*** simon-AS5591 has quit IRC21:24
*** ansmith has joined #puppet-openstack21:30
*** derekh has quit IRC21:57
*** thegreenhundred has quit IRC22:11
*** derekh has joined #puppet-openstack22:17
*** vnogin has quit IRC22:25
openstackgerritMerged openstack/puppet-ironic stable/ocata: Fixes for Ironic-inspector introspection when client using UEFI  https://review.openstack.org/50175622:37
*** xarses_ has quit IRC22:48
*** alee has joined #puppet-openstack22:51
*** vnogin has joined #puppet-openstack23:32
*** xarses_ has joined #puppet-openstack23:40
*** vnogin has quit IRC23:49

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