Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-09-18

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

All times shown according to UTC.

Time Nick Message
00:14 thumpba joined #puppet-openstack
00:19 xingchao joined #puppet-openstack
00:36 aimon joined #puppet-openstack
00:38 gildub joined #puppet-openstack
00:53 cjellick joined #puppet-openstack
01:25 rmoe joined #puppet-openstack
01:26 xingchao joined #puppet-openstack
01:50 hogepodge left #puppet-openstack
02:04 xingchao joined #puppet-openstack
02:21 openstackgerrit joined #puppet-openstack
03:07 hogepodge joined #puppet-openstack
03:57 gildub joined #puppet-openstack
04:17 mmaglana joined #puppet-openstack
05:30 sputnik13 joined #puppet-openstack
05:30 alokj joined #puppet-openstack
06:08 gchamoul joined #puppet-openstack
06:46 gcha joined #puppet-openstack
06:51 sanjayu joined #puppet-openstack
06:55 arnaud_orange joined #puppet-openstack
06:56 tfz joined #puppet-openstack
06:57 pieterl joined #puppet-openstack
07:09 alokj joined #puppet-openstack
07:17 Alremovi4 joined #puppet-openstack
07:23 berendt joined #puppet-openstack
07:26 dachary joined #puppet-openstack
07:31 mmagr joined #puppet-openstack
07:33 francois joined #puppet-openstack
07:58 francois1 joined #puppet-openstack
08:00 peterstac joined #puppet-openstack
08:00 mmagr sbadia, mgagne, EmilienM: Hi guys, anyone from you is online?
08:25 derekh joined #puppet-openstack
08:31 daneyon joined #puppet-openstack
08:35 daneyon_ joined #puppet-openstack
08:56 daneyon joined #puppet-openstack
09:22 qba73 joined #puppet-openstack
10:05 tfz joined #puppet-openstack
10:05 sputnik13 joined #puppet-openstack
10:27 openstackgerrit Martin Mágr proposed a change to stackforge/puppet-heat: Implement Keystone domain creation  https://review.openstack.org/103127
10:33 openstackgerrit Martin Mágr proposed a change to stackforge/puppet-nova: Disable file injection when using RBD as compute ephemeral storage  https://review.openstack.org/113164
10:44 gildub joined #puppet-openstack
11:20 openstackgerrit Martin Mágr proposed a change to stackforge/puppet-nova: Disable file injection when using RBD as compute ephemeral storage  https://review.openstack.org/113164
11:20 openstackgerrit Martin Mágr proposed a change to stackforge/puppet-nova: Install ceph client libraries when using rbd  https://review.openstack.org/105649
11:43 7GHAA33FA joined #puppet-openstack
11:48 walkleyn joined #puppet-openstack
11:58 igajsin joined #puppet-openstack
11:59 EmilienM mmagr: o/
12:06 eflynn joined #puppet-openstack
12:24 mmagr EmilienM, Hi Emilien, I've got one question for you, can you please check comment in https://review.openstack.org/#/c/115652/3
12:24 mmagr ?
12:27 EmilienM mmagr: ok
12:27 mmagr thx
12:28 EmilienM mmagr: yes, to me it makes sense that tests are for all classes
12:28 EmilienM mmagr: I know it's dupplicated code but...
12:29 mmagr EmilienM, so I should proceed with refactor?
12:29 EmilienM mmagr: yes I think so
12:30 mmagr EmilienM, ok, thanks
12:43 alokj joined #puppet-openstack
12:47 openstackgerrit A change was merged to stackforge/puppet-glance: Make user creation optional when creating service.  https://review.openstack.org/116263
12:49 openstackgerrit A change was merged to stackforge/puppet-cinder: Make user creation optional when creating service.  https://review.openstack.org/116264
13:04 morazi joined #puppet-openstack
13:13 derekh joined #puppet-openstack
13:21 rharrison joined #puppet-openstack
13:46 derekh joined #puppet-openstack
13:53 tfz joined #puppet-openstack
13:55 imcsk8 joined #puppet-openstack
13:57 admin0 joined #puppet-openstack
14:08 pradk joined #puppet-openstack
14:12 ericpeterson joined #puppet-openstack
14:16 sputnik13 joined #puppet-openstack
14:37 gildub joined #puppet-openstack
14:38 openstackgerrit Eric Peterson proposed a change to stackforge/puppet-horizon: adding db paramaters and session engine config  https://review.openstack.org/122256
14:43 EmilienM ericpeterson: hey, could you add unit tests in your patch?
14:48 ericpeterson sure, will take a bit to figure out how to do that ;)
14:57 mdorman joined #puppet-openstack
15:06 EmilienM mmagr: https://review.openstack.org/#/c/103127 is really good.
15:09 mmagr EmilienM, thanks, but it was only copy/paste of nova_admin_id_setter from puppet-neutron :)
15:21 mmagr EmilienM, btw wrt puppet-ceilometer manage_service, I found cleaner way to implement tests, will submit next patchset in the meantime
15:22 EmilienM ok
15:24 cjellick joined #puppet-openstack
15:44 mmaglana joined #puppet-openstack
15:49 tfz joined #puppet-openstack
16:03 openstackgerrit joined #puppet-openstack
16:04 rmoe joined #puppet-openstack
16:05 openstackgerrit A change was merged to stackforge/puppet-nova: Add quotes for bareword instances of 'type'  https://review.openstack.org/122262
16:05 openstackgerrit A change was merged to stackforge/puppet-keystone: Duplicate definition of parameters in ldap.pp  https://review.openstack.org/122155
16:13 aimon joined #puppet-openstack
16:18 daneyon joined #puppet-openstack
16:35 openstackgerrit Eric Peterson proposed a change to stackforge/puppet-horizon: adding db paramaters and session engine config  https://review.openstack.org/122256
16:37 ericpeterson EmilienM I attempted to get unit tests working, but I am missing something ^   if you find my problem, please publicly shame me here
16:37 ericpeterson (new to puppet and unit testing this stuff)
16:44 mdorman ericpeterson:  are you saying the unit test doesn’t work?
16:44 ericpeterson I am saying my attempt at adding unit tests was a partial failure
16:45 mdorman i mean, it seems pretty good to me <?>   or were you wanting to test each new parameter individually?
16:46 ericpeterson It seemed like what I needed, but for some reason the tests fail for me.  seems like there is some test case magic I am missing
16:48 mdorman i see.  but the tests succeed in jenkins after you git review, right?
16:50 ericpeterson I don't think so.  they fail locally, and I would expect them to fail in jenkins as well.  The failure I am getting from the tests don't seem to make sense (to me)
16:51 mdorman ok
16:52 mdorman i’ll try to pull it down in a little bit here and check it out
16:52 mdorman i am not the greatest at rspec tests, either.   but have worked my way through a few weird failures in the past
16:52 mdorman i think you and i met at the openstack meetup last night, right?
16:52 ericpeterson in FC, right?
16:53 ericpeterson I think so, yeah :D
16:53 mdorman yup :)
17:14 photocyte joined #puppet-openstack
17:29 mdorman ericpeterson:  pretty sure the trouble here is just matching on the whitespace correctly
17:32 ericpeterson thanks mdorman.  I will count my spaces
17:33 mdorman i’mtrying to get it to work on my end, too, just to see.  i think it might be an issue of tabs vs. spaces, too.
17:33 openstackgerrit joined #puppet-openstack
17:33 ericpeterson yeah, that's what the console output led me to believe.... but I couldn't find where tabs were used
18:02 photocyte joined #puppet-openstack
18:06 tfz joined #puppet-openstack
18:15 mdorman ericpeterson:  having any luck?
18:17 ericpeterson not so far, but had to grab lunch
18:20 openstackgerrit Mike Dorman proposed a change to stackforge/puppet-ceilometer: Stringify $evaluation_interval for validate_re cal  https://review.openstack.org/122490
18:20 openstackgerrit Mike Dorman proposed a change to stackforge/puppet-ceilometer: Stringify $evaluation_interval for validate_re call  https://review.openstack.org/122490
18:22 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-neutron: Enable DVR + HA support  https://review.openstack.org/122491
18:22 mdorman ericpeterson:  cool.  i’ve been messing with it off an on here and am still having trouble.  not quite sure what’s up
18:23 admin0 joined #puppet-openstack
18:28 hogepodge joined #puppet-openstack
18:29 openstackgerrit Mike Dorman proposed a change to stackforge/puppet-ceilometer: Stringify $evaluation_interval for validate_re call  https://review.openstack.org/122490
18:49 photocyte joined #puppet-openstack
18:55 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-neutron: Enable DVR + HA support  https://review.openstack.org/122491
19:03 Mosibi joined #puppet-openstack
19:06 mdorman ericpeterson:  think i figured it out.  needed to squash the whitespace in front of the <% if and <% end tags in the template.  b/c it was including that extra whitespace in the config file, so not matching correctly in the spec
19:07 openstackgerrit Clayton O'Neill proposed a change to stackforge/puppet-horizon: Adding db paramaters and session engine config  https://review.openstack.org/122256
19:07 ericpeterson cool.  I was trying <%- and -%> variations
19:07 ericpeterson I think Clayton just got us step farther as well
19:08 mdorman yup i just saw his patchset come through
19:08 mdorman well glad it got figured out.
19:10 ericpeterson thanks to both of you
19:22 openstackgerrit Clayton O'Neill proposed a change to stackforge/puppet-horizon: Adding db paramaters and session engine config  https://review.openstack.org/122256
19:23 openstackgerrit Eric Peterson proposed a change to stackforge/puppet-horizon: Adding db paramaters and session engine config  https://review.openstack.org/122256
20:04 prad joined #puppet-openstack
20:36 mfisch can anyone explain why the local settings.py creation in puppet horizon is wrapped by a random variable conditional?
20:36 mfisch https://github.com/stackforge/puppet-horizon/blob/master/manifests/init.pp#L307
20:36 mfisch I'd think you always want to make that file and/or that test seems wrong
20:50 admin0 joined #puppet-openstack
21:05 mdorman seems like a bug
21:07 mdorman https://github.com/stackforge/puppet-horizon/commit/589a9ddf7db6a2468435b42d22808589f6617017
21:08 openstackgerrit joined #puppet-openstack
21:18 openstackgerrit Eric Peterson proposed a change to stackforge/puppet-horizon: Adding db paramaters and session engine config  https://review.openstack.org/122256
21:19 cjellick joined #puppet-openstack
21:20 ericpeterson I amended previous checkin so that puppet would automatically run the django db sync command for you, fwiw
21:28 gaudenz joined #puppet-openstack
21:28 snguye2 joined #puppet-openstack
21:28 fvollero joined #puppet-openstack
21:29 jamespd joined #puppet-openstack
21:29 Qu4tre joined #puppet-openstack
21:29 sbadia joined #puppet-openstack
21:49 snguye2 joined #puppet-openstack
21:53 dvorak ericpeterson: and now you have to write more tests, what were you thinking?
21:53 ericpeterson I have defeated myself, it seems
21:54 dvorak that line around if $compress_offline is a bug in the existing module I think
21:54 dvorak I suspect no one has ever tried to use that.
21:55 dvorak it's set to "True" so I assume the opposite value would be "False"
21:55 dvorak which would still be true from a puppet perspective
21:58 rharrison joined #puppet-openstack
21:58 mdorman anyone ever seen this error?   I’m on 3.7.1 with future parser.  Error: This Array Expression is not productive. A non productive construct may only be placed last in a block/sequence
21:59 mdorman it references the first character on the first line of a manifest file.  it doens’t make any sense to me
21:59 mfisch dvorak: that line in the existing module is in response to an old debian bug which has likely since been fixed but we should ask the original guy who's probably asleep
21:59 dvorak you have a link?
22:00 mfisch yeah one sec
22:00 dvorak if so, that should be conditionalized
22:00 dvorak if I turn off compress_offline, I should still get a config file
22:00 mfisch https://github.com/stackforge/puppet-horizon/commit/720ce5cf845d57ce04d9d602cbd58b45f3144121
22:00 mfisch bug referenced here
22:00 dvorak so, adding that exec is completely reasonable
22:00 mfisch I was at a loss to explain it to eric but that commit explains it a bit, although it seems like not the most clear solution to the bug
22:01 ericpeterson the bug we saw where horizon's css was all messed up the other day, it's very related to the compress offline (funfact)
22:01 mfisch mdorman: I hope you are taking notes that we can copy when we move to 3.7
22:01 dvorak so, putting a trigger in place so that the cache is updated when the config file is updated is reasonable
22:01 mfisch new^h^h^h existing strategy: let mdorman find all bugs before we get to them…
22:02 dvorak making the config file not get generated unless you're using compress_offline seems non-sensical
22:02 mfisch the moving into the conditional Im still not sure about
22:02 mfisch we should ping goneri tomorrow if he's on IRC
22:02 dvorak it also has the exec subscribing to the file changes, and the file notifying the exec
22:02 cjellick joined #puppet-openstack
22:02 dvorak you only need one of them
22:03 dvorak that's what is making eric's change so gross
22:03 ericpeterson plus I am not too shabby at making it gross too
22:03 ericpeterson just sayin'
22:04 mdorman mfisch: :)
22:05 mdorman do you guys use the future parser?
22:05 dvorak no, we're not that brave :)
22:05 mdorman i have been using it for a long time b/c i wanted to use the ‘each’ construct
22:05 dvorak I've been thinking about it, but I assumed we'd have lot sof problems with third party modules
22:05 mfisch foreach loops??????  SIGN ME UP!
22:05 dvorak haha
22:06 mfisch no im serious
22:06 mfisch I miss them
22:06 dvorak yeah, there are a bunch of nice things like that
22:06 mfisch mdorman: are you going to post some notes to the puppet-openstack ML when you figure some of it out?
22:06 dvorak mdorman: have you had problems with 3rd party modules?
22:06 mfisch when do I get inline python?
22:06 mfisch 3.9?
22:06 dvorak 999.never
22:07 ericpeterson list comprehension with a generator
22:07 dvorak I suspect it'll go 3.7 -> 3.8 (maybe) -> 4.0
22:07 mfisch I'll put in a good word for it at the conference
22:08 mdorman haha
22:08 mdorman yeah i can post some stuff.  so far i’m not having too much trouble with 3.7.  pretty much all my gerrit reviews from the last few days are related to 3.7 stuff
22:09 mdorman though, i suspect most of that is becuase i’m using the future parser (approximately the puppet 4.0 language) insated of the current parser
22:11 dvorak I've been heads down w/gerrit and jenkins for the past month or so, so not much puppet work outside of automating those two.  I want to upgrade to 3.7.1 when we get back from puppetconf
22:11 mdorman well, i take that back.  a lot of it is purely 3.7 stuff
22:12 mdorman hopefully by that time most of my stuff will be merged so you won’t have too much pain
22:12 mdorman are you guys using stable/icehouse modules, or master?
22:12 dvorak stable/icehouse
22:13 mdorman k.  i guess i should also backport this stuff there
22:13 dvorak we'd appreciate that :)
22:13 mdorman i have never done a backport, so i gotta figure out how to do that.  so you just branch from stable/icehouse instead of master before you do your review commit?
22:16 dvorak you can just cherry-pick it to the stable branch right from gerrit
22:16 dvorak it's super easy
22:27 rharrison joined #puppet-openstack
22:29 openstackgerrit Eric Peterson proposed a change to stackforge/puppet-horizon: Adding db paramaters and session engine config  https://review.openstack.org/122256
22:29 ericpeterson dvorak: one line test case added, I think I'm done here
23:00 mdorman fwiw that weird Array Expression error was caused by trailing commas on lines where variables were being assigned.  like:   $var = ‘value’,
23:03 ashp markvoelker: Hey, do you know the other people doing puppet stuff at cisco.  Specifically the nexus switch guys?
23:03 ashp they are located in MA if that helps
23:03 dvorak mdorman: weird. must be a cut and paste error
23:06 mdorman yeah i think originally they had been class parameters, but turned into local variables at one point
23:57 gildub joined #puppet-openstack

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