Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-10-07

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

All times shown according to UTC.

Time Nick Message
00:24 ducttape_ joined #puppet-openstack
00:32 zhangjn joined #puppet-openstack
00:44 EmilienM gildub: I will, but I'm at puppetconf this week, might be slow in reviews.
00:46 xarses joined #puppet-openstack
00:47 EmilienM Hunner: not a big deal for us I think
00:48 EmilienM Hunner: once the release will be done, we will try to bump our Puppetfile to it, and see if CI pass
00:50 EmilienM dmsimard: no, I can't because the manifest is using puppetdb, so it collect first and then realize
00:57 jfluhmann joined #puppet-openstack
01:11 dmsimard EmilienM: didn't you get it to work in integration testing ?
01:11 gildub EmilienM, I understand, although it's the backward compatibility exception we need to get through.
01:13 dmsimard EmilienM: happy puppet conf :)
01:22 richm joined #puppet-openstack
01:29 lman4821 joined #puppet-openstack
01:32 rahulait joined #puppet-openstack
01:59 jfluhmann joined #puppet-openstack
02:00 zhangjn joined #puppet-openstack
02:06 zhangjn joined #puppet-openstack
02:27 ducttape_ joined #puppet-openstack
02:47 Hunner EmilienM: I actually backed it out. We can re-merge again if you want to go for 6.0.0
02:47 Hunner https://github.com/puppetlabs​/puppetlabs-rabbitmq/pull/399 is ready for merge to release rabbitmq 5.3.1 :)
02:48 Hunner If any cores could merge ^ (or if you're puppet openstack core and don't have access, let me know)
03:19 markvoelker joined #puppet-openstack
03:23 markvoelker_ joined #puppet-openstack
03:29 rahulait joined #puppet-openstack
03:33 markvoelker joined #puppet-openstack
04:06 jamespag` joined #puppet-openstack
04:40 markvoelker joined #puppet-openstack
04:46 saravanankr joined #puppet-openstack
04:46 Mosibi joined #puppet-openstack
04:50 markvoelker joined #puppet-openstack
05:02 markvoelker joined #puppet-openstack
05:03 fedexo joined #puppet-openstack
05:13 markvoelker joined #puppet-openstack
05:28 markvoelker joined #puppet-openstack
05:33 sanjayu joined #puppet-openstack
05:36 zhangjn joined #puppet-openstack
05:41 markvoelker joined #puppet-openstack
05:42 ducttape_ joined #puppet-openstack
05:42 markvoelker_ joined #puppet-openstack
05:58 markvoelker joined #puppet-openstack
06:00 markvoelker_ joined #puppet-openstack
06:19 markvoelker joined #puppet-openstack
06:25 EmilienM Hunner: done, thanks
06:25 EmilienM dmsimard: yes, I need to refactor acceptance tests for swift
06:26 EmilienM gildub: I'm not the only one who can review, it's open to other core members too but I'll look this week.
06:28 paramite joined #puppet-openstack
06:31 mtreinish joined #puppet-openstack
06:33 gchamoul joined #puppet-openstack
06:47 markvoelker joined #puppet-openstack
06:48 gildub EmilienM, sure but you got involved initially, sort of restore -1 into something +
06:52 jtomasek joined #puppet-openstack
07:02 markvoelker joined #puppet-openstack
07:16 markvoelker joined #puppet-openstack
07:17 dgurtner joined #puppet-openstack
07:18 gchamoul joined #puppet-openstack
07:22 gchamoul joined #puppet-openstack
07:31 markvoelker joined #puppet-openstack
07:32 gchamoul joined #puppet-openstack
07:33 lman4821 joined #puppet-openstack
07:34 gchamoul joined #puppet-openstack
07:37 jpena joined #puppet-openstack
07:46 markvoelker joined #puppet-openstack
08:00 markvoelker joined #puppet-openstack
08:04 sergmelikyan joined #puppet-openstack
08:05 rochaporto joined #puppet-openstack
08:15 markvoelker joined #puppet-openstack
08:26 imcsk8 joined #puppet-openstack
08:29 markvoelker joined #puppet-openstack
08:36 jistr joined #puppet-openstack
08:36 markvoelker joined #puppet-openstack
08:44 markvoelker joined #puppet-openstack
08:45 dsutyagin joined #puppet-openstack
08:47 openstackgerrit Yanis Guenane proposed openstack/puppet-horizon: Create horizon::keystone::auth  https://review.openstack.org/230949
08:51 markvoelker joined #puppet-openstack
08:57 sergmelikyan joined #puppet-openstack
08:59 imcsk8 joined #puppet-openstack
09:01 markvoelker joined #puppet-openstack
09:16 markvoelker joined #puppet-openstack
09:18 sergmelikyan joined #puppet-openstack
09:27 markvoelker joined #puppet-openstack
09:37 markvoelker joined #puppet-openstack
09:52 markvoelker joined #puppet-openstack
09:55 sergmelikyan joined #puppet-openstack
10:06 markvoelker joined #puppet-openstack
10:12 openstackgerrit Denis Egorenko proposed openstack/puppet-nova: Use OpenstackClient for nova providers auth  https://review.openstack.org/226862
10:21 markvoelker joined #puppet-openstack
10:35 LimorStotland joined #puppet-openstack
10:35 markvoelker joined #puppet-openstack
10:39 mdavidson joined #puppet-openstack
10:45 markvoelker joined #puppet-openstack
10:52 zhangjn joined #puppet-openstack
10:54 darkxploit joined #puppet-openstack
11:00 markvoelker joined #puppet-openstack
11:01 openstackgerrit Denis Egorenko proposed openstack/puppet-nova: Use OpenstackClient for nova providers auth  https://review.openstack.org/226862
11:14 markvoelker joined #puppet-openstack
11:21 cgfbee joined #puppet-openstack
11:23 sergmelikyan joined #puppet-openstack
11:25 IvanBerezovskiy joined #puppet-openstack
11:29 markvoelker joined #puppet-openstack
11:42 darkxploit joined #puppet-openstack
11:43 dprince joined #puppet-openstack
11:44 markvoelker joined #puppet-openstack
11:50 markvoelker joined #puppet-openstack
12:00 sergmelikyan joined #puppet-openstack
12:00 iurygregory good morning people :D
12:07 markvoelker joined #puppet-openstack
12:09 morazi joined #puppet-openstack
12:09 morazi_ joined #puppet-openstack
12:09 khushbu joined #puppet-openstack
12:10 richm joined #puppet-openstack
12:21 markvoelker joined #puppet-openstack
12:23 ducttape_ joined #puppet-openstack
12:28 zhangjn joined #puppet-openstack
12:29 richm joined #puppet-openstack
12:31 markvoelker joined #puppet-openstack
12:39 zhangjn joined #puppet-openstack
12:40 zhangjn joined #puppet-openstack
12:45 zhangjn joined #puppet-openstack
12:45 markvoelker joined #puppet-openstack
12:48 zhangjn joined #puppet-openstack
12:49 markvoelker_ joined #puppet-openstack
12:56 markvoelker joined #puppet-openstack
13:03 sergmelikyan joined #puppet-openstack
13:03 lman482 joined #puppet-openstack
13:10 lman4823 joined #puppet-openstack
13:10 lman4821 joined #puppet-openstack
13:13 vinsh joined #puppet-openstack
13:16 ducttape_ joined #puppet-openstack
13:31 samuelBartel joined #puppet-openstack
13:34 bklei joined #puppet-openstack
13:35 bklei emillienm are you cool with https://review.openstack.org/#/c/231687/?
13:36 linkedinyou joined #puppet-openstack
13:37 bklei or crinkle ^^
13:41 ducttape_ joined #puppet-openstack
13:41 ducttape_ joined #puppet-openstack
13:44 Hunner EmilienM: Thanks :). rabbitmq 5.3.1 released
13:52 zhangjn joined #puppet-openstack
13:52 sergmelikyan joined #puppet-openstack
13:53 zhangjn joined #puppet-openstack
13:54 zhangjn joined #puppet-openstack
14:04 markvoelker_ joined #puppet-openstack
14:05 degorenko hello people :)
14:06 lman4821 Not sure this is the right place to ask but I am struggling to get either Kilo OR Liberty to install using puppet code.  Seem to keep running into version issues between Kilo and Puppet modules (version 6) failing on NIL issues.  Any suggestions on where I can pull matching Openstack repos for Centos and puppet modules?  I was able to get the RDP packstack to install completely, but I see that that uses OLDER ve
14:08 mwhahaha lman4821: what packages are you using?
14:08 mwhahaha lman4821: https://github.com/opensta​ck/puppet-openstack_extras has the repo configuration that is used for testing
14:12 _ducttape_ joined #puppet-openstack
14:12 lman4821 We had used 6.0.0 versions by doing a specigi download
14:14 lman4821 using the RDO kilo RPMS.
14:14 lman4821 Kept getting Error: /Stage[main]/Keystone::Roles::Ad​min/Keystone_user[admin]/ensure:
14:14 lman4821 change from absent to present failed: Could not set 'present' on
14:14 lman4821 ensure: undefined method `[]' for nil:NilClass at
14:14 lman4821 Have not been able to debug but there were lots of deprecation warnings…
14:30 jasondotstar EmilienM: ping
14:31 khushbu_ joined #puppet-openstack
14:32 markvoelker joined #puppet-openstack
14:39 mattymo_ joined #puppet-openstack
14:40 mjblack joined #puppet-openstack
14:41 mattymo_ joined #puppet-openstack
14:45 v1k0d3n joined #puppet-openstack
14:46 severion joined #puppet-openstack
14:47 bklei_ joined #puppet-openstack
14:47 ducttape_ joined #puppet-openstack
14:47 skolekonov Hi guys. Please review https://review.openstack.org/#/c/230633/. I faced this problem on an environment
14:50 darkxploit joined #puppet-openstack
14:51 hogepodge joined #puppet-openstack
14:51 khushbu_ joined #puppet-openstack
14:55 mwhahaha lman4821: wonder if your credentials are wrong or something. I know i've seen that error in providers when the response comes back bad and there's a [] access on a nil class. was there a line number with that error?
14:55 EmilienM good morning
14:56 EmilienM jasondotstar: pong
14:56 jasondotstar EmilienM: touching base w/ you re: assisting w/ the release work
14:57 EmilienM jasondotstar: have you read https://wiki.openstack.org/wiki/Puppet/releases ?
14:57 EmilienM it's a good place to start
14:57 jasondotstar not yet....
14:57 * jasondotstar opens
14:57 trown why would the beaker-upgrade tests have different package versions then the regular beaker tests?
14:59 dfisher joined #puppet-openstack
15:03 fedexo joined #puppet-openstack
15:06 mdorman joined #puppet-openstack
15:13 degorenko EmilienM, take a look please: https://review.openstack.org/219275 and https://review.openstack.org/#/q/status:open+p​roject:openstack/puppet-nova+branch:master+top​ic:bp/use-openstackclient-in-module-resources,n,z
15:13 degorenko crinkle, clayton sbadia ^
15:18 paramite joined #puppet-openstack
15:22 sergmelikyan joined #puppet-openstack
15:34 lman4821 mwhahaha: interesting.  so those puppet commands can fail if something is wrong with the running Keystone setup then?  So it could be a configuration issue then, not a puppet code problem?
15:34 mwhahaha might be both
15:34 mwhahaha :D
15:35 mwhahaha i've seen that error get thrown for something completely unrelated before
15:37 ducttape_ joined #puppet-openstack
15:44 lman4821 mwhahaha: in case you're curious, this was the line no.
15:44 lman4821
15:44 lman4821 Error: /Stage[main]/Keystone::Roles::Ad​min/Keystone_user[admin]/ensure:
15:44 lman4821 change from absent to present failed: Could not set 'present' on
15:44 lman4821 ensure: undefined method `[]' for nil:NilClass at
15:44 lman4821 153:/etc/puppet/modules/thirdparty​/keystone/manifests/roles/admin.pp
15:46 mwhahaha which is of course is probably keystone_user line which doesn't help :D
15:47 lman4821 If I omit this puppet code, I don't get the error, but I also don't get everything setup (I think, as I'm still trying to grok all this).
15:47 lman4821 class { 'keystone::roles::admin':
15:47 lman4821 email        => 'mailto:yepper@yeppercorp.com',
15:47 lman4821 password     => 'a_big_secret',
15:47 lman4821 }
15:47 mwhahaha i wonder if it's line 41 in lib/puppet/provider/keystone_user/openstack.rb
15:47 mwhahaha project_id = Puppet::Resource.indirection.find("Keys​tone_tenant/#{resource[:tenant]}")[:id]
15:47 mwhahaha if that resource.indirection.find returns nil, that'll throw that error
15:48 mwhahaha in other words, bad tenant
15:49 lman4821 interesting.. OK, so maybe I do have an issue with the puppet code
15:49 mwhahaha the puppet default is 'openstack' for the tenant
15:53 lman4821 mind taking a look at my basic config?  http://pastebin.com/zUCfMDCe I think I may have removed the tenant lines earlier since they gave me trouble as well.  TIA
15:55 bklei joined #puppet-openstack
15:56 lman4821 I'm assuming I should stick with Kilo for now until Liberty is released?
15:57 bklei_ joined #puppet-openstack
16:04 openstackgerrit Merged openstack/puppet-monasca: Refresh monasca-api service when startup script changes  https://review.openstack.org/231687
16:04 openstackgerrit Merged openstack/puppet-monasca: Refresh monasca-api service when startup script changes  https://review.openstack.org/231687
16:04 cdelatte joined #puppet-openstack
16:05 ducttape_ joined #puppet-openstack
16:08 iurygregory EmilienM, when you have time this week or next, can you take a look at the acceptance tests in https://review.openstack.org/#/c/208054/ ? ^^
16:30 EmilienM crinkle, nibalizer: I'm with clayton at the OpenStack table, in Contributor meetup room
16:30 clayton it's near the door on the right, if you're here
16:32 crinkle EmilienM: clayton i'm not there yet, on my way
16:34 nibalizer me too
16:36 clayton eric is still doing the intro, talking about community state, etc
16:39 EmilienM spredzy: what is the use case of horizon endpoint?
16:40 spredzy EmilienM, backward compatibility with what python-tripleoclient does
16:40 EmilienM spredzy: so it's only for tripleo?
16:41 EmilienM spredzy: just curious, why tripleo needs an horizon endpoint?
16:41 spredzy EmilienM, nop everyone can benefit from it, tripleo has probably a use for it, I just don't know which one
16:42 EmilienM spredzy: I would like to know why
16:42 EmilienM spredzy: also, the :80 is not necessary, it's default port in http
16:43 EmilienM but it's not a big deal probably
16:44 spredzy EmilienM, can you comment for the use of the horizon endpoint in the review
16:44 spredzy this way I can send the link to the right person
16:44 EmilienM spredzy: well, my first comment is already asking
16:44 EmilienM spredzy: sure
16:45 EmilienM spredzy: done
16:45 spredzy EmilienM, thx
16:52 openstackgerrit Merged openstack/puppet-cinder: Add posibility to specify privileged user for Cinder  https://review.openstack.org/219275
17:01 linkedinyou|2 joined #puppet-openstack
17:07 openstackgerrit Merged openstack/puppet-keystone: Support for Keystone as Identity Provider  https://review.openstack.org/208054
17:14 openstackgerrit Yanis Guenane proposed openstack/puppet-horizon: Create horizon::keystone::auth  https://review.openstack.org/230949
17:26 xarses joined #puppet-openstack
17:32 trown anyone able to review a puppet-ironic patch to add support for inspector? has a couple +1 and a +2
17:32 trown https://review.openstack.org/#/c/223690/
17:35 jfluhmann joined #puppet-openstack
17:52 IvanBerezovskiy joined #puppet-openstack
17:53 EmilienM clayton: https://review.openstack.org/#/c/231570/
17:54 IvanBerezovskiy EmilienM: hi, can ask you to review this important patch https://review.openstack.org/#/c/230633/? it fixes issue with keystone under apache
17:55 EmilienM clayton: https://review.openstack.org/#/c/226624/
17:57 EmilienM IvanBerezovskiy: have you patched openstacklib first?
17:57 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Switch cinder to use os_service_default fact  https://review.openstack.org/229095
18:01 IvanBerezovskiy EmilienM: no, does it make sense? I've also found ansible issue for this https://bugs.launchpad.net/op​enstack-ansible/+bug/1481339
18:01 openstack Launchpad bug 1481339 in openstack-ansible trunk "Keystone import errors and upstream config mismatch" [High,Fix released] - Assigned to Jesse Pretorius (jesse-pretorius)
18:01 EmilienM clayton: https://docs.google.com/forms/d/1Z6IGeJRNmX7x​x0Ggmr5Pmpzq7BudphDkZE-3t4Q5G1k/viewanalytics
18:01 EmilienM IvanBerezovskiy: yes, you should patch openstacklib
18:03 EmilienM richm: talking with clayton about indirection patch
18:03 EmilienM richm: we think if we have to do some non-backward compatible stuff about v3, it's time to do it now, because we guess not a lot of people are using it right now
18:07 EmilienM chem, richm: I think this patch is going to break CI for all puppet openstack modules, because you'll have to patch  ::keystone::auth
18:08 openstackgerrit Merged openstack/puppet-openstacklib: Fix fact for puppet facter 2.0.1+  https://review.openstack.org/231570
18:09 sergmelikyan joined #puppet-openstack
18:09 EmilienM actually it's in manifests/resource/service_identity.pp
18:09 IvanBerezovskiy joined #puppet-openstack
18:09 IvanBerezovskiy EmilienM: I got it, thanks, will do
18:19 rahulait joined #puppet-openstack
18:21 EmilienM clayton: https://review.openstack.org/231171 and https://review.openstack.org/231175
18:23 IvanBerezovskiy left #puppet-openstack
18:33 morazi joined #puppet-openstack
18:34 linkedinyou joined #puppet-openstack
18:34 dprince joined #puppet-openstack
18:36 sergmelikyan joined #puppet-openstack
18:38 openstackgerrit Merged openstack/puppet-openstack-integration: Create functions for common bits  https://review.openstack.org/231171
18:40 sergmelikyan joined #puppet-openstack
18:43 openstackgerrit Merged openstack/puppet-openstack-integration: Create install_modules_unit.sh  https://review.openstack.org/231175
18:53 EmilienM mwhahaha: is it on purpose https://review.openstack.org/#/c​/229095/10/manifests/backup/nfs.pp,cm backup_container is not set?
18:54 mwhahaha no idea
18:54 mwhahaha was like that when i got there :D
18:55 mwhahaha it might be safe to move over to service defaults
19:09 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/226830
19:11 EmilienM clayton: https://etherpad.openstack.org/p/HND-puppet
19:24 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Switch cinder to use os_service_default fact  https://review.openstack.org/229095
19:27 khappone joined #puppet-openstack
19:29 jasondotstar gchamoul: ping re: https://bugs.launchpad.net/​puppet-cinder/+bug/1440401
19:29 openstack Launchpad bug 1440401 in puppet-cinder "Missing or incomplete coverage of the rspec unit tests for Types and Providers" [Low,In progress] - Assigned to liaonanhai (nanhai-liao)
19:38 sergmelikyan joined #puppet-openstack
19:39 rahulait joined #puppet-openstack
19:42 _ducttape_ joined #puppet-openstack
19:43 ducttap__ joined #puppet-openstack
19:45 ducttape_ joined #puppet-openstack
19:49 _ducttape_ joined #puppet-openstack
19:55 markvoelker joined #puppet-openstack
20:07 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Switch cinder to use os_service_default fact  https://review.openstack.org/229095
20:09 sergmelikyan joined #puppet-openstack
20:26 dgurtner joined #puppet-openstack
20:33 arnaud_orange joined #puppet-openstack
20:47 off_rhoden joined #puppet-openstack
20:50 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Switch cinder to use os_service_default fact  https://review.openstack.org/229095
20:55 rahulait Hi,
20:57 rahulait I am right now stuck in trying to figure out which parameter I need to change to modify the OS_AUTH_URL from 127.0.0.1 to some IP-address. By default, if I run puppet, the OS_AUTH_URL is set to 127.0.0.1, I want to make it point to my controller's DNS name/IP-address so that I don't have any dependency on 127.0.0.1
20:58 rahulait How does puppet sources the OS_AUTH_URL when it tries to execute any openstack command on the nodes?
21:02 mwhahaha I think it leverages environment variables
21:02 mwhahaha so you could just export that prior to running puppet
21:03 mwhahaha i think it also will try and source the creds from your ~/openrc as well so you could just put those in place
21:04 rahulait Nope, it didn't worked
21:06 rahulait Even without exporting the variables before we run puppet, its able to configure the services, this means its setting them internally...
21:07 rahulait I tried manually setting it to my ip-address, but it again starts pointing to 127.0.0.1
21:13 angdraug joined #puppet-openstack
21:14 rahulait Modifying the keystone/lib/puppet/provider/keystone.rb did the trick....
21:14 mwhahaha you shouldn't need to as there should be a way to do it via a file somewhere
21:15 mwhahaha you'll just run into issues if you need to apply updates
21:15 rahulait yeah
21:15 rahulait I am thinking if there is any way I can add it to hiera.yaml file....
21:15 rahulait not sure which parameter I should set..
21:16 mwhahaha in openstacklib, ./lib/puppet/provider/openstack/auth.rb there's an RCFILENAME
21:17 mwhahaha that's what is supposed to get used
21:17 mwhahaha but i think sometimes there's an issue with that being properly evaluated and you might be running into to that
21:19 rahulait yeah, I don't think we have any openrc file created
21:19 mwhahaha if you create one it'll use that
21:19 mwhahaha that's the cleaner way than modifying the providers
21:20 rahulait oh ok... let me try that
21:23 ducttape_ joined #puppet-openstack
21:29 mfisch joined #puppet-openstack
21:29 mfisch EmilienM: did you have other topics we needed to cover before tokyo?
21:38 mfisch EmilienM: crinkle do you want us to plug your talk during ours when we briefly discuss openstack?
21:45 Lokesh_Jain Can anybody take a look at puppet-nova code review for making libvirt_vif_driver configurable at https://review.openstack.org/#/c/231564/
21:52 mfisch Lokesh_Jain: lookig
21:54 mfisch Lokesh_Jain: a few comments but its a good start
21:55 spredzy_ joined #puppet-openstack
21:55 vinsh joined #puppet-openstack
22:03 crinkle mfisch: sure
22:04 mfisch crinkle: what time? 1:30?
22:05 vinsh joined #puppet-openstack
22:06 crinkle mfisch: yep, on friday
22:06 mfisch cool
22:06 mfisch did we figure out later plans? we'restill  practicing
22:07 crinkle mfisch: i think the summit ends at 5:30, so dinner around 6?
22:08 mfisch sure
22:09 bklei joined #puppet-openstack
22:11 crinkle mfisch: i'm thinking http://www.theeastburn.com/
22:11 crinkle it's a not-too-terrible walk from the conference
22:11 mfisch wfm
22:14 sergmelikyan joined #puppet-openstack
22:16 vinsh joined #puppet-openstack
22:24 crinkle mfisch: actually let's do 7 in case any of the authors stuck in the meet the authors session want to come
22:24 mfisch sure
22:32 EmilienM mfisch: I've look at https://review.openstack.org/#/c/203220/ - it's really good
22:32 EmilienM crinkle: wfm
22:33 markvoelker joined #puppet-openstack
22:38 EmilienM richm: have you seen my comment on https://review.openstack.org/#/c/226624/ ?
22:43 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002: add dashboard service  https://review.openstack.org/224974
22:45 richm EmilienM: You said " I would like to see patches in others modules that Depends-On this patch" ?
22:45 richm What other modules depend on this patch?
22:51 EmilienM richm: all modules using service_identity?
22:53 richm EmilienM: Why do they need to change?
22:53 EmilienM richm: I thought they would have
22:53 EmilienM I'm probably wrong, I'm testing it again now
22:55 openstackgerrit Emilien Macchi proposed openstack/puppet-neutron: Test https://review.openstack.org/#/c/226624/  https://review.openstack.org/232262
23:00 EmilienM crinkle: if you're interested, we could hack on https://github.com/openstack/​puppet-openstack_spec_helper this week
23:00 bklei joined #puppet-openstack
23:01 richm EmilienM: in the puppet-heat case, they could pass in the user as 'heat_admin::heat_stack'
23:02 sanjayu joined #puppet-openstack
23:02 richm EmilienM: once chem gets his code working, they wouldn't even have to do that, just pass in the user_domain
23:02 EmilienM richm: ok
23:02 richm EmilienM: They have to pass in user_domain anyway
23:02 EmilienM richm: do we pass it by default?
23:02 richm $user_domain           = undef,
23:03 richm EmilienM: ^^^
23:04 vinsh joined #puppet-openstack
23:05 EmilienM richm: so by default, the class will fail, right?
23:08 markvoelker joined #puppet-openstack
23:08 richm EmilienM: I'm not sure what you mean
23:09 bklei_ joined #puppet-openstack
23:10 EmilienM richm: I'm just wondering about default behaviors
23:11 EmilienM richm: I suggst we go ahead with that patch and see how it behaves
23:12 EmilienM richm: +2
23:12 richm EmilienM: ack - thanks
23:16 gildub joined #puppet-openstack
23:16 bklei joined #puppet-openstack
23:17 gildub michchap, hi, can you please re-review https://review.openstack.org/226624 ?
23:18 EmilienM gildub: we were discussing about that
23:18 dgurtner joined #puppet-openstack
23:18 gildub EmilienM, great
23:18 dgurtner joined #puppet-openstack
23:18 EmilienM gildub: I would rather ask mfisch reviewing it too
23:18 xarses joined #puppet-openstack
23:18 gildub EmilienM, sure
23:19 EmilienM though michchap's feedback is valuable, I would like feedback from people deploying that stuff already
23:19 xarses joined #puppet-openstack
23:19 EmilienM and i know they do
23:19 gildub mfisch, can you please review https://review.openstack.org/226624 ?
23:20 EmilienM gildub: you going to tokyo?
23:20 EmilienM I'm reading https://etherpad.openstack.org/p/HND-puppet
23:20 gildub EmilienM, nope, not on the second short list somehow, not happy about that
23:21 EmilienM "How to get faster +2 reviews" > Are you seriously saying reviews are slow?
23:21 gildub EmilienM, I think that the number of core reviewer against the number of patches needs to scale
23:22 EmilienM gildub: we are promoting new contributors "core" during every cycle
23:22 gildub EmilienM, or maybe the need to have more core members across time zones
23:23 EmilienM gildub: timezone is not a critical thing here - we're not running critical applications AFIK
23:23 EmilienM gildub: it's not that easy to be core reviewer
23:24 gildub EmilienM, I can imagine ^
23:25 EmilienM gildub: it's an heavy responsability, we rely each others, and that's something people get involved after some time contributing
23:26 EmilienM contributing not only by sending code, but also doing good reviews, understand our best practices, participate to meetings / mailing lists
23:26 EmilienM I mean, I wish we could have more core members
23:27 gildub EmilienM, are you offering me a job? :)
23:28 EmilienM gildub: you can still send an e-mail to ML about that topic
23:28 gildub EmilienM, seriously, I understand it's not an easy task. What I see from my end is that: 1. I have pretty much only michchap
23:29 gildub 2. I have to hassle him, not because is not helping, but because he's the only one in my timezone, and because if we don't ask core reviewers then nothing happens
23:30 EmilienM gildub: nothing's happen?
23:30 gildub EmilienM, therefore the short-circuit in my head => we need more core reviewers, preferably across the globe.
23:30 EmilienM I'm spending 2 hours per day at reviewing code
23:30 EmilienM I'm sure others also spend some time
23:31 gildub EmilienM, sorry I mean, there are more in the plate then needed
23:31 EmilienM what we need is to train our contributors to quickly get involved and become core
23:31 gildub EmilienM, sure
23:31 EmilienM what I'm trying to do with https://wiki.openstack.org/wiki/Puppet
23:40 frozenfoxx joined #puppet-openstack
23:40 mfisch gildub: yeah
23:40 mfisch man I forgot about humidity
23:40 mfisch but extra O2 is a blessing
23:41 mfisch gildub: oh yeah this one
23:41 mfisch I'm not happy about breaking backwards compat which was agreed to
23:41 mfisch and I dont think that Richard's email changes that by fiat
23:41 mfisch it certainly wasnt the plan
23:42 gildub mfisch, well, delivering broken keystone/domain feature was probably not the plan either
23:43 markvoelker joined #puppet-openstack
23:44 _ducttape_ joined #puppet-openstack
23:44 gildub mfisch, this is blocking from moving forward with domains other features, trusts, groups, etc
23:47 EmilienM mfisch: the first implementation was not perfect, we are adjusting it
23:48 openstackgerrit Vasyl Saienko proposed openstack/puppet-keystone: refactor keystone_user, keystone_user_role providers  https://review.openstack.org/213906
23:51 EmilienM gildub: I dropped +2 just because I want to recheck, since now we have integration jobs testing everything
23:51 EmilienM mfisch: I think it's kind of fair if our users are aware of that
23:51 EmilienM I would expect some documentation from gildub and richm about that, but richm already did a good job on the ML to inform our users
23:51 gildub EmilienM, ok
23:54 bklei joined #puppet-openstack

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