Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
23:20 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Cleanup rspec tests of nova::objectstore  https://review.openstack.org/70250
23:20 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Cleanup rspec tests of nova::scheduler  https://review.openstack.org/70249
23:27 ryanycoleman joined #puppet-openstack
23:30 hogepodge joined #puppet-openstack
00:05 ryanycoleman joined #puppet-openstack
00:07 starmer joined #puppet-openstack
00:08 starmer joined #puppet-openstack
00:10 xarses joined #puppet-openstack
00:30 dmsimard joined #puppet-openstack
00:32 dmsimard1 joined #puppet-openstack
00:42 starmer joined #puppet-openstack
01:04 sputnik13net joined #puppet-openstack
01:05 starmer joined #puppet-openstack
01:13 starmer joined #puppet-openstack
01:25 bodepd mjblack: pong
01:25 bodepd tzn: pong (although you already bailed :) )
01:27 dmsimard1 bodepd is alive !?
01:34 bodepd I still exist.and even try to respond to this channel.t
01:34 bodepd I was just giving this guy Kudos > mgagne
01:34 bodepd for fixing up the tests I screwed up before I knew what I was doing
01:52 dmsimard1 bodepd: It's all good, I remember coming across some funny comment of yours in puppet-swift :)
02:04 terryw joined #puppet-openstack
02:21 kev joined #puppet-openstack
02:49 rmoe joined #puppet-openstack
02:57 bodepd dmsimard1: want to remind me of the comment?
03:07 mgagne bodepd: probably "I don't know what I'm doing" or "Is this really needed?"
03:24 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: apt-get dist-upgrade needed after adding repos  https://review.openstack.org/69876
03:39 starmer joined #puppet-openstack
03:55 badiane_ka joined #puppet-openstack
04:07 xarses joined #puppet-openstack
04:36 rmoe joined #puppet-openstack
04:58 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet_openstack_builder: Add a bunch of inline documentation  https://review.openstack.org/70042
05:28 mgagne1 joined #puppet-openstack
05:39 starmer joined #puppet-openstack
05:42 openstackgerrit A change was merged to stackforge/puppet_openstack_builder: Add a bunch of inline documentation  https://review.openstack.org/70042
07:55 EmilienM good morning
07:56 tnoor joined #puppet-openstack
08:35 openstackgerrit A change was merged to stackforge/puppet-nova: Cleanup rspec tests of nova  https://review.openstack.org/70241
08:47 sileht joined #puppet-openstack
08:48 francois1 joined #puppet-openstack
09:00 tzn joined #puppet-openstack
09:07 bauzas joined #puppet-openstack
09:07 bauzas joined #puppet-openstack
09:27 mmagr joined #puppet-openstack
10:04 openstackgerrit A change was merged to stackforge/puppet-horizon: Refactor rspec tests  https://review.openstack.org/69645
10:05 bauzas joined #puppet-openstack
10:06 openstackgerrit A change was merged to stackforge/puppet-horizon: Deprecation of keystone_scheme,host,port params  https://review.openstack.org/69646
10:23 britthouser joined #puppet-openstack
10:27 fc__ mjblack: could you update https://review.openstack.org/#/c/69511/ the dependency has been updated !
10:27 openstackgerrit A change was merged to stackforge/puppet-horizon: Add ability to disable configuration of apache  https://review.openstack.org/69990
10:49 bauzas joined #puppet-openstack
11:39 bauzas joined #puppet-openstack
12:02 bauzas joined #puppet-openstack
12:06 michchap joined #puppet-openstack
12:42 bogdando joined #puppet-openstack
12:46 tzn_ joined #puppet-openstack
13:07 markvoelker joined #puppet-openstack
13:08 morazi joined #puppet-openstack
13:19 markvoelker joined #puppet-openstack
13:33 michchap joined #puppet-openstack
13:39 openstackgerrit Thomas Bechtold proposed a change to stackforge/puppet-nova: Add Nova Aggregate support  https://review.openstack.org/70344
13:54 bauzas joined #puppet-openstack
14:11 kev joined #puppet-openstack
14:18 morazi joined #puppet-openstack
14:27 bauzas joined #puppet-openstack
14:57 morazi joined #puppet-openstack
15:05 ryanycoleman joined #puppet-openstack
15:05 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-horizon: local_settings: Allow to manage can_set_password option  https://review.openstack.org/68433
15:05 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-horizon: Parameterize OPENSTACK_NEUTRON_NETWORK settings support  https://review.openstack.org/70054
15:08 EmilienM markvoelker: I was working on my patch : https://review.openstack.org/68433 - I did a rebase to your patch and also to master, and I accidentally commit on your patch, please check I did not wrong :-)
15:09 EmilienM and jenkins is not happy... grrrr
15:09 markvoelker EmilienM: oops. =)
15:09 EmilienM markvoelker: for my patch, jenkins is happy though
15:09 EmilienM markvoelker: I broke it, I fix it. Sorry for that, let me some time
15:10 kkirkpatrick__ joined #puppet-openstack
15:10 dgollub joined #puppet-openstack
15:10 EmilienM markvoelker: too many things have changed :-) it's hard to have 4 patches on the same file :)
15:10 markvoelker No worries.  I believe I can revert is from my local copy actually....save you some trouble
15:11 terryw joined #puppet-openstack
15:11 EmilienM markvoelker: you will still have to rebase.
15:11 markvoelker EmilienM: yep
15:12 EmilienM markvoelker: let me try to fix it
15:12 EmilienM markvoelker: if I fail, I'll ask you some help or revert :)
15:12 markvoelker EmilienM: sure, go for it
15:13 EmilienM if you don't mind of course
15:13 markvoelker not a bit
15:13 rwsu joined #puppet-openstack
15:16 rmoe joined #puppet-openstack
15:19 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-horizon: Parameterize OPENSTACK_NEUTRON_NETWORK settings support  https://review.openstack.org/70054
15:19 EmilienM markvoelker: should be good. I did the rebase to master and tests pass ^
15:19 EmilienM to be honest I still don't know how I did lol
15:20 EmilienM markvoelker: when jenkins is happy, I think your patch is ready for review, I'll +2
15:24 dmsimard joined #puppet-openstack
15:29 prad_ joined #puppet-openstack
15:32 mjblack joined #puppet-openstack
15:34 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Cleanup rspec tests of nova::compute  https://review.openstack.org/70246
15:34 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Cleanup rspec tests of nova::consoleauth  https://review.openstack.org/70247
15:34 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Cleanup rspec tests of nova::compute::libvirt  https://review.openstack.org/70244
15:34 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Cleanup rspec tests of nova::cert  https://review.openstack.org/70245
15:34 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Cleanup rspec tests of nova::objectstore  https://review.openstack.org/70250
15:34 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Cleanup rspec tests of nova::conductor  https://review.openstack.org/70248
15:34 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Cleanup rspec tests of nova::scheduler  https://review.openstack.org/70249
15:35 EmilienM mgagne1: good morning
15:35 mgagne EmilienM: hi
15:36 dmsimard jesus mgagne
15:36 EmilienM mgagne: i rebased on master https://review.openstack.org/68433 and the one of markvoelker which is a dependency - they are ready for review I think
15:37 mgagne EmilienM: hmmm it isn't what I expected
15:37 mgagne EmilienM: this also breaks backward compatibility for someone with a custom template
15:37 EmilienM damns
15:38 mgagne What fvollero suggested it what I was looking for
15:38 mgagne EmilienM: ^
15:38 mgagne EmilienM: and bc
15:38 mgagne si*
15:38 mgagne is*
15:39 EmilienM markvoelker: sounds like we have to do better this time :-)
15:39 michchap joined #puppet-openstack
15:45 mgagne EmilienM: will create bug for all this public_url, internal_url and admin_url rework
15:45 mgagne EmilienM: this will ease backporting process and offer greater visibility
15:46 kev joined #puppet-openstack
15:47 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ceilometer: Introduce public_url, internal_url and admin_url  https://review.openstack.org/69006
15:49 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-glance: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70011
15:50 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-heat: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70024
15:51 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ironic: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70160
15:52 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-keystone: Introduce public_url, internal_url and admin_url  https://review.openstack.org/68991
15:56 EmilienM mgagne: what is the final word about "inline_template" use in all your patches.
15:56 EmilienM mgagne: is it ok?
15:56 mgagne EmilienM: it's ok, people abused it to generate files instead of using real template
15:56 mgagne EmilienM: using it to compute a variable is ok
15:57 EmilienM mgagne: ack, I'm testing it now.
16:01 mjblack fc__: sure, I expected that I'd probably have to do some updates, will update today
16:01 fc__ mjblack: I added a note too, you should add back the bind_address parameter I think
16:03 starmer joined #puppet-openstack
16:19 xarses joined #puppet-openstack
16:21 kev joined #puppet-openstack
16:30 fvollero mgagne: :) :) I knew that my 8 years of ruby would be useful :)
16:34 dmsimard 8 years of ruby, I didn't know it was that old
16:37 dmsimard Going to try puppet-swift against newly released 1.12, let's hope it doesn't blow up in my face
16:37 mjblack joined #puppet-openstack
16:37 mjblack fc__: I took out bind address because if its set, it will cause apache module to error out
16:38 mjblack fc__: the only time you would need bind_address in apache is if you wanted it to listen on a specific ip address instead of all addresses
16:38 fc__ mjblack: we need this
16:38 mjblack fc__: ok I'll see what I can do to make it work
16:39 fc__ mjblack: to not break apache, you can set bind_address to 'undef' instead of '0.0.0.0'
16:39 fc__ mjblack: to no break the *default* puppet-apache settings
16:40 mjblack fc__: right, but I think I ran into an issue with the rspec on that
16:41 mjblack fc__: what I might do to get around it is this https://github.com/puppetlabs/puppetlabs-apache/blob/master/manifests/vhost.pp#L326
16:41 mjblack fc__: basically check if its already defined and set add_listen to false
16:42 fc__ mjblack: that won't be a problem if bind_ip is undef
16:42 mjblack fc__: its not that an issue if its undef, but if someone sets a value it will be
16:42 mjblack fc__: hence why I'd want to do that check ;)
16:43 fc__ mjblack: then if bind_ip is not undef, it's an issue if there's one virtualhost that binds to an ip and one virtualhost that binds to all ips
16:43 fc__ mjblack: but in that case apache won't even start
16:43 mjblack fc__: right but if I set add_listen to the vhost define to false it wont be an issue
16:43 fc__ mjblack: but we need add_listen
16:44 mjblack fc__: you are right, but if its already defined thats where puppet will complain and fail
16:44 fc__ mjblack: the fix when binding to an ip address is to disable the default virtualhost
16:44 fc__ mjblack: yes, but it's a configuration choice to be made
16:44 mjblack fc__: thats assuming that there is nothing else on that host though
16:45 mjblack fc__: I think checking if the listen is already defined and then setting add_listen based on that is the way to go
16:45 fc__ mjblack: in any case apache (the server) won't allow to have a wildcard bind and a specific ip address bind
16:45 fc__ mjblack: nope, the add_listen should be set anyway
16:46 mjblack fc__: heh, what I mean is if its already set, somewhere else, then we want to set it to false on the vhost
16:46 fc__ mjblack: nope, no need, the problem is only when some vhosts listen on a defined ip and other vhosts don't listen to a specific ip (and both on the same port)
16:47 fc__ mjblack: this might be documented, but the listen directive should be set IMHO
16:48 fc__ mjblack: if one vhost listens to 10.0.0.1:80 and another to 10.0.0.2:80 and NO vhost listens to *:80, this won't be a problem. even if another third vhost listens to 10.0.0.1:80 (the same as the first one) this won't be ap roblem
16:48 mjblack fc__: right, but if the default vhost creates *:80 listen, then the horizon vhost should not try to create it either
16:48 fc__ mjblack: the apache module handles this case
16:49 mjblack fc__: the add_listen isnt for the vhost, its the Listen <ip>:<port> directive
16:49 fc__ https://github.com/puppetlabs/puppetlabs-apache/blob/master/manifests/vhost.pp#L329-L331
16:49 fc__ mjblack: the listen directives will be added only if they don't already exist
16:49 mjblack fc__: yes, but if you look at the lines above it, if its already defined it will fail
16:50 fc__ mjblack: you mean https://github.com/puppetlabs/puppetlabs-apache/blob/master/manifests/vhost.pp#L326 ?
16:50 mjblack yes
16:50 fc__ mjblack: it will fail only if ip *and* non-ip listen directives are mixed
16:51 fc__ mjblack: which is not possible for apache (and IP in general)
16:52 mjblack fc__: ok so the listen directive is being done for just the port
16:53 mjblack fc__: we can look at removing the default vhost I think
16:53 fc__ mjblack: in a production environment this is a good practice indeed
16:53 fc__ mjblack: but this should not be the role of puppet-horizon
16:54 mjblack fc__: only concern I have is that we're taking over apache, which is basically stating that where ever they install is only for horizon
16:54 mjblack fc__: I'd much rather take out the ::apache, put a doc stating what the requirements are and let the user implement it their way
16:55 fc__ mjblack: +1
16:55 mjblack fc__: still keep the wsgi changes though
16:55 fc__ mjblack: +1 on that, you absolutely remove the base ::apache and keep the remaining
16:57 mjblack I will update the commit to reflect that decision too
16:57 fc__ mjblack: thank you very much :)
16:59 mjblack fc__: heh, I'm also going to remove mgagne's change of automatically calling the apache, I thought about it today that ::horizon shouldnt call ::horizon::wsgi::apache
17:00 mjblack because if someone comes up with nginx class it will need to be done anyway
17:00 mgagne mjblack: it was meant to preserve bc
17:01 fc__ mjblack: with the $configure_apache parameter it's no big deal, it allows to have only one entry point to configure horizon
17:01 mgagne mjblack: you can disable it if you want to setup apache/wsgi yourself
17:01 fc__ (which is just one component)
17:01 mjblack mgagne: right but I think maybe with icehouse we should make that change
17:01 mgagne mjblack: most users would expect horizon to work out of the box
17:02 mjblack mgagne: I totally agree with you, but it should be like cinder volume drivers, the user should implement it
17:02 kkirkpatrick__ joined #puppet-openstack
17:02 mjblack mgagne: for havana I wouldnt change it, but for icehouse I think we should
17:03 ryanycoleman joined #puppet-openstack
17:08 mjblack if its not the best way to handle it I'll forget it
17:09 dtalton joined #puppet-openstack
17:16 ryanycoleman joined #puppet-openstack
17:16 rmoe joined #puppet-openstack
17:25 angdraug joined #puppet-openstack
17:30 ryanycoleman joined #puppet-openstack
17:43 dgollub joined #puppet-openstack
17:56 mjblack_ joined #puppet-openstack
17:58 mjblack_ joined #puppet-openstack
18:00 mjblack__ joined #puppet-openstack
18:01 mjblack__ joined #puppet-openstack
18:02 mjblack__ joined #puppet-openstack
18:05 mjblack joined #puppet-openstack
18:16 dtalton is anyone doing live migration
18:16 dtalton i noticed that puppet-nova doesn't have any params to set uid/gid for the nova account and group
18:17 tzn joined #puppet-openstack
18:19 tnoor joined #puppet-openstack
18:21 tnoor1 joined #puppet-openstack
18:22 michchap joined #puppet-openstack
18:30 mgagne dtalton: true, I guess most people rely on the packaging system to create the user with the correct uid
18:32 dtalton hmm this is hairy. IIRC nova needs to match across all compute nodes, but libvirt needs to be unique
18:32 ryanycoleman joined #puppet-openstack
18:38 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ceilometer: Introduce public_url, internal_url and admin_url  https://review.openstack.org/69006
18:42 ryanycoleman joined #puppet-openstack
18:43 hogepodge joined #puppet-openstack
18:43 hogepodge fc__
18:46 fc__ hogepodge: _o/
18:46 hogepodge Want to take a look at https://github.com/puppetlabs/puppetlabs-mongodb/pull/88/files?
18:46 pzimmer joined #puppet-openstack
18:46 hogepodge Looks good to me. I think it will fix the Ruby 1.8.7 problems.
18:47 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-designate: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70393
18:48 fc__ hogepodge: seems good
18:49 hogepodge Cool. I'll merge and publish to the forge. We should get some more robust testing in there too for replica sets.
18:49 hogepodge (by we I probably mean me) :-)
18:49 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-designate: Fix wrong rspec tests filename  https://review.openstack.org/70394
18:53 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-designate: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70393
18:53 fc__ hogepodge: thx. i was thinking just the same about having more robust tests ;)
18:54 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-designate: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70393
18:54 dtalton mgagne, I think that works on rhel, not so sure about on ubuntu
18:55 dtalton I just checked the postinst scripts for nova and there is no uid/gid being specified
18:56 mgagne dtalton: our live migration mostly works fine. (issues are with volume backed instances) Could be that we provision everything is the right order each time, creating the user with the same UID on each server each time
18:56 dtalton that's my fear :D
18:57 mgagne dtalton: I don't think puppet should mess with the uid anyway. If packages create a user with a UID and puppet then changes the UID of this user later on, a lot of files could now be owned by a user (uid) that not longer exist.
18:57 dtalton good point
18:57 mgagne no*
18:57 mgagne dtalton: I guess this problem should/could be addressed with other means outside puppet
18:58 mgagne dtalton: maybe with a centralized user management
19:14 aryan joined #puppet-openstack
19:15 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-glance: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70011
19:18 aryan hi guys,
19:18 aryan have a question regarding neutron ml2 plugin
19:18 aryan I can see some stuff for ML2 plugin in the code
19:18 aryan but the docs says ovs and linuxbridge are only supported
19:18 aryan https://github.com/stackforge/puppet_openstack_builder/tree/master/data
19:25 ryanycoleman joined #puppet-openstack
19:25 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-neutron: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70411
19:28 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ironic: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70160
19:31 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ironic: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70160
19:32 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-glance: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70011
19:43 kev joined #puppet-openstack
19:53 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-horizon: Fixed Horizon SSL issues  https://review.openstack.org/69511
19:54 ryanycoleman joined #puppet-openstack
19:56 openstackgerrit Matthew J Black proposed a change to stackforge/puppet-horizon: Fixed Horizon SSL issues  https://review.openstack.org/69511
20:08 hogepodge joined #puppet-openstack
20:11 michchap joined #puppet-openstack
20:24 otherwiseguy joined #puppet-openstack
20:32 ryanycoleman joined #puppet-openstack
20:38 thumpba joined #puppet-openstack
20:46 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet_openstack_builder: Add inline docs for common.yaml  https://review.openstack.org/70438
20:49 dmsimard joined #puppet-openstack
20:51 ryanycoleman joined #puppet-openstack
20:59 openstackgerrit David Moreau Simard proposed a change to stackforge/puppet-swift: Remove deprecated parameter 'address'  https://review.openstack.org/70440
21:03 ryanycoleman joined #puppet-openstack
21:23 bauzas joined #puppet-openstack
21:26 ryanycoleman joined #puppet-openstack
21:31 xarses joined #puppet-openstack
21:51 starmer joined #puppet-openstack
21:53 openstackgerrit joined #puppet-openstack
22:00 michchap joined #puppet-openstack
22:06 mjblack joined #puppet-openstack
22:07 starmer joined #puppet-openstack
22:24 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Introduce public_url, internal_url and admin_url (and ec2)  https://review.openstack.org/70458
22:34 hogepodge joined #puppet-openstack
22:34 dtalton2 joined #puppet-openstack
22:41 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-neutron: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70411
22:41 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ironic: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70160
22:42 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-glance: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70011
22:42 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ironic: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70160
22:48 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-designate: Introduce public_url, internal_url and admin_url  https://review.openstack.org/70393
22:50 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-keystone: Introduce public_url, internal_url and admin_url  https://review.openstack.org/68991
22:50 tnoor joined #puppet-openstack
22:52 tnoor1 joined #puppet-openstack
22:53 tnoor1 hey guys.. good evening, i had a question.. we are using grizzly modules to install openstack via puupet, needed some suggestion on running puppet via cron? is it really required as i see it does lot of config checks ( takes almost 40 seconds for just checking), if not then what is the standard and best practices to manage your configs etc?
22:58 mgagne tnoor1: it really depends on your needs. You could run puppet through cron or the agent if you want "compliance" mode: where you want to make sure the config is right in case someone changes it.
22:59 mgagne tnoor1: otherwise you can run it once per deployment and be done with it. (no cron or agent) and suppose/hope no one will ever change the configs until the next deployment.
22:59 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Introduce public_url, internal_url and admin_url (and ec2)  https://review.openstack.org/70458
22:59 tnoor1 mgagne: agent means running "puppet agent -t" right i think same command needs to be run via cron also right?

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