Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-04-25

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

All times shown according to UTC.

Time Nick Message
00:25 thumpba_ joined #puppet-openstack
00:52 rmoe joined #puppet-openstack
00:56 kkirkpatrick joined #puppet-openstack
00:56 m0ikz joined #puppet-openstack
01:00 kkirkpatrick joined #puppet-openstack
01:03 kkirkpatrick joined #puppet-openstack
01:13 ashp joined #puppet-openstack
01:28 kkirkpatrick joined #puppet-openstack
01:35 hogepodge joined #puppet-openstack
01:45 Aimon joined #puppet-openstack
01:45 Aimon joined #puppet-openstack
01:50 Aimon joined #puppet-openstack
02:02 xarses joined #puppet-openstack
02:17 xingchao joined #puppet-openstack
02:31 higgins joined #puppet-openstack
02:56 thumpba joined #puppet-openstack
03:03 thumpba joined #puppet-openstack
03:09 badiane_ka joined #puppet-openstack
03:27 shakamunyi joined #puppet-openstack
03:53 kkirkpatrick joined #puppet-openstack
03:58 m0ikz joined #puppet-openstack
04:10 sputnik13 joined #puppet-openstack
04:11 sputnik13 joined #puppet-openstack
04:31 sputnik13 joined #puppet-openstack
04:36 sputnik13 joined #puppet-openstack
04:47 sputnik13 joined #puppet-openstack
04:56 sputnik13 joined #puppet-openstack
05:29 thumpba joined #puppet-openstack
05:31 starmer joined #puppet-openstack
05:34 sanjayu joined #puppet-openstack
06:36 openstackgerrit Bogdan Dobrelya proposed a change to stackforge/puppet-glance: Add multiple rabbit hosts support  https://review.openstack.org/90022
06:37 social joined #puppet-openstack
06:57 tfz joined #puppet-openstack
06:57 EmilienM good morning
07:00 dachary morning
07:04 artem_panchenko joined #puppet-openstack
07:07 mmagr joined #puppet-openstack
07:12 bauzas joined #puppet-openstack
07:33 shakamunyi joined #puppet-openstack
07:39 shakamunyi joined #puppet-openstack
07:45 qba73 joined #puppet-openstack
08:09 derekh joined #puppet-openstack
08:13 starmer joined #puppet-openstack
08:40 shakamunyi joined #puppet-openstack
09:07 fvollero morning :)
09:41 shakamunyi joined #puppet-openstack
10:05 openstackgerrit Lukas Bezdicka proposed a change to stackforge/puppet-neutron: Fix neutron subnets with empty values.  https://review.openstack.org/88305
10:06 openstackgerrit Lukas Bezdicka proposed a change to stackforge/puppet-neutron: Fix neutron subnets with empty values.  https://review.openstack.org/88305
10:41 shakamunyi joined #puppet-openstack
10:44 waterkinfe joined #puppet-openstack
10:50 bauzas joined #puppet-openstack
11:07 openstackgerrit Bogdan Dobrelya proposed a change to stackforge/puppet-glance: Add multiple rabbit hosts support  https://review.openstack.org/90022
11:09 bauzas joined #puppet-openstack
11:11 bogdando https://review.openstack.org/#/c/90022 now fixes one more bug discovered. It is amazing how many new bugs could be discovered in the 10 lines of code..
11:11 bogdando see https://bugs.launchpad.net/​puppet-glance/+bug/1312658
11:31 morazi joined #puppet-openstack
11:34 kkirkpatrick joined #puppet-openstack
11:39 bogdando that do you think guys, should I also include the rabbit_host & rabbit_port into the deprecation stanza, like I did for durable_queues?
11:40 bogdando or we should avoid deprecating the parameters unless it was done in the OS project as well?
11:42 shakamunyi joined #puppet-openstack
11:43 bogdando I'm asking cuz looks like the https://github.com/openstack/glanc​e/blob/master/etc/glance-api.conf wasn't synced as appropriate and there are nothing yet about oslo.messaging support and rabbit_durable_queues / host/ port deprecation...
11:50 toMeloos joined #puppet-openstack
12:00 tfz joined #puppet-openstack
12:12 prad joined #puppet-openstack
12:41 tfz hi all, is somebody using cinder and mysql and could verify that https://review.openstack.org/#/c/88303/ does not break the setup ?
12:43 shakamunyi joined #puppet-openstack
12:53 Pax joined #puppet-openstack
12:55 dkranz joined #puppet-openstack
13:00 dprince joined #puppet-openstack
13:05 m0ikz joined #puppet-openstack
13:07 toMeloos tfz: are you considering doing the same for the other modules?
13:08 tfz toMeloos which other modules do you mean ?
13:08 toMeloos cinder isn't the only module with a mysql_module parameter outside the xxxxxxx::db::mysql class
13:09 tfz toMeloos really ?
13:09 tfz I did not catch any other yet.
13:09 tfz and I only use postgres.
13:10 toMeloos joined #puppet-openstack
13:10 toMeloos yes definitely
13:12 tfz toMeloos: I did not hit any in heat, glance, nova, neutron, ceilometer, keystone
13:12 toMeloos keystone, ceilometer::db, heat, glance::api, glance::registry, neutron::server, nova and cinder
13:12 toMeloos basically all the same story
13:12 tfz let me check, I cannot imagine.
13:12 toMeloos would be interesting if all of that could be moved to their db::mysql subclass
13:13 toMeloos it would definitely make it easier to add non-mysql support to all these modules
13:13 toMeloos I'm pretty sure, trying to get icehouse working on Ubuntu Trusty using the puppet scripts right now ;)
13:14 toMeloos by the way, I wanted to ask the room if anybody was working on the various package issues with the neutron agents classes
13:15 tfz toMeloos I looked at keystone, indeed, it is there, but there is an if statement around. so it's only required if($sql_connection =~ /mysql: ...
13:15 tfz so that's ok.
13:15 toMeloos you could do the same for cinder
13:16 toMeloos still it seems cleaner to me to move ALL mysql-related code to the db::mysql subclass
13:16 toMeloos e.g. to do your approach
13:17 toMeloos unfortunately I can't test your patch right now
13:19 tfz actually I also think that the move is the cleaner solution. but both ways are acceptable for me.
13:22 toMeloos if this move works (and I don't see why it wouldn't) then I would recommend doing the same for the other classes that I just mentioned
13:22 toMeloos for consistency
13:26 tfz toMeloos ACK.
13:43 social Hi, I have this pull to puppetlabs-firewall https://github.com/puppetlabs​/puppetlabs-firewall/pull/346 and it failed on missing function is_integer which is provided by stdlib, yet the tests seem to be using stdlib. Can someone give me hint for good solution to this?
13:44 shakamunyi joined #puppet-openstack
13:44 kkirkpatrick joined #puppet-openstack
13:47 prad joined #puppet-openstack
14:19 dmassey joined #puppet-openstack
14:27 marun joined #puppet-openstack
14:27 fvollero social: I think this is a question more for Hunner or hodgepodge :)
14:27 shakamunyi joined #puppet-openstack
14:49 toMeloos say anybody here working on the puppet-neutron module?
14:49 openstackgerrit Martin Mágr proposed a change to stackforge/puppet-cinder: Use lioadm as iscsi helper on RHEL 7  https://review.openstack.org/90389
14:49 bauzas joined #puppet-openstack
14:50 hesco joined #puppet-openstack
15:00 shakamunyi joined #puppet-openstack
15:22 blentz joined #puppet-openstack
15:41 xarses joined #puppet-openstack
15:46 bauzas joined #puppet-openstack
15:46 ashp joined #puppet-openstack
15:46 Aimon joined #puppet-openstack
15:46 bauzas joined #puppet-openstack
15:49 mfisch joined #puppet-openstack
15:49 mfisch joined #puppet-openstack
15:56 rmoe joined #puppet-openstack
15:58 angdraug joined #puppet-openstack
16:08 openstackgerrit Martin Mágr proposed a change to stackforge/puppet-nova: RHEL should have default provider  https://review.openstack.org/90416
16:21 sputnik13 joined #puppet-openstack
16:27 badiane_ka joined #puppet-openstack
16:36 markvoelker hodgepodge: a request for your weekly status update to the ML. Is there a master list of Puppet-related sessions/locations for Atlanta?
16:37 markvoelker hogepodge: Hunner: ^^
16:42 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Fix to specify the endpoint url directly  https://review.openstack.org/87372
16:45 hogepodge joined #puppet-openstack
16:54 Hunner social: Which version of stdlib is is_integer in? Do you know?
16:57 openstackgerrit A change was merged to stackforge/puppet-neutron: Fix neutron subnets with empty values.  https://review.openstack.org/88305
16:58 Hunner social: Actually, it doesn't look like stdlib is a dependency of firewall, and it is also not in .fixtures.yml, so it won't have stdlib for the tests
16:59 markvoelker Hunner: 2.2ish I think
17:10 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-nova: support for managing ssh keys  https://review.openstack.org/89620
17:33 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-nova: support for managing ssh keys  https://review.openstack.org/89620
17:34 larsks markvoelker: Sorry, you commented in between updates.  I was over enthusiastic in typing 'git review' :)
17:34 markvoelker larsks: heh, you're quick =)
17:34 larsks markvoelker: There are explciit failures now, and tests for the failures as well.
17:35 larsks markvoelker: Also, that default => is gone.
17:35 markvoelker larsks: nice!  I'll take another look as soon as I finish this other review I'm working on
17:36 larsks Hold off a moment because I'm going to address your most recent comments...
17:39 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-nova: support for managing ssh keys  https://review.openstack.org/89620
17:39 kkirkpatrick joined #puppet-openstack
17:41 tfz would be nice if somebody could review https://review.openstack.org/#/c/88303/ without this, puppet-cinder can only be used with mysql (no postgresql possible).
17:43 kkirkpatrick joined #puppet-openstack
17:45 kkirkpatrick joined #puppet-openstack
18:06 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-nova: support for managing ssh keys  https://review.openstack.org/89620
18:15 markvoelker tfz: I will try to have a look this afternoon.
18:20 xarses joined #puppet-openstack
18:33 starmer joined #puppet-openstack
18:35 m0ikz joined #puppet-openstack
18:38 dmsimard dachary: can you let me know once you are done with all the changes ? This way I don't need to review too many times :(
18:43 openstackgerrit A change was merged to stackforge/puppet-nova: fix report_interval configuration  https://review.openstack.org/89852
18:45 starmer joined #puppet-openstack
19:01 xarses joined #puppet-openstack
19:03 larsks markvoelker: There was also a missing test case for the situation you highlited in your recent comments.  Fixed that and added the error message as suggested.
19:03 * markvoelker looks
19:05 xarses joined #puppet-openstack
19:05 markvoelker larsks: I had one further suggestion about improving an error message (see my latest comment), but it's one I can live without.  LGTM
19:06 larsks markvoelker: I updated the error message per your suggestion on patchset 7.  If you've got another suggestion I'm all ears, because they've been good ones so far...
19:07 markvoelker larsks: is there a patchset 8 uploaded?  I don't see it...
19:08 larsks markvoelker: Erg, git review yelled at me about uncomitted changes and I missed it.  Fixing...
19:08 markvoelker larsks: =)
19:11 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-nova: support for managing ssh keys  https://review.openstack.org/89620
19:32 francois1 joined #puppet-openstack
19:40 markvoelker larsks: whoops...that last update made jenkins mad
19:41 larsks markvoelker: checking...
19:41 markvoelker larsks: https://review.openstack.org/#/c/8​9620/8/spec/classes/nova_init_spec.rb
19:41 markvoelker Line 135
19:41 larsks Yup :)
19:44 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-nova: support for managing ssh keys  https://review.openstack.org/89620
19:44 fandi joined #puppet-openstack
19:53 tfz markvoelker thanks for the review. I see you're point. what do you think about changing it to something like in https://github.com/stackforge/puppet-key​stone/blob/master/manifests/init.pp#L295 ?
19:54 markvoelker tfz: that sounds like a good idea
19:54 tfz ok, I will implement it this way.
19:54 tfz thanks.
19:54 markvoelker tfz: thank you!
20:09 starmer joined #puppet-openstack
20:26 daneyon joined #puppet-openstack
20:27 daneyon joined #puppet-openstack
20:28 dachary dmsimard: your reviews are not lost ;-)
20:28 dachary dmsimard: in the past 24h it was just rebasing, no change
20:31 openstackgerrit Benedikt Trefzer proposed a change to stackforge/puppet-keystone: fix typo in comment.  https://review.openstack.org/90463
20:31 openstackgerrit Benedikt Trefzer proposed a change to stackforge/puppet-cinder: require mysql binding for mysql db only  https://review.openstack.org/88303
20:33 tfz markvoelker: done. (including a typo found in keystone :))
21:05 dmassey any suggestions on best methods to use puppet for inventory and configuration reporting?
21:07 openstackgerrit Loic Dachary proposed a change to stackforge/puppet-ceph: ceph::mon : control client.admin key generation  https://review.openstack.org/89351
21:10 thumpba joined #puppet-openstack
21:14 openstackgerrit Benedikt Trefzer proposed a change to stackforge/puppet-glance: add kombu_ssl parameters to notify::rabbitmq  https://review.openstack.org/90037
21:29 evilstephen joined #puppet-openstack
21:43 bauzas joined #puppet-openstack
21:56 openstackgerrit Benedikt Trefzer proposed a change to stackforge/puppet-glance: fix rspec tests in backend::rbd and notify::qpid  https://review.openstack.org/90489
22:03 openstackgerrit Benedikt Trefzer proposed a change to stackforge/puppet-glance: fix rspec tests in backend::rbd and notify::qpid  https://review.openstack.org/90489
22:12 thumpba joined #puppet-openstack
22:17 xarses joined #puppet-openstack
22:37 daneyon joined #puppet-openstack
22:49 tfz joined #puppet-openstack
22:57 xarses joined #puppet-openstack
23:11 m0ikz joined #puppet-openstack
23:24 derekh joined #puppet-openstack
23:55 shakamunyi joined #puppet-openstack

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