Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2012-11-07

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

All times shown according to UTC.

Time Nick Message
11:56 _ilbot joined #puppet-openstack
12:02 moritz hi
12:02 moritz by request from pdurbin I've put up public logs of this channel at http://irclog.perlgeek.de/puppet-openstack/today
12:02 moritz if there's anything you want or need regarding the logs, or want the logging to stop, please /msg me
12:03 moritz it would also be nice if you could mention the logs in the /topic, so that newcomers see that this channel is logged
12:03 * moritz waves
12:03 moritz left #puppet-openstack
12:31 aimon nice! :)
12:35 pdurbin :)
12:35 pdurbin i describe the next steps here: https://groups.google.com/a/puppetlabs.com/forum/?fromgroups=#!topic/puppet-openstack/SB7ZU3TG9lg
12:36 pdurbin 1. we should add "logs at http://irclog.perlgeek.de/puppet-openstack/today " to the /topic
12:36 pdurbin 2. we can pick a link to appear at the top of the logs
13:55 Elwell joined #puppet-openstack
15:48 JoeJulian joined #puppet-openstack
16:20 JoeJulian bodepd: Passing by class parameter is the most efficient. Everything else is going to require adding a bunch of logic which just adds cycles and, imho, makes for less readable code. Personally, I don't see a problem with humongous lists of parameters as long as there are only a few that are required.
16:55 Topic for #puppet-openstack is now Place to collaborate on Puppet/OpenStack tools: logs at http://irclog.perlgeek.de/puppet-openstack/today
17:20 dachary joined #puppet-openstack
17:21 dachary joined #puppet-openstack
17:31 JoeJulian bodepd: Another idea I had would be to do a ::params class and allow that class to be overridden.
17:32 JoeJulian You also suggested that I was working against an old copy with my poc yesterday. It's current per the puppetlabs git tree. Would you prefer I patch against yours?
17:44 JoeJulian or maybe I'm on a different machine looking at a completely different checkout...
17:52 JoeJulian bodepd: It looks like the only place you build urls is in puppetlabs-openstack. I can't find any in -nova, -rabbitmq, or -glance. Any thoughts on other modules that might need the uriescape?
18:05 comptona joined #puppet-openstack
18:25 bodepd JoeJulian: please patch against master from puppetlabs
18:26 bodepd everything that has a sql_connnection param, I thikn
18:26 bodepd I am happy if that validation only occurs in the openstack modules.
18:29 JoeJulian Hrm... You're really trying to make me work for this aren't you... ;)
18:29 JoeJulian You're right, of course.
18:39 jtopjian joined #puppet-openstack
19:22 JoeJulian I walked away, made myself a mocha, and mulled on that a bit. I'm no longer sure I agree. How can you tell the difference between a valid url, already uri escaped, and an invalid one that still needs escaping? I think it's reasonable to expect the url to be valid.
19:55 bodepd where in the code do you expect that urls will be valid?
20:01 JoeJulian In the class parameters that accept urls, like nova sql_connection.
20:02 bodepd and not in the openstack module?
20:03 JoeJulian The openstack module builds the url from username and password and that password needs to be used in various places unescaped.
20:08 JoeJulian For instance: class { 'nova': sql_connection => 'mysql://myuser:DUykXDJ5hAeCTjLUn%2B16AkmkXErx9QHMq7YbghWUMts%3D@127.0.0.1/db1' } should be expected vs class{'openstack': glance_db_password => "DUykXDJ5hAeCTjLUn+16AkmkXErx9QHMq7YbghWUMts="}
20:09 JoeJulian This has come up with several of my peers. It seems a lot of us use SHA hashes to make random passwords.
20:15 bodepd and is there a problem with just doing the encoding in the openstack module only?
20:17 JoeJulian Ah, never mind... I missed the single line of this conversation that has caused way too much miscommunication. I apologize. I read "everything that has a sql_connnection param, I think" and so I was thinking you wanted sql_connection validated in every module. I completely missed the next line. My only excuse is that I hadn't had my coffee yet. :/
20:21 JoeJulian Yep, I don't need to include the class. It's only the parser that's required. I also updated the Modulefile to reflect that dependency.
20:25 Elwell joined #puppet-openstack
20:29 JoeJulian By the way, if you haven't looked at Jeff McCune's stdlib, it's a great tool to be able to make use of.
21:31 e1mer joined #puppet-openstack
22:11 aimon hoped pardon my git lameness..
22:11 aimon lol auto correct!
22:11 aimon *boped
22:12 aimon have a rebased PR in
22:22 bodepd aimon: I saw it. Can you add a test?
22:22 bodepd JoeJulian: I have contributed lots of stuff to it.
22:22 bodepd JoeJulian: did he get your patch merged in?
22:23 comptona bodepd: I got my boss to approve my submitting puppet module patches
22:23 comptona once I get the bugs shaken out of folsom on centos6 here, I'll sent you pulls with my changes
22:24 bodepd comptona: great.
22:24 bodepd I would also get derekh involved as well.
22:25 comptona noted
22:32 JoeJulian Yes. He said he'll be pushing a minor version upgrade to forge today that includes it.
22:36 aimon Boped.. yes I have a test I am putting in separate PR for
22:37 aimon .. part of my rebase snafu.. I need practice on that
22:50 comptona interesting
22:51 comptona I'm having a problem with horizon not being able to track down the admin endpoint with keystone
22:52 comptona it looks like it's calling keystone /v2.0/tokens without specifying a tenant, to which keystone says "no service catalogs here"
22:52 comptona I'm not sure what the right behavior ought to be here
22:58 bodepd that is weird. it works perfectly for me.
22:59 comptona I suspect I've done something wrong with the setup of keystone
22:59 comptona I had to blow it away at some point and I'll bet a bit of it didn't get recreated
23:00 comptona although; my essex install does the same thing with you don't specify a tenant
23:03 comptona s/with/when
23:56 comptona hm, yeah
23:57 comptona built a new machine and tried the modules from upstream, and I'm getting http://pastebin.com/Z4JNYCpY
23:58 comptona particularly, "unable to retrieve service catalog with token" is printed by keystoneclient when you try to get the service catalog URL without specifying a tenant
23:58 comptona that behavior hasn't changed from essex to folsom, so something about horizon must have done
23:58 comptona *dig dig*

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