Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-05-21

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

All times shown according to UTC.

Time Nick Message
00:11 gildub joined #puppet-openstack
00:17 britthouser joined #puppet-openstack
00:17 sbadia \o/
00:17 openstackgerrit Matthew Kirby proposed stackforge/puppet-nova: Enable nova service by default  https://review.openstack.org/184656
00:19 openstackgerrit Cody Herriges proposed stackforge/puppet-neutron: Make tests pass on Puppet 4.x  https://review.openstack.org/184646
00:40 ferest joined #puppet-openstack
00:48 ducttape_ joined #puppet-openstack
00:56 ducttape_ joined #puppet-openstack
01:06 gildub richm, ok, I fixed service token, you're right this was not working, I had keystone_request taken out!
01:07 richm gildub: ok
01:07 gildub richm, thanks!
01:07 richm gildub: sure - just trying to get this code working as soon as possible
01:08 gildub richm, absolutely
01:09 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: keystone_service resource using api Version 3  https://review.openstack.org/184195
01:09 gildub richm, ^ the packstack test should pass
01:10 gildub richm, now looking at inifile issue for v2
01:10 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: Fix file ressource (need an octale for mode param)  https://review.openstack.org/184666
01:10 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: proxy/cache: Fix logic for swift::proxy::cache  https://review.openstack.org/184667
01:10 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: Fix spec tests for RSpec 3.x and Puppet 4.x  https://review.openstack.org/184668
01:13 simon-AS559 joined #puppet-openstack
01:13 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: Refined authentication sources  https://review.openstack.org/181299
01:13 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: keystone_service resource using api Version 3  https://review.openstack.org/184195
01:14 gildub richm, sorry I had to rebase
01:17 gildub richm, something's wrong with the patch set
01:18 simon-AS559 joined #puppet-openstack
01:23 simon-AS5591 joined #puppet-openstack
01:24 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: Refined authentication sources  https://review.openstack.org/181299
01:28 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: Refined authentication sources  https://review.openstack.org/181299
01:29 gildub richm, ok, fixed, feel free to test
01:30 gildub imcsk8, are you there?
01:35 gildub richm, re inifile - per your comments it confirms me we're gonna drop it for now.
01:38 richm gildub: ok
01:39 richm gildub: shouldn't https://review.openstack.org/#/c/18​4195/3/lib/puppet/provider/keystone.rb be in https://review.openstack.org/#/c/181299 ?
01:40 richm gildub: ah, nevermind - just got the notice
01:41 simon-AS559 joined #puppet-openstack
01:45 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: keystone_service resource using api Version 3  https://review.openstack.org/184195
01:52 richm gildub: still not working
01:53 gildub richm, do you get the same message about lack of credentials?
01:53 richm looks like something is putting a string with a single space " " at the end of the command
01:53 richm it says "error: unrecognized arguments"
01:54 richm gildub: so looks like the credentials are correct, but something else is wrong
01:54 gildub richm, hmm
01:55 gildub richm, do you have the trace/log?
01:55 richm gildub: copying now
01:56 gildub richm, this is obviously an openstack (OSC) error
01:56 gildub richm, ok
01:57 richm gildub: http://paste.openstack.org/show/229965/
01:57 richm look at line 4 - there is a space between set and the single quote
01:57 richm line 7 - same thing - space between csv and '
01:58 richm If I do this: openstack role list --quiet --format csv " "
01:58 richm I get the same error
01:58 gildub richm, ok
02:01 gildub richm, checking the code
02:02 gildub richm, I get same error when testing keystone_role
02:03 richm gildub:  I think this is the problem
02:04 richm list = request('role', 'list', nil)
02:04 gildub richm, yeah find it too
02:04 richm I think it should just be
02:04 richm list = request('role', 'list')
02:04 gildub richm, remove the nil
02:04 richm yes
02:04 gildub richm, that's it
02:04 richm also in the output above - line 4
02:04 richm project set with no arguments?
02:05 gildub richm, saw that too
02:05 richm I guess openstack just ignores that, so no error, but looks wrong
02:05 gildub richm, that's the tenant - yes it's wrong
02:06 gildub richm, I tested that, but request propagation has changed, need to retest
02:06 richm ok
02:12 gildub richm, although I'm not sure where that 'project set' is coming from, it seems to be the tenant, but I check the resource_tenant works fine including update (set)
02:13 richm yeah - I'm not sure how @property_flush could be set
02:14 gildub richm, also there is no error message about it in the above log propagated
02:15 richm right - openstack must ignore it - it's probably a no-op
02:16 richm user set has the same problem in the log
02:17 gildub richm, @property_flush is set up in the instance method and get populate when properties change
02:18 gildub ^populated
02:18 richm right - but in a new install with empty database, I would not expect @property_flush to be set
02:19 richm gildub:  problem with user role list too
02:19 richm http://paste.openstack.org/show/229986/
02:20 richm gildub: error: unrecognized arguments: --user
02:20 gildub richm, I get that line in debug mode
02:21 gildub Debug: Executing '/usr/bin/openstack project set '
02:21 richm I always run packstack with debug mode on puppet
02:21 gildub richm, that's fine, it's not an error
02:22 richm gildub: I only see it with project set and user set - with empty string
02:24 gildub richm, right
02:24 gildub richm, investigating...
02:30 gildub richm, definitely related to flush, but create and destroy shouldn't be kicking the set part
02:44 gildub richm, ok fixed it! Had to add a test to verify when property_flush is empty to not operate
02:44 richm ok
02:51 gildub richm, good find though! The usage of flush was already there, before patch, but it's not consistent across keystone providers, I've tried to realign it a bit!
02:51 richm ok
02:52 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: Refined authentication sources  https://review.openstack.org/181299
02:53 gildub richm, can you please give it a go against your packstack test set?
02:55 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: keystone_service resource using api Version 3  https://review.openstack.org/184195
02:55 richm gildub: running
02:55 gildub richm, thanks
02:58 richm gildub: still need to fix user role list
02:59 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-openstacklib: Refined source of authentication information  https://review.openstack.org/180407
03:03 gildub richm, you mentioned that earlier, but what do you get?
03:05 richm I get the usage error
03:06 gildub richm, of course! Darn!
03:07 richm two places
03:09 gildub richm, yeah, I know, I just skipped it :(
03:11 richm ok - with my local fixes for those two issues, keystone installs!
03:12 richm and cinder, glance, and nova_api so far
03:14 gildub richm, that's good, thanks for that
03:14 richm nova is good
03:15 richm ok - gotta go - a demain
03:15 gildub richm, I found a small issue in the user provider, the field in self.instances method must be project and not tenant!
03:18 gildub richm, a demain
03:19 simon-AS559 joined #puppet-openstack
03:22 panda joined #puppet-openstack
03:34 simon-AS559 joined #puppet-openstack
03:40 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: Refined authentication sources  https://review.openstack.org/181299
03:41 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: keystone_service resource using api Version 3  https://review.openstack.org/184195
03:43 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: Refined authentication sources  https://review.openstack.org/181299
03:43 openstackgerrit Gilles Dubreuil proposed stackforge/puppet-keystone: keystone_service resource using api Version 3  https://review.openstack.org/184195
04:34 saneax joined #puppet-openstack
04:44 sks joined #puppet-openstack
05:20 simon-AS559 joined #puppet-openstack
05:36 gildub imcsk8, hi
06:12 tfz joined #puppet-openstack
06:15 openstackgerrit Merged stackforge/puppet-cinder: Synchronize LICENSE file with OpenStack projects  https://review.openstack.org/184354
06:22 fedexo joined #puppet-openstack
06:39 jtomasek joined #puppet-openstack
06:43 tdb joined #puppet-openstack
06:43 aarefiev joined #puppet-openstack
06:43 plopix joined #puppet-openstack
06:43 fvollero joined #puppet-openstack
06:44 dachary joined #puppet-openstack
06:52 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: Fix file resource (need an octal for mode param)  https://review.openstack.org/184666
06:52 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: proxy/cache: Fix logic for swift::proxy::cache  https://review.openstack.org/184667
06:52 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: Fix spec tests for RSpec 3.x and Puppet 4.x  https://review.openstack.org/184668
06:52 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: Disable lint check about quoted_booleans  https://review.openstack.org/184723
07:21 panda joined #puppet-openstack
07:24 jistr joined #puppet-openstack
07:26 saneax joined #puppet-openstack
07:31 paramite joined #puppet-openstack
07:42 dachary joined #puppet-openstack
07:48 aimon joined #puppet-openstack
07:50 jpena joined #puppet-openstack
08:01 gfidente joined #puppet-openstack
08:02 aimon joined #puppet-openstack
08:21 aimon joined #puppet-openstack
08:39 openstackgerrit Martin Mágr proposed stackforge/puppet-neutron: Bring Redhat support to acceptance tests  https://review.openstack.org/183610
09:04 stamak joined #puppet-openstack
09:22 saneax joined #puppet-openstack
09:38 openstackgerrit Lukas Bezdicka proposed stackforge/puppet-trove: Sync amqp support with other openstack modules  https://review.openstack.org/169808
10:17 openstackgerrit Martin Mágr proposed stackforge/puppet-neutron: Bring Redhat support to acceptance tests  https://review.openstack.org/183610
10:51 simon-AS559 joined #puppet-openstack
11:03 simon-AS559 joined #puppet-openstack
11:21 panda joined #puppet-openstack
11:48 larsks joined #puppet-openstack
11:55 saneax joined #puppet-openstack
12:17 richm joined #puppet-openstack
12:18 openstackgerrit Martin Mágr proposed stackforge/puppet-sahara: Fixed /etc/sahara handling  https://review.openstack.org/184261
12:18 openstackgerrit Martin Mágr proposed stackforge/puppet-sahara: Beaker: install APT repo with openstack_extras  https://review.openstack.org/184054
12:18 openstackgerrit Martin Mágr proposed stackforge/puppet-sahara: Bring Redhat support to acceptance tests  https://review.openstack.org/184055
12:29 simon-AS559 joined #puppet-openstack
12:43 openstackgerrit Martin Mágr proposed stackforge/puppet-swift: Bring Redhat support to acceptance tests  https://review.openstack.org/182621
12:43 openstackgerrit Martin Mágr proposed stackforge/puppet-swift: Beaker: install APT repo with openstack_extras  https://review.openstack.org/184016
12:57 openstackgerrit Martin Mágr proposed stackforge/puppet-trove: CentOS 7 compatibility  https://review.openstack.org/184464
12:57 openstackgerrit Martin Mágr proposed stackforge/puppet-trove: Beaker: install APT repo with openstack_extras  https://review.openstack.org/184246
12:57 openstackgerrit Martin Mágr proposed stackforge/puppet-trove: Bring Redhat support to acceptance tests  https://review.openstack.org/184247
13:17 ducttape_ joined #puppet-openstack
13:50 ferest joined #puppet-openstack
13:55 tfz joined #puppet-openstack
14:11 openstackgerrit Martin Mágr proposed stackforge/puppet-trove: CentOS 7 compatibility  https://review.openstack.org/184464
14:11 openstackgerrit Martin Mágr proposed stackforge/puppet-trove: Beaker: install APT repo with openstack_extras  https://review.openstack.org/184246
14:11 openstackgerrit Martin Mágr proposed stackforge/puppet-trove: Bring Redhat support to acceptance tests  https://review.openstack.org/184247
14:12 tfz joined #puppet-openstack
14:20 kriskrishna__ joined #puppet-openstack
14:22 ducttape_ joined #puppet-openstack
14:22 ducttape_ joined #puppet-openstack
14:25 openstackgerrit Giulio Fidente proposed stackforge/puppet-nova: Add parameter to optionally use rbd for the ephemeral storage  https://review.openstack.org/119093
14:26 chem joined #puppet-openstack
14:27 saneax joined #puppet-openstack
14:31 openstackgerrit Martin Mágr proposed stackforge/puppet-sahara: Fixed /etc/sahara handling  https://review.openstack.org/184261
14:31 openstackgerrit Martin Mágr proposed stackforge/puppet-sahara: Beaker: install APT repo with openstack_extras  https://review.openstack.org/184054
14:31 openstackgerrit Martin Mágr proposed stackforge/puppet-sahara: Bring Redhat support to acceptance tests  https://review.openstack.org/184055
14:43 rcallawa joined #puppet-openstack
14:52 openstackgerrit Martin Mágr proposed stackforge/puppet-trove: CentOS 7 compatibility  https://review.openstack.org/184464
14:52 openstackgerrit Martin Mágr proposed stackforge/puppet-trove: Beaker: install APT repo with openstack_extras  https://review.openstack.org/184246
14:52 openstackgerrit Martin Mágr proposed stackforge/puppet-trove: Bring Redhat support to acceptance tests  https://review.openstack.org/184247
14:57 openstackgerrit sac proposed stackforge/puppet-manila: Add manifests for GlusterFS backend.  https://review.openstack.org/184452
15:08 ferest joined #puppet-openstack
15:11 jistr joined #puppet-openstack
15:18 rcallawa joined #puppet-openstack
15:20 openstackgerrit sac proposed stackforge/puppet-manila: Add manifests for GlusterFS backend.  https://review.openstack.org/184452
15:22 panda joined #puppet-openstack
15:22 rcallawa joined #puppet-openstack
15:37 britthouser joined #puppet-openstack
15:38 openstackgerrit Arne Wiebalck proposed stackforge/puppet-cinder: NetApp: use $name for configuration group name  https://review.openstack.org/184804
15:39 britthou_ joined #puppet-openstack
15:39 ferest joined #puppet-openstack
15:42 openstackgerrit sac proposed stackforge/puppet-manila: Add manifests for GlusterFS backend.  https://review.openstack.org/184452
15:52 arnaud_orange joined #puppet-openstack
16:01 openstackgerrit sac proposed stackforge/puppet-manila: Add manifests for GlusterFS backend.  https://review.openstack.org/184452
16:07 simon-AS559 joined #puppet-openstack
16:13 rhefner joined #puppet-openstack
16:20 cwolferh joined #puppet-openstack
16:22 dalgaaf joined #puppet-openstack
16:33 __nick joined #puppet-openstack
16:40 sks joined #puppet-openstack
16:44 jpena joined #puppet-openstack
16:52 britthouser joined #puppet-openstack
16:54 ducttape_ joined #puppet-openstack
17:00 mdorman joined #puppet-openstack
17:14 aimon joined #puppet-openstack
17:16 ferest joined #puppet-openstack
17:24 openstackgerrit Richard Megginson proposed stackforge/puppet-openstacklib: Refined source of authentication information  https://review.openstack.org/180407
17:24 aimon joined #puppet-openstack
17:38 aimon joined #puppet-openstack
17:39 rcallawa joined #puppet-openstack
17:41 openstackgerrit Richard Megginson proposed stackforge/puppet-openstacklib: Refined source of authentication information  https://review.openstack.org/180407
17:42 rcallawa_ joined #puppet-openstack
17:44 britthouser joined #puppet-openstack
18:30 openstackgerrit Arne Wiebalck proposed stackforge/puppet-cinder: NetApp: use $name for configuration group name  https://review.openstack.org/184804
18:40 ferest joined #puppet-openstack
18:41 ducttape_ joined #puppet-openstack
18:47 arnaud_orange joined #puppet-openstack
18:55 aimon joined #puppet-openstack
18:59 morazi joined #puppet-openstack
19:02 aimon joined #puppet-openstack
19:07 aimon joined #puppet-openstack
19:19 openstackgerrit Arne Wiebalck proposed stackforge/puppet-cinder: NetApp: use $name for configuration group name  https://review.openstack.org/184804
19:22 panda joined #puppet-openstack
19:26 danieru_ joined #puppet-openstack
19:27 Mosibi_ joined #puppet-openstack
19:32 nigelk joined #puppet-openstack
19:34 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: Fix spec tests for RSpec 3.x and Puppet 4.x  https://review.openstack.org/184668
19:55 iurygregory left #puppet-openstack
20:05 openstackgerrit Arne Wiebalck proposed stackforge/puppet-cinder: NetApp: use $name for configuration group name  https://review.openstack.org/184804
20:13 openstackgerrit Ivan Chavero proposed stackforge/puppet-neutron: Add support for keystone v3 API  https://review.openstack.org/184856
20:29 gildub joined #puppet-openstack
20:31 gildub richm, Hi - I just saw you comment about ENV precedence, I'd say it's the same issue whether in puppet or not, I don't think we should step into that
20:33 gildub richm, say you uploaded a patch set, I haven't done diff yet (just waking up), what's about?
20:35 marvin0815 joined #puppet-openstack
20:53 gildub richm, ok thanks for removing the inifile part and specs
21:03 richm gildub: yeah, it was just a couple of minor bug fixes - the tricky part was the spec file, having to create a class to test the module code
21:03 gildub richm, I'm looking at it now, great
21:04 simon-AS559 joined #puppet-openstack
21:08 arnaud_orange joined #puppet-openstack
21:11 dalgaaf joined #puppet-openstack
21:14 gildub crinkle, could you please review bp/auth-consolidation?
21:14 richm gildub: It's probably going to be hard to get the +2 guys to review code this week since they are all in Vancouver
21:18 gildub richm, ah yeah, true
21:19 gildub richm, when are you back?
21:19 gildub imcsk8, ping
21:20 richm gildub: I'll be back June 1
21:20 britthouser joined #puppet-openstack
21:22 gildub richm, ok
21:47 gildub richm, per your comment on set 26, I don't see an issue with OS_* overlapping each others
21:48 gildub richm, with or without puppet this is something that can happen
21:48 EmilienM richm: which patch do you need review now?
21:51 richm gildub: The issue I have is - this assumes the operators expect puppet to behave exactly like the openstackclient - that is, we are not hiding the implementation _at all_ - we are tying forever the implementation of puppet with the implementation of openstackclient
21:53 richm But, if that's ok, all of the operators expect puppet will always behave with the same semantics as openstackclient, then I guess that's just the way it is
21:55 richm EmilienM: https://review.openstack.org/#/c/180407/ and https://review.openstack.org/#/c/181299/
21:57 ducttape_ joined #puppet-openstack
22:02 wabu joined #puppet-openstack
22:05 britthouser joined #puppet-openstack
22:17 richm when is prefetch called on a resource?
22:25 richm never mind - http://garylarizza.com/blog/2013/12/15/​seriously-what-is-this-provider-doing/
22:31 aimon joined #puppet-openstack
22:39 cwolferh joined #puppet-openstack
22:44 openstackgerrit Sebastien Badia proposed stackforge/puppet-modulesync-configs: rake: Add a rake task to easily run acceptance tests  https://review.openstack.org/184894
22:45 simon-AS559 joined #puppet-openstack
22:55 britthouser joined #puppet-openstack
23:01 arnaud_orange joined #puppet-openstack
23:15 saneax joined #puppet-openstack
23:17 aimon joined #puppet-openstack
23:21 panda joined #puppet-openstack
23:44 britthouser joined #puppet-openstack
23:47 gildub richm, what are the choices we have between:
23:48 gildub 1. Pass the credentials to the ENV - This is an atomic operation thanks to withenv
23:49 gildub 2. Pass the credentials to openstack client via cli interface - --os-*
23:49 gildub richm, ^
23:50 richm are those my only two choices?
23:50 gildub richm, we have opt out 2. because insecure
23:50 gildub richm, you tell me what's 3 then?
23:50 richm then there is no choice - only 1
23:51 gildub richm, well I'm happy to use 3. but what that could be?
23:51 gildub richm, well, ATM
23:51 richm pass the credentials via ENV from Puppet::Provider::Openstack.request to /usr/bin/openstack
23:51 richm yes, that is option 1, and that's the only secure way afaik
23:52 richm note that doesn't mean we have to use ENV to pass the credentials from higher level ruby/puppet code into Puppet::Provider::Openstack.request
23:53 gildub richm, as someone suggested, having some sort of AAA, like puppet talking to IPA or Kerberos
23:53 richm that's what v3 will bring us - the foundation for being able to use a keytab for auth or certs or federation or whatever
23:53 gildub richm, I don't understand ^^^
23:54 richm gildub: https://review.openstack.org/#/c/181299/17/​lib/puppet/provider/keystone_user/openstack.rb - line 114
23:55 richm the keystone_user password code has to know that Puppet::Provider::Openstack.request uses ENV
23:56 richm gildub: and, please see my last comment at https://review.openstack.org/#/c/181299/
23:57 gildub richm, I did ^
23:58 richm again, I do not object to passing the auth credentials from the puppet process to the openstack process using ENV
23:59 gildub richm, ok
23:59 richm I do object to passing the auth credentials from keystone_user/openstack.rb password to Puppet::Provider::Openstack.request using ENV
23:59 richm however, if everyone thinks that's ok, then it's ok by me

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