Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
23:01 EmilienM mgagne: because of templates?
23:01 mgagne EmilienM: because of poorly structured tests =)
23:02 mgagne EmilienM: sure, templates are always a pain though
23:08 pzimmer left #puppet-openstack
23:27 fvollero EmilienM: go sleep
23:27 EmilienM fvollero: how you know I'm here
23:27 fvollero EmilienM: ^_^
23:28 fvollero EmilienM: I felt a french aura around #puppet-openstack :D
23:28 EmilienM fvollero: end of sprint tomorrow, working late...
23:28 starmer joined #puppet-openstack
23:29 fvollero mgagne: meh to the test... Make code not spec :P
23:29 EmilienM crazy italian
23:29 mgagne fvollero: refactor mostly completed
23:29 fvollero mgagne: \o/
23:29 mgagne fvollero: having issues with shared_examples and verify_contents helper
23:30 EmilienM glad to see mgagne having free time :-)
23:30 fvollero mgagne: sigh... seems the story of my life
23:30 mgagne EmilienM: I don't in fact =)
23:35 fvollero mgagne: the comment on the horizon COMPRESS_OFFLINE about the lack of testing and the -1 was right no?
23:36 mgagne fvollero: ?
23:36 fvollero mgagne: 68977
23:37 mgagne fvollero: I know, I don't understand the meaning of what you said
23:37 mgagne fvollero: you mean adding tests?
23:38 mgagne fvollero: ok, saw your comment
23:38 fvollero mgagne: that the lack of test in the patchset
23:45 EmilienM 'night guys
23:47 xarses_ joined #puppet-openstack
23:48 fvollero EmilienM: Bonne nuit
23:48 * fvollero go to sleep as well
23:48 fvollero night fellas
23:48 ianw joined #puppet-openstack
23:51 fvollero ianw: howdy!!
23:54 starmer joined #puppet-openstack
23:59 dtalton joined #puppet-openstack
00:20 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-horizon: Fixed Horizon SSL issues  https://review.openstack.org/69511
00:26 fvollero mgagne: there's something weird, in the puppetlabs-apache they're using concat::setup just in balancer.pp and they're using from init.pp i'm doing something wrong?
00:26 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-horizon: Fixed Horizon SSL issues  https://review.openstack.org/69511
00:27 fvollero mgagne: because i get puppet errors... :/
00:29 mjblack joined #puppet-openstack
00:39 dtalton2 joined #puppet-openstack
00:41 dtalton3 joined #puppet-openstack
01:48 badiane_ka joined #puppet-openstack
02:00 ianw joined #puppet-openstack
02:08 pzimmer joined #puppet-openstack
02:08 pzimmer left #puppet-openstack
02:24 sputnik13 joined #puppet-openstack
02:33 rmoe joined #puppet-openstack
03:01 pzimmer joined #puppet-openstack
03:01 pzimmer left #puppet-openstack
03:51 saneax_ joined #puppet-openstack
04:12 tnoor joined #puppet-openstack
04:14 tnoor1 joined #puppet-openstack
04:35 parstac_pete joined #puppet-openstack
05:28 thumpba_ joined #puppet-openstack
05:30 thumpba_ joined #puppet-openstack
07:11 starmer joined #puppet-openstack
07:21 starmer_ joined #puppet-openstack
07:48 toabctl joined #puppet-openstack
07:48 toabctl joined #puppet-openstack
08:09 openstackgerrit Kevin Benton proposed a change to stackforge/puppet-horizon: Adds COMPRESS_OFFLINE option to local_settings  https://review.openstack.org/68977
08:09 EmilienM good morning o/
08:43 mmagr joined #puppet-openstack
08:52 starmer joined #puppet-openstack
09:03 piliman974 joined #puppet-openstack
09:05 derekh joined #puppet-openstack
10:19 dgollub joined #puppet-openstack
10:47 dgollub joined #puppet-openstack
11:19 tzn joined #puppet-openstack
11:19 piliman974 joined #puppet-openstack
11:19 tzn Hi there
11:20 tzn bodepd: ping
11:22 mattymo tzn, he's on US west coast time, but I haven't seen any chat messages from him in a week
11:44 dgollub joined #puppet-openstack
12:24 tzn mottymo: anyone else taking care of puppet-tempest?
12:24 tzn mattymo: anyone else taking care of puppet-tempest (sorry for previous)
12:25 EmilienM tzn: i started to write unit tests, but stopped because of timing
12:25 EmilienM tzn: i'm not sure if somebody is using it to be honest
12:26 markvoelker joined #puppet-openstack
12:32 mmagr joined #puppet-openstack
12:50 mattymo sorry I haven't worked on it
12:57 csschwe joined #puppet-openstack
12:57 bauzas joined #puppet-openstack
12:59 dprince joined #puppet-openstack
13:00 fvollero EmilienM: you've an idea who is here that work/contributed on puppetlabs-apache ?
13:02 EmilienM fvollero: not really. Why?
13:02 EmilienM i think chris does
13:04 fvollero EmilienM: is that there's something weird afaik, on the concat, because it's included in balancer.pp but not in init.pp so i was wondering how could it work in init if the private concat::setup is not called there first
13:05 EmilienM fvollero: not sure it's the right place to ask
13:06 EmilienM fvollero: I mean, probably on another puppet IRC chan
13:10 fvollero EmilienM: sure thing
13:15 morazi joined #puppet-openstack
13:31 michchap joined #puppet-openstack
13:45 prad joined #puppet-openstack
13:52 mattymo EmilienM, did you decide to go to FOSDEM?
13:52 EmilienM mattymo: i won't this year
13:53 mattymo ok :(
14:10 fvollero EmilienM: http://developerblog.redhat.com/2014/01/28/​deploy-to-upgrade-puppet-openstack-modules/
14:10 fvollero EmilienM: i'm thinking to skip as well
14:10 rharrison joined #puppet-openstack
14:13 mjblack joined #puppet-openstack
14:18 mjblack joined #puppet-openstack
14:23 EmilienM fvollero: thanks!
14:25 fvollero EmilienM: you know, i like u to be updated :)
14:27 EmilienM fvollero: heh!!
14:27 mjblack so quick question
14:28 mjblack if someone submitted a change for review and they say arent working on it, can I pull down the change update it and push it back for review?
14:32 michchap joined #puppet-openstack
14:33 fvollero mjblack: i guess so
14:38 dgollub joined #puppet-openstack
14:39 michchap joined #puppet-openstack
14:42 dmsimard joined #puppet-openstack
14:55 prad joined #puppet-openstack
15:02 mfisch joined #puppet-openstack
15:05 openstackgerrit François Charlier proposed a change to stackforge/puppet-keystone: Allow binding apache to a specific IP address  https://review.openstack.org/69621
15:06 openstackgerrit François Charlier proposed a change to stackforge/puppet-keystone: Allow binding apache to a specific IP address  https://review.openstack.org/69621
15:12 rwsu joined #puppet-openstack
15:31 fc__ mjblack: if you're around, I just added few notes about https://review.openstack.org/#/c/69511/
15:32 fc__ mjblack: about your earlier question, you can absolutely do this if the someone doesn't answer or is ok with this after trying to get in touch !
15:33 rharrison joined #puppet-openstack
15:39 michchap joined #puppet-openstack
15:51 mjblack fc__: one thing I'm dealing with the apache module, addressing the comments you made, its with the listen statement and ip for vhosts. It looks like it might be an issue with the keystone wsgi if someone does indeed try to use 80/443.
15:51 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-horizon: Fixed Horizon SSL issues  https://review.openstack.org/69511
15:52 fc__ mjblack: oh, the add_listen should be set to true
15:53 mjblack fc__: in the case of 80/443 it should be set to false
15:53 fc__ mjblack: why ?
15:53 fc__ mjblack: oh
15:53 mjblack fc__: cause they're already added
15:54 fc__ mjblack: oh, with the default virtualhost you mean ?
15:54 mjblack fc__: this is the condition statement: if $ip and defined(Apache::Listen[$port])
15:55 mjblack and it throws an error when its already defined
15:56 fc__ mjblack: ok, I see, let me look at the puppetlabs-apache module
15:57 mjblack its not an issue right now so long as the port for keystone is not set to 80/443, I'm just thinking back to adam young's comments about him wanting to default the apache wsgi part to be 80/443
15:59 pzimmer joined #puppet-openstack
15:59 fc__ mjblack: ok, maybe then the default bind_address can be set to 'undef' rather than '0.0.0.0', that should fix the problem for the simple case when the default virtual host is not disabled
15:59 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-horizon: Fixed Horizon SSL issues  https://review.openstack.org/69511
16:00 fc__ mjblack: in fact the error happens because of the default virtual host not being disabled
16:00 mjblack right but we cant force that because in most cases the keystone server is also the dashboard server, and other functions too
16:02 starmer joined #puppet-openstack
16:04 fc__ mjblack: in fact as long as all virtualhost using the same port are either binding just the port or an ip+port, there won't be problems
16:04 fc__ mjblack: there will be problems if one virtualhost binds on just the port (and wildcard ip) and one virtualhost tries to bind on ip+port
16:05 fc__ mjblack: there's a patch currently in review for the same thing in keystone
16:05 mjblack fc__: yeah I saw
16:06 mjblack :)
16:06 mjblack but for that change its a little different since the ports are uncommon ports with apache.
16:07 fc__ mjblack: so I think we should have bind_ip=undef as a default (same meaning as 0.0.0.0 but will please the apache class) and "add_listen => true"
16:07 fc__ mjblack: what do you think about this ?
16:09 mjblack if we leave add_listen true, the only concern with that is if they do specify an ip address
16:09 tnoor joined #puppet-openstack
16:09 mjblack it can create a situation where it will error out
16:11 tnoor1 joined #puppet-openstack
16:11 badiane_ka joined #puppet-openstack
16:13 fc__ mjblack: yes of course, but it'll be a decision to make and apache itself cannot bind *:80 and 10.0.0.1:80
16:14 mjblack yeah
16:14 openstackgerrit François Charlier proposed a change to stackforge/puppet-keystone: Allow binding apache to a specific IP address  https://review.openstack.org/69621
16:16 fc__ mjblack: I update keystone's patch to use undef as default bind ip too ↑
16:16 * fc__ has to leave now. Will review this later tonight
16:17 mjblack yeah I'm doing the same here too
16:20 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-horizon: Fixed Horizon SSL issues  https://review.openstack.org/69511
16:24 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-horizon: Fixed Horizon SSL issues  https://review.openstack.org/69511
16:32 michchap joined #puppet-openstack
16:33 dgollub joined #puppet-openstack
16:34 otherwiseguy joined #puppet-openstack
16:34 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-horizon: Fixed Horizon SSL issues  https://review.openstack.org/69511
16:35 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-horizon: Refactor rspec tests  https://review.openstack.org/69645
16:35 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-horizon: Deprecation of keystone_scheme,host,port params  https://review.openstack.org/69646
16:40 michchap joined #puppet-openstack
16:43 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-horizon: Refactor rspec tests  https://review.openstack.org/69645
16:43 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-horizon: Deprecation of keystone_scheme,host,port params  https://review.openstack.org/69646
16:44 EmilienM mgagne: nice one (horizon)
16:44 mgagne EmilienM: we are testing havana this week so I'll mostly contribute backward compatibility fixes this week =)
16:45 EmilienM mgagne: same here, cool
16:45 mgagne EmilienM: and other stuff triggering my ocd
16:45 EmilienM mgagne: we are running an issue with keystone/wsgi & horizon when running on same node
16:45 mgagne EmilienM: I want to deprecate as much stuff as possible before the icehouse release so we can get ride of them asap.
16:46 EmilienM mgagne: seems like https://review.openstack.org/#/c/6962 fix it
16:46 mgagne EmilienM: don't run them on the same node :D
16:46 mgagne EmilienM: big security issue IMO
16:46 EmilienM mgagne: +1 for deprec
16:47 EmilienM mgagne: it's not production
16:47 mgagne EmilienM: =)
16:48 mgagne EmilienM: do you guys maintain or contribute to Ubuntu packages for OpenStack?
16:48 EmilienM mgagne: no, only debian
16:49 mgagne EmilienM: I guess it's a good news or not depending on the POV
16:49 mgagne EmilienM: dependencies are a mess with Ubuntu packages
16:49 mgagne EmilienM: you can't properly upgrade from Grizzly to Havana
16:49 mjblack EmilienM: what issue are you running into?
16:49 EmilienM mgagne: yeah, and we have the packager working on the dev team, very close from our work. He fix stuff before people can see the bug :)
16:50 EmilienM mgagne: we upgrade our public cloud last week from F to G
16:50 EmilienM and G to H is close
16:50 EmilienM mjblack: keystone API requests are redirected to /horizon vhist
16:50 EmilienM vhost*
16:51 mjblack EmilienM: try this https://review.openstack.org/#/c/69511
16:51 EmilienM mjblack: yeah it's related
16:51 EmilienM mjblack: and https://review.openstack.org/#/c/6962
16:52 EmilienM mjblack: we don't use SSL btw on our test
16:52 mjblack EmilienM: sure but the change turns the non-ssl into its own vhost and removes the redirect :D
16:52 francois1 joined #puppet-openstack
16:52 EmilienM mjblack: I'm reviewing it right now
16:52 EmilienM since it's a quite important bug, maybe mgagne could also have a look
16:53 mjblack EmilienM: 6962 is not related to puppet modules?
16:53 EmilienM mjblack: wrong URL
16:53 mgagne mjblack, EmilienM: I'll refer you to this bug then: https://bugs.launchpad.net/​puppet-horizon/+bug/1190282 =)
16:53 EmilienM mjblack: https://review.openstack.org/#/c/69621
16:54 mjblack ah ok I know your problem
16:54 EmilienM mgagne: yeah, feel free
16:54 mjblack EmilienM: the issue is keystone and horizon share the same ServerName
16:54 EmilienM mjblack: yeah
16:55 openstackgerrit David Moreau Simard proposed a change to stackforge/puppet-ceilometer: Remove log_dir from params and make logs configurable in init  https://review.openstack.org/69659
16:55 mgagne mjblack: issue is that both modules try to take over apache global configurations
16:55 mjblack EmilienM: hmm I didnt run into an issue, but then again my keystone vhost has a different ServerName
16:57 EmilienM mjblack: i see
16:57 EmilienM mjblack: thanks for your input
16:59 mjblack EmilienM: no problem, I spent a lot of time working with the keystone wsgi.
17:03 mjblack mgagne: I'm fine with providing more options but I'm more worried about things like https://review.openstack.org/#/c/69646 :)
17:03 marun joined #puppet-openstack
17:04 mgagne mjblack: why?
17:05 mjblack mgagne: I wish it was there sooner cause I updated the module and the issue it would fix took down my dashboard
17:05 mjblack or rather prevented me from authenticating
17:06 mgagne mjblack: sorry, is my fix breaking your stuff or would it fix it?
17:06 mjblack no no
17:06 mjblack nothing wrong with your fix
17:06 mjblack it fixes this https://review.openstack.org/61177
17:07 mgagne mjblack: yes, I had the same problem when upgrading, backward compatibility wasn't properly done
17:07 mgagne mjblack: you HAD to set keystone_url to false to "keep" backward compat... that's not how back comp is done...
17:08 mjblack mgagne: lol I know I wasnt happy when I saw that
17:08 marun joined #puppet-openstack
17:09 mgagne mjblack: I guess people don't really test those cases...
17:10 mjblack mgagne: or this https://review.openstack.org/63154
17:10 mgagne mjblack: =)
17:11 mgagne mjblack: I didn't have time to test changes in the previous months. I guess I'll have to pay the price now =)
17:12 mjblack mgagne: lol I'm starting to think I need to get involved more too since it does cause me some issues
17:13 mgagne mjblack: yep
17:13 mjblack mgagne: so whats the official stance on the puppet-openstack module?
17:14 mgagne mjblack: "deprecated" unless someone step in and wants to maintain it. I don't bother reviewing stuff anymore
17:36 EmilienM mjblack: I'm looking at your patch
17:37 angdraug joined #puppet-openstack
17:41 michchap joined #puppet-openstack
17:42 michchap joined #puppet-openstack
17:43 ryanycoleman joined #puppet-openstack
17:50 mgagne anyone from puppetlabs?
17:53 starmer joined #puppet-openstack
17:59 EmilienM mjblack: I did a review
17:59 EmilienM I'm very curious to read mgagne review and thoughts about https://review.openstack.org/#/c/69511/
18:00 mgagne EmilienM: this requires the latest puppetlabs-apache module. what's the state on upgrade to latest version? is it stable enough now?
18:01 mgagne EmilienM: also consider that other puppet modules of ours will need to update to latest version too otherwise there will be conflicting dependencies.
18:01 EmilienM mgagne: we use this one currently, I have to check with our infra team, but no bug reported yet
18:01 EmilienM mgagne: makes sense
18:03 ryanycoleman joined #puppet-openstack
18:05 tnoor joined #puppet-openstack
18:06 dtalton joined #puppet-openstack
18:08 tnoor1 joined #puppet-openstack
18:08 otherwiseguy joined #puppet-openstack
18:12 openstackgerrit Kevin Benton proposed a change to stackforge/puppet-horizon: Adds COMPRESS_OFFLINE option to local_settings  https://review.openstack.org/69684
18:16 ryanycoleman joined #puppet-openstack
18:28 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-neutron: Prefix database parameters with database  https://review.openstack.org/69686
18:30 EmilienM mgagne: are you going to backport ^ ?
18:30 mgagne EmilienM: yes
18:31 mgagne EmilienM: waiting for comments and approval
18:31 EmilienM good to know
18:31 EmilienM mgagne: i'll review it tomorrow
18:31 openstackgerrit Kevin Benton proposed a change to stackforge/puppet-horizon: Adds COMPRESS_OFFLINE option to local_settings  https://review.openstack.org/69684
18:32 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-glance: Add availability to configure show_image_direct_url  https://review.openstack.org/69688
18:39 openstackgerrit Kevin Benton proposed a change to stackforge/puppet-horizon: Adds COMPRESS_OFFLINE option to local_settings  https://review.openstack.org/69684
18:42 michchap joined #puppet-openstack
18:44 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-neutron: Prefix database parameters with database  https://review.openstack.org/69686
18:44 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-glance: Add availability to configure show_image_direct_url  https://review.openstack.org/69688
18:46 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-neutron: Remove use of deprecated include_class matcher  https://review.openstack.org/69694
18:46 rowleyaj joined #puppet-openstack
18:50 openstackgerrit Kevin Benton proposed a change to stackforge/puppet-horizon: Adds COMPRESS_OFFLINE option to local_settings  https://review.openstack.org/69684
18:52 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-neutron: Prefix database parameters with database  https://review.openstack.org/69697
18:56 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-glance: Add availability to configure show_image_direct_url  https://review.openstack.org/69688
18:56 starmer joined #puppet-openstack
18:57 openstackgerrit Kevin Benton proposed a change to stackforge/puppet-horizon: Adds COMPRESS_OFFLINE option to local_settings  https://review.openstack.org/69684
19:02 mgagne Is Kevin on IRC?
19:03 ryanycoleman joined #puppet-openstack
19:07 openstackgerrit Kevin Benton proposed a change to stackforge/puppet-horizon: Adds COMPRESS_OFFLINE option to local_settings  https://review.openstack.org/69684
19:16 ryanycoleman joined #puppet-openstack
19:29 openstackgerrit Kevin Benton proposed a change to stackforge/puppet-horizon: Adds COMPRESS_OFFLINE option to local_settings  https://review.openstack.org/69684
19:32 ryanycoleman joined #puppet-openstack
19:40 rwsu joined #puppet-openstack
19:43 michchap joined #puppet-openstack
19:43 michchap joined #puppet-openstack
19:50 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-glance: Add availability to configure show_image_direct_url  https://review.openstack.org/69688
19:54 ryanycoleman joined #puppet-openstack
19:59 ryanycoleman joined #puppet-openstack
20:11 ryanycoleman joined #puppet-openstack
20:19 ryanycoleman joined #puppet-openstack
20:43 michchap joined #puppet-openstack
20:44 michchap joined #puppet-openstack
20:58 starmer joined #puppet-openstack
21:08 unixpro1970 joined #puppet-openstack
21:32 dmsimard joined #puppet-openstack
21:34 dmsimard1 joined #puppet-openstack
21:44 michchap joined #puppet-openstack
21:45 michchap joined #puppet-openstack
21:58 ryanycoleman joined #puppet-openstack
22:14 ryanycoleman joined #puppet-openstack
22:34 ryanycoleman joined #puppet-openstack
22:45 michchap joined #puppet-openstack
22:45 michchap joined #puppet-openstack
22:55 ryanycoleman joined #puppet-openstack

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