Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2013-05-22

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

All times shown according to UTC.

Time Nick Message
00:10 mgagne Don't forget to add puppet-ceilometer to your filter if you have any ;)
00:19 openstackgerrit Dan Bode proposed a change to stackforge/puppet-cinder: make sql_connection optional  https://review.openstack.org/30026
00:23 openstackgerrit A change was merged to stackforge/puppet-keystone: Add puppet-lint to Gemfile  https://review.openstack.org/30017
00:24 openstackgerrit A change was merged to stackforge/puppet-glance: Add puppet-lint to Gemfile  https://review.openstack.org/30015
00:25 openstackgerrit A change was merged to stackforge/puppet-swift: Add puppet-lint to Gemfile  https://review.openstack.org/30021
00:25 openstackgerrit A change was merged to stackforge/puppet-quantum: Add puppet-lint to Gemfile  https://review.openstack.org/30020
00:25 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Add puppet-lint to Gemfile  https://review.openstack.org/30024
00:26 openstackgerrit A change was merged to stackforge/puppet-horizon: Add puppet-lint to Gemfile  https://review.openstack.org/30016
00:26 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Add .gitreview  https://review.openstack.org/30022
00:28 openstackgerrit A change was merged to stackforge/puppet-openstack: Add puppet-lint to Gemfile  https://review.openstack.org/30018
00:41 openstackgerrit Terry Wilson proposed a change to stackforge/puppet-quantum: Add log_dir and use by default instead of log_file  https://review.openstack.org/29992
00:45 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstack_dev_env: Remove nova::conductor from site.pp  https://review.openstack.org/30027
00:45 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstack_dev_env: Remove nova::conductor from site.pp  https://review.openstack.org/30027
01:03 openstackgerrit Dan Bode proposed a change to stackforge/puppet-nova: added qemu.conf config file requirement when using OVS libvirt_vif_driver  https://review.openstack.org/30029
01:30 openstackgerrit A change was merged to stackforge/puppet-openstack_dev_env: Remove nova::conductor from site.pp  https://review.openstack.org/30027
01:48 openstackgerrit Dan Bode proposed a change to stackforge/puppet-openstack: refactor quantum configuration  https://review.openstack.org/29160
03:44 openstackgerrit Adam Compton proposed a change to stackforge/puppet-openstack: Adding an "openstack::repo" class which automatically sets up repos  https://review.openstack.org/29977
04:21 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Capitalize boolean values in config files  https://review.openstack.org/30023
04:48 openstackgerrit Dan Bode proposed a change to stackforge/puppet-openstack: update cinder config in controller/compute  https://review.openstack.org/29452
06:47 JoeJulian joined #puppet-openstack
07:00 JoeJulian joined #puppet-openstack
07:14 dachary joined #puppet-openstack
08:40 dachary joined #puppet-openstack
09:11 MxG joined #puppet-openstack
09:32 mruza joined #puppet-openstack
09:43 needcoffee joined #puppet-openstack
10:38 derekh joined #puppet-openstack
11:57 dachary joined #puppet-openstack
12:11 dprince joined #puppet-openstack
12:43 openstackgerrit Radim K proposed a change to stackforge/puppet-swift: Customizable allowed_sync_hosts in container-server.conf  https://review.openstack.org/29932
15:48 dachary joined #puppet-openstack
16:00 dachary joined #puppet-openstack
18:01 dprince joined #puppet-openstack
18:07 openstackgerrit A change was merged to stackforge/puppet-keystone: Capitalize boolean values in config files  https://review.openstack.org/30010
18:22 openstackgerrit A change was merged to stackforge/puppet-glance: Capitalize boolean values in config files  https://review.openstack.org/30009
18:29 comptona in the horizon module, there's a line that adds a "Listen ${bind_address}:80"; this conflicts with the apache module's ports.conf
18:31 comptona is the horizon copy of that line still necessary?
18:38 openstackgerrit Dan Prince proposed a change to stackforge/puppet-nova: Don't require flat_inferface.  https://review.openstack.org/30145
18:38 dachary1 joined #puppet-openstack
18:39 openstackgerrit Dan Prince proposed a change to stackforge/puppet-nova: Set compute_driver in xenserver.pp.  https://review.openstack.org/30146
18:46 comptona bodepd: I'm not clear on how to file a bug to fix this; I have a working, tested, simple patch, but the gerrit_workflow doc says to make a branch with a bug number and I can't figure out where bugs for the puppet modules live
18:46 comptona any suggestions?
18:47 mgagne bugs are located on Launchpad: https://launchpad.net/puppet-openstack/
18:47 mgagne you can still submit patch without a bug
18:48 comptona ah
18:48 comptona should I file a bug for completeness' sake? or is just a patch OK?
18:48 mgagne I usually don't fill bugs myself so...
18:48 comptona haha, ok
18:49 comptona do you name your git branches "bugfix/descriptive-name" then?
18:49 comptona instead of "bug/NNNNNN"?
18:49 mgagne Any meaningful name will be ok. Can be fix_that_thing
18:50 mgagne If you don't plan on fixing it yourself but want to inform people of the existence of the bug and hope someone will fix it, you should open a bug =)
18:51 comptona gotcha :)
18:51 comptona thanks again for your help
18:51 mgagne any time ;)
18:54 openstackgerrit Adam Compton proposed a change to stackforge/puppet-horizon: Fix for duplicate Listen directive in horizon apache config  https://review.openstack.org/30150
18:59 dachary joined #puppet-openstack
18:59 openstackgerrit Dan Prince proposed a change to stackforge/puppet-nova: Soft link /usr/bin/pip (fix xenapi install).  https://review.openstack.org/30151
19:11 mruza joined #puppet-openstack
19:16 openstackgerrit A change was merged to stackforge/puppet-cinder: Capitalize boolean values in config files  https://review.openstack.org/30008
19:28 openstackgerrit Adam Compton proposed a change to stackforge/puppet-nova: Added package requirement to install dnsmasq-utils for RHEL-alikes  https://review.openstack.org/30156
19:28 openstackgerrit Adam Compton proposed a change to stackforge/puppet-nova: Added package requirement to install dnsmasq-utils for RHEL-alikes  https://review.openstack.org/30156
19:30 comptona sorry, error in the commit message :-P
19:33 mgagne1 joined #puppet-openstack
19:37 bodepd_ joined #puppet-openstack
19:57 openstackgerrit A change was merged to stackforge/puppet-cinder: Use boolean instead of string  https://review.openstack.org/28758
20:56 otherwiseguy joined #puppet-openstack
21:07 comptona bodepd_: are you using rabbitmq with RHEL in your quantum module testing?
21:07 comptona I'm getting a protocol mismatch when I try to set it up by hand
21:11 comptona never mind; the default quantum.conf has two copies of the rpc_backend parameter in different places and I changed only one of them. :-P
21:11 bodepd_ comptona: I havent been doing that much quantum testing
21:11 bodepd_ comptona: sorry, not testing quantum on redhat atm
21:11 comptona I thought the tasks in the redhat-grizzly blueprint were to get a) quantum working and b) redhat working with quantum?
21:21 mgagne bodepd: Found out that the logical architecture overview schema has been updated for Grizzly: http://docs.openstack.org/trunk/openstack-compute/admin/content/logical-architecture.html =)
21:27 EmilienM joined #puppet-openstack
21:31 bodepd_ comptona: I'm on the get quantum working task. I am still reviewing code from redhat that gets quantum working on redhat
21:31 bodepd_ comptona: but it'll be a while before I personally am working on quantum/redhat
21:32 bodepd_ mgagne: that is really cool.
21:32 bodepd_ mgagne: maybe we should have a ticket that someone should go through the module and make sure we conform to this
21:32 mgagne bodepd: it seems cinder-volume still require db access :P
21:33 mgagne bodepd: conform to what? the schema? which aspect of it?
21:34 bodepd_ mgagne: mgagne just to make sure its consistent with our own configurations
21:34 bodepd_ mgagne: it says that quantum agent connects to the database...
21:35 mgagne bodepd: agent(s). They don't say which ones.
21:35 bodepd_ mgagne: what do you think about the proposed code freeze?
21:36 mgagne bodepd: I'm not using l3 or dhcp so I don't know if they are referring to those ones because I found out that I don't need db access from quantum-plugin-linuxbrige-agent.
21:36 bodepd_ the timing seems a bit arbitrary given the amount of features being added
21:36 mgagne yes
21:36 bodepd_ I'm so excited that I won't have to manage backports!
21:36 mgagne :P
21:38 bodepd_ mgagne: speaking of last minute features add.
21:38 mgagne If it's arbitrary so we can stabilize the code and release it, be it. It will always be possible to add fixes, we just won't be able to change signature without caring about backward compatibility.
21:38 bodepd_ think you could review that cinder patch and get it merged?
21:38 mgagne checking
21:38 bodepd_ mgagne: also, what do you think about that quantum patch?
21:38 bodepd_ it's massive isnt it?
21:38 bodepd_ given that quantum didnt work in the first place...
21:38 mgagne I'll check, didn't have much time today
21:44 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-keystone: Use boolean instead of string  https://review.openstack.org/28763
21:47 openstackgerrit A change was merged to stackforge/puppet-keystone: Use boolean instead of string  https://review.openstack.org/28763
21:48 devicenull the quantum agents do need db access (at least the OVS one does)
21:57 mgagne devicenull: Did you test it or have doc to confirm this? I don't see any db access in the code of the agent, not the plugin.
21:57 openstackgerrit A change was merged to stackforge/puppet-nova: Set compute_driver in xenserver.pp.  https://review.openstack.org/30146
21:58 devicenull sec
21:59 devicenull hm, maybe it doesnt
21:59 devicenull ya guess I was wrong
22:00 mgagne but I don't know about L3 and DHCP, I'm not using those.
22:03 mgagne bodepd: I'll try to find some time to add rspec tests to most manifests.
22:04 mgagne bodepd: so people cannot blame the lack of tests for not updating them =)
22:04 mgagne bodepd: and clean them up a bit
22:10 openstackgerrit Dan Bode proposed a change to stackforge/puppet-openstack: update cinder config in controller/compute  https://review.openstack.org/29452
22:12 mgagne why can't we diff between 2 patch sets =)
22:19 openstackgerrit Dan Bode proposed a change to stackforge/puppet-openstack: update cinder config in controller/compute  https://review.openstack.org/29452
22:19 otherwiseguy mgagne: bodepd_: so I know we are soon dropping puppet 2.6 support. we're trying to get puppet 2.7 into the RHOS/RDO repos, but it is taking a bit. Anyone going to get irritated if I post a patch removing a bunch of trailing commas from argument lists?
22:20 mgagne I don't mind
22:20 mgagne yesterday I spammed gerrit so I can't complain =)
22:20 otherwiseguy I see some discussion here: https://groups.google.com/a/puppetlabs.com/forum/#!topic/puppet-openstack/4L_HyvWiIUs about the puppet 2.6 test actually running with 2.7.
22:21 mgagne would it be easy to fix?
22:21 otherwiseguy mgagne: Maybe? I'm not sure who can/how to fix the CI stuff.
22:22 mgagne anyone can
22:22 mgagne just commit to openstack-infra/config I think
22:22 otherwiseguy Great, I'm 50% of the way there, then. :) Now to figure out how. :)
22:22 mgagne it's self-service
22:23 otherwiseguy I've just been going through and running: find . -name *.pp|xargs pcregrep -r -M ',$\n\s*\)' and fixing where I find stuff since that is the most common issue.
22:23 mgagne ok
22:23 otherwiseguy I'll check everything out on a RHEL box, fix the easy stuff, run the tests, and if I don't find anything broken, see about fixing the test.
22:24 * otherwiseguy can't wait until we finally move away from 2.6
22:24 bodepd_ mgagne: I agree that sucks.
22:24 otherwiseguy Hopefully not too long now.
22:24 bodepd_ otherwiseguy: go ahead
22:24 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Add rspec tests for nova::compute::xenserver  https://review.openstack.org/30177
22:24 bodepd_ but really, you should fix the 2.6.x tests
22:26 openstackgerrit Terry Wilson proposed a change to stackforge/puppet-quantum: Remove trailing commas for Puppet 2.6 compatibility  https://review.openstack.org/30178
22:26 mgagne https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/jenkins_job_builder/config/puppet-module-jobs.yaml
22:26 mgagne check for puppet-module-unit
22:27 mgagne and check console output in one build job to see what's wrong in it
22:27 bodepd_ the version specification in the Gemfile was incorrect.
22:27 bodepd_ or something.
22:27 bodepd_ there was an email aboout it...
22:28 openstackgerrit A change was merged to stackforge/puppet-quantum: Remove trailing commas for Puppet 2.6 compatibility  https://review.openstack.org/30178
22:29 mgagne http://logs.openstack.org/30178/1/check/gate-puppet-quantum-puppet-unit-2.6/125/console.html
22:29 mgagne Installing puppet (2.7.21)
22:29 openstackgerrit Terry Wilson proposed a change to stackforge/puppet-nova: Remove trailing comma for Puppet 2.6 compatibility  https://review.openstack.org/30179
22:29 bodepd_ otherwiseguy: I just forwarded the thread about it back to the mailing list
22:30 otherwiseguy mgagne:  bodepd_: thx
22:30 mgagne I think it's all broken
22:30 mgagne http://logs.openstack.org/30178/1/check/gate-puppet-quantum-puppet-unit-3.0/125/console.html
22:30 mgagne Installing puppet (3.2.1)
22:31 bodepd_ yeah, that is what that thread mentions
22:31 mgagne alright, I should read about it =)
22:32 bodepd_ basically, the tests are submitting a bad format to bundler
22:33 mgagne Updating the job-group will change the jenkins job name from 2.6 to 2.6.0
22:33 mgagne I got an idea
22:34 mgagne https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/jenkins_job_builder/config/puppet-module-jobs.yaml#L11
22:34 mgagne Change {puppet_version} to {puppet_version}.0
22:34 bodepd_ https://github.com/openstack-infra/config/blob/master/modules/openstack_project/files/jenkins_job_builder/config/puppet-module-jobs.yaml#L52
22:34 bodepd_ mgagne: your too fast :)
22:35 mgagne bodepd: fix to openstack-infra/config merged already =)
22:35 mgagne j/k
22:46 bodepd_ mgagne: want to give this cinder patch hopefully the last review
22:46 mgagne will check
22:50 openstackgerrit A change was merged to stackforge/puppet-nova: Remove trailing comma for Puppet 2.6 compatibility  https://review.openstack.org/30179
22:51 openstackgerrit A change was merged to stackforge/puppet-nova: Add rspec tests for nova::compute::xenserver  https://review.openstack.org/30177
22:58 dachary joined #puppet-openstack
23:10 openstackgerrit Dan Bode proposed a change to stackforge/puppet-openstack: refactor quantum configuration  https://review.openstack.org/29160
23:12 bodepd_ mgagne: thanks.
23:13 bodepd_ mgagne: once I rebase, I can take the WIP off the quantum patch
23:13 mgagne ok
23:16 openstackgerrit A change was merged to stackforge/puppet-openstack: update cinder config in controller/compute  https://review.openstack.org/29452
23:43 openstackgerrit Dan Bode proposed a change to stackforge/puppet-openstack: refactor quantum configuration  https://review.openstack.org/29160
23:48 mgagne bodepd: https://review.openstack.org/30187
23:50 mgagne bodepd: At first view, it generates the correct Jenkins config: http://logs.openstack.org/30187/1/check/config-compare-xml/1279/console.html
23:55 mgagne bodepd: I feel we should add puppet-syntax to catch manifests without rspec tests (and yha, puppet-lint)

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