Thursday, 2016-09-15

*** ducttape_ has joined #puppet-openstack00:02
*** hjensas has quit IRC00:06
*** dgurtner has quit IRC00:10
*** lucas-dinner has quit IRC00:10
*** lucasagomes has joined #puppet-openstack00:16
*** johnmilton has quit IRC00:44
*** johnmilton has joined #puppet-openstack00:57
*** evidex has quit IRC01:06
*** itsuugo has quit IRC01:08
*** itsuugo has joined #puppet-openstack01:10
*** itsuugo has quit IRC01:15
*** itsuugo has joined #puppet-openstack01:16
*** itsuugo has quit IRC01:22
*** itsuugo has joined #puppet-openstack01:22
*** cdelatte has quit IRC01:25
openstackgerritMarcus Furlong proposed openstack/puppet-ceilometer: fix broken join that stops polling service from starting  https://review.openstack.org/36609801:27
*** saneax is now known as saneax-_-|AFK01:27
*** itsuugo has quit IRC01:27
*** itsuugo has joined #puppet-openstack01:28
*** ducttape_ has quit IRC01:37
*** ilbot3 has quit IRC01:44
*** ilbot3 has joined #puppet-openstack01:48
*** itsuugo has quit IRC01:48
*** itsuugo has joined #puppet-openstack01:50
*** ducttape_ has joined #puppet-openstack01:55
*** itsuugo has quit IRC01:55
*** ducttape_ has quit IRC01:56
*** itsuugo has joined #puppet-openstack01:57
*** richm has quit IRC01:59
openstackgerritEmilien Macchi proposed openstack/puppet-zaqar: dbsync: don't require User resource  https://review.openstack.org/37051302:02
*** itsuugo has quit IRC02:08
*** itsuugo has joined #puppet-openstack02:09
*** itsuugo has quit IRC02:16
*** itsuugo has joined #puppet-openstack02:17
*** itsuugo has quit IRC02:22
*** itsuugo has joined #puppet-openstack02:23
*** michchap has quit IRC02:26
*** michchap has joined #puppet-openstack02:26
*** michchap_alt has joined #puppet-openstack02:27
*** michapma_alt has quit IRC02:27
*** itsuugo has quit IRC02:29
*** ducttape_ has joined #puppet-openstack02:30
*** itsuugo has joined #puppet-openstack02:30
*** itsuugo has quit IRC02:35
*** itsuugo has joined #puppet-openstack02:36
*** itsuugo has quit IRC02:43
*** itsuugo has joined #puppet-openstack02:44
*** jperry has joined #puppet-openstack02:44
*** ducttape_ has quit IRC02:47
*** itsuugo has quit IRC02:48
*** itsuugo has joined #puppet-openstack02:50
*** itsuugo has quit IRC03:01
*** itsuugo has joined #puppet-openstack03:03
*** itsuugo has quit IRC03:08
*** itsuugo has joined #puppet-openstack03:10
*** itsuugo has quit IRC03:17
*** itsuugo has joined #puppet-openstack03:18
*** itsuugo has quit IRC03:23
*** itsuugo has joined #puppet-openstack03:24
*** itsuugo has quit IRC03:33
*** itsuugo has joined #puppet-openstack03:36
*** itsuugo has quit IRC03:41
*** itsuugo has joined #puppet-openstack03:42
*** ducttape_ has joined #puppet-openstack03:48
*** itsuugo has quit IRC03:51
*** itsuugo has joined #puppet-openstack03:52
*** ducttape_ has quit IRC03:53
*** itsuugo has quit IRC03:59
*** itsuugo has joined #puppet-openstack04:00
*** jperry has quit IRC04:04
*** itsuugo has quit IRC04:10
*** itsuugo has joined #puppet-openstack04:12
*** itsuugo has quit IRC04:16
*** itsuugo has joined #puppet-openstack04:18
*** itsuugo has quit IRC04:23
*** itsuugo has joined #puppet-openstack04:24
*** itsuugo has quit IRC04:29
*** itsuugo has joined #puppet-openstack04:29
*** itsuugo has quit IRC04:34
*** itsuugo has joined #puppet-openstack04:35
*** itsuugo has quit IRC04:40
*** itsuugo has joined #puppet-openstack04:41
*** jaosorior has joined #puppet-openstack04:47
*** ducttape_ has joined #puppet-openstack04:49
*** itsuugo has quit IRC04:53
*** itsuugo has joined #puppet-openstack04:54
*** ducttape_ has quit IRC04:55
*** saneax-_-|AFK is now known as saneax04:58
*** jaosorior has quit IRC05:01
*** jaosorior has joined #puppet-openstack05:02
*** itsuugo has quit IRC05:03
*** itsuugo has joined #puppet-openstack05:04
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add VNC Proxy common manifest  https://review.openstack.org/37054105:09
*** itsuugo has quit IRC05:10
*** itsuugo has joined #puppet-openstack05:10
*** rcernin has joined #puppet-openstack05:17
*** skramaja has joined #puppet-openstack05:18
*** itsuugo has quit IRC05:20
*** itsuugo has joined #puppet-openstack05:21
*** itsuugo has quit IRC05:26
*** itsuugo has joined #puppet-openstack05:28
*** itsuugo has quit IRC05:37
*** itsuugo has joined #puppet-openstack05:38
*** markvoelker has quit IRC05:41
*** itsuugo has quit IRC05:43
*** itsuugo has joined #puppet-openstack05:43
*** itsuugo has quit IRC05:51
*** itsuugo has joined #puppet-openstack05:51
*** itsuugo has quit IRC05:56
*** itsuugo has joined #puppet-openstack05:58
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Add VNC Proxy common manifest  https://review.openstack.org/37054106:00
*** itsuugo has quit IRC06:03
*** itsuugo has joined #puppet-openstack06:03
*** karthiks has joined #puppet-openstack06:06
*** karthiks has quit IRC06:10
*** jtomasek has joined #puppet-openstack06:12
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Fix wrong flag name for VNC Proxy in HAProxy  https://review.openstack.org/37055206:17
*** itsuugo has quit IRC06:18
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Fix wrong flag name for VNC Proxy in HAProxy  https://review.openstack.org/37055206:19
*** itsuugo has joined #puppet-openstack06:19
*** evidex has joined #puppet-openstack06:21
*** itsuugo has quit IRC06:28
*** itsuugo has joined #puppet-openstack06:30
*** itsuugo has quit IRC06:37
*** itsuugo has joined #puppet-openstack06:38
*** itsuugo has quit IRC06:44
*** rwsu has joined #puppet-openstack06:44
*** itsuugo has joined #puppet-openstack06:45
*** ducttape_ has joined #puppet-openstack06:52
*** ducttape_ has quit IRC06:56
*** itsuugo has quit IRC07:00
*** itsuugo has joined #puppet-openstack07:02
*** paramite has joined #puppet-openstack07:03
*** itsuugo has quit IRC07:17
*** evidex has quit IRC07:17
*** itsuugo has joined #puppet-openstack07:18
*** shardy has joined #puppet-openstack07:20
*** samuelBartel has joined #puppet-openstack07:22
*** itsuugo has quit IRC07:23
*** jpena|off is now known as jpena07:24
*** itsuugo has joined #puppet-openstack07:24
openstackgerritSven Anderson proposed openstack/puppet-tempest: Fix ec2 credential provider.  https://review.openstack.org/36999307:29
*** itsuugo has quit IRC07:29
*** itsuugo has joined #puppet-openstack07:30
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Fix dependencies for HAProxy when certmonger is used  https://review.openstack.org/37057707:33
*** moravec has quit IRC07:33
*** amoralej|off is now known as amoralej07:34
*** itsuugo has quit IRC07:35
*** itsuugo has joined #puppet-openstack07:35
*** chem has joined #puppet-openstack07:36
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Fix dependencies for HAProxy when certmonger is used  https://review.openstack.org/37057707:36
*** moravec has joined #puppet-openstack07:37
*** cgfbee has quit IRC07:41
*** markvoelker has joined #puppet-openstack07:42
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Fix dependencies for HAProxy when certmonger is used  https://review.openstack.org/37057707:43
*** itsuugo has quit IRC07:47
*** markvoelker has quit IRC07:48
openstackgerritOpenStack Proposal Bot proposed openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/37058707:49
*** itsuugo has joined #puppet-openstack07:49
*** dtantsur|afk is now known as dtantsur07:55
*** basilAB has quit IRC08:03
*** basilAB has joined #puppet-openstack08:05
ansiwenchem: I updated your change... a file has to be given as last argument, obvioudly08:11
chemansiwen: hum ... don't get it08:20
ansiwenthe optional argument is the properties array, tha last argument, a file, is not optional.08:20
ansiwenchem: ^08:21
ansiwenchem: wait...08:21
chemansiwen: def self.request(service, action, properties, credentials=nil, options={})08:21
ansiwenchem: http://logs.openstack.org/42/356442/4/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/07761e7/console.html#_2016-09-14_10_25_08_93900008:22
chemansiwen: is the function signature, I believe that file_path is not credentials08:22
ansiwenchem: https://github.com/openstack/puppet-tempest/blob/master/lib/puppet/provider/tempest.rb#L1608:23
ansiwenit's this request function I think08:23
ansiwenchem: well, actually I know... from the traceback ;-)08:24
chemansiwen: good catch, I missed that08:24
chemansiwen: that's kinda why I prefer composition over inheritance ...08:26
ansiwenchem: yes, inheritance is out... ;-)08:26
chemansiwen: anyway, that should do it now.  But this really lack a rspec test.  We would have catch all this before.08:27
chemansiwen: and thanks for the fix08:27
ansiwenchem: I also added a depends-on footer in the hope that it's tested with the updated CI already, so we see that it actually works.08:27
chemansiwen: to which patch08:28
ansiwenthe CI patch which uses this provider08:28
ansiwenchem: https://review.openstack.org/#/c/356442/08:28
chemansiwen: perfect :)  That will exercice the code and catch any failure.  Very good job there :)08:30
ansiwenchem: would a provider unit-test have catched that?08:30
chemansiwen: it depends on how you do it.08:31
chemansiwen: the best way to catch this is : don't mock anything and call the function08:32
*** itsuugo has quit IRC08:32
chemansiwen: it fails and then you mock the required call08:32
chemansiwen: at the it fails step, you would have seen the interface08:32
chemansiwen: and mock it apropriatly08:33
chemansiwen: does it make sens ?08:33
cheme08:33
*** itsuugo has joined #puppet-openstack08:34
ansiwenwell, the function at which level?08:34
*** dgurtner has joined #puppet-openstack08:34
*** dgurtner has quit IRC08:34
*** dgurtner has joined #puppet-openstack08:34
ansiwenchem: I'm off for today (I'm UTC+8 at the moment), let's hope it will work08:38
chemansiwen: will keep a look on it, thanks08:42
*** itsuugo has quit IRC08:46
*** derekh has joined #puppet-openstack08:46
*** itsuugo has joined #puppet-openstack08:46
*** itsuugo has quit IRC08:51
*** itsuugo has joined #puppet-openstack08:52
*** evidex has joined #puppet-openstack09:02
*** itsuugo has quit IRC09:06
*** itsuugo has joined #puppet-openstack09:08
*** jtomasek has quit IRC09:08
*** jtomasek has joined #puppet-openstack09:09
*** itsuugo has quit IRC09:18
*** itsuugo has joined #puppet-openstack09:19
*** markvoelker has joined #puppet-openstack09:44
*** markvoelker has quit IRC09:49
*** gfidente has joined #puppet-openstack09:57
*** gfidente has quit IRC10:00
*** itsuugo has quit IRC10:00
*** itsuugo has joined #puppet-openstack10:01
*** itsuugo has quit IRC10:11
*** itsuugo has joined #puppet-openstack10:12
*** gfidente has joined #puppet-openstack10:12
*** gfidente has quit IRC10:12
*** gfidente has joined #puppet-openstack10:12
openstackgerritPeter Zhurba proposed openstack/puppet-glare: Move glare puppet support into separate module (params and db parts)  https://review.openstack.org/37067010:17
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: [WIP] Pass boot_mode=uefi to kernel on introspection with UEFI  https://review.openstack.org/37067310:21
dtantsuranyone wants to tell me that this ^^^ is a ridiculous approach?10:23
*** itsuugo has quit IRC10:25
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: [WIP] Pass boot_mode=uefi to kernel on introspection with UEFI  https://review.openstack.org/37067310:25
*** itsuugo has joined #puppet-openstack10:27
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: [WIP] Pass boot_mode=uefi to kernel on introspection with UEFI  https://review.openstack.org/37067310:31
*** itsuugo has quit IRC10:32
*** _degorenko|afk is now known as degorenko10:32
*** hjensas has joined #puppet-openstack10:32
*** itsuugo has joined #puppet-openstack10:32
*** dtantsur is now known as dtantsur|bbl10:39
*** itsuugo has quit IRC10:39
*** itsuugo has joined #puppet-openstack10:41
*** itsuugo has quit IRC10:46
*** itsuugo has joined #puppet-openstack10:46
openstackgerritCarlos Camacho proposed openstack/puppet-ceph: Fix puppet-lint before upgrading gem  https://review.openstack.org/36994810:47
*** chem` has joined #puppet-openstack10:49
*** chem has quit IRC10:50
*** shardy has quit IRC10:57
*** lucasagomes is now known as lucas-hungry11:06
*** paramite has quit IRC11:14
openstackgerritCarlos Camacho proposed openstack/puppet-watcher: Fix puppet-lint before upgrading gem  https://review.openstack.org/37006111:20
*** rlaurika has joined #puppet-openstack11:22
*** itsuugo has quit IRC11:23
*** itsuugo has joined #puppet-openstack11:25
openstackgerritMerged openstack/puppet-openstack-integration: Promote RDO repository to latest consistent URL  https://review.openstack.org/37058711:25
*** skramaja has quit IRC11:30
*** skramaja has joined #puppet-openstack11:30
*** skramaja_ has joined #puppet-openstack11:34
*** skramaja has quit IRC11:35
*** skramaja_ is now known as skramaja11:36
*** dgurtner has quit IRC11:37
openstackgerritCarlos Camacho proposed openstack/puppet-watcher: Fix puppet-lint before upgrading gem  https://review.openstack.org/37006111:37
*** dgurtner has joined #puppet-openstack11:37
*** dgurtner has quit IRC11:37
*** dgurtner has joined #puppet-openstack11:37
*** skramaja_ has joined #puppet-openstack11:42
*** skramaja has quit IRC11:42
openstackgerritCarlos Camacho proposed openstack/puppet-ceph: Fix puppet-lint before upgrading gem  https://review.openstack.org/36994811:43
*** itsuugo has quit IRC11:43
*** itsuugo has joined #puppet-openstack11:44
*** markvoelker has joined #puppet-openstack11:45
*** dgurtner has quit IRC11:46
*** skramaja has joined #puppet-openstack11:49
*** skramaja_ has quit IRC11:49
*** markvoelker has quit IRC11:50
*** trown|outtypewww is now known as trown11:52
*** jpena is now known as jpena|lunch12:02
EmilienMhello12:07
*** amoralej is now known as amoralej|lunch12:07
*** pradk has joined #puppet-openstack12:13
openstackgerritMaksym Yatsenko proposed openstack/puppet-nova: Test commit  https://review.openstack.org/36991412:14
*** ducttape_ has joined #puppet-openstack12:14
*** lucas-hungry is now known as lucasagomes12:17
*** itsuugo has quit IRC12:23
*** itsuugo has joined #puppet-openstack12:23
*** paramite has joined #puppet-openstack12:27
*** itsuugo has quit IRC12:28
*** jayg|g0n3 is now known as jayg12:28
*** itsuugo has joined #puppet-openstack12:29
*** richm has joined #puppet-openstack12:30
*** markvoelker has joined #puppet-openstack12:38
*** dgurtner has joined #puppet-openstack12:38
*** dgurtner has quit IRC12:38
*** dgurtner has joined #puppet-openstack12:38
*** ducttape_ has quit IRC12:42
openstackgerritMerged openstack/puppet-tempest: Renamed python-sahara-tests to python-sahara-tests-tempest  https://review.openstack.org/36987812:45
*** itsuugo has quit IRC12:45
iurygregorygood morning o/12:46
*** itsuugo has joined #puppet-openstack12:46
*** shardy has joined #puppet-openstack12:50
*** morazi has joined #puppet-openstack12:51
*** dtantsur|bbl is now known as dtantsur12:51
*** itsuugo has quit IRC13:01
*** jpena|lunch is now known as jpena13:02
*** jaosorior has quit IRC13:02
*** itsuugo has joined #puppet-openstack13:02
*** jaosorior has joined #puppet-openstack13:03
*** StraubTW has joined #puppet-openstack13:04
openstackgerritEmilien Macchi proposed openstack/puppet-mistral: Add rpc_response_timeout option for Oslo Messaging  https://review.openstack.org/37079813:06
*** paramite has quit IRC13:06
*** iberezovskiy|off is now known as iberezovskiy13:08
*** vinsh has quit IRC13:08
*** fultonj has quit IRC13:09
*** fultonj has joined #puppet-openstack13:09
*** moravec has quit IRC13:11
*** moravec has joined #puppet-openstack13:12
*** amoralej|lunch is now known as amoralej13:15
*** paramite has joined #puppet-openstack13:17
*** shardy has quit IRC13:22
*** bogdando has quit IRC13:22
*** bogdando has joined #puppet-openstack13:23
openstackgerritPeter Zhurba proposed openstack/puppet-glare: Move glare puppet support into separate module (params and db parts)  https://review.openstack.org/37067013:27
openstackgerritPeter Zhurba proposed openstack/puppet-glare: Move glare puppet support into separate module (params and db parts)  https://review.openstack.org/37067013:29
*** evidex has quit IRC13:30
*** evidex has joined #puppet-openstack13:30
*** trown is now known as trown|afk13:40
*** jperry has joined #puppet-openstack13:41
*** ducttape_ has joined #puppet-openstack13:42
*** degorenko has quit IRC13:46
*** itsuugo has quit IRC13:46
*** degorenko has joined #puppet-openstack13:47
*** ducttape_ has quit IRC13:47
*** itsuugo has joined #puppet-openstack13:48
openstackgerritPeter Zhurba proposed openstack/puppet-glare: Move glare puppet support into separate module (params and db parts)  https://review.openstack.org/37067013:58
openstackgerritChristopher Brown proposed openstack/puppet-keystone: Implement chase referrals parameter  https://review.openstack.org/37085313:59
*** ducttape_ has joined #puppet-openstack14:05
*** ducttape_ has quit IRC14:09
*** andreww has quit IRC14:10
*** ccamacho has joined #puppet-openstack14:13
ccamachoHello mwhahaha14:14
mwhahahaccamacho: so the reason why i recommended the ignore for the commands is that i don't necessarily want to introduce something that could introduce errors just to cleanup the 140 char limit14:14
mwhahahaccamacho: my concern is that by splitting and now  using joins, etc it's harder to read and we'll introduce a typo or something14:15
ccamachoyour right there, righ now we are in "release mode"14:15
ccamachoso lets ignore those lines14:15
mwhahahayea so my preference is to use the ignore comment for only the exec commands14:15
ccamachoand then we can investigate the most elegant way of writing them14:15
mwhahahaeverything else we can split or deal with in a different way :)14:16
ccamachomwhahaha agreed14:16
ccamacho :)14:16
mwhahahathanks14:16
ccamachonp man14:16
ccamachomwhahaha first time in this channel Ill stay here then :)14:16
*** hjensas has quit IRC14:16
mwhahahasounds good :)14:17
*** dfisher has joined #puppet-openstack14:17
*** ducttape_ has joined #puppet-openstack14:18
openstackgerritCarlos Camacho proposed openstack/puppet-watcher: Fix puppet-lint before upgrading gem  https://review.openstack.org/37006114:20
*** ducttape_ has quit IRC14:23
openstackgerritCarlos Camacho proposed openstack/puppet-cinder: Fix puppet-lint before upgrading gem  https://review.openstack.org/37003014:26
beaglesccamacho: it's a good channel, it's always open on my IRC client - even if I don't always get to spend as much time on puppet stuff as I'd like14:28
*** devnull0 has joined #puppet-openstack14:28
ccamachobeagles +114:28
*** vinsh has joined #puppet-openstack14:29
*** andreww has joined #puppet-openstack14:30
openstackgerritCarlos Camacho proposed openstack/puppet-swift: Fix puppet-lint before upgrading gem  https://review.openstack.org/37005514:30
*** trown|afk is now known as trown14:30
openstackgerritCarlos Camacho proposed openstack/puppet-glance: Fix puppet-lint before upgrading gem  https://review.openstack.org/37004514:33
*** snecklifter has joined #puppet-openstack14:36
snecklifterhello, hi, so is fuel ci broken or is it just me?14:36
mwhahahasmoke_neutron is14:36
openstackgerritCarlos Camacho proposed openstack/puppet-monasca: Fix puppet-lint before upgrading gem  https://review.openstack.org/37003314:37
mwhahahasnecklifter: which review? i'll check it out and make sure you didn't break it :D14:37
sneckliftermwhahaha: pretty sure I didn't14:37
snecklifterhttps://review.openstack.org/#/c/370853/14:37
sneckliftermwhahaha: thanks, btw14:38
mwhahahasnecklifter: no looks like connectivity issues. fyi you can use 'fuel: recheck' to rerun just those tests14:38
EmilienMjust a general FYI14:39
sneckliftermwhahaha: ah, didn't know that, perfect, thanks14:39
EmilienMtripleo ci is currently broken14:39
mwhahahaboo tripleo ci14:39
EmilienMand tripleo folks are on it14:39
EmilienMif situation is not unblocked today, we'll disable voting on the job14:39
EmilienMplease do not recheck if your patch is failing tripleo jobs14:39
sneckliftergotcha14:41
snecklifteri'll just recheck on all the other ci that are failing :/14:42
*** pradk has quit IRC14:42
sneckliftercan't we get rid of CI and just wing it? It does seem broken a _lot_ :D14:43
EmilienMwe don't get rid of CI14:43
snecklifteryes, was joke14:44
EmilienM:)14:44
iurygregoryif only the ooo job is failling people can still +1 +2 if they are fine XD14:44
EmilienMdon't make jokes about CI! :P14:44
snecklifterunderstood!14:45
*** pradk has joined #puppet-openstack14:45
iurygregoryTripleO is in the process to align the stars14:46
*** ducttape_ has joined #puppet-openstack14:47
*** chem` is now known as chem14:50
*** jperry has quit IRC14:55
*** saneax is now known as saneax-_-|AFK14:56
openstackgerritMerged openstack/puppet-openstack-integration: Enable sahara testing in scenario003  https://review.openstack.org/37004814:59
openstackgerritMarios Andreou proposed openstack/puppet-tripleo: Fixup manila-cephfs native backend defaults  https://review.openstack.org/36676015:03
*** jperry has joined #puppet-openstack15:06
EmilienMI'm disabling voting of tripleo job https://review.openstack.org/#/c/370922/15:12
EmilienMso we'll be good in a few min hopefully15:12
EmilienMour team is still debugging what's wrong, but it takes time15:12
*** lutzb has quit IRC15:17
*** jaosorior has quit IRC15:18
*** paramite has quit IRC15:22
*** lutzb has joined #puppet-openstack15:24
openstackgerritMarios Andreou proposed openstack/puppet-tripleo: Add manila-netapp backend to manila class and tidy up generic  https://review.openstack.org/35401415:28
openstackgerritMarios Andreou proposed openstack/puppet-tripleo: Fixup manila-cephfs native backend defaults  https://review.openstack.org/36676015:31
*** lutzb has quit IRC15:32
*** lutzb has joined #puppet-openstack15:34
*** ducttape_ has quit IRC15:37
openstackgerritMerged openstack/puppet-openstack-integration: Stop using tox  https://review.openstack.org/36954215:39
openstackgerritDimitri Savineau proposed openstack/puppet-tripleo: Cinder: Add Nexenta Support  https://review.openstack.org/37095615:42
mfischdegorenko: can I get +A on 34446415:45
openstackgerritAlex Schultz proposed openstack/puppet-ceph: Honor rgw_enable and mon_enable for systemd units  https://review.openstack.org/36913615:45
*** lutzb has left #puppet-openstack15:47
mwhahahamfisch: i'll do it15:47
mfischthx15:47
mwhahahalet me actually review it tho :D15:47
*** itsuugo has quit IRC15:48
*** ducttape_ has joined #puppet-openstack15:49
*** priteau has joined #puppet-openstack15:49
*** itsuugo has joined #puppet-openstack15:49
*** andreww is now known as xarses15:56
*** michchap_alt has quit IRC16:00
*** samuelBartel has quit IRC16:08
*** devnull0 has quit IRC16:14
*** fultonj_ has joined #puppet-openstack16:20
*** lucasagomes is now known as lucas-dinner16:24
*** dtantsur is now known as dtantsur|afk16:25
*** rcernin has quit IRC16:26
*** itsuugo has quit IRC16:32
*** evidex has quit IRC16:32
*** itsuugo has joined #puppet-openstack16:34
*** priteau has quit IRC16:44
*** granlibakken has joined #puppet-openstack16:48
*** derekh has quit IRC16:49
*** darkxploit has joined #puppet-openstack16:50
*** _ducttape_ has joined #puppet-openstack17:00
*** ducttape_ has quit IRC17:03
*** fultonj_ has quit IRC17:07
*** darkxploit has quit IRC17:09
*** jpena is now known as jpena|off17:10
*** _ducttape_ has quit IRC17:22
*** ducttape_ has joined #puppet-openstack17:23
*** snecklifter has quit IRC17:24
*** trown is now known as trown|lunch17:28
*** chem has quit IRC17:29
*** chem has joined #puppet-openstack17:30
*** chem has quit IRC17:30
*** chem has joined #puppet-openstack17:30
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Add designate test coverage to scenario003  https://review.openstack.org/33786017:32
*** ducttape_ has quit IRC17:34
*** degorenko is now known as _degorenko|afk17:35
*** iberezovskiy is now known as iberezovskiy|off17:39
EmilienMtripleo CI job is now fixed17:39
EmilienMyou can now do recheck17:40
*** amoralej is now known as amoralej|off17:43
*** gfidente has quit IRC17:44
mwhahaharechecks for all17:48
EmilienMI went through most of them17:49
*** ducttape_ has joined #puppet-openstack17:55
*** hjensas has joined #puppet-openstack18:05
openstackgerritMerged openstack/puppet-cinder: Cinder hooks support  https://review.openstack.org/34446418:42
*** itsuugo has quit IRC18:42
*** xarses has quit IRC18:42
*** trown|lunch is now known as trown18:43
*** itsuugo has joined #puppet-openstack18:44
*** AnarchyAo has joined #puppet-openstack18:48
vinshHmm.19:04
vinsh  service { 'tftpd-hpa':19:04
vinsh    ensure => 'stopped',19:04
vinsh    enable => false,19:04
vinsh  }19:04
vinshpuppet-ironic pxe.pp  has that.. that will fail if you havn't ever installed tftpd-hpa19:04
*** hjensas has quit IRC19:04
mwhahahathat's unfortunate19:06
vinshI think it's an assumption that doesn't stand for all users.19:06
vinshJust asking incase there is some obvious oversight on my part :)19:06
mwhahahano that's a problem19:08
vinshI think we could either ensure tftpd-hpa package is absent..  or just remove this block and not mess with it.19:11
vinshLeave it up to the user to decide.19:11
mwhahahawell we shouldn't manage services that aren't installed by a package19:12
mwhahahaso if we don't install that package we shouldn't manage that service19:12
mwhahahai guess19:12
vinshYeah, I bet it was added to prevent tftpd-hpa from interfering with xinted tftp19:12
vinshYeah. I agree with your logic. I'll post a review.19:12
openstackgerritAdam Vinsh proposed openstack/puppet-ironic: Ironic pxe support, Don't manage tftpd-hpa service  https://review.openstack.org/37106919:16
mwhahahauh oh i wonder if tempest just dropped something that broke everything19:22
mwhahahahttps://review.openstack.org/#/c/369996/ all the scenarios just failed on the gate19:23
mwhahahahttps://review.openstack.org/#/c/341098/19:24
mwhahahainsert explative here19:24
vinsh:-o19:24
mwhahahawhere do those guys hang out19:24
vinshthe local tire-fire19:24
mtreinishmwhahaha: #openstack-qa19:24
mtreinishbut that shouldn't change anything iirc, the flag to do extra stuff should default false19:25
mwhahahayea well it does19:26
mwhahahahttp://logs.openstack.org/98/341098/7/check/gate-puppet-openstack-integration-3-scenario001-tempest-centos-7-nv/cafe952/console.html.gz#_2016-08-04_15_07_01_64229019:26
mwhahahaunrecognized attribute19:26
dmsimardmwhahaha: oh man I just got that in RDO too19:32
dmsimardin trunk19:33
mwhahahayea it landed between a check and a gate :D19:33
dmsimardmtreinish: ^19:35
dmsimardhalp19:35
dmsimardoh, wait, didn't see you already replied above sorry19:35
mtreinishdmsimard: heh, no worries. We've identified the problem mriedem is gonna work on a fix19:36
dmsimardmtreinish: all three puppet scenarios failed on that patch, should've been an indicator :(19:37
mwhahahai think for next cycle I shall push for voting in tempest :D19:38
dmsimardmwhahaha: not gonna work I think19:39
mwhahahai know i'm kidding19:39
dmsimardmwhahaha: at least so long as upstream doesn't implement features in puppet out of the bat19:39
mwhahahabut it really should19:39
dmsimardmwhahaha: they make it work in devstack (tm)19:39
dmsimardbrb19:40
mwhahahawe don't have to implement their features, they need to not break the defaults19:40
mwhahahaie backwards compatibility19:40
dmsimardhah19:40
dmsimardyou're funny19:40
dmsimard:D19:40
mwhahahapeople who don't have to deploy19:41
* mwhahaha shakes fist and demands people get off his lawn19:41
iurygregorylol19:42
openstackgerritMatt Fischer proposed openstack/puppet-cinder: Add deps to authtoken  https://review.openstack.org/37107719:44
openstackgerritDimitri Savineau proposed openstack/puppet-cinder: Nexenta Edge Support  https://review.openstack.org/37023819:45
openstackgerritDimitri Savineau proposed openstack/puppet-cinder: Nexenta: Update parameters  https://review.openstack.org/37023719:47
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Pin tempest  https://review.openstack.org/37107819:47
openstackgerritAlex Schultz proposed openstack/puppet-openstack-integration: Pin tempest  https://review.openstack.org/37107819:48
*** rcernin has joined #puppet-openstack19:53
*** itsuugo has quit IRC19:53
*** itsuugo has joined #puppet-openstack19:55
*** ducttape_ has quit IRC20:03
dmsimardmwhahaha: can you put related-bug: in https://review.openstack.org/#/c/371078/ ?20:04
mwhahahai can, but i also proposed the tempest revert :D20:04
EmilienMwhat20:05
EmilienMlet me go complain20:06
EmilienMon #openstack-qa20:06
*** xarses has joined #puppet-openstack20:07
*** xarses has quit IRC20:21
*** jayg is now known as jayg|g0n320:22
*** ducttape_ has joined #puppet-openstack20:27
*** tdb_ has joined #puppet-openstack20:36
*** tdb has quit IRC20:36
*** xarses has joined #puppet-openstack20:37
*** morazi has quit IRC20:42
*** morazi has joined #puppet-openstack20:42
*** morazi has quit IRC20:43
*** itsuugo has quit IRC20:47
*** trown is now known as trown|outtypewww20:48
*** granlibakken has quit IRC20:49
*** itsuugo has joined #puppet-openstack20:50
*** xarses has quit IRC20:55
mwhahahaso https://review.openstack.org/#/c/371078/ passed if we want to unblock stuff20:56
EmilienMmwhahaha: if we go that way, tempest won't be able to test an actual fix20:56
EmilienMjust fyi20:56
EmilienMI'm happy to approve it though20:56
mwhahahawell we can wait for the outstanding proposed fixes20:57
mwhahahajust saying it's an option20:57
EmilienMif it's approved tonight, let's wait20:57
mwhahahasure20:57
EmilienMif it doesn't pass tempest CI and is not approved, let's take this option20:57
dmsimardeh20:58
dmsimarddesignate doesn't use [oslo_messaging_rabbit] apparently20:59
dmsimardeither a transport_url string or standarlone params in [DEFAULT]20:59
dmsimard2014 called20:59
iurygregorywow20:59
dmsimardI was troubleshooting http://logs.openstack.org/60/337860/31/check/gate-puppet-openstack-integration-3-scenario003-tempest-centos-7/0f1ed57/logs/designate/api.txt.gz#_2016-09-15_18_11_58_39120:59
dmsimardOf course it worked in devstack http://logs.openstack.org/24/360924/1/check/gate-tempest-dsvm-designate-bind9/142edcf/logs/etc/designate/designate.conf.txt.gz20:59
dmsimardand standalone params in their get-started config http://docs.openstack.org/developer/designate/getting-started.html#configuring-designate21:00
dmsimardmwhahaha: FWIW I'll rebase on top of that pin so I'm not blocked in testing stuff :D21:01
mwhahahaha ok21:01
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Add designate test coverage to scenario003  https://review.openstack.org/33786021:08
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Add designate test coverage to scenario003  https://review.openstack.org/33786021:08
dmsimardiurygregory: /me shrugs it looks like it's there in their sample file but it may be broken since they're not testing it that way ? https://github.com/openstack/designate/blob/c8fa7ef289ed2cb722d803f377199982ddcceb7b/etc/designate/designate.conf.sample#L4321:10
dmsimardthat wouldn't be the first bug our CI helped uncover in designate and it's not even fully implemented yet21:11
mwhahahathats why we should always be voting :D21:12
*** _stowa has joined #puppet-openstack21:12
mwhahahaif it can't actually be deployed, is it actually working? :D21:13
dmsimardp-o-i probably has a better testing matrix than devstack21:13
mwhahahanone of that devstack stuff21:13
*** _stowa has quit IRC21:16
*** ccamacho has quit IRC21:17
openstackgerritBradley Klein proposed openstack/puppet-monasca: Better vertica partition defaults, new drop script  https://review.openstack.org/37116821:28
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: WIP - install puppet4 from fedora  https://review.openstack.org/37117021:34
EmilienMmwhahaha, dmsimard ^ fyi21:34
EmilienMthe final goal is to have puppet packaging gated in Fedora / RDO21:35
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Remove nova::memcached_servers  https://review.openstack.org/37117121:35
dmsimardEmilienM: what about Ubuntu ?21:36
EmilienMdmsimard: what can I do?21:36
* dmsimard shrugs21:36
dmsimardinstalling from a copr is dirty though :(21:36
EmilienMdmsimard: it's a first testing21:37
EmilienMnumber80 gave me this repo to start21:37
EmilienMwe won't use it in production21:37
openstackgerritDavid Moreau Simard proposed openstack/puppet-openstack-integration: Remove nova::memcached_servers  https://review.openstack.org/37117121:43
*** AndyWojo has quit IRC21:45
*** ntpttr_ has quit IRC21:45
*** llua` has joined #puppet-openstack21:45
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: WIP - install puppet4 from fedora  https://review.openstack.org/37117021:46
*** llua has quit IRC21:46
*** bmjen has quit IRC21:46
*** mjb has quit IRC21:46
*** saneax-_-|AFK has quit IRC21:46
*** pradk has quit IRC21:48
*** dhollinger has quit IRC21:48
EmilienMvinsh: no need to do recheck, ci is busted by tempest21:48
*** dhollinger has joined #puppet-openstack21:51
*** bmjen has joined #puppet-openstack21:51
*** AndyWojo has joined #puppet-openstack21:51
vinshstill eh? ok :)21:51
*** mjb has joined #puppet-openstack21:52
vinshI wish mwhahaha would stop breaking things.21:52
* mwhahaha did no such thing21:52
vinsh;)21:52
*** saneax-_-|AFK has joined #puppet-openstack21:56
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: WIP - install puppet4 from fedora  https://review.openstack.org/37117021:58
*** jperry has quit IRC22:04
*** iurygregory_ has joined #puppet-openstack22:09
*** itsuugo has quit IRC22:15
*** itsuugo has joined #puppet-openstack22:17
*** llua` is now known as llua22:19
*** llua has joined #puppet-openstack22:19
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: WIP - install puppet4 from fedora  https://review.openstack.org/37117022:27
*** ntpttr has quit IRC22:31
*** itsuugo has quit IRC22:34
*** ntpttr has joined #puppet-openstack22:35
*** itsuugo has joined #puppet-openstack22:37
*** tdb_ has quit IRC22:41
*** ntpttr_ has joined #puppet-openstack22:41
*** itsuugo has quit IRC22:41
*** itsuugo has joined #puppet-openstack22:42
*** ducttape_ has quit IRC22:42
*** chem` has joined #puppet-openstack22:49
*** chem has quit IRC22:51
dfisherok, i need some puppet error translation.22:53
dfisherhttp://paste.openstack.org/show/577740/22:53
*** tdb has joined #puppet-openstack22:54
dfisherthe error is coming from lines 6-8.  I only know that because I commented it out and didn't get an error any more :)22:54
EmilienMit's pretty clear22:54
EmilienMpuppet can't find your service name because you run on solaris22:54
EmilienMand in params.pp there is no service name for solaris systems22:55
EmilienMso puppet try to instanciate service[nil] multiple times22:55
* dfisher sighs at you22:55
dfisherhttp://paste.openstack.org/show/577741/ — updated params.pp for Solaris22:55
iurygregory_poor dfisher =(22:55
dfisherpoor dfisher :((22:56
EmilienMI'm out, ttyl folks22:57
*** StraubTW has quit IRC22:57
mwhahahai like that service nil22:58
mwhahahawe need more service nil22:58
dfisheri'm fine with it22:58
mwhahahadfisher: did  you figure it out?23:01
dfishernope.23:01
dfisherbut > 5pm23:01
dfisherso23:01
dfishertomorrow23:01
dfishernow?  sushi.23:01
mwhahahai may look, maybe23:01
dfisherk23:01
dfisheranything you find would be appreciated.23:01
mwhahahathe swift module is terrible23:01
dfisher:)23:01
mwhahahakinda like swift actually23:01
dfisherseriously.  if only had zfs.23:02
dfisher:)23:02
mwhahahawe should rewrite it in go23:02
dfisheryou* only had zfs23:02
mwhahahait's ok i read it with the missing word23:02
mwhahahatry not to die of mercury poisioning23:02
dfisherthat's good.  i type good much many do.23:02
dfisherthanks!23:02
dfisherttyl23:02
mwhahahawe'll bury you in a sun cabinet23:02
dfisheri'll have lots of cores to keep me warm23:03
*** dfisher has quit IRC23:03
mwhahahaoh and i figured it out23:05
mwhahahaoh well maybe i'll tell him tomorrow23:05
*** tdb has quit IRC23:05
* mwhahaha wanders off23:06
mwhahahai case i forget proxy_service_name -> proxy_server_service_name23:06
mwhahahaand a few other23:07
mwhahahas23:07
*** itsuugo has quit IRC23:08
*** itsuugo has joined #puppet-openstack23:09
vinshmwhahaha: which items in the swift module don't you like?23:10
EmilienMthe fact is doesn't use ini_file provider but erb templates23:11
EmilienMit23:11
*** ducttape_ has joined #puppet-openstack23:12
EmilienMprobably more things23:12
vinshSomeone /could/ convert it23:12
EmilienMbut swift is super complex to configure23:12
EmilienMvinsh: yeah someone23:12
vinsh:D23:12
vinshSO I have 3 WIP's that add major features to that module23:12
iurygregory_swift is awesome when you are a user =P23:13
vinshand I agree they are some complex things to figure out.23:13
* EmilienM afk_real from now23:13
* EmilienM will read later23:13
vinsh;)23:13
vinshI'm knee deep in getting ironic up over here for us.. a week from now I will have good time blocked off to improve swift23:14
vinshI like the ini idea23:14
vinshIt's hard to guy/rewrite modules while trying to be backward compatible... but I guess that goes without saying.23:16
vinsh*gut23:16
mwhahahait's also really hard to rewrite and maintain backwards compatibility :D23:22
*** itsuugo has quit IRC23:23
*** itsuugo has joined #puppet-openstack23:24
*** dgurtner has quit IRC23:25
*** vinsh has quit IRC23:28
*** ducttape_ has quit IRC23:29
*** itsuugo has quit IRC23:35
*** johnmilton has quit IRC23:37
*** itsuugo has joined #puppet-openstack23:37
*** markvoelker has quit IRC23:39
*** AnarchyAo has quit IRC23:41
*** tdb has joined #puppet-openstack23:46
*** chem` has quit IRC23:49
*** itsuugo has quit IRC23:49
*** itsuugo has joined #puppet-openstack23:51

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