Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2017-09-07

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
00:57 dixiaoli joined #puppet-openstack
01:54 ilbot3 joined #puppet-openstack
01:54 Topic for #puppet-openstack is now CI Status: GREENish (occasional timeouts) | Puppet OpenStack modules https://docs.openstack.org/puppet-openstack-guide/latest/
02:24 xarses_ joined #puppet-openstack
02:26 andreww joined #puppet-openstack
02:29 andreww joined #puppet-openstack
02:50 dmsimard Fedora definitely not
02:50 dmsimard It's not supported by RDO since... liberty I think ?
02:51 dmsimard Debian I would drop if no one steps up to maintain the things
02:51 dmsimard It's already challenging to keep ubuntu and centos working without maintaining a third distro without volounteers (imo)
03:10 iurygregory joined #puppet-openstack
03:11 guimaluf joined #puppet-openstack
03:11 markvoelker joined #puppet-openstack
03:42 ykarel joined #puppet-openstack
03:50 udesale joined #puppet-openstack
04:19 ntpttr_ joined #puppet-openstack
04:21 oyrogerg joined #puppet-openstack
04:24 jistr joined #puppet-openstack
04:34 openstackgerrit Merged openstack/puppet-openstack_spec_helper master: Drop Fedora 24 from supported operating systems  https://review.openstack.org/501481
04:36 iurygregory joined #puppet-openstack
04:43 markvoelker joined #puppet-openstack
04:43 asilenkov joined #puppet-openstack
04:46 yprokule joined #puppet-openstack
04:51 jtomasek joined #puppet-openstack
05:03 skramaja joined #puppet-openstack
05:05 andreww joined #puppet-openstack
05:08 xarses_ joined #puppet-openstack
05:18 ratailor joined #puppet-openstack
05:39 jaosorior joined #puppet-openstack
05:40 udesale__ joined #puppet-openstack
05:42 udesale joined #puppet-openstack
06:04 agurenko joined #puppet-openstack
06:04 openstackgerrit Merged openstack/puppet-ironic stable/pike: Fixes for Ironic-inspector introspection when client using UEFI  https://review.openstack.org/501445
06:13 markvoelker joined #puppet-openstack
06:24 ccamacho joined #puppet-openstack
06:30 xarses_ joined #puppet-openstack
06:32 andreww joined #puppet-openstack
06:50 simon-AS559 joined #puppet-openstack
06:51 indistylo joined #puppet-openstack
06:53 rcernin joined #puppet-openstack
07:00 yprokule_ joined #puppet-openstack
07:01 jaganathan joined #puppet-openstack
07:06 agurenko joined #puppet-openstack
07:13 tesseract joined #puppet-openstack
07:22 shardy joined #puppet-openstack
07:44 markvoelker joined #puppet-openstack
07:49 openstackgerrit OpenStack Proposal Bot proposed openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/501613
08:17 indistylo joined #puppet-openstack
08:17 jaosorior joined #puppet-openstack
08:18 vnogin joined #puppet-openstack
08:23 mwarad joined #puppet-openstack
08:31 rmart04 joined #puppet-openstack
08:32 yprokule_ joined #puppet-openstack
08:45 indistylo joined #puppet-openstack
08:45 derekh joined #puppet-openstack
09:10 simon-AS559 joined #puppet-openstack
09:12 vnogin joined #puppet-openstack
09:14 markvoelker joined #puppet-openstack
09:47 jaosorior joined #puppet-openstack
10:22 openstackgerrit Merged openstack/puppet-nova master: Ensure novajoin's log directory ownership is correct  https://review.openstack.org/500690
10:25 vnogin joined #puppet-openstack
10:31 openstackgerrit Juan Antonio Osorio Robles proposed openstack/puppet-nova stable/pike: Ensure novajoin's log directory ownership is correct  https://review.openstack.org/501665
10:42 openstackgerrit Javier Peña proposed openstack/puppet-nova master: Sanitize nova_manage output in provider  https://review.openstack.org/501669
10:45 markvoelker joined #puppet-openstack
11:57 vnogin joined #puppet-openstack
12:07 jaganathan joined #puppet-openstack
12:15 markvoelker joined #puppet-openstack
12:18 vnogin joined #puppet-openstack
12:24 markvoelker joined #puppet-openstack
12:39 ccamacho joined #puppet-openstack
12:39 mnaser mwhahaha thanks :)
12:40 mwhahaha np
12:42 mnaser jpena is it possible to get stdout only when we run commands so we don't have that warning issue
12:42 mnaser assuming that these warnings go to stderr
12:44 jaosorior joined #puppet-openstack
12:48 mnaser mwhahaha 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 mwhahaha no
12:48 mwhahaha you may need to logout/login
12:49 jtomasek joined #puppet-openstack
12:49 vnogin joined #puppet-openstack
12:50 mnaser nope, still nothing, that's weird (because i can see that i can do it for other changes for example)
12:50 mnaser ill ask infra
12:50 mwhahaha you may try removing your +1 first
12:50 mwhahaha and refreshing
12:50 mnaser mwhahaha that was it.. the "turn it off and on" of gerrit :>
12:51 mwhahaha :D
12:51 mnaser https://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:53 jpena mnaser: 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 modules
12:53 mnaser jpena: 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 overall
12:54 dprince joined #puppet-openstack
12:58 mwhahaha mnaser: yea we can merge that, we need to cut a release for it
12:59 mwhahaha mnaser: that's all we need to do as a prereq for releasing the modules
13:00 mnaser mwhahaha done.  would it be okay if i tried my hand at the release commit.. just to have a first hand experience
13:00 mwhahaha mnaser: eventually we can drop the reno config bump as there's a way to not require that
13:00 mwhahaha mnaser: yea there's a command in the releases repo to do it
13:00 mnaser mwhahaha thats what i thought, i guess it was EmilienM with EmilienM big bulk of patches
13:00 mwhahaha yea
13:01 jpena mnaser: 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 command
13:01 mwhahaha mnaser: i think the command is: tox -e venv -- new-release --stable-branch independent puppet-ceph bug
13:01 jpena I'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-L21
13:02 mwhahaha bbl
13:02 mnaser jpena ouch
13:02 mnaser https://github.com/puppetlabs/puppet/blob/master/lib/puppet/provider.rb#L259 looks like they still merge it
13:03 mnaser jpena: i guess the nice thing is we don't support puppet 3 anymore so we dont need that little ternary operation
13:03 mnaser Puppet::Util::Execution seems like a public API as well
13:03 mnaser https://github.com/puppetlabs/puppet/blob/master/lib/puppet/util/execution.rb#L14-L16
13:04 ansmith joined #puppet-openstack
13:04 mnaser the 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 mnaser i think not, because command provider would fail if it doesn't exit with anything other than 0
13:06 mnaser jpena: 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 go
13:07 udesale joined #puppet-openstack
13:07 mnaser (assuming these warnings are going to stderr lol)
13:12 jpena it seems to go to stderr, fortunately
13:13 mnaser \o/
13:19 shardy joined #puppet-openstack
13:41 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack_spec_helper stable/pike: Drop Fedora 24 from supported operating systems  https://review.openstack.org/501735
13:41 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack_spec_helper stable/ocata: Drop Fedora 24 from supported operating systems  https://review.openstack.org/501736
13:44 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack_spec_helper stable/newton: Drop Fedora 24 from supported operating systems  https://review.openstack.org/501740
13:50 thegreenhundred joined #puppet-openstack
14:06 evidex joined #puppet-openstack
14:10 ansmith joined #puppet-openstack
14:10 agrebennikov joined #puppet-openstack
14:18 openstackgerrit Mohammed Naser proposed openstack/puppet-openstack-integration master: Add puppet-archive to external modules  https://review.openstack.org/501752
14:29 mwarad joined #puppet-openstack
14:31 openstackgerrit Bob Fournier proposed openstack/puppet-ironic stable/ocata: Fixes for Ironic-inspector introspection when client using UEFI  https://review.openstack.org/501756
14:45 vnogin joined #puppet-openstack
14:48 openstackgerrit Moshe Levi proposed openstack/puppet-neutron master: ovs agent: add support for ovs hw-offload option  https://review.openstack.org/501760
14:56 xarses_ joined #puppet-openstack
15:01 simon-AS559 joined #puppet-openstack
15:12 openstackgerrit Bob Fournier proposed openstack/puppet-ironic stable/ocata: Fixes for Ironic-inspector introspection when client using UEFI  https://review.openstack.org/501756
15:15 openstackgerrit Merged openstack/puppet-cinder master: add parameter to overwrite/add wsgi process options  https://review.openstack.org/500578
15:28 simon-AS5591 joined #puppet-openstack
15:29 mnaser mwhahaha EmilienM https://review.openstack.org/#/c/501774/1 if you have a few minutes
15:33 vnogin joined #puppet-openstack
16:02 rmart04 joined #puppet-openstack
16:06 mwynne joined #puppet-openstack
16:08 ansmith joined #puppet-openstack
16:10 simon-AS559 joined #puppet-openstack
16:34 rmart04 joined #puppet-openstack
16:43 openstackgerrit Javier Peña proposed openstack/puppet-nova master: Sanitize nova_manage output in provider  https://review.openstack.org/501669
16:46 rmart04_ joined #puppet-openstack
17:04 dprince joined #puppet-openstack
18:01 EmilienM mnaser: replied
18:19 vnogin joined #puppet-openstack
18:53 dprince joined #puppet-openstack
19:16 mnaser EmilienM mwhahaha thank you, hope i got it right this time hah
19:22 derekh joined #puppet-openstack
19:36 vnogin joined #puppet-openstack
19:48 derekh joined #puppet-openstack
20:38 openstackgerrit Merged openstack/puppet-magnum stable/pike: Fix magnum cluster TLS cert generation  https://review.openstack.org/501141
20:38 openstackgerrit Merged openstack/puppet-magnum stable/ocata: Fix magnum cluster TLS cert generation  https://review.openstack.org/501142
20:38 openstackgerrit Merged openstack/puppet-magnum stable/newton: Fix magnum cluster TLS cert generation  https://review.openstack.org/501143
21:05 mwarad joined #puppet-openstack
21:06 openstackgerrit Merged openstack/puppet-nova stable/pike: Ensure novajoin's log directory ownership is correct  https://review.openstack.org/501665
21:07 simon-AS5591 joined #puppet-openstack
21:07 vnogin joined #puppet-openstack
21:30 ansmith joined #puppet-openstack
22:17 derekh joined #puppet-openstack
22:37 openstackgerrit Merged openstack/puppet-ironic stable/ocata: Fixes for Ironic-inspector introspection when client using UEFI  https://review.openstack.org/501756
22:51 alee joined #puppet-openstack
23:32 vnogin joined #puppet-openstack
23:40 xarses_ joined #puppet-openstack

| Channels | #puppet-openstack index | Today | | Search | Google Search | Plain-Text | summary