Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2013-08-01

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

All times shown according to UTC.

Time Nick Message
00:16 michchap joined #puppet-openstack
02:23 michchap joined #puppet-openstack
02:57 michchap joined #puppet-openstack
03:00 michchap joined #puppet-openstack
03:24 michchap joined #puppet-openstack
03:47 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-glance: Update allowed_hosts conditional statement  https://review.openstack.org/39263
03:55 otherwiseguy joined #puppet-openstack
04:12 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-glance: Update allowed_hosts conditional statement  https://review.openstack.org/39263
05:08 openstackgerrit A change was merged to stackforge/puppet-glance: Update allowed_hosts conditional statement  https://review.openstack.org/39263
05:45 openstackgerrit A change was merged to stackforge/puppet-glance: Add glance-cache-cleaner and glance-cache-pruner  https://review.openstack.org/38373
05:50 openstackgerrit A change was merged to stackforge/puppet-openstack: Horizon needs this selinux boolean set  https://review.openstack.org/39202
05:56 michchap joined #puppet-openstack
06:00 michchap joined #puppet-openstack
06:10 e1mer joined #puppet-openstack
06:38 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-nova: Update allowed_hosts conditional statement  https://review.openstack.org/39260
06:45 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-cinder: Update allowed_hosts conditional statement  https://review.openstack.org/39261
06:49 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-keystone: Update allowed_hosts conditional statement  https://review.openstack.org/39256
06:50 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-cinder: Update allowed_hosts conditional statement  https://review.openstack.org/39261
06:58 starmer joined #puppet-openstack
07:08 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-quantum: Update allowed_hosts conditional statement  https://review.openstack.org/39639
07:23 openstackgerrit Robert Starmer proposed a change to stackforge/puppet-glance: Fix the glance_image type example so that it can actually be used.  https://review.openstack.org/39642
07:30 e1mer joined #puppet-openstack
07:44 e1mer joined #puppet-openstack
07:47 e1mer joined #puppet-openstack
08:27 derekh joined #puppet-openstack
11:06 openstackgerrit Xingchao Yu proposed a change to stackforge/puppet-nova: Add nova::scheduler::filter for managing scheduler.filter  https://review.openstack.org/36971
13:03 _ilbot joined #puppet-openstack
13:03 Topic for #puppet-openstack is now Place to collaborate on Puppet/OpenStack tools: logs at http://irclog.perlgeek.de/puppet-openstack/today
13:47 openstackgerrit loganmc10 proposed a change to stackforge/puppet-nova: Use the correct vif_driver default  https://review.openstack.org/39132
13:47 dmsimard joined #puppet-openstack
13:52 prad_ joined #puppet-openstack
13:55 otherwiseguy joined #puppet-openstack
14:52 michchap joined #puppet-openstack
14:53 michchap joined #puppet-openstack
15:10 EmilienM mgagne: o/
15:27 mgagne EmlienM: bonjour
16:12 starmer joined #puppet-openstack
17:10 openstackgerrit Francesco Vollero proposed a change to stackforge/puppet-ceilometer: Adding support for qpid in ceilometer, rspec for qpid part in ceilometer  https://review.openstack.org/39689
17:20 openstackgerrit A change was merged to stackforge/puppet-swift: Add signing dir  https://review.openstack.org/38877
17:44 openstackgerrit loganmc10 proposed a change to stackforge/puppet-cinder: Setup test volume fails without lvm2  https://review.openstack.org/39761
18:26 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet-openstack: Fix Keystone 'token-get' Error  https://review.openstack.org/39566
18:53 otherwiseguy joined #puppet-openstack
19:01 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet-quantum: Adds root_helper parameter for DHCP Agent  https://review.openstack.org/39400
19:01 mgagne bodepd: "Add signing dir" got merged
19:39 otherwiseguy joined #puppet-openstack
19:58 mgagne joined #puppet-openstack
20:06 mgagne bodepd: ping
20:09 starmer joined #puppet-openstack
20:20 otherwiseguy joined #puppet-openstack
20:22 guimaluf joined #puppet-openstack
20:44 prad_ joined #puppet-openstack
20:55 ayoung joined #puppet-openstack
20:56 ayoung bodepd, how would I go about specifying a change to the Keystone puppet module to be able to install Keystone in HTTPD as opposed to the Eventlet approach?
20:58 EmilienM mgagne: bonsoir
20:59 EmilienM I was wondering if I miss something for pushing puppet-heat into stackforge
20:59 bodepd ayoung: https://review.openstack.org/#/c/29059/
20:59 bodepd ayoung: did you notice that patch?
20:59 bodepd ayoung: it's really been languishing
21:00 bodepd ayoung: can you try it out and then yell at us to merge it?
21:00 bodepd mgagne: I saw that
21:00 bodepd mgagne: that patch does not work as-is for cisco atm, but I could not recreate
21:00 bodepd mgagne: are you pinging me to tell me to run CI on your swift patch and merge it )
21:00 mgagne bodepd: is it a permission issue?
21:00 bodepd mgagne: yes
21:00 mgagne bodepd: well try to see if the CI issue is fixed by this change or not
21:02 ayoung bodepd, awesomesauce
21:03 bodepd mgagne: I meant the swift patch (not the mod_wsgi path does not work)
21:03 bodepd mgagne: I already did.
21:03 bodepd mgagne: that patch gets CI up and running
21:04 bodepd (ran a test this morning)
21:04 mgagne bodepd: *cough* https://review.openstack.org/#/c/35219/ *cough*
21:04 bodepd https://github.com/CiscoSyste​ms/puppet-swift/pull/6/files
21:04 bodepd probably needs to be sorted first
21:05 mgagne bodepd: ok. is the manual giving any hint on which permissions or ownership to set?
21:05 ayoung bodepd, dolphm and bknudson are Keystone core.  Their comments are correct on that patch.
21:06 ayoung fc__, let me know when you are around and I can work with you on https://review.openstack.org/#/c/29059/11
21:07 mgagne bodepd: juju sets ownership to swift:swift
21:08 mgagne bodepd: http://docs.openstack.org/developer​/swift/howto_installmultinode.html
21:08 mgagne bodepd: "Change the ownership of the directory /var/cache/swift to the user and group which Swift account, container or object services will run under."
21:09 bodepd ayoung: he's CET, so it'll be a while :)
21:17 fc__ ayoung: hi
21:18 fc__ ayoung: how can I help you about this patch ?
21:19 fc__ ayoung: won't be available too long tonight & will be afk for the next weeks as I'm on vacation until august 19th
21:25 openstackgerrit joined #puppet-openstack
21:37 ayoung fc__, are there any question on the review comments?  THe big one is, I think, picking up the wsgi file from Keystone as opposed to keeping it in the puppet module
21:37 ayoung fc__ just drop this https://review.openstack.org/#/​c/29059/11/files/httpd/keystone.py
21:39 fc__ ayoung: indeed, this file was here just because it was not packaged at the time I wrote this patch. I know it's packaged for Debian now, but I'm not sure about other distros.
21:39 ayoung fc__, Fedora better have it...
21:39 ayoung let me look
21:42 starmer joined #puppet-openstack
21:43 ayoung fc__, nope, you are right.  It ain't there.  Grumple Gurumumle
21:43 fc__ ayoung: ^_^
21:44 ayoung fc__, where does debian stick it?
21:44 fc__ ayoung: let me check
21:44 EmilienM mgagne: still here ?
21:48 mgagne EmilienM: yes
21:51 fc__ ayoung: Debia installs it in /usr/lib/$python-version/dist-pac​kages/keystone/httpd/keystone.py
21:53 ayoung fc, hmmm, that puts it into the keystone namespace.  Is that right for HTTPD?
21:53 fc__ ayoung: basically, with other keystone .py files
21:53 ayoung I had it as top level for apache
21:53 fc__ ayoung: I didn't have the occation to test this package yet :(
21:53 ayoung fc__, where were you putting the file?
21:53 EmilienM mgagne: do you know if I could do something else to push puppet-heat in stackforge, or just wait that my 2 patchs are merged ?
21:54 ayoung When I ran it, I put it in /var/www/cgi-bin
21:54 ayoung fc__, let me see what some other projects do.  I think that FreeIPA put the comparable file into /usr/share
21:54 mgagne EmilienM: You have to wait for approval from a bunch of core reviewers. You can poke them on #openstack-infra.
21:55 EmilienM mgagne: I mean, do I have to patch something else ?
21:57 fc__ ayoung: In fact I'm currently copying this file into /var/www/cgi-bin/ as it has to be named 'admin' or 'main' for each endpoint type
21:58 fc__ ayoung: but installing it in a a directory _not_ depending on python version should be easier
21:58 ayoung /usr/share/ipa/wsgi.py
21:59 ayoung fc__, FreeIPA puts the comparable file ^^
21:59 mgagne EmilienM: no =)
21:59 fc__ ayoung: then /usr/share/keystone/wsgi.py should be a good idea, what's your opinion about this ?
22:01 ayoung fc__, I think that I need to update https://bugzilla.redhat.com/show_bug.cgi?id=892420
22:02 fc__ ayoung: sorry, I don't have an account to see this bug apparently
22:03 ayoung really?  Didn't think it was private.  Let me see
22:04 EmilienM mgagne: thx
22:04 ayoung Ah, that is the RHOS one.  I need to file it against Fedora.  Same Bugzilla, different project.
22:12 ayoung fc__, https://bugzilla.redhat.com/show_bug.cgi?id=991226
22:15 fc__ ayoung: btw, about permissions on /etc/keystone/keystone.conf, I worked these around by running the mod_wsgi workers as the keystone user, what's your opinion about this ?
22:16 fc__ ayoung: Debian packaging updated: http://anonscm.debian.org/gitweb/?p=o​penstack/keystone.git;a=commit;h=ad1a​58fb4a0ad6d4e2cd578d7f71ade4a5a51687
22:23 dmsimard joined #puppet-openstack
22:44 openstackgerrit A change was merged to stackforge/puppet-openstack: Ensure that test networks are shared  https://review.openstack.org/39211
22:45 openstackgerrit Donald Talton proposed a change to stackforge/puppet-cinder: check for init override file existence  https://review.openstack.org/39535
22:46 openstackgerrit A change was merged to stackforge/puppet-glance: Fix the glance_image type example so that it can actually be used.  https://review.openstack.org/39642
22:52 openstackgerrit A change was merged to stackforge/puppet-quantum: Adds root_helper parameter for DHCP Agent  https://review.openstack.org/39400
22:53 bodepd mgagne: I'm not too fussed about it. Cisco is exploring if it might be a packaging issue on their end
22:53 openstackgerrit A change was merged to stackforge/puppet-nova: Add nova::scheduler::filter for managing scheduler.filter  https://review.openstack.org/36971
22:53 bodepd let me test Yu's code and try to get it merged first
22:53 mgagne bodepd: context plz =)
22:53 bodepd and I'll try to get a test running tonight for you
22:53 bodepd ah, the swift perms issue
22:54 bodepd and also, you're poor sad swift patch :()
22:54 mgagne bodepd: does it need rebase?
22:55 bodepd mgagne: I'm sure it does :)
23:03 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet-openstack: Adds Support for Keystone SSL and Token Format  https://review.openstack.org/39831
23:04 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet-openstack: Adds Support for Keystone SSL and Token Format  https://review.openstack.org/39831
23:19 openstackgerrit A change was merged to stackforge/puppet-cinder: Update allowed_hosts conditional statement  https://review.openstack.org/39261
23:22 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-quantum: Improve the L3 Agent support  https://review.openstack.org/39836
23:43 dmsimard1 joined #puppet-openstack
23:46 dmsimard joined #puppet-openstack
23:56 openstackgerrit A change was merged to stackforge/puppet-nova: Update allowed_hosts conditional statement  https://review.openstack.org/39260
23:59 ayoung fc__, had to put the kids to bed.  Keystone should run as apache, and the config file should be owned by by root, and readable by apache.
23:59 ayoung maybe make it group read able?

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