Friday, 2016-02-19

« Thursday, 2016-02-18 Index Saturday, 2016-02-20 »
richmayoung: you should be able to exercise self.instances with puppet resource00:01
*** johnmilton has joined #puppet-openstack00:01
ayoungrichm, got the syntax for that?00:02
richmpuppet resource --help00:03
richmayoung: it's been too long since I did this, and I blew away my test VMs :P00:05
richmI think the argument to modulepath is the _parent_ directory, and the directory name must be 'keystone' not 'puppet-keystone'00:06
richmThat is, if you have --modulepath=/some/path00:06
richmthen /some/path must contain 'keystone', 'openstacklib', etc. etc.00:06
richmyou can have multiple modulepath00:06
richmyou can either use --modulepath=/usr/share/openstack-puppet/module:/path/to/my/local/one00:07
richmor use --modulepath=/usr/share/openstack-puppet/module --modulepath /path/to/my/local/one00:07
richmI can't remember which00:07
richmthen - puppet resource --modulepath=something keystone_federation_protocol00:08
richmand00:08
richmpuppet resource --modulepath=something keystone_federation_protocol somename00:08
richmthat should exercise self.instances00:08
richmor maybe it's capital K - Keystone_federation_protocol00:09
*** alee has joined #puppet-openstack00:13
openstackgerritMerged openstack/puppet-nova: Properly document deprecated parameter  https://review.openstack.org/28153800:15
*** nlahouti1 has joined #puppet-openstack00:28
*** nlahouti has quit IRC00:28
ayoungrichm, was not able to find an incantation that worked00:32
ayoungI have the original code in /etc/puppet00:32
ayoungso00:32
ayoung puppet resource    keystone_federation_protocol00:32
ayoungshould work but gets Error: Could not run: Could not find type keystone_federation_protocol00:32
richmayoung: try puppet resource keystone_user00:33
richmthen try puppet resource Keystone_user00:34
ayoungnope for all00:34
ayoungrichm, lets cut it here for now.00:34
ayoungToo much trial and error, not enough progress to warrant00:35
ayoungI'm going to look in to EmilienM 's recommendation00:35
richmhttp://specs.openstack.org/openstack/puppet-openstack-specs/specs/kilo/auth-restructure.html00:35
richmpuppet resource keystone_user foo email=foo@example.com password=test”00:35
ayounghttp://my1.fr/blog/puppet-module-functional-testing-with-vagrant-openstack-and-beaker/00:35
richmayoung: what do you have under /etc/puppet ?00:35
ayoungrichm, pretty much everything:00:36
ayoungrichm, http://paste.openstack.org/show/487517/00:36
ayoungand a lot below each of those:  find is acting wonky here00:36
richmI guess the modules were moved from /usr/share/openstack-puppet . . .00:37
ayounghttp://paste.openstack.org/show/487518/00:37
ayoungrichm, I think that happens when they deploy the controller.  They are not intalled via RPM I think00:38
richmok00:38
ayoungrichm, it might be a tripleoism...I see00:38
ayoung/etc/puppet/modules/keystone/.git00:38
richmIf puppet resource keystone_user is not working . . . then I don't know00:38
richmare you logged in as root?00:39
richmand is there an /etc/keystone/keystone.conf with admin_token set?00:39
richmotherwise, I don't know what credentials it is using00:39
*** johnmilton has quit IRC00:40
*** gildub has quit IRC00:40
*** gildub has joined #puppet-openstack00:40
openstackgerritJames Slagle proposed openstack/puppet-cinder: Revert "Allow customization of DEFAULT/host"  https://review.openstack.org/28209800:40
richmpuppet resource is the fastest, easiest way I know to test self.instances00:41
*** xarses has quit IRC00:41
openstackgerritJames Slagle proposed openstack/puppet-cinder: Revert "Allow customization of DEFAULT/host"  https://review.openstack.org/28209800:42
*** xarses has joined #puppet-openstack01:06
*** ducttape_ has quit IRC01:06
*** social has quit IRC01:08
*** social has joined #puppet-openstack01:08
*** johnmilton has joined #puppet-openstack01:12
*** cwolferh has quit IRC01:34
*** cwolferh has joined #puppet-openstack01:35
*** nlahouti1 has quit IRC01:39
*** cwolferh has quit IRC01:42
*** cwolferh has joined #puppet-openstack01:42
*** rahulait has joined #puppet-openstack01:44
openstackgerritMatt Fischer proposed openstack/puppet-keystone: Remove deprecated tenant_ LDAP parameters  https://review.openstack.org/28207301:50
*** kindjal has joined #puppet-openstack01:53
*** morazi has quit IRC01:55
*** rstarmer has joined #puppet-openstack02:01
*** cwolferh has quit IRC02:02
*** johnmilton has quit IRC02:20
*** markvoelker_ has quit IRC02:34
*** ducttape_ has joined #puppet-openstack02:34
*** ilbot3 has quit IRC02:45
*** ilbot3 has joined #puppet-openstack02:48
openstackgerritXingchao Yu proposed openstack/puppet-glance: Use stores to replace known_stores  https://review.openstack.org/26584602:51
openstackgerritXingchao Yu proposed openstack/puppet-oslo: Add define oslo::messaging::amqp  https://review.openstack.org/27476402:54
*** alee has quit IRC03:06
*** kindjal has quit IRC03:09
*** ducttape_ has quit IRC03:12
*** alee has joined #puppet-openstack03:19
*** ducttape_ has joined #puppet-openstack03:20
*** ducttape_ has quit IRC03:21
*** nlahouti has joined #puppet-openstack03:30
*** richm has quit IRC03:33
*** nlahouti has quit IRC03:33
*** nlahouti has joined #puppet-openstack03:33
*** ducttape_ has joined #puppet-openstack03:34
*** markvoelker has joined #puppet-openstack03:34
*** markvoelker has quit IRC03:39
*** v1k0d3n has quit IRC03:41
*** v1k0d3n_ has joined #puppet-openstack03:41
*** ducttape_ has quit IRC03:51
*** nlahouti1 has joined #puppet-openstack03:55
*** nlahouti has quit IRC03:57
*** cwolferh has joined #puppet-openstack03:58
*** rahulait has quit IRC04:18
openstackgerritMerged openstack/puppet-sahara: Add the capability to configure api-paste.ini with config.pp  https://review.openstack.org/28175604:22
*** saneax_AFK is now known as saneax04:25
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/25454804:26
openstackgerritAlex Schultz proposed openstack/puppet-zaqar: Switch zaqar to use rspec-puppet-facts  https://review.openstack.org/28216504:35
*** markvoelker has joined #puppet-openstack05:36
openstackgerritMerged openstack/puppet-vswitch: Remove duplicate loading of constants  https://review.openstack.org/28093705:36
*** rcernin has joined #puppet-openstack05:37
*** markvoelker has quit IRC05:40
openstackgerritAlex Schultz proposed openstack/puppet-glance: Switch glance to rspec-puppet-facts  https://review.openstack.org/28218206:00
*** rstarmer has quit IRC06:12
*** jaosorior has joined #puppet-openstack06:48
*** gildub has quit IRC06:58
*** jtomasek has joined #puppet-openstack07:02
*** nlahouti1 has quit IRC07:09
*** markvoelker has joined #puppet-openstack07:36
*** markvoelker has quit IRC07:40
*** andybotting has joined #puppet-openstack07:55
*** dgurtner has joined #puppet-openstack08:02
*** dgurtner has quit IRC08:02
*** dgurtner has joined #puppet-openstack08:02
*** foxhunt_away is now known as foxhunt7208:23
*** jpena has joined #puppet-openstack08:37
*** blkperl has quit IRC08:40
*** Hunner has quit IRC08:40
*** mentat has quit IRC08:40
*** bmjen has quit IRC08:40
*** davideagle_ has joined #puppet-openstack08:52
*** gfidente has joined #puppet-openstack09:09
openstackgerritSergey Kolekonov proposed openstack/puppet-neutron: Add support for modern auth to neutron providers  https://review.openstack.org/28122709:12
*** gildub has joined #puppet-openstack09:12
openstackgerritGiulio Fidente proposed openstack/puppet-cinder: Allow customization of DEFAULT/host  https://review.openstack.org/28199809:16
*** derekh has joined #puppet-openstack09:18
*** furlongm has quit IRC09:21
*** hjensas has joined #puppet-openstack09:35
*** markvoelker has joined #puppet-openstack09:37
*** markvoelker has quit IRC09:42
*** davideagle_ has quit IRC10:01
*** _degorenko|afk is now known as degorenko10:07
*** davideagle_ has joined #puppet-openstack10:09
openstackgerritIvan Berezovskiy proposed openstack/puppet-ceilometer: Deprecate ceilometer alarm classes  https://review.openstack.org/28225810:09
*** andybotting has quit IRC10:24
degorenkosbadia, spredzy hey, are you around?10:36
sbadiayup!10:37
sbadiahi here o/10:37
degorenkosbadia, hey10:37
degorenkowe have this one https://review.openstack.org/28209810:37
degorenkotake a look on my comments10:37
sbadiaack, /me take a look10:37
degorenkoit is related to tripleo10:37
sbadiahéhé, I think you're right :) I'll ping Derek10:43
*** gildub has quit IRC10:44
degorenkosbadia, thanks o/10:45
sbadianp ;)10:45
openstackgerritJerzy Mikolajczak proposed openstack/puppet-neutron: Remove API configuration values from metadata agent  https://review.openstack.org/28227610:59
*** rmart04 has joined #puppet-openstack11:14
*** Kvisle has joined #puppet-openstack11:15
Kvisledoes anyone have any experience with the quality of the puppet modules on el7? (centos in this case)11:15
openstackgerritLukas Bezdicka proposed openstack/puppet-nova: Supports IPv6 in vnc proxy.  https://review.openstack.org/28134611:22
*** linkmark has joined #puppet-openstack11:27
*** rain has joined #puppet-openstack11:38
*** rain is now known as leanderthal11:38
*** markvoelker has joined #puppet-openstack11:38
*** markvoelker has quit IRC11:42
*** karimb has joined #puppet-openstack11:45
openstackgerritJerzy Mikolajczak proposed openstack/puppet-openstack-integration: Remove auth_password from neutron metadata agent  https://review.openstack.org/28229411:47
*** karimb has quit IRC11:57
*** davideagle_ has quit IRC11:59
*** davideagle_ has joined #puppet-openstack12:13
*** rmart04 has quit IRC12:18
*** cdelatte has joined #puppet-openstack12:22
openstackgerritSergey Kolekonov proposed openstack/puppet-neutron: Add support for modern auth to neutron providers  https://review.openstack.org/28122712:24
openstackgerritSergey Kolekonov proposed openstack/puppet-neutron: Set v3 auth related options by default  https://review.openstack.org/28230912:24
*** davideagle_ has quit IRC12:27
*** davideagle_ has joined #puppet-openstack12:34
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/25454812:39
*** jaosorior has quit IRC12:40
*** jaosorior has joined #puppet-openstack12:41
*** jpena is now known as jpena|lunch12:44
*** jaosorior has quit IRC12:46
*** jaosorior has joined #puppet-openstack12:46
*** johnmilton has joined #puppet-openstack12:46
*** markvoelker has joined #puppet-openstack12:54
*** markvoelker has quit IRC12:58
*** markvoelker has joined #puppet-openstack13:04
*** jayg|g0n3 is now known as jayg13:21
*** 32NAABCSV has joined #puppet-openstack13:23
*** 32NAABCSV has quit IRC13:38
skolekonovLooks like Trove deployment is broken now in integration jobs, db-sync fails. Does someone know the root cause?13:40
*** davideagle_ has quit IRC13:42
*** kindjal has joined #puppet-openstack13:46
*** davideagle_ has joined #puppet-openstack13:53
*** dgurtner has quit IRC13:57
*** jpena|lunch is now known as jpena14:09
openstackgerritDougal Matthews proposed openstack/puppet-ironic: Change Baremetal to Bare Metal to be consistent with Ironic.  https://review.openstack.org/28235014:14
*** dalees has quit IRC14:17
*** hjensas has quit IRC14:37
*** saneax is now known as saneax_AFK14:38
*** dfisher has joined #puppet-openstack14:39
*** openstackgerrit_ has quit IRC14:43
*** bmjen has joined #puppet-openstack14:45
*** Hunner has joined #puppet-openstack14:45
*** Hunner has quit IRC14:45
*** Hunner has joined #puppet-openstack14:45
*** mentat has joined #puppet-openstack14:45
*** richm has joined #puppet-openstack14:54
*** ducttape_ has joined #puppet-openstack15:00
*** openstackgerrit_ has joined #puppet-openstack15:00
*** linkmark has quit IRC15:02
*** davideagle_ has quit IRC15:12
*** xarses has quit IRC15:12
*** v1k0d3n_ has quit IRC15:12
*** davideagle_ has joined #puppet-openstack15:14
*** v1k0d3n has joined #puppet-openstack15:15
*** davideagle_ has quit IRC15:15
*** hjensas has joined #puppet-openstack15:16
*** hjensas has joined #puppet-openstack15:16
openstackgerritKeith Schincke proposed openstack/puppet-ceph: Add support for civetweb  https://review.openstack.org/25454815:20
*** baker has joined #puppet-openstack15:26
mwhahahalooks like glance beaker tests have a dependency cycle somehow, http://logs.openstack.org/46/265846/3/check/gate-puppet-glance-puppet-beaker-rspec-dsvm-centos7/23a1596/console.html#_2016-02-19_03_09_05_47415:34
mwhahahaand http://logs.openstack.org/82/282182/1/check/gate-puppet-glance-puppet-beaker-rspec-dsvm-trusty/24f6703/console.html15:34
*** xarses has joined #puppet-openstack15:35
mfischdegorenko: can you look at this?  https://review.openstack.org/#/c/282073/  its not as bad as it looks15:40
mfischpromise15:40
mfischoh15:40
mfischnm I see you commented15:40
degorenkomfisch, :)15:40
*** jaosorior has quit IRC15:44
*** mdorman has joined #puppet-openstack15:52
*** dalees has joined #puppet-openstack16:08
*** leanderthal is now known as leanderthal|afk16:10
*** xarses has quit IRC16:25
mfischlong term I'm going to move fernet support and pki support to separate classes16:25
mfischkeystone's init pp is a hot mess of crap16:25
*** xarses has joined #puppet-openstack16:26
mfischdegorenko: thoughts ^?16:26
degorenkomfisch, good idea16:26
mfischI'll do it after I land this dep class stuff16:26
degorenkodo not forget to use include for init.pp for backward compatiblity16:26
mfischI might have a preview of it today hopefully16:27
mfischyep16:27
degorenkoyou can add me sure16:27
mfischwhen I get a preview of the dep class I'll ping you16:27
degorenkomfisch, we have non-working days in russia 22-23 feb, i will be unavailable here in irc, but will check review sometime16:27
*** ddmitriev has joined #puppet-openstack16:28
mfisch"Defender of the Fatherland Day"16:28
mfischlooks like a good holiday for guys16:28
degorenko:D yep16:29
mfischright after valentines too16:29
degorenkoand then we will have international women day 8 march16:29
mfischspecial forces day sounds interesting too16:29
mfischthis website could be making stuff up too16:30
degorenkomfisch, we have a loooooot of some days, like day of fool :)16:31
mfischapril 1?16:32
*** foxhunt72 is now known as foxhunt_away16:32
degorenkomfisch, yep :)16:34
mfischyeah we have that one too16:35
openstackgerritMiles Gould proposed openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/27618716:37
richmayoung: how goes the keystone federation stuff?16:45
*** blkperl has joined #puppet-openstack16:51
*** duritong has quit IRC16:52
*** duritong has joined #puppet-openstack16:53
ayoungrichm, I bumped it down in priority until after aI get a manual setup working16:57
ayoungrichm, but I think I know what is going on16:57
*** dhollinger has joined #puppet-openstack16:57
ayoungthere is some sort of spec that says list can have only a few params, and by adding something, we go beyond what the spec allows.16:57
ayoungI think it is this:16:58
*** rcernin has quit IRC16:58
ayoungdmnit lost the link...one sec16:58
*** degorenko is now known as _degorenko|afk16:59
ayoungrichm, https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack.rb#L43  calls the openstack_command with args.  I put tracing in and we were good up to this call17:01
ayoungrichm, Heh.  I think I was looking at the unit tests now that I reread it17:03
ayounghttps://github.com/openstack/puppet-openstacklib/blob/master/spec/unit/provider/openstack_spec.rb17:03
richmayoung: yeah - the spec mock for the openstack() method expects arrays in the arguments17:04
ayoungrichm, so...I think that the list command is, somehere, limited in what it can accept17:05
richmayoung: what's not clear to me is if the openstack command will flatten array arguments when passed to openstack_command https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack.rb#L4317:05
ayoungrichm, to fix it, it looks like I have to find where the arguments are counted. And I can't find that17:05
richmayoung: I don't understand what is different about how you are using request() - there are other places in the code where request is used like this: request('thing', 'list', [args]) - and they all work fine17:07
ayoungrichm, example?17:08
richmayoung: https://github.com/openstack/puppet-keystone/blob/master/lib/puppet/provider/keystone_user/openstack.rb#L12517:08
richmchem: Any ideas about how we can debug this?17:09
chemrichm: I was trying to catch up with the problem17:09
chemrichm ayoung, what is the error :)17:09
richmchem: https://review.openstack.org/#/c/282103/17:11
chemrichm: ack17:11
richmayoung: is ^^^ the latest code you are using?17:11
ayoungchem, OK, when trying to list federation protocols17:12
ayoungI need to pass --identity-provider <idp>17:12
ayounglet me recreate the error17:12
ayounglist = request('federation protocol', 'list', ['--identity-provider', 'keycloak'])17:13
ayounggives me17:13
ayoungError: Could not prefetch keystone_federation_protocol provider 'openstack': wrong number of arguments (5 for 2..3)17:13
richmchem: This is from self.instances so it's using self.request from the base class17:14
chemayoung: oki, I get it.  Is the review that richm pointed out is the last code ?17:14
ayoungchem, yeah17:14
chemayoung:thank for the clear explanation17:14
ayounglots of trial and error17:14
chemayoung:I'm looking into it17:14
*** hjensas has quit IRC17:14
ayoungchem, I've tried a few variations17:14
ayoungand it seems something is choking on the number of params specific to list17:15
ayoungif I fake it out so that the openstack provider doesn;'t think it is a list, I can get get back something that looks like the CLI processed it correctly17:15
richmchem: What I don't understand is that the code is using request('thing', 'list', [args]) in exactly the same way as used elsewhere - in keystone_user/openstack.rb et. al.17:16
ayoungchem, for example, if I run: list = request('federation protocol', ['list', '--quiet', '--format', 'csv', '--identity-provider', 'keycloak'])17:17
ayoung    puts "#{list}"17:17
ayoungI get17:17
ayoung"id","mapping"17:17
ayoung"saml2","keycloak-saml2"17:17
ayoungwhich is what the list command would then parse.  So, I could force it by cut and pasting the lower level code, but would be cleaner to get it right17:17
chemrichm ayoung the form is correct, that's for sure, the splat operator is working.  The error is comming from the openstack command.17:19
ayoungchem, I can run the command from the CLI like this:17:20
chemrichm ayoung (I think)17:20
ayoung/bin/openstack federation protocol list --identity-provider keycloak -f csv --quiet17:20
ayoungchem, I get a different error if I make openstack CLI angry17:21
richmayoung: does it work if you do /bin/openstack federation protocol list -f csv --quiet --identity-provider keycloak ?17:21
ayounglet's see17:23
ayoungrichm, yes it does17:23
ayoungif I mess up the params to the openstack command like this17:23
ayoungError: Could not prefetch keystone_federation_protocol provider 'openstack': Execution of '/bin/openstack federation protocol list --quiet --format csv' returned 2: usage: openstack federation protocol list [-h]17:23
ayoungI get back a trace from the output of the command itself17:24
ayoungso I don't think it is the CLI that is angry here.  I think it is whatever is calling the CLI. And I can't trace down to where that is17:24
chemayound, richm: humm ... so it looks like a problem inside the module, then.  Let me do a quick spec for the provider.  The error should show up in there as well17:25
chemayoung: can you paste me a working output from the openstack federation protocol list ... command ?17:29
ayoung$  /bin/openstack federation protocol list -f csv --quiet --identity-provider keycloak17:29
ayoung"id","mapping"17:29
ayoung"saml2","keycloak-saml2"17:29
*** rcernin has joined #puppet-openstack17:30
ayoungchem, Ok so first create an IdP like this17:30
chemayoung:for what I'm doing only the list output will do :)17:30
*** haplo37 has joined #puppet-openstack17:30
ayoungopenstack identity provider create keycroak17:31
ayoungexcept not keycroak for the rest of my stuff to work17:31
ayoungkeycloak17:31
ayounganyways.17:31
ayoungah, then you need a mapping17:31
ayoungthat is the tough part, which needsd a file, and why I skipped right to protocol17:32
ayoungI'll go back for thampping once we get this17:32
ayoungbut..17:32
ayounghttp://paste.openstack.org/show/487608/17:32
ayoungand create using17:32
ayoungopenstack mapping create mapping_keycloak_saml2.json keycloak_saml217:32
ayoungum...17:32
ayoungI think that the mapping needs to be17:33
ayoungkeycloak-saml217:33
ayoungno underscope17:33
ayoungthe underscore is messing things up so that should have been17:33
ayoungopenstack mapping create --rules mapping_keycloak_saml2.json keycloak-saml217:34
ayoungsorry, like I said, a bit of trial and error17:34
ayoungchem, but now we have it in IRC for posterity17:34
chemayoung: that's ok, I have what I need :)17:34
ayoungchem, so, I can parse it if I need17:35
chemayoung ... sorry I'm lost here ... one problem at a time :)17:37
chemayoung I could reproduce the error "wrong number of arguments (5 for 2..3)" using rspec17:37
chemayoung: a matter of minutes to find out the reason, now :)17:38
ayoungok17:38
ayoungI'll let you run.  chem there is a WIP review https://review.openstack.org/#/c/282103/17:39
chemoki, I've got it :)17:39
chemayoung: ^ I'll comment on the review17:40
*** gfidente has quit IRC17:41
ayoungchem, what was it?17:41
chemayoung: do you mind if I push the spec for the provider ?17:41
ayoungchem, not at all17:41
chemayoung: it's basic, but you can work from it after17:41
ayoungchem, so, I need two of these:17:41
ayoungthis one and the one for the mapping17:41
ayoungand the mapping one needs a file.  I assume I can copy what the identity provider does for that, but I might be back to bother you when I get stuck17:42
chemayoung: actually richm had already the answer.  Look at its comment line 8117:43
ayoungbut both require the list using the --identity-provider flag,17:43
ayoungchem, that is for show...and I wonder if I even needed the list in the first place17:43
*** jpena has quit IRC17:43
chemayoung: the error comes from this line not the one on line 7617:43
ayoung?17:44
ayounghmm, OK , let me try that17:44
richmthe error is from the 'show' command, not the 'list' command?17:44
chemyep17:44
richmchem: how can you tell?17:44
chemrichm: because the list command did not fail on my spec test17:44
richmok17:45
chemayoung: richm request('federation protocol', 'show', ['--identity-provider','keycloak',  federation_protocol[:id]])17:46
chemwill work17:46
ayoungError: Could not prefetch keystone_federation_protocol provider 'openstack': No resource and no name in property hash in openstack instance17:47
ayounggood!17:47
ayoungI think I know why17:48
chemayoung: this one is another story :)17:48
richmchem: Thanks!17:48
ayoungchem, hmmm.  Ok I get that part to run with no errors but now it is still trying to create the pre-existing resource17:49
ayoungcloser17:49
chemrichm: np17:49
ayoungbut that is probably cuz I am doing unspeakable things here17:49
chemayoung: it's because you're not matching the existing one properly17:50
ayounglet me clean up my code...17:50
chemayoung: in your prefetch you matching on the name while in the instances class method you're not adding the name property to the created object17:51
chemayoung: it can never match17:51
ayoungchem, ok...17:51
*** bnemec has quit IRC17:52
chemayoung: can you paste me the ouptup of the show command for "federation protocol', 'show', '--format', 'shell', ['--identity-provider', 'keycloak', 'saml2']" on the real systemm ?17:54
ayoungchem, so the way these work is that the idp owns the mapping, an the two together are used by the protocol17:54
ayoungso do I need to have a global list of all protocols to do this \right?17:54
ayoungthis one here is hacked to be scoped to one specific identity provider and mapping17:54
*** derekh has quit IRC17:57
chemayoung: ouch, that's a mouthfull for a friday evening :)17:58
ayoungchem, its ok.  If I get this one working, the other should be prett much the same17:59
chemayoung: I can get more of the functional side of this on monday.  I just wanted the output of the above command to add it to the spec of the self.instances method17:59
ayoungchem, was is 'instances' supposed to have?  The complete list of instances of a particular class?18:00
chemayoung: the list of existing object on the system18:01
chemayoung: associated with your type18:01
ayoungif that is the case, then the right thing is probably to have nested loops, with the outer looping through the identity providers and the inner looping through the  protocols per identity provider18:01
chemayoung: all that is required to get all the fields of your type filled correctly18:03
chemayoung: self.instances is used in two cases : when you run "puppet resource keystone_federation_protocol" and when you use self.prefetch18:03
ayoungchem, this would be a lot easier if I didn't need to learn a new language to do it18:03
chemayoung: common, that's even more fun :)18:04
ayoungchem, this is just one of my many responsibilities18:05
ayoungI did a little bit off ruby 6 years ago18:05
ayoungnot enough to remember18:05
*** bnemec has joined #puppet-openstack18:08
chemayoung ruby is not too hard but puppet type and provider logic is harder, lot of idioms to learn and processes to understand.  Especially in the keystone's one.  I will keep on reviewing this one.  You're not going to be alone :)18:08
chemayoung I'm going to push the spec file I've got.  It has blank to fill but it can help you getting started.18:10
chemayoung: is that ok ?18:10
ayoungchem, excellent18:12
openstackgerritAthlan-Guyot sofer proposed openstack/puppet-keystone: first hack at keystone_federation_protocol  https://review.openstack.org/28210318:13
chemayoung: oki, done.  I've left a comment on this one.18:14
chemayoung: I'm leaving now, I'll review it again on monday.18:15
ayoungchem, what do you mean by :"You have to fill the blank here with the actual output of the command.18:17
ayoung"18:17
ayoungdisregard18:18
chemayoung: you need to run this command in your test env and put the output there18:18
chemayoung: I let another comment, refresh :)18:18
chemayoung: when you paste the output, be *very* mindful of extra blank line, it can lead to very strange error18:19
openstackgerritayoung proposed openstack/puppet-keystone: first hack at keystone_federation_protocol  https://review.openstack.org/28210318:20
chemayoung: you have to put id line right next to the opening single quote18:22
chemayoung: not on a new line18:22
chemayoung: and you should adjust id_provider_attrs to match correct parameter for your provider18:23
ayoungchem, will do. Go now.  Get out while you still can.18:23
chemayoung: leaaaaving, bye and have a good one.18:24
*** imcsk8 has quit IRC18:25
*** chem has quit IRC18:25
*** bodepd has quit IRC18:25
*** jesusaur has quit IRC18:25
*** abramley has quit IRC18:25
*** skolekonov has quit IRC18:25
*** dmburmistrov has quit IRC18:25
*** Aegil has quit IRC18:25
*** pasquier-s has quit IRC18:25
*** ikkeT has quit IRC18:25
*** fc__ has quit IRC18:25
*** clayton has quit IRC18:25
*** _ody has quit IRC18:25
*** psanchez has quit IRC18:25
*** Aegil has joined #puppet-openstack18:25
*** _ody has joined #puppet-openstack18:25
*** fc__ has joined #puppet-openstack18:25
*** skolekonov has joined #puppet-openstack18:25
*** bodepd has joined #puppet-openstack18:25
*** dmburmistrov has joined #puppet-openstack18:25
*** imcsk8 has joined #puppet-openstack18:25
*** ikkeT has joined #puppet-openstack18:25
*** pasquier-s has joined #puppet-openstack18:26
*** abramley has joined #puppet-openstack18:26
*** jesusaur has joined #puppet-openstack18:26
*** clayton has joined #puppet-openstack18:26
*** psanchez has joined #puppet-openstack18:27
*** leanderthal|afk has quit IRC18:27
*** _degorenko|afk has quit IRC18:27
*** mflobo has quit IRC18:27
*** llua has quit IRC18:27
*** llua` has joined #puppet-openstack18:27
*** mflobo has joined #puppet-openstack18:28
*** _degorenko|afk has joined #puppet-openstack18:28
*** clayton_ has joined #puppet-openstack18:28
*** rleander_ has joined #puppet-openstack18:28
*** clayton has quit IRC18:28
*** clayton_ is now known as clayton18:29
*** haplo37 has quit IRC18:37
*** rcernin has quit IRC18:56
*** cwolferh has quit IRC19:03
*** cwolferh has joined #puppet-openstack19:09
*** hjensas has joined #puppet-openstack19:20
*** hjensas has joined #puppet-openstack19:20
*** rcernin has joined #puppet-openstack19:23
*** rcernin has quit IRC19:45
*** cwolferh_ has joined #puppet-openstack19:51
*** cwolferh has quit IRC19:52
*** llua` is now known as llua20:02
*** llua has joined #puppet-openstack20:02
*** hjensas has quit IRC20:32
*** haplo37 has joined #puppet-openstack20:37
*** baker has quit IRC20:51
*** ayoung has quit IRC20:52
*** baker has joined #puppet-openstack20:57
*** foxhunt_away is now known as foxhunt7221:08
*** brandon has joined #puppet-openstack21:11
*** kindjal has quit IRC21:17
*** jayg is now known as jayg|g0n321:21
*** johnmilton has quit IRC21:23
*** kindjal has joined #puppet-openstack22:01
*** ducttape_ has quit IRC22:15
*** ducttape_ has joined #puppet-openstack22:16
*** ducttape_ has quit IRC22:22
*** francois has quit IRC22:29
*** francois has joined #puppet-openstack22:37
*** ducttape_ has joined #puppet-openstack22:46
*** ducttape_ has quit IRC22:47
*** mdorman has quit IRC22:47
*** dhollinger has quit IRC22:54
*** haplo37 has quit IRC23:05
*** foxhunt72 is now known as foxhunt_away23:11
*** ducttape_ has joined #puppet-openstack23:15
*** baker has quit IRC23:16
*** baker has joined #puppet-openstack23:16
*** jtomasek has quit IRC23:19
*** baker has quit IRC23:20
*** ducttape_ has quit IRC23:28
*** ducttape_ has joined #puppet-openstack23:30
*** saneax_AFK is now known as saneax23:31
*** ducttape_ has quit IRC23:39
*** dfisher has quit IRC23:53
« Thursday, 2016-02-18 Index Saturday, 2016-02-20 »

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