Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
01:08 digicloseup joined #puppet-openstack
01:41 digicloseup joined #puppet-openstack
02:02 xingchao joined #puppet-openstack
02:24 xingchao_ joined #puppet-openstack
02:35 xingchao joined #puppet-openstack
04:26 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-openstack: Add standalone network node class  https://review.openstack.org/41334
04:29 openstackgerrit Mark T. Voelker proposed a change to stackforge/puppet-openstack: Add standalone network node class  https://review.openstack.org/41334
07:55 derekh joined #puppet-openstack
10:01 ashp joined #puppet-openstack
10:06 xingchao joined #puppet-openstack
10:12 xingchao joined #puppet-openstack
12:16 dprince joined #puppet-openstack
12:59 xingchao_ joined #puppet-openstack
13:04 EmilienM hello there
13:11 dmsimard joined #puppet-openstack
13:13 xingchao joined #puppet-openstack
13:20 dmsimard joined #puppet-openstack
13:26 mgagne joined #puppet-openstack
13:30 ersmith joined #puppet-openstack
13:32 EmilienM mgagne: hello :)
13:32 mgagne mgagne: Hi
13:32 EmilienM mgagne: I was not able to add the puppet-* into the new project page
13:33 mgagne EmilienM: I did so already =)
13:33 EmilienM mgagne: how ?
13:33 EmilienM I'm interested in :P
13:33 mgagne EmilienM: I just sent an email
13:33 EmilienM right
13:34 mgagne EmilienM: you cannot add a project to a group from the group page. You have to do so from the project page. I guess it's a question of rights and authorization or maybe just UX.
13:34 EmilienM oh ok
13:34 mgagne EmilienM: On the project page: Edit Details => Part Of field.
13:34 EmilienM mgagne: I created the first blueprint > https://blueprints.launchpad.net/puppet-nova/+spec/cells-support
13:35 EmilienM waiting for approbation from core :P
13:35 mgagne EmilienM: nice, and now we will see if gerrit will update the blueprint =)
13:35 EmilienM yep
13:35 EmilienM mgagne: my patch is not ready yet
13:35 mgagne EmilienM: you have my blessing
13:35 EmilienM ahah
13:35 EmilienM glad
13:37 EmilienM mgagne: I'm wondering if we could simply configure releases belong openstack projects
13:37 mgagne EmilienM: I'm still trying to figure how we should handle releases, especially with forge/module versions
13:37 EmilienM oh right
13:38 mgagne EmilienM: openstack does pre-release (like beta) and I'm not sure how forge handles such versions (and librarian-puppet)
13:38 mgagne EmilienM: and how a user can decide to install (or not) development/pre-release versions
13:40 mgagne EmilienM: I know for sure that librarian-puppet doesn't handle suffixes well, an upstream patch might be required
13:41 EmilienM ok
13:41 mgagne EmilienM: this has to be merged first for updates from gerrit to work on LP: https://review.openstack.org/#/c/41128/
13:41 EmilienM yep
13:41 EmilienM I know
13:41 EmilienM we miss one approved :/
13:42 EmilienM mgagne: maybe should I ask to Monty ?
13:42 mgagne EmilienM: =) also I have to check for ACL, not sure who I have to add to allow updates
13:42 EmilienM mgagne: already done
13:42 mgagne EmilienM: you can try
13:42 EmilienM it's hudson-openstack
13:42 mgagne EmilienM: ok, will (double) check
13:43 EmilienM mgagne: done
13:43 EmilienM I don't like "asking"
13:45 mgagne EmilienM: they sure won't poke you and ask if you need anything ;)
13:45 EmilienM ahah
13:45 EmilienM mgagne: why not :P
13:47 otherwiseguy joined #puppet-openstack
13:47 prad_ joined #puppet-openstack
13:48 EmilienM mgagne: merged
13:48 mgagne EmilienM: saw it ;)
13:56 morazi joined #puppet-openstack
14:05 mgagne1 joined #puppet-openstack
14:05 mgagne1 joined #puppet-openstack
14:06 EmilienM mgagne: cells are a pain to configure. I think reviews will be difficult :)
14:06 EmilienM please be kind with me this time, ahahah
14:06 mgagne1 EmilienM: I'm not familiar with the concept =)
14:07 EmilienM mgagne: a bit of doc here > http://docs.openstack.org/trunk/openstack-compute/admin/content/ch_cells.html
14:09 prad_ joined #puppet-openstack
14:10 mgagne1 EmilienM: I will need more than 1 coffee for that one =)
14:11 EmilienM \o/
14:24 openstackgerrit joined #puppet-openstack
14:31 mjblack joined #puppet-openstack
14:40 openstackgerrit joined #puppet-openstack
14:55 technolo-g joined #puppet-openstack
14:55 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet-quantum: Add ordering and include quantum-plugin-cisco package resource  https://review.openstack.org/41438
15:21 ersmith joined #puppet-openstack
15:22 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet-quantum: Add ordering and include quantum-plugin-cisco package resource  https://review.openstack.org/41438
15:24 prad_ joined #puppet-openstack
15:38 EmilienM mgagne: before if you have some time today, I have some review for core devs here
15:38 EmilienM https://review.openstack.org/#/c/40122/ -  https://review.openstack.org/#/c/41103/ -  https://review.openstack.org/#/c/41147/
15:41 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet-glance: Include iso as supported disk_format  https://review.openstack.org/41452
15:49 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet-glance: Include iso as supported disk_format  https://review.openstack.org/41454
15:57 otherwiseguy joined #puppet-openstack
16:02 EmilienM mgagne: how do you test a value with it could be "None" ?
16:02 mgagne None or absent?
16:02 EmilienM mgagne: for oscert, absent I think
16:03 mgagne .with_ensure('absent') I would say
16:03 mgagne or .with(:ensure => 'absent')
16:03 EmilienM ok
16:07 marun joined #puppet-openstack
16:07 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-ceilometer: Add support for CA authentication in Keystone  https://review.openstack.org/41103
16:08 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-ceilometer: Add support for CA authentication in Keystone  https://review.openstack.org/41103
16:29 mgagne EmilienM: you can nest contexts in rspec tests to test more specific cases: https://review.openstack.org/#/c/41103/8
16:29 mgagne EmilienM: you can add new context at the end of the shared_examples block to test override of parameters for example
16:30 mgagne EmilienM: I'll show an example
16:31 EmilienM mgagne: thx !
16:37 mgagne EmilienM: https://gist.github.com/mgagne/6212607#file-gistfile1-rb-L56-L61
16:39 mgagne before block modifies params before the execution of the context. merge! merges in-place instead of returning a new hash with the merged value. I used params[:auth_cacert] because I'm lazy and want to reuse the value I already have.
16:39 mgagne the new params value is contained/limited to the context and context add a description/context to the output
16:42 EmilienM mgagne: got it
16:42 EmilienM mgagne: thx. I'm about leaving now. See you later maybe
16:43 mgagne EmilienM: alright cya
17:04 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet-quantum: Add ordering and include quantum-plugin-cisco package resource  https://review.openstack.org/41438
17:23 bodepd mgagne: pre-release is usually marked as rc or alpha I think in the forge
17:24 bodepd mgagne: this causes librarian-puppet to explode, but it works with the module tool
17:24 mgagne bodepd: true, I'm not sure how they are handled by dependency resolution
17:25 mgagne bodepd: With >=1.0.0  Would 1.0.0-rc1 be installed?
17:26 bodepd EmilienM: mgagne I hope not :)
17:26 bodepd EmilienM:  you should ask ryancolement that in #puppet
17:26 bodepd ryancoleman
17:29 bodepd sorry, I mean to send that to mgagne
17:29 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet-quantum: Add missing quantum-plugin-cisco package resource  https://review.openstack.org/41438
17:33 openstackgerrit Dan Bode proposed a change to stackforge/puppet-openstack: Added force_config_drive option with default set to false for compute  https://review.openstack.org/41173
17:34 openstackgerrit Dan Bode proposed a change to stackforge/puppet-openstack: Added force_config_drive param for compute  https://review.openstack.org/41173
17:37 openstackgerrit A change was merged to stackforge/puppet-tempest: Add Modulefile  https://review.openstack.org/41145
17:45 openstackgerrit A change was merged to stackforge/puppet-nova: Improve Neutron support in networking  https://review.openstack.org/41147
17:49 blkperl bodepd: so I think we got all the fixture files for grizzly, is there other low hanging fruit besides the updating the concat references?
17:49 blkperl which is apparently being released on wednesday
17:51 bodepd blkperl: the vswitch module on the forge is confusing.
17:52 bodepd blkperl: the docs aren't rendering properly, and there is no link to the github url
17:52 blkperl Yeah I bugged Ryan about README formatting a while ago.... for a differnet module
17:52 bodepd blkperl: if you want somethign more complicated task...
17:53 bodepd blkperl: (I speak English, trust me)
17:53 bodepd blkperl: I need integration tests running against havana.
17:53 blkperl ha, intergration tests is not exactly friendly for my crew of new people :)
17:53 bodepd blkperl: yeah.
17:53 bodepd blkperl: but it's fun!
17:54 blkperl tests fun? maddness! :)
17:54 bodepd mgagne: you got any intern friendly tasks in mind
17:54 bodepd blkperl: no, building test framework to automate running tests is fun :)
17:55 bodepd mgagne: one of those patches needs to be abandoned
17:55 bodepd uggh. I mean blkperl ^^^
17:55 blkperl which patch?
17:57 blkperl bodepd: ?
17:57 bodepd sorry, I suck at irc today.
17:58 bodepd mgagne: nevermind that. and do you have any easy intern friendly tasks in mind that blkperl can delegate to his army of cheap student labor ?
17:58 blkperl Hunner: look at 10:51
17:59 bodepd blkperl: https://review.openstack.org/#/c/41197/ can you get this abandoned if it doesn't need to get it?
18:00 blkperl yea
18:00 bodepd and I'm going to resign from irc for the day and practice typing :)
18:03 Hunner Don't re-invent another integration framework! Use an existing one...
18:04 Hunner blkperl: vswitch updated to the github page. I'm not dealing with the readme atm
18:05 blkperl Hunner: just yell at Ryan, readme is broken other places too, I don't think it likes some of githubs markdown
18:07 bodepd Hunner: I already wrote a vagrant wrapper, it just isn't running atm
18:07 bodepd Hunner: for havana
18:08 bodepd Hunner: how is your vagrant provider for openstack coming?
18:12 blkperl anyone else running swift with puppet-swift? My proxy is refusing to start and there is a lack of useful logs
18:12 blkperl IOError: [Errno 2] No such file or directory: '/etc/swift/object.ring.gz'
18:12 blkperl doesnt the module create that?
18:16 hogepodge joined #puppet-openstack
18:19 bodepd blkperl: it has some nasty orchestration issues.
18:20 bodepd you have to run the storage nodes twice
18:20 bodepd 1. storage nodes (they fail). 2. proxy/ringbuilder. 3. storage nodes again (and they pass)
18:20 bodepd blkperl: ^
18:22 blkperl oh
18:22 blkperl I remember you saying something about that now
18:23 blkperl Another local or imported resource exists with the type and title Swift::Ringsync[account]
18:23 blkperl sigh....
18:32 openstackgerrit A change was merged to stackforge/puppet-glance: Include iso as supported disk_format  https://review.openstack.org/41452
18:32 openstackgerrit A change was merged to stackforge/puppet-glance: Include iso as supported disk_format  https://review.openstack.org/41454
18:33 mgagne bodepd: Regarding puppet-lint, I couldn't care less about 80chars and default parameters. Can we disable these checks?
18:39 openstackgerrit Daneyon Hansen proposed a change to stackforge/puppet-swift: Improve Proxy Signing Directory Support  https://review.openstack.org/40356
18:43 Hunner bodepd: Oh I'm not working on it yet. I am working on vagrant tools and will use a vagrant-openstack provider once the other integration tools are built up
18:44 blkperl mgagne: +1 to that :)
18:45 mgagne blkperl: should only be a matter of adding those lines to Rakefile: https://gist.github.com/mgagne/6213775
18:46 blkperl yep
18:50 openstackgerrit joined #puppet-openstack
18:57 bodepd mgagne: default parameters, for sure.
18:57 bodepd mgagne: I kind of like hte 80 char limit as a warning
18:58 mgagne bodepd: unless we take aggressive measure to fix it, I want it disabled
18:58 mgagne bodepd: 80chars (with puppet) can lead to unreadable code
18:59 mgagne bodepd: or weird indentation just for the sake of respecting the 80chars rule
18:59 bodepd Hunner: I'd be happy to look at what you guys come up with
18:59 bodepd mgagne: can we bump it to 100 or 120?
18:59 bodepd mgagne: I'd like to have some kind of limit
18:59 bodepd mgagne: surely the limit should be 80 < x < 1000
19:00 bodepd mgagne: that being said, I don't really care if you just disable it
19:00 mgagne bodepd: sure, I hope people won't sh*t 1000 one-liner
19:01 bodepd mgagne: you can't trust anyone >_>
19:01 mgagne bodepd: the 80chars rule is hardcoded (of course)
19:04 bodepd mgagne: sounds like a reasonable patch to me
19:05 mgagne bodepd: I'll change puppet-lint to voting
19:05 mgagne bodepd: making some last tests
19:08 blkperl bodepd: so are you going to deprecate the openstack module or not? because theres somethings I could fix but I don't want to make wasted effort
19:12 blkperl i.e swift support is missing things so I can either add it to the swift module as smoke tests or fix openstack
19:16 bodepd mgagne: oh, I thought it already was :)
19:16 bodepd blkperl: I'm not sure.
19:16 mgagne bodepd: writing email and sending review to infra in a couple of minutes
19:17 bodepd blkperl: I am going to propose a replacement
19:17 bodepd blkperl: but I suspect that the openstack module will live at least for a little while
19:17 technolo-g +1 on the > 80 limit for lint
19:17 blkperl bodepd: :) where will you be proposing the replacement? is there a mailing list?
19:18 mgagne blkperl: head@bodepd.com
19:18 mgagne blkperl: https://github.com/CiscoSystems/openstack-cosi-blueprints/tree/master/blueprints
19:19 bodepd blkperl: yes. I'm going to move the blueprint under launchpad
19:19 bodepd blkperl: https://github.com/bodepd/kickstack
19:20 bodepd blkperl: it's still too early to really share
19:20 bodepd blkperl: puppet-openstack@puppetlabs.com is the project mailing list, and it should be a blueprint in launchpad
19:21 bodepd mgagne: https://github.com/bodepd/kickstack/blob/master/manifests/cinder/all.pp
19:21 bodepd mgagne: that is what it's all about right there :)
19:21 mgagne bodepd: I love this file :D
19:21 blkperl ty
19:22 bodepd blkperl: it's really early, I should also remove the README, b/c it is irrelevant atm
19:23 technolo-g i recommend a little more abstraction:
19:23 technolo-g http://fpaste.org/31588/37633536/
19:23 bodepd my target is to have an installer, and CI running against openstack by havana
19:23 technolo-g :)
19:26 mgagne bodepd: https://review.openstack.org/#/c/41502/
19:29 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet-openstack: Expose nova security_group_api parameter  https://review.openstack.org/41112
19:36 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: Install bridge-utils only if needed  https://review.openstack.org/40122
19:36 EmilienM mgagne: yes, sometimes I disagree :P ^
19:37 mgagne EmilienM: I don't mind opposition =)
19:37 EmilienM mgagne: I know :)
19:38 mgagne voting puppet-lint jobs are coming :D
19:40 bodepd mgagne: thanks for all of your hard work to make that happen.
19:41 mgagne =)
19:41 bodepd mgagne: probably about 40 patches over 2 months
19:41 bodepd mgagne: does that sound about right :)
19:41 mgagne I don't know, I didn't count :P
19:42 mgagne When you are on your balcony, under the sun, doing puppet-lint cleanup, you don't see the time pass ;)
20:04 EmilienM mgagne: thx for https://review.openstack.org/#/c/41103/ :)
20:04 mgagne =)
20:05 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-ceilometer: Add support for CA authentication in Keystone  https://review.openstack.org/41103
20:05 EmilienM mgagne: I made a good progress on cells support
20:05 EmilienM mgagne: I would like to push a WIP tomorrow
20:05 mgagne EmilienM: cool =)
20:06 EmilienM mgagne: with a "don't merge" in the commit message
20:06 EmilienM just to share what I did, and get some feedback as soon as possible
20:06 EmilienM rather than doing evental mistakes
20:06 mgagne EmilienM: Marking it as WIP should be enough I would say
20:06 EmilienM eventual *
20:06 EmilienM ok
20:07 mgagne EmilienM: here is the results of the use of context: http://logs.openstack.org/03/41103/9/check/gate-puppet-ceilometer-puppet-unit-3.1/7283dcb/console.html
20:08 EmilienM mgagne: I see
20:08 EmilienM mgagne: tests should cover any situation, even in puppet
20:09 mgagne EmilienM: we should aim for such coverage =)
20:09 mgagne EmilienM: btw, we should document which deployment scenarios/plugins/drivers are supported by the modules
20:10 EmilienM +1
20:10 EmilienM and dachary suggested to document also a deployment using modules
20:10 mgagne cool
20:11 EmilienM mgagne: where would you like to see the doc ?
20:12 mgagne EmilienM: I'm not sure yet, where do you think it should go? Wiki? Sphinx generated doc? Markdown files? And in which repo?
20:13 EmilienM I would love to see that in docs.openstack.org, but I'm not sure
20:19 mgagne I think it would imply an endorsement of our project by openstack
20:19 mgagne EmilienM: where are other stackforge projects hosting their manuals/docs ?
20:20 EmilienM mgagne: yes
20:20 EmilienM mgagne: other projects usually use readthedocs
20:29 mjblack joined #puppet-openstack
20:49 mgagne bodepd: ping
20:49 mgagne bodepd: is the github namespace and its tasks still used? https://github.com/stackforge/puppet-openstack/blob/master/Rakefile
20:53 bodepd mgagne: no, I thought I removed that when I fixed the upstream branches...
20:53 bodepd mgagne: it now uses the Puppetfile
21:02 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet-openstack: Expose nova security_group_api parameter  https://review.openstack.org/41112
21:04 otherwiseguy joined #puppet-openstack
21:07 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstack: Cleanup Rakefile by removing unused tasks  https://review.openstack.org/41530
21:08 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstack: Cleanup Rakefile by removing unused tasks  https://review.openstack.org/41531
21:11 openstackgerrit Logan McNaughton proposed a change to stackforge/puppet-openstack: Allow Controller to manage volumes  https://review.openstack.org/37277
21:11 mgagne bodepd: can you approve? =)
21:18 blkperl grrr swift errors are so frustrating, http://fpaste.org/31622/34225213/ anyone have an idea? looks like im passing in '' somewhere
21:19 mgagne incoming
21:19 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ceilometer: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41532
21:19 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-cinder: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41533
21:19 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-glance: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41534
21:19 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-heat: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41535
21:19 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-horizon: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41536
21:19 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-keystone: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41537
21:19 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41538
21:19 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstack: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41539
21:19 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-quantum: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41540
21:19 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-swift: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41541
21:20 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-tempest: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41542
21:24 openstackgerrit A change was merged to stackforge/puppet-openstack: Cleanup Rakefile by removing unused tasks  https://review.openstack.org/41530
21:42 mgagne damn... =(
21:47 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ceilometer: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41546
21:47 openstackgerrit Pradeep Kilambi proposed a change to stackforge/puppet-openstack: Expose nova security_group_api parameter  https://review.openstack.org/41117
21:48 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ceilometer: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41532
21:50 bodepd blkperl: what verson of swift?
21:51 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-cinder: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41533
21:51 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-glance: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41534
21:51 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-heat: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41535
21:51 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-horizon: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41536
21:51 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-keystone: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41537
21:51 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41538
21:51 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstack: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41539
21:51 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-quantum: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41540
21:51 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-swift: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41541
21:51 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-tempest: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41542
21:51 mgagne sorry for the spam
21:52 bodepd ßn
21:54 blkperl bodepd: 1.8.0-0ubuntu1.2~cloud0
21:58 mgagne bodepd: 2 modules without warnings :D http://logs.openstack.org/42/41542/2/check/gate-puppet-tempest-puppet-lint/2efbb73/console.html & http://logs.openstack.org/32/41532/2/check/gate-puppet-ceilometer-puppet-lint/248ff95/console.html
21:58 openstackgerrit A change was merged to stackforge/puppet-openstack: Cleanup Rakefile by removing unused tasks  https://review.openstack.org/41531
22:02 openstackgerrit A change was merged to stackforge/puppet-tempest: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41542
22:02 openstackgerrit A change was merged to stackforge/puppet-swift: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41541
22:03 dmsimard joined #puppet-openstack
22:03 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41532
22:03 bodepd mgagne: manifests/python.pp - WARNING: class inheriting from params class on line 8
22:04 bodepd mgagne: that is a strange warning.
22:04 openstackgerrit A change was merged to stackforge/puppet-quantum: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41540
22:06 mgagne bodepd: it wasn't recommended to inherit from params because puppet 2.6 didn't full supported it
22:06 mgagne bodepd: http://docs.puppetlabs.com/guides/style_guide.html#class-parameter-defaults
22:06 mgagne bodepd: I guess we can disable this check too, we don't care about 2.6 anymore
22:07 mgagne bodepd: or send a PR to puppet-lint to remove the check
22:07 mgagne bodepd: is it the only module having such warnings?
22:08 openstackgerrit A change was merged to stackforge/puppet-openstack: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41539
22:09 openstackgerrit A change was merged to stackforge/puppet-cinder: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41533
22:10 bodepd mgagne: I didn't go through all of them
22:10 mgagne bodepd: ok, I'll grep
22:11 bodepd mgagne: wow. 2.6.2 compat
22:11 openstackgerrit A change was merged to stackforge/puppet-keystone: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41537
22:11 openstackgerrit A change was merged to stackforge/puppet-nova: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41538
22:11 bodepd that is ancient
22:11 mgagne only nova and keystone are concerned by this warning
22:12 openstackgerrit A change was merged to stackforge/puppet-glance: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41534
22:12 openstackgerrit A change was merged to stackforge/puppet-heat: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41535
22:12 mgagne bodepd: should we disable it globally or just for those modules?
22:12 openstackgerrit A change was merged to stackforge/puppet-horizon: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41536
22:17 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ceilometer: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41555
22:18 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-cinder: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41556
22:18 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-glance: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41557
22:18 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-horizon: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41558
22:18 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-keystone: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41559
22:18 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41560
22:18 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstack: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41561
22:18 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-quantum: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41562
22:18 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-swift: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41563
22:18 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-tempest: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41564
22:24 openstackgerrit joined #puppet-openstack
22:27 openstackgerrit A change was merged to stackforge/puppet-swift: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41563
22:27 openstackgerrit A change was merged to stackforge/puppet-quantum: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41562
22:29 openstackgerrit A change was merged to stackforge/puppet-keystone: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41559
22:29 openstackgerrit A change was merged to stackforge/puppet-nova: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41560
22:29 openstackgerrit A change was merged to stackforge/puppet-horizon: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41558
22:30 openstackgerrit A change was merged to stackforge/puppet-glance: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41557
22:30 openstackgerrit A change was merged to stackforge/puppet-cinder: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41556
22:31 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41555
22:32 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-tempest: Add Modulefile  https://review.openstack.org/41566
22:34 openstackgerrit A change was merged to stackforge/puppet-openstack: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41561
22:34 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-tempest: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41564
22:44 openstackgerrit A change was merged to stackforge/puppet-tempest: Add Modulefile  https://review.openstack.org/41566
22:44 openstackgerrit A change was merged to stackforge/puppet-tempest: Disable 80chars/class_parameter_defaults checks  https://review.openstack.org/41564
22:44 mgagne finally :D
22:46 nibalizer haha
22:47 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ceilometer: DO NOT MERGE - Testing puppet-lint gate  https://review.openstack.org/41570
22:51 openstackgerrit Jonathan Hong proposed a change to stackforge/puppet-nova: updated fixtures to correct postgresql link  https://review.openstack.org/41571
22:51 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-ceilometer: DO NOT MERGE - Testing puppet-lint gate  https://review.openstack.org/41570
22:58 openstackgerrit A change was merged to stackforge/puppet-nova: updated fixtures to correct postgresql link  https://review.openstack.org/41571
22:59 openstackgerrit Jonathan Hong proposed a change to stackforge/puppet-keystone: corrected the postgresql link in .fixtures.yml  https://review.openstack.org/41572
23:00 openstackgerrit A change was merged to stackforge/puppet-keystone: corrected the postgresql link in .fixtures.yml  https://review.openstack.org/41572
23:27 openstackgerrit Jonathan Hong proposed a change to stackforge/puppet-nova: updated fixtures to correct postgresql link  https://review.openstack.org/41574
23:30 bodepd mgagne: it's up to you
23:43 blkperl bodepd: in case you didn't see amongst the spam, its swift 1.8.0-0ubuntu1.2~cloud0
23:43 bodepd blkperl: yeah, idk :)
23:43 blkperl its super frustrating docs for the command suck
23:45 blkperl i've changed the arguement values and it still fails
23:57 openstackgerrit William Van Hevelingen proposed a change to stackforge/puppet-openstack: Add swift to openstack::all  https://review.openstack.org/41577

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