Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-10-24

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

All times shown according to UTC.

Time Nick Message
00:07 michchap EmilienM: mgagne crinkle I don't think we need a class that creates the rabbit server. We are all using profiles for that aren't we? What we might benefit from is a defined type in openstacklib that creates a (user,pw,vhost) for that service, similar to what we do for mysql.
00:07 mgagne michchap: sure
00:08 mgagne michchap: I do however like the idea of providing just enough to start something
00:08 michchap mgagne: and yes, nova::rabbitmq should not be instantiating the rabbit class.
00:08 michchap mgagne: just 'include rabbitmq' is enough to start something
00:09 mgagne michchap: fine with me then
00:09 ric` joined #puppet-openstack
00:09 mgagne michchap: https://github.com/stackforge/cookbook-openstack-ops-messaging
00:10 EmilienM ok, make sense then
00:10 mgagne michchap: I'm not sure what this cookbook does that any other rabbitmq cookbook doesn't
00:10 mgagne michchap: looks to do the same as nova::rabbitmq
00:10 EmilienM mgagne: michchap: but thus type in openstacklib would be useless I think since there is nothing depending of OpenStack, I would directly use Rabbit type
00:11 michchap EmilienM: so each module should have a neutron::rabbitmq and a nova::rabbitmq, and those classes can be used to make their own vhost,user,pw for that service, with some sane defaults.
00:11 mgagne EmilienM, michchap: I think we each moved away from nova::rabbitmq at some point right?
00:11 EmilienM mgagne: I never used it.
00:12 mgagne EmilienM: hehe
00:12 michchap mgagne: yep, I just include rabbitmq directly
00:12 mgagne EmilienM: ok, maybe for 2 days :P
00:12 EmilienM ahah
00:12 EmilienM and I gave upp
00:12 michchap haha
00:12 mgagne michchap: I guess that would be a sane solution
00:12 mgagne michchap: btw, weren't you looking for a manifest to manage rabbitmq policies?
00:13 michchap mgagne: EmilienM because today, making neutron on its own vhost we don't have a class for that.
00:13 EmilienM mgagne: in that case, I won't push any BP for firewall code.
00:13 michchap mgagne: are you about to link me to the github PR adding federation types?
00:13 EmilienM michchap: no you can't :(
00:14 mgagne michchap: what's this PR you are talking about?
00:14 michchap mgagne: https://github.com/puppetlabs/puppetlabs-rabbitmq/pull/192
00:15 mgagne michchap: how am I supposed to review that PR...
00:15 mgagne michchap: 19 commits and counting
00:15 michchap mgagne: you posted some code for it in an issue, which I turned into a PR. That was merged briefly, and then reverted without the linked PR being merged, so there was a breif regression and it pissed me off quite a lot
00:15 mgagne michchap: my code introduced a regression?
00:16 michchap mgagne: because I was able to work off upstream for like a week, and then suddenly it was gone
00:16 mgagne michchap: I rewrote it from scratch a couple of weeks ago, not sure if it's that code that was used or not
00:16 michchap mgagne: no, it was reverted, which was a regression
00:16 michchap mgagne: https://github.com/puppetlabs/puppetlabs-rabbitmq/commit/d32e501e3889d20bfceeb9b77554a794b36d8610
00:17 mgagne michchap: haha, very old code
00:17 mgagne michchap: I rewrote it
00:17 michchap mgagne: yep, but it worked =/
00:17 mgagne michchap: =(
00:18 mgagne hold on
00:22 mgagne michchap: https://gist.github.com/mgagne/65fdf555299544e58a40
00:22 mgagne michchap: main point is: "# rabbitmq<3.2 does not support the applyto field"
00:22 michchap mgagne: much neater :D
00:23 mgagne michchap: code is used in production
00:24 rmoe joined #puppet-openstack
00:26 mgagne michchap: :broken => true is for my jenkins setup. I have conflicting facter versions. about the same issue we had on openstack ci a couple of weeks ago
00:28 michchap mgagne: ah
00:34 openstackgerrit A change was merged to stackforge/puppet-horizon: Add selection of Django session engine to this module  https://review.openstack.org/130403
00:35 mgagne michchap: alright, I have to go. let me know if rabbitmq_policy works (or not) for you
00:36 ibravo joined #puppet-openstack
00:44 michchap mgagne: I'dll give it a go
00:57 svetmy joined #puppet-openstack
00:59 xarses joined #puppet-openstack
00:59 svetmy joined #puppet-openstack
01:28 topshare joined #puppet-openstack
01:32 topshare_ joined #puppet-openstack
02:23 topshare joined #puppet-openstack
02:32 topshare joined #puppet-openstack
02:40 topshare joined #puppet-openstack
02:51 topshare joined #puppet-openstack
03:10 sputnik13 joined #puppet-openstack
03:14 topshare joined #puppet-openstack
03:30 xingchao joined #puppet-openstack
04:13 sputnik13 joined #puppet-openstack
06:14 sputnik13 joined #puppet-openstack
06:49 rwsu_ joined #puppet-openstack
07:00 svetmy joined #puppet-openstack
07:00 ric` joined #puppet-openstack
07:00 jamespd joined #puppet-openstack
07:00 Guest28430 joined #puppet-openstack
07:00 michchap joined #puppet-openstack
07:00 tchaypo joined #puppet-openstack
07:00 anteaya joined #puppet-openstack
07:00 underyx|off joined #puppet-openstack
07:02 arnaud_orange joined #puppet-openstack
07:07 vb joined #puppet-openstack
07:10 markvoelker joined #puppet-openstack
07:10 dachary joined #puppet-openstack
07:10 sbadia joined #puppet-openstack
07:32 underyx|off joined #puppet-openstack
07:32 Guest28430 joined #puppet-openstack
07:32 jamespd joined #puppet-openstack
07:32 ric` joined #puppet-openstack
07:32 markvoelker joined #puppet-openstack
07:32 dachary joined #puppet-openstack
07:32 sbadia joined #puppet-openstack
07:37 svetmy joined #puppet-openstack
07:37 michchap joined #puppet-openstack
07:37 tchaypo joined #puppet-openstack
07:37 anteaya joined #puppet-openstack
07:39 Mosibi joined #puppet-openstack
07:40 mmagr joined #puppet-openstack
07:40 ric` joined #puppet-openstack
07:40 jamespd joined #puppet-openstack
07:40 Guest28430 joined #puppet-openstack
07:40 underyx|off joined #puppet-openstack
07:40 mmagr joined #puppet-openstack
08:08 qba73 joined #puppet-openstack
08:17 dgurtner joined #puppet-openstack
08:31 derekh joined #puppet-openstack
08:51 derekh Anybody know when the juno branches will be created for the modules ?
08:56 michchap derekh: we don't usually have a schedule, it's just kind of 'wait until a few people running trunk have it working and are happy with it'
09:00 derekh michchap: ok, cool, roughly are we talking days or week ?
09:01 derekh *weeks
09:01 michchap derekh: usually about a month after upstream release, but that's not to say it can't be faster if things are in good shape.
09:03 derekh michchap: ok, thanks, just trying to get a rough idea
09:13 kbyrne joined #puppet-openstack
09:53 arnaud_orange joined #puppet-openstack
09:56 mmagr joined #puppet-openstack
09:56 ric` joined #puppet-openstack
09:56 jamespd joined #puppet-openstack
09:56 Guest28430 joined #puppet-openstack
09:56 underyx|off joined #puppet-openstack
09:58 kbyrne joined #puppet-openstack
09:58 marcel joined #puppet-openstack
09:58 Madkiss joined #puppet-openstack
09:58 aimon_ joined #puppet-openstack
09:58 hogepodge joined #puppet-openstack
09:58 tlbr joined #puppet-openstack
09:58 abramley joined #puppet-openstack
09:58 bodepd_ joined #puppet-openstack
10:59 rcallawa_ joined #puppet-openstack
11:42 dgurtner joined #puppet-openstack
11:53 dgurtner joined #puppet-openstack
11:53 dgurtner joined #puppet-openstack
12:16 EmilienM good morning
12:52 rcallawa joined #puppet-openstack
13:13 dgurtner joined #puppet-openstack
13:26 openstackgerrit John Patterson proposed a change to stackforge/puppet-neutron: Adding neutron_service_provider type  https://review.openstack.org/129360
13:39 morazi joined #puppet-openstack
13:50 morazi joined #puppet-openstack
14:00 michchap EmilienM: I looked that that change ^ does service_provider have to be present on both neutorn-server nodes and agent nodes?
14:02 EmilienM michchap: only server
14:02 michchap EmilienM: propose add it to server.pp as a param as well?
14:02 michchap EmilienM: feels weird only having it in config.pp
14:03 ericpeterson joined #puppet-openstack
14:03 dvorak EmilienM: not sure I understand your comment on my nova_admin_tenant_id_setter patch
14:03 dvorak and to be honest, it's been so long that I forgot I had even submitted it :)
14:04 dvorak is the question about where I put the dependency, or whether or not we need it at all?
14:04 peterstac joined #puppet-openstack
14:09 EmilienM dvorak: why did you put this dep I mean?
14:09 EmilienM michchap: yes, server.pp makes sense
14:10 dvorak EmilienM: ah, because neutron_config requires the neutron packages to already be installed, because it will not create /etc/neutron, or the config files that live there
14:11 dvorak and nova_admin_tenant_id_setter is just a wrapper around neutron_config
14:11 richm joined #puppet-openstack
14:11 dvorak but the collector in the line above doesn't gather it, since it's technically another type
14:11 EmilienM dvorak: yes, but when you declare neutron::server::notifications, neutron::server is also declared, right?
14:13 dvorak you mean included?  They're both included, but there is no dependecy between the two that I see
14:13 dvorak from an ordering standpoint I mean
14:14 dvorak the issue I'm trying to address is that when I bring up a new server that has neutron and nova_admin_tenant_id_setter on it, sometimes the id_setter is applied before the neutron package is installed, and that fails because /etc/neutron/* isn't there yet
14:21 dvorak does that make any else?
14:21 dvorak err, sense
14:25 EmilienM dvorak: yes
14:25 EmilienM indeed
14:38 * larsks just wants puppetlabs-mysql to be able to create databases on a remote host.
14:39 EmilienM derekh: I'm just scared about putting this dep in init
14:40 derekh EmilienM: ?, meant for dvorak I guess
14:40 EmilienM derekh: int he case the nova_admin_tenant_id_setter is not in the catalog for one node
14:40 EmilienM dvorak: ^
14:40 EmilienM derekh: sorry :-)
14:40 derekh EmilienM: no prob
14:43 dvorak EmilienM: it should fail silently.  the collector won't error out if it finds nothing
14:44 EmilienM dvorak: +2
14:48 openstackgerrit A change was merged to stackforge/puppet-swift: Allow to customize log_name for services  https://review.openstack.org/128385
14:50 xingchao joined #puppet-openstack
14:58 xingchao joined #puppet-openstack
15:05 cjellick joined #puppet-openstack
15:07 cjellick joined #puppet-openstack
15:16 aarefiev joined #puppet-openstack
15:19 xingchao joined #puppet-openstack
15:28 mschmitt joined #puppet-openstack
15:31 peterstac joined #puppet-openstack
15:39 ddmitriev joined #puppet-openstack
15:54 mmaglana joined #puppet-openstack
15:56 stamak joined #puppet-openstack
15:58 xingchao joined #puppet-openstack
15:58 ericpeterson joined #puppet-openstack
15:59 mschmitt1 joined #puppet-openstack
16:02 xarses joined #puppet-openstack
16:02 morazi joined #puppet-openstack
16:10 svetmy joined #puppet-openstack
16:20 hogepodge EmilienM should be able to +2 this now. https://review.openstack.org/#/c/128675/
16:20 hogepodge sbadia ^^
16:22 EmilienM hogepodge: done
16:36 xingchao joined #puppet-openstack
16:37 sbadia hi
16:37 sbadia crinkle: thanks for the comments (I'll update this)
16:37 sbadia for vswitch the original patch was owned by gilles https://review.openstack.org/#/c/123632/
16:38 sbadia but indeed i've missed openstacklib and extra
16:38 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-openstacklib: Switch to metadata.json for Puppet forge releases  https://review.openstack.org/130848
16:38 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-openstack_extras: Switch to metadata.json for Puppet forge releases  https://review.openstack.org/130849
16:40 sbadia hogepodge: done, thanks!
16:40 aimon joined #puppet-openstack
16:41 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-keystone: Switch to metadata.json for Puppet forge releases  https://review.openstack.org/124579
16:42 openstackgerrit A change was merged to stackforge/puppet-sahara: Added sahara.conf provider and unit tests.  https://review.openstack.org/128675
16:43 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-manila: Switch to metadata.json for Puppet forge releases  https://review.openstack.org/125247
16:46 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-ceph: Switch to metadata.json for Puppet forge releases  https://review.openstack.org/125248
16:56 rwsu joined #puppet-openstack
17:08 cjellick joined #puppet-openstack
17:08 dgurtner joined #puppet-openstack
17:09 rcallawa_ joined #puppet-openstack
17:09 cjellick joined #puppet-openstack
17:37 xarses joined #puppet-openstack
17:52 mmaglana joined #puppet-openstack
17:53 rmoe joined #puppet-openstack
17:53 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-horizon: Add selection of Django session engine to this module  https://review.openstack.org/130863
18:01 xingchao joined #puppet-openstack
18:05 rcallawa joined #puppet-openstack
18:33 ericpeterson mgagne:  you are more than welcome to steal the contents of this patch too:  https://review.openstack.org/#/c/122256/
18:34 mgagne ericpeterson: will check as soon as I can
19:41 dmsimard joined #puppet-openstack
19:43 rcallawa joined #puppet-openstack
19:48 dgurtner joined #puppet-openstack
19:57 xingchao joined #puppet-openstack
20:48 svetmy joined #puppet-openstack
21:05 xingchao joined #puppet-openstack
21:44 openstackgerrit Sergii Golovatiuk proposed a change to stackforge/puppet-nova: Set force_snat_rang parameter  https://review.openstack.org/130488
21:48 EmilienM have a great week-end everyone !
21:49 crinkle o/
22:00 holser Have a great weekend EmilienM
22:05 dgurtner joined #puppet-openstack
22:05 dgurtner joined #puppet-openstack
22:09 xingchao joined #puppet-openstack
22:10 ashp left #puppet-openstack
23:07 bitblt joined #puppet-openstack
23:17 xingchao joined #puppet-openstack
23:27 rmoe joined #puppet-openstack
23:44 mmaglana_ joined #puppet-openstack
23:52 aimon joined #puppet-openstack

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