Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2013-09-11

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

All times shown according to UTC.

Time Nick Message
00:14 xingchao joined #puppet-openstack
00:17 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-openstack: Add ceilometer support about mysql configuration  https://review.openstack.org/45576
00:42 ianw EmilienM: hi, would you be able to take a look at https://review.openstack.org/#/c/45380/?  I think it's good to go
01:14 xingchao joined #puppet-openstack
01:48 ari joined #puppet-openstack
01:55 xingchao joined #puppet-openstack
02:36 lnxnut joined #puppet-openstack
02:49 otherwiseguy joined #puppet-openstack
05:43 mjblack joined #puppet-openstack
05:50 badiane_ka joined #puppet-openstack
06:43 openstackgerrit Dan Bode proposed a change to stackforge/puppet-keystone: Update default db username to keystone  https://review.openstack.org/46011
06:52 openstackgerrit Dan Bode proposed a change to stackforge/puppet-keystone: Update default db username to keystone  https://review.openstack.org/46011
06:53 mmagr joined #puppet-openstack
07:44 openstackgerrit A change was merged to stackforge/puppet-neutron: Update invalid sql_connection error message  https://review.openstack.org/45973
07:46 openstackgerrit A change was merged to stackforge/puppet-neutron: Update invalid sql_connection error message  https://review.openstack.org/45972
07:51 openstackgerrit A change was merged to stackforge/puppet-openstack: Add ceilometer support about mysql configuration  https://review.openstack.org/45576
07:52 francois1 joined #puppet-openstack
08:28 bauzas hi folks,
08:29 bauzas shall I want to deploy Heat-API-CFN, which class should I choose ?
08:29 bauzas because there are both heat-api-cfn classes in the module... :)
08:30 bauzas https://github.com/stackforge/puppet-heat/blob/master/manifests/api-cfn.pp
08:30 bauzas https://github.com/stackforge/puppet-heat/blob/master/manifests/api_cfn.pp
08:30 bauzas aiui, api-cfn is calling api_cfn, correct ?
08:31 bauzas sounds like a wrapper class on top of api-cfn.pp
08:31 bauzas s/api-cfn/api_cfn/
08:31 bauzas oh dear, I doesn't read the code...
08:32 bauzas warning('heat::api-cfn is deprecated. Use heat::api_cfn instead.')
08:43 fc__ joined #puppet-openstack
08:51 bauzas one last question, I can't find any heat::keystone::auth class
08:52 bauzas I assume nobody yet provided it ?
08:55 e1mer joined #puppet-openstack
08:56 bauzas EmilienM: ping ?
09:00 EmilienM bauzas: pong
09:00 bauzas EmilienM: cool :)
09:01 bauzas EmilienM: I'm reviewing the puppet heat module
09:01 EmilienM bauzas: not yet for keystone afik
09:01 EmilienM nice !
09:01 bauzas EmilienM: tho the rspec is assuming the endpoint is configured
09:02 bauzas https://github.com/stackforge/puppet-heat/blob/master/spec/classes/heat_keystone_endpoint_spec.rb#L3
09:03 bauzas EmilienM: and others: no objections I'm filing a bug and working on providing a patch then ?
09:03 bauzas EmilienM: shouldn't be a big dela
09:03 bauzas s/dela/deal
09:05 EmilienM bauzas: sure !! go ahead
09:05 bauzas ok
09:06 bauzas the rspec is actually already done :)
09:08 EmilienM bauzas: indeed :)
09:14 bauzas EmilienM: well, I'm nearly done but I still need to provide also the Heat-cfn service and endpoint
09:42 mjblack_ joined #puppet-openstack
09:52 bauzas could I get support from someone who is OK with writing rspecs ?
09:52 bauzas I want to provide the heat::keystone::auth class and I tried to run the rake spec
09:53 bauzas it raised an error confusing for me :
09:53 bauzas Pending:
09:53 bauzas heat::keystone::auth on Debian platforms configures heat keystone auth
09:53 bauzas # Not yet implemented
09:53 bauzas # ./spec/classes/heat_keystone_endpoint_spec.rb:69
09:53 bauzas heat::keystone::auth on RedHat platforms configures heat keystone auth
09:53 bauzas # Not yet implemented
09:53 bauzas # ./spec/classes/heat_keystone_endpoint_spec.rb:77
09:54 bauzas I guess I have to rewrite the specfile but it's out of my skills :)
09:56 dguitarbite joined #puppet-openstack
10:25 bauzas noone could help in rspec'ing the class ?
10:26 bauzas I can also provide a patch but it'll probably have a -1
10:26 bauzas if the testcase is not present
11:30 mjblack joined #puppet-openstack
11:56 fvollero Good morning :)
11:56 fvollero s/morning/afternoon/
11:57 fvollero bauzas: just to have an idea, whats the problem ?
11:59 mjblack joined #puppet-openstack
12:05 morazi joined #puppet-openstack
12:21 bauzas fvollero|afk: I see you're afk, but once you're back, I need help for adding the heat::keystone::auth class upstream by rewriting the rspec files
12:31 dprince joined #puppet-openstack
12:42 fvollero bauzas: that's a huge thing
12:43 fvollero bauzas: no kidding, i can help you go thru it but not writing it from scratch. Usually, it's more easy to take a look on the others rspec and get 'inspired'
12:47 bauzas fvollero: I don't want to redo things :)
12:48 bauzas the rspec is already written, but for some reason, the heat::keystone::auth class was missing
12:48 bauzas I just added it
12:48 bauzas https://github.com/stackforge/puppet-heat/blob/master/spec/classes/heat_keystone_endpoint_spec.rb#L3
12:49 bauzas that does the work
12:49 bauzas I tested it against my dev environment
12:49 bauzas the only issue is that I can't provide the patch as the rspec is having 2 steps pending :
12:49 bauzas Pending:
12:49 bauzas heat::keystone::auth on Debian platforms configures heat keystone auth
12:49 bauzas # Not yet implemented
12:49 bauzas # ./spec/classes/heat_keystone_endpoint_spec.rb:69
12:49 bauzas heat::keystone::auth on RedHat platforms configures heat keystone auth
12:49 bauzas # Not yet implemented
12:49 bauzas # ./spec/classes/heat_keystone_endpoint_spec.rb:77
12:49 bauzas Finished in 5.51 seconds
12:49 bauzas 134 examples, 0 failures, 2 pending
12:57 fc__ bauzas: could you explain me what's your problem with these tests ?
12:58 bauzas I'm not a Ruby writer, I know Puppet as operator
12:59 bauzas so, writing manifests is in my veins, writing rspecs not :)
12:59 bauzas I can't figure what the trouble is :-)
13:00 bauzas as said, I know the class I wrote is working as expected, I can't publish it upstream tho
13:00 bauzas b/c the tests are failing
13:00 fc__ bauzas: can you share the code you have currently ?
13:00 bauzas for sure, pasting it
13:01 bauzas http://paste.openstack.org/show/46644/
13:02 bauzas the file is called auth.pp and located in heat/manifests/keystone dir
13:02 bauzas I'm calling the class thru Foreman atm
13:02 bauzas by specifying 'password' and other parameters
13:03 bauzas once the review get merged, I'll publish a patch for having the class called by openstack::keystone as the other ones
13:03 bauzas (provided the operator will set the 'heat' parameter to true
13:03 bauzas )
13:04 bauzas http://paste.openstack.org/show/46647/ is the result of the tests
13:05 * fc__ looking
13:05 fc__ fg
13:05 bauzas you'll find the rspec file already in master https://github.com/stackforge/puppet-heat/blob/master/spec/classes/heat_keystone_endpoint_spec.rb#L3
13:05 * bauzas having an unattended meeting right now, brb in 30 mins or so
13:06 bauzas let's discuss that later if you want :)
13:08 fc__ bauzas: for a start, try this patch to the tests: http://paste.openstack.org/show/46649/
13:08 * fc__ having a meeting too
13:14 mjblack_ joined #puppet-openstack
13:20 otherwiseguy joined #puppet-openstack
13:32 morazi joined #puppet-openstack
13:44 ikke-t joined #puppet-openstack
13:55 dmsimard joined #puppet-openstack
14:05 bauzas fc__: THANKS ! your patch leaved myself with 2 errors, but now I understand these errors !
14:05 bauzas fc__: I only have to modify the asserts, that's it
14:06 fc__ bauzas: great :)
14:06 bauzas apologies again about that, but I'm definitely more skilled on JUnit and PHPUnit :)
14:06 morazi joined #puppet-openstack
14:07 bauzas Python and Ruby worlds are still obscure to me
14:46 ari_ joined #puppet-openstack
15:00 technolo-g joined #puppet-openstack
15:06 ari_ joined #puppet-openstack
15:08 ari_ joined #puppet-openstack
15:38 prad joined #puppet-openstack
15:40 xingchao joined #puppet-openstack
16:18 morazi joined #puppet-openstack
16:37 dguitarbite joined #puppet-openstack
16:46 marun joined #puppet-openstack
16:48 dachary joined #puppet-openstack
17:35 ari joined #puppet-openstack
18:01 openstackgerrit Dan Prince proposed a change to stackforge/puppet-nova: Move sql_ config opts into [database] section.  https://review.openstack.org/46111
18:20 lnxnut joined #puppet-openstack
18:27 dachary joined #puppet-openstack
19:29 otherwiseguy joined #puppet-openstack
21:43 openstackgerrit Dan Prince proposed a change to stackforge/puppet-keystone: Switch from signing/format --> token/provider.  https://review.openstack.org/45933
22:41 e1mer joined #puppet-openstack
22:41 e1mer joined #puppet-openstack
22:57 e1mer joined #puppet-openstack
22:57 e1mer joined #puppet-openstack
23:08 prad joined #puppet-openstack
23:11 badiane_ka joined #puppet-openstack
23:12 openstackgerrit Francesco Vollero proposed a change to stackforge/puppet-heat: Enable execution of heat-manage db_sync  https://review.openstack.org/46152
23:21 openstackgerrit A change was merged to stackforge/puppet-heat: Remove heat::db class  https://review.openstack.org/45179
23:38 xingchao joined #puppet-openstack
23:39 openstackgerrit Ian Wienand proposed a change to stackforge/puppet-heat: Add heat::keystone::auth class  https://review.openstack.org/45380
23:52 openstackgerrit Francesco Vollero proposed a change to stackforge/puppet-heat: Enable execution of heat-manage db_sync  https://review.openstack.org/46152
23:54 otherwiseguy joined #puppet-openstack

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