Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-01-13

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

All times shown according to UTC.

Time Nick Message
23:30 dmsimard joined #puppet-openstack
23:33 michchap radmiraal: if you're on Ubuntu, use the ubuntu cloud archive.
23:34 dmsimard joined #puppet-openstack
00:35 tnoor joined #puppet-openstack
01:34 dmsimard joined #puppet-openstack
01:39 xingchao joined #puppet-openstack
01:42 dmsimard1 joined #puppet-openstack
01:47 xingchao_ joined #puppet-openstack
01:51 rongze joined #puppet-openstack
02:13 thumpba_ joined #puppet-openstack
02:15 nigelk joined #puppet-openstack
02:24 openstackgerrit A change was merged to stackforge/puppet-cinder: Update manifest and spec to use new NetApp unified driver config options.  https://review.openstack.org/65953
02:27 openstackgerrit A change was merged to stackforge/puppet-neutron: Add support for allow_pagination and allow_sorting  https://review.openstack.org/65308
02:28 openstackgerrit A change was merged to stackforge/puppet-nova: Removes unnecessary api-paste.ini configuration  https://review.openstack.org/65439
03:11 tnoor joined #puppet-openstack
04:00 dmsimard joined #puppet-openstack
04:47 sanman joined #puppet-openstack
05:20 rongze joined #puppet-openstack
05:32 rongze joined #puppet-openstack
05:48 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Expose time zone in user.common.yaml  https://review.openstack.org/65866
05:48 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Some minor enhancements to install scripts  https://review.openstack.org/65559
05:49 openstackgerrit A change was merged to stackforge/puppet-neutron: VXLAN support  https://review.openstack.org/66138
06:32 rongze joined #puppet-openstack
06:38 otherwiseguy joined #puppet-openstack
07:00 rongze joined #puppet-openstack
07:47 jkyle joined #puppet-openstack
08:13 bauzas joined #puppet-openstack
09:07 derekh joined #puppet-openstack
09:09 toMeloos joined #puppet-openstack
09:12 bogdando joined #puppet-openstack
09:21 EmilienM good morning
09:21 dgollub joined #puppet-openstack
09:22 mmagr joined #puppet-openstack
09:28 toMeloos joined #puppet-openstack
09:57 toMeloos joined #puppet-openstack
10:11 piliman974 joined #puppet-openstack
10:44 rongze joined #puppet-openstack
11:02 rongze joined #puppet-openstack
11:29 rongze joined #puppet-openstack
11:31 michchap joined #puppet-openstack
11:46 dgollub joined #puppet-openstack
11:55 starmer joined #puppet-openstack
12:04 rongze joined #puppet-openstack
12:57 dgollub1 joined #puppet-openstack
13:02 rongze joined #puppet-openstack
13:03 markvoelker joined #puppet-openstack
13:07 michchap joined #puppet-openstack
13:08 bauzas joined #puppet-openstack
13:32 rongze joined #puppet-openstack
13:35 dgollub joined #puppet-openstack
13:47 otherwiseguy joined #puppet-openstack
13:47 rcrit joined #puppet-openstack
13:51 prad_ joined #puppet-openstack
13:52 dprince joined #puppet-openstack
14:05 BroadcastStorm left #puppet-openstack
14:08 piliman974 joined #puppet-openstack
14:32 rongze joined #puppet-openstack
14:34 toMeloos joined #puppet-openstack
14:41 britthouser joined #puppet-openstack
14:42 finnx joined #puppet-openstack
14:48 pzimmer joined #puppet-openstack
14:51 rongze joined #puppet-openstack
14:55 dmsimard joined #puppet-openstack
15:16 rwsu joined #puppet-openstack
15:33 EmilienM fc__: I need a quick review on https://review.openstack.org/#/c/66200/
15:33 EmilienM michchap: if you are still here, ^
15:37 mgagne joined #puppet-openstack
15:37 morazi joined #puppet-openstack
15:54 fvollero EmilienM: Bonjour!
15:59 EmilienM fvollero: hey
16:00 tnoor joined #puppet-openstack
16:04 angdraug joined #puppet-openstack
16:05 dtalton joined #puppet-openstack
16:23 starmer joined #puppet-openstack
16:27 sputnik13net joined #puppet-openstack
16:48 pzimmer joined #puppet-openstack
16:50 pzimmer1 joined #puppet-openstack
16:58 dmsimard rongze: ping
16:59 rongze dmsimard: pong
17:03 dmsimard rongze: The 3.0 gate matters more or less, it's non-voting
17:04 dmsimard rongze: I asked a recheck mostly because I thought the lint test was broken - it's not though, I am able to reproduce the error here
17:04 dmsimard rongze: It's complaining about a case without defaults
17:04 rongze yes, it is a warning
17:04 rongze manifests/mon.pp - WARNING: case statement without a default case on line 67
17:05 dmsimard rongze: We want to either put a default or add a rake lint exclusion for that kind of check
17:05 rongze ok
17:06 rongze I am curious that a warning lead to a failure...
17:06 openstackgerrit A change was merged to stackforge/puppet-nova: Update gitignore file  https://review.openstack.org/65830
17:06 dmsimard rongze: Looking at various puppet modules, it looks like the default case is enforced.. Ex: https://github.com/stackforge/p​uppet-nova/blob/master/Rakefile .. https://github.com/stackforge/pu​ppet-cinder/blob/master/Rakefile .. https://github.com/stackforge/pupp​et-openstack/blob/master/Rakefile
17:07 mgagne PuppetLint.configuration.fail_on_warnings = true
17:07 rongze thank your remind, I fix it right now
17:07 openstackgerrit A change was merged to stackforge/puppet-heat: Updated mysql dependency to correct range  https://review.openstack.org/65757
17:08 dmsimard rongze: I guess that means we have to set a default (probably fail ?) and an additional spec test for that
17:09 rongze there is "PuppetLint.configuration.fail_on_warnings = true" in puppet-ceph/Rakefile,
17:11 rongze set $init = fail in default ?
17:13 rongze I make a try
17:19 angdraug joined #puppet-openstack
17:21 unixpro1970 joined #puppet-openstack
17:25 xarses joined #puppet-openstack
17:30 dmsimard rongze: Maybe we can remove fail_on_warnings, I don't know who put it there
17:31 rongze I think it is copied by some one
17:32 dmsimard rongze: It was put there by bodepd when the repository was first created. Let me do a quick commit to remove it and you can rebase on it ?
17:33 rongze I think we can keep it
17:34 rongze I just add a default and spec test for that
17:35 dmsimard It's a bit draconian. We should be aware of the warnings when we run lint tests in our respective development environments. I can understand why warnings are non-blocking for the other projects
17:38 openstackgerrit Rongze Zhu proposed a change to stackforge/puppet-ceph: Add ceph::mon RedHat supprt  https://review.openstack.org/64388
17:38 rongze https://review.openstack.org/64388
17:40 rongze gate-puppet-ceph-puppet-lint SUCCESS in 2s
17:42 rongze Can your puppet-lint check the warning for lacking default ?
17:48 hogepodge joined #puppet-openstack
17:50 openstackgerrit Rongze Zhu proposed a change to stackforge/puppet-ceph: correct option name in class ceph  https://review.openstack.org/66380
17:55 sputnik13net joined #puppet-openstack
17:56 rmoe joined #puppet-openstack
18:40 hogepodge joined #puppet-openstack
18:47 ilbot3 joined #puppet-openstack
18:47 topic for #puppet-openstack is now Place to collaborate on Puppet/OpenStack tools: logs at http://irclog.perlgeek.de/puppet-openstack/today
18:48 unixpro1970 joined #puppet-openstack
18:49 xarses joined #puppet-openstack
19:03 unixpro1970 joined #puppet-openstack
19:05 unixpro1970 joined #puppet-openstack
19:13 unixpro1970 joined #puppet-openstack
19:31 openstackgerrit joined #puppet-openstack
19:36 thumpba joined #puppet-openstack
19:46 openstackgerrit A change was merged to stackforge/puppet-cinder: Release 3.0.0  https://review.openstack.org/65780
19:46 openstackgerrit A change was merged to stackforge/puppet-glance: Release 3.0.0  https://review.openstack.org/65791
19:47 openstackgerrit A change was merged to stackforge/puppet-horizon: Release 3.0.0-rc3  https://review.openstack.org/65794
19:48 openstackgerrit A change was merged to stackforge/puppet-keystone: Release 3.0.0-rc3  https://review.openstack.org/65795
19:48 openstackgerrit A change was merged to stackforge/puppet-neutron: Release 3.0.0  https://review.openstack.org/65796
19:49 openstackgerrit A change was merged to stackforge/puppet-nova: Release 3.0.0  https://review.openstack.org/65797
19:50 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Fixed mysql dependency to match other modules  https://review.openstack.org/65756
19:51 openstackgerrit A change was merged to stackforge/puppet-swift: Release 3.0.0-rc3  https://review.openstack.org/65800
19:52 openstackgerrit A change was merged to stackforge/puppet-neutron: Configure security group when using ML2 plugin  https://review.openstack.org/66200
19:54 openstackgerrit A change was merged to stackforge/puppet-swift: Add /var/cache/swift to the list of directories that must be owned by swift.  https://review.openstack.org/65717
19:55 EmilienM hogepodge: massive review :-)
19:55 hogepodge Yeah. Just releasing code already released.
19:55 hogepodge Not good form, but we're very late behind on this.
19:55 hogepodge Wouldn't have done it in master. :-/
19:56 hogepodge by we're I mean I'm
19:56 hogepodge You're working on Ceil, right?
20:00 rmoe_ joined #puppet-openstack
20:01 openstackgerrit A change was merged to stackforge/puppet-openstack: Release 3.0.0  https://review.openstack.org/65798
20:01 dmsimard1 joined #puppet-openstack
20:05 dmsimard2 joined #puppet-openstack
20:06 EmilienM hogepodge: puppet-ceilometer ? not too much
20:06 hogepodge ah, ok.
20:06 EmilienM hogepodge: but I can help
20:07 hogepodge When I install the compute ceil client all of the compute metrics disappear.
20:08 EmilienM hogepodge: ?
20:08 EmilienM hogepodge: could you be more precise?
20:08 EmilienM when you install ceilometer compute agent?
20:08 hogepodge When you look at the ceilometer panel in Horizon, you can no longer select the ones associated with the compute agent.
20:19 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-neutron: Ensure ML2 plugin package is installed  https://review.openstack.org/66413
20:40 openstackgerrit A change was merged to stackforge/puppet-neutron: Ensure ML2 plugin package is installed  https://review.openstack.org/66413
20:59 dmsimard joined #puppet-openstack
21:04 tnoor joined #puppet-openstack
21:20 badiane_ka joined #puppet-openstack
21:25 bauzas joined #puppet-openstack
21:44 hogepodge joined #puppet-openstack
21:56 sputnik13net joined #puppet-openstack
21:58 mgagne hogepodge: ping
21:58 hogepodge hey
21:58 mgagne hogepodge: https://review.openstack.org/#/c/65379/1
21:58 hogepodge yeah
21:58 mgagne hogepodge: what do you think about the -1 ?
21:58 hogepodge I think it's wrong
21:59 hogepodge +> is used to append to an existing array, and no such array exists.
21:59 mgagne fc__: ping
21:59 hogepodge So => is the correct code
21:59 mgagne hogepodge: unless fc__ has a use case for it, I'll merge it
21:59 hogepodge Ok. -1 isn't a blocker for merging?
21:59 mgagne hogepodge: no, -2 is
22:00 hogepodge Oh yeah, I see. :-)
22:02 EmilienM mgagne: hogepodge: i don't think fc__ is here at this time, it's 11pm by his time
22:02 hogepodge yeah. late
22:02 hogepodge do you have opinions on the patch emilienm?
22:02 EmilienM hogepodge: i do agree with you
22:03 EmilienM mgagne: do you think i should shake something to have the puppet 3.4 patch merged into openstack-infra ?
22:03 mgagne EmilienM: just poke people in infra
22:03 EmilienM mgagne: already did, but i don't like repeat myself
22:03 mgagne EmilienM: I've nothing else to suggest =)
22:05 EmilienM mgagne: i don't know why but i thought you was infra/core
22:05 EmilienM s/was/were/
22:05 mgagne EmilienM: I'm not, I just hangout too much in infra =)
22:05 EmilienM mgagne: heh
22:05 dmsimard EmilienM: He already has enough things to do :(
22:06 xarses joined #puppet-openstack
22:06 mgagne yep, I kind of stack overflowed recently
22:07 EmilienM being core in infra is like... being root on all openstack
22:07 dmsimard It's a full time job :)
22:13 dmsimard EmilienM: Have a question for you
22:13 mgagne EmilienM: merged =)
22:13 EmilienM dmsimard: tell me
22:13 EmilienM mgagne: w00t
22:14 dmsimard EmilienM: When using the ceilometer middleware with swift, swift needs access to /etc/ceilometer (configuration files) as well as /var/log/ceilometer. Do you think we should implement something directly in swift::proxy::ceilometer ?
22:14 EmilienM dmsimard: why does it should?
22:14 EmilienM IMO, user permissions are up to packaging stuff.
22:14 EmilienM mgagne: ^
22:15 mgagne EmilienM: I don't think the middleware runs as ceilometer anyway...
22:15 EmilienM yep
22:15 EmilienM mgagne: puppet-swift should not configure ceilometer
22:15 dmsimard EmilienM: Well, we already add the swift user to ceilometer group: https://github.com/stackforge/puppet-swift/blo​b/master/manifests/proxy/ceilometer.pp#L22-L24
22:15 mgagne EmilienM: it should run as swift or whatever swift-proxy is running as. ceilometer middleware for swift isn't an agent/daemon, it's a middleware
22:15 mgagne dmsimard: it's not the same
22:16 EmilienM mh i see
22:16 dmsimard EmilienM: But that does not grant write permissions
22:16 mgagne dmsimard: oh.... I see
22:16 mgagne EmilienM: ceilometer middleware looks to need read access to ceilometer config
22:17 mgagne EmilienM: and as it's running under swift, you need to add it to ceilometer group
22:17 dmsimard mgagne / EmilienM: proxy without access to /var/log/ceilometer: http://paste.openstack.org/show/61146/
22:17 EmilienM mgagne: read is disabled ? it's like 750 ?
22:17 EmilienM it makes sense at all I think
22:17 mgagne EmilienM: should be 750
22:18 dmsimard mgagne / EmilienM: proxy without access to /etc/ceilometer: Jan 13 22:10:48 proxy01 proxy-server STDOUT: 2014-01-13 22:10:48 CRITICAL [ceilometer] coercing to Unicode: need string or buffer, NoneType found
22:18 EmilienM sounds like dmsimard has a patch in the pipe :P
22:18 dmsimard Permissions for /etc/ceilometer are 755 and swift writes 3 files in it
22:19 dmsimard It seems like it reads the ceilometer config file and writes pipeline.yaml, policy.json and sources.json
22:19 tnoor joined #puppet-openstack
22:19 dmsimard Well, obviously I need to do something about this cause it's bothering my setup — but my question was where should we be handling this
22:20 dmsimard I think what makes sense is in swift::proxy::ceilometer
22:20 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: puppet-3-4 test: do not review  https://review.openstack.org/66450
22:20 mgagne /etc/ceilometer and /var/log/ceilometer are 750
22:21 dmsimard ceilometer:ceilometer
22:21 dmsimard Adding swift to ceilometer group does not grant write permissions with 750
22:23 EmilienM mgagne: looks like puppet-3.4 gate is not ready yet > https://review.openstack.org/#/c/66450/
22:23 EmilienM maybe should we wait something? like a periodic update or..?
22:23 mgagne EmilienM: just give it some time
22:23 dmsimard Yeah they're both 750, not 755. My bad.
22:23 mgagne EmilienM: JJB hasn't run yet
22:24 EmilienM mgagne: sure.
22:24 EmilienM mgagne: here, we use a git hook
22:24 mgagne EmilienM: git hook doesn't scale =)
22:24 EmilienM mgagne: for that, it's ok, but I agree otherwise
22:25 mgagne EmilienM: imagine 10 people committing at the same time, JJB update jobs would just pile-up
22:25 EmilienM yeah
22:25 EmilienM mgagne: but not the case here :-)
22:25 dmsimard So, should I be setting the directories as 770 inside swift::proxy::ceilometer ? Make swift the folder owner and leave it 750 ?
22:33 dmsimard mgagne / EmilienM ^ Do you have an opinion ?à
22:33 mgagne dmsimard: no
22:33 mgagne dmsimard: don't
22:34 mgagne dmsimard: what's the official procedure to install ceilometer on swift?
22:35 dmsimard mgagne: http://docs.openstack.org/developer/ceilome​ter/install/manual.html#installing-manually
22:35 thumpba joined #puppet-openstack
22:38 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: puppet-3-4 test: do not review  https://review.openstack.org/66450
22:40 EmilienM ok puppet 3.4 is not part of gate. great
22:41 mgagne is part*
22:41 mgagne is now*
22:42 EmilienM mgagne: i'm jetlaged, sorry :P
22:42 EmilienM mgagne: i'm in MTL :)
22:43 mgagne EmilienM: oh fun =)
22:43 dmsimard Oh, we should meet then. :P
22:43 mgagne EmilienM: at least, it's not -40C atm
22:43 EmilienM mgagne: my first fall on ice though
22:44 mgagne EmilienM: haha, yes, glaze ice :D
22:44 EmilienM dmsimard: when you want :-)
22:44 EmilienM i still wonder how you manage that stuffs :)
22:44 dmsimard If you're at Enovance's office you're like 15 minutes away
22:44 EmilienM dmsimard: so we should take lunch together
22:45 EmilienM like, tomorrow?
22:45 dmsimard I dunno, I hardly have time to lunch recently :P
22:48 EmilienM dmsimard: let's see tomorrow morning
22:48 EmilienM cya
22:53 thumpba joined #puppet-openstack
22:55 mattymo joined #puppet-openstack
22:57 thumpba joined #puppet-openstack
22:57 rharrison joined #puppet-openstack

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