Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-06-03

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

All times shown according to UTC.

Time Nick Message
00:11 dtalton2 joined #puppet-openstack
00:14 kkirkpatrick joined #puppet-openstack
00:17 mdorman joined #puppet-openstack
00:29 aimon joined #puppet-openstack
00:30 rmoe joined #puppet-openstack
00:51 djm68 joined #puppet-openstack
00:53 michchap I don't have strong opinions on the module name. As I noted on the pad I don't want to push up what I have right now, instead I think it would be better if I work to make it encompass Emilien's work as a sanity check against me pushing up something that's too opinionated.
00:55 michchap For example. right now I make assumptions about the network layout which feeds into how I create endpoints and load balancing rules.
01:05 starmer joined #puppet-openstack
01:07 thumpba joined #puppet-openstack
01:08 djm68 joined #puppet-openstack
01:14 thumpba joined #puppet-openstack
01:38 xarses joined #puppet-openstack
01:40 djm68 joined #puppet-openstack
01:41 thumpba joined #puppet-openstack
01:51 xingchao joined #puppet-openstack
01:57 openstackgerrit Lars Kellogg-Stedman proposed a change to stackforge/puppet-ceilometer: Fixes agent_notification_service_name  https://review.openstack.org/95519
01:59 thumpba joined #puppet-openstack
02:00 starmer joined #puppet-openstack
02:10 kitp joined #puppet-openstack
02:12 kitp_ joined #puppet-openstack
02:17 djm68 joined #puppet-openstack
02:44 djm68 joined #puppet-openstack
03:13 kitp joined #puppet-openstack
03:20 badiane_ka joined #puppet-openstack
03:22 thumpba joined #puppet-openstack
03:50 kkirkpatrick joined #puppet-openstack
04:06 djm68 joined #puppet-openstack
04:10 rmoe joined #puppet-openstack
04:10 starmer joined #puppet-openstack
04:14 kitp joined #puppet-openstack
04:22 kitp joined #puppet-openstack
04:53 sanjayu joined #puppet-openstack
05:19 djm68_ joined #puppet-openstack
05:45 sputnik13 joined #puppet-openstack
05:54 gcha joined #puppet-openstack
05:57 aldavud joined #puppet-openstack
06:29 kitp joined #puppet-openstack
06:31 EmilienM good morning
06:40 aldavud joined #puppet-openstack
07:04 artem_panchenko1 joined #puppet-openstack
07:07 arnaud_orange morning
07:13 ilbot3 joined #puppet-openstack
07:13 Topic for #puppet-openstack is now Place to collaborate on Puppet/OpenStack tools: logs at http://irclog.perlgeek.de/puppet-openstack/today
07:30 kitp joined #puppet-openstack
07:35 EmilienM I have no idea about tests failures on https://review.openstack.org/#/c/82353/
07:51 aldavud joined #puppet-openstack
07:59 starmer joined #puppet-openstack
08:02 admin0 joined #puppet-openstack
08:20 derekh_ joined #puppet-openstack
08:24 francois EmilienM: hi, looks like those tests are failing even with your patch reverted
08:25 francois i see the same failure with the backports I've sent yesterday
08:25 francois https://review.openstack.org/#/c/97279/
08:25 francois for instance
08:27 francois hmmm, not exactly the same failures actually...
08:31 kitp joined #puppet-openstack
08:31 EmilienM francois: interesting
08:31 EmilienM maybe fc__ has an idea
08:32 EmilienM i was thinking at a new version of rspec
08:32 EmilienM francois: tests pass locally, I would need to recheck maybe
08:37 francois rspec version sounds like a likely culprit
08:38 francois PASS with rspec 2.14 http://logs.openstack.org/10/88410​/26/gate/gate-puppet-neutron-puppe​t-unit-2.7/682b8f2/console.html.gz
08:38 francois FAIL with rspec 3 http://logs.openstack.org/79/97279/1/check/gate-p​uppet-cinder-puppet-unit-2.7/7add666/console.html
08:51 EmilienM ouch
08:57 mattymo reading scrollback
08:57 mattymo I like the glue theme
08:57 mattymo puppet-openstack-superglue
09:26 bogdando joined #puppet-openstack
09:27 mmagr joined #puppet-openstack
09:30 openstackgerrit A change was merged to stackforge/puppet-swift: Improve readability of swift::xfs  https://review.openstack.org/94867
09:31 openstackgerrit A change was merged to stackforge/puppet-swift: Bump stdlib dependency from >=2.3.0 to >=3.2.0  https://review.openstack.org/97243
09:32 kitp joined #puppet-openstack
09:57 openstackgerrit A change was merged to stackforge/puppet-keystone: Introduce keystone:config  https://review.openstack.org/97278
10:20 xingchao joined #puppet-openstack
10:33 kitp joined #puppet-openstack
10:59 Mr_D joined #puppet-openstack
11:18 morazi joined #puppet-openstack
11:20 NuclearW0rm joined #puppet-openstack
11:33 kitp joined #puppet-openstack
11:42 morazi joined #puppet-openstack
11:43 artem_panchenko joined #puppet-openstack
12:34 kitp joined #puppet-openstack
12:49 djm68 joined #puppet-openstack
12:56 dprince joined #puppet-openstack
13:03 [4]Mr_D joined #puppet-openstack
13:07 openstackgerrit joined #puppet-openstack
13:17 beddari lib is lib but the "other stuff" to me is roles and profiles in a wide sense, has been my way of thinking (on other projects) a good year or so
13:18 beddari I think it works OK within its own limitations hehe
13:22 michchap the other stuff is explicitly not roles and profiles
13:22 beddari I got that, but that is my "wide" sense of the term
13:25 beddari so the actual config data (which in my case comes from Hiera, service discovery) lives in the profile layer for things like haproxy clusters, firewalling … anything more "dynamic"
13:29 djm68 joined #puppet-openstack
13:31 mjblack joined #puppet-openstack
13:32 mjblack joined #puppet-openstack
13:35 kitp joined #puppet-openstack
13:50 kkirkpatrick joined #puppet-openstack
13:51 shakamunyi joined #puppet-openstack
14:00 openstackgerrit Lukas Bezdicka proposed a change to stackforge/puppet-swift: Add swift::proxy::crossdomain class  https://review.openstack.org/97259
14:09 prad joined #puppet-openstack
14:12 mdorman joined #puppet-openstack
14:32 kitp joined #puppet-openstack
14:44 mgagne test
14:45 dmsimard mgagne: ^ received
14:46 EmilienM mgagne: could you please have a look at my comment: https://review.openstack.org/#/c/92993/
14:46 mgagne EmilienM: ok
14:46 EmilienM i'm also trying to figure out why tests are failing for some current patches, included this one: https://review.openstack.org/#/c/82353/
14:46 mgagne EmilienM: how can you have 2 cinder-volume with each an AZ?
14:48 EmilienM mgagne: ok so you want to me that I move parameters in api & backup classes, to support multiple regions, right?
14:48 mgagne EmilienM: With Nova, AZ is configured in nova-compute. I don't know why it would be different with Cinder.
14:48 EmilienM mgagne: it makes sense
14:49 mgagne EmilienM: but
14:49 EmilienM mgagne: that's this kind of comment I was waiting. Thanks
14:49 mgagne EmilienM: Cinder has cinder-backup too
14:49 mgagne EmilienM: So I don't know how to handle it without creating duplicated ressources
14:49 mgagne EmilienM: btw, I know it's a minor change but how do you feel about a change submitted, reviewed and approved exclusively by people from enovance? https://review.openstack.org/#/c/97243/
14:50 EmilienM I don't see the problem if you run multiple AZ
14:51 EmilienM mgagne: sbadia and me did not talk each others about this patch, so our consideration was not affected by the company we are working for
14:51 mgagne EmilienM: it's not something external know about
14:51 EmilienM if this patch is not good for you, let us know why and maybe revert it if we broke something.
14:51 EmilienM but again, we just do reviews
14:51 EmilienM maybe something few people is doing here
14:52 EmilienM mgagne: cinder: I think we should let it in init.pp and people using multiple AZ can handle it with Hiera and override.
14:52 EmilienM isn't i?
14:52 EmilienM isn't it?
14:53 mgagne EmilienM: maybe, we should however make it consistent across our modules
14:54 mgagne EmilienM: about review: it's not about the change being broken or not, it's about external observers seeing 3 people from the same company approving a change and not knowing if it's a coordinated effort to fast-forward a change or not.
14:54 sbadia I'm very careful with this (with the same company in review), and mathieu pin-point this change
14:54 abramley joined #puppet-openstack
14:55 sbadia but anyway we must maybe define a « policy » about this
14:55 EmilienM sbadia: we did not talk each other, you could have been another guy, it would have been the same
14:56 EmilienM mgagne: you know we lake in reviews...
14:56 sbadia EmilienM: yep yep i agree, but it can only be better
14:56 EmilienM mgagne: so if you are not happy with this, let's define a rule and this won't happen again.
14:56 EmilienM but I believe it won't work... again, few people does reviews every days like us.
14:57 sbadia EmilienM: I think it's not a remark for you, but a general remark
14:57 sbadia mgagne: no?
14:57 EmilienM sbadia: I did not take this for me
14:57 sbadia ok :)
14:57 EmilienM sbadia: I'm just concerned about this rule which could affect review process
14:58 EmilienM "With great power comes great responsibility"
14:58 sbadia ^^'
14:58 EmilienM so we should make reviews without consideration of the company we work for
14:59 EmilienM sbadia: I won't give any example in OpenStack, but it's the case for many patches/projects... :(
15:02 EmilienM mgagne: final words on puppet-cinder? :-)
15:14 tfz joined #puppet-openstack
15:14 sputnik13 joined #puppet-openstack
15:19 tfz joined #puppet-openstack
15:19 mgagne damn, I'm really having serious issues with my IRC client...
15:29 larsks Hi all.  I have the distinct feeling I've asked this before, but: for a module like the puppetlabs "firewall" module, that really wants certain packages installed in order to work correctly, is there a way to make sure those packages are installed before realizing custom resource types?
15:30 dmsimard larsks: you can add a require => Package['requiredpackage'] ?
15:31 larsks dmsimard: To a custom provider?  The problem is they can be used without ever including the classes defined in the module manifests.
15:31 larsks So, e.g., people can create firewall {...} resources without ever "include"ing the "firewall" class (because puppet automatically loads things from lib/).
15:32 larsks dmsimard: We're hitting an issue where the firewall module wants iptables-services installed, but the resources are being realized before the package is installed leading to errors.
15:32 mgagne larsks: your defined resource could include the firewall class
15:33 mgagne larsks: hmm, it's not a defined resource :-/
15:33 larsks mgagne: No, custom types/providers.
15:33 dmsimard Oh, perhaps something like Package['requiredpackage'] -> custom_provider <||> ?
15:33 mgagne larsks: https://github.com/puppetlabs/puppet​labs-firewall/blob/master/lib/puppet​/provider/firewall/iptables.rb#L33
15:33 mgagne larsks: doesn't look optional to me
15:35 larsks mgagne: Hmm, worth looking at, although  I think a different problem (we actually need /usr/libexec/iptables/iptables.init, provided by iptables-services package).
15:35 larsks mgagne: But yeah, those don't look optional :)
15:35 larsks mgagne: Is there a way to assert dependencies from within a custom provider (for if people do not explicitly include the class)?
15:36 mgagne larsks: tbh, I'm not familiar enough with custom provider to be able to answer you
15:36 kkirkpatrick joined #puppet-openstack
15:36 openstackgerrit A change was merged to stackforge/puppet-nova: Adds vnc_keymap support  https://review.openstack.org/97244
15:36 larsks mgagne: Yeah.  My problem has been that very few people appear to be intimately familiar with custom providers :).  I will keep looking. Thanks for your thoughts!
15:36 mgagne larsks: http://docs.puppetlabs.com/guides/cust​om_types.html#automatic-relationships but require isn't enforced
15:37 mgagne larsks: ask bodepd, he wrote a book about it
15:37 larsks mgagne: Nice. Thanks for the suggestion.
15:38 EmilienM mgagne: have you read the backlog?
15:38 mgagne EmilienM: about cinder?
15:38 EmilienM mgagne: about everything we talked about.
15:38 mgagne EmilienM: +/-
15:38 EmilienM yeah about cinder first
15:38 mgagne EmilienM: reading
15:39 EmilienM mgagne: then about the review process
15:39 mgagne EmilienM: I'll have to fix my dashboard, I can't find the review without browsing the project directly :-/
15:40 EmilienM mgagne: about https://review.openstack.org/#/c/92993/ ?
15:40 mgagne EmilienM: thanks
15:41 mgagne EmilienM: ok, there is a logic to assign storage_availability_zone to default_availability_zone if it's not defined. Why? What happens if it's not defined?
15:42 EmilienM if default_availability_zone is empty, it will takes storage_availability_zone
15:42 EmilienM default value is "nova
15:43 mgagne EmilienM: I meant in Cinder vanilla
15:43 EmilienM mgagne: "nova"
15:44 tfz1 joined #puppet-openstack
15:44 mgagne EmilienM: https://review.openstack.org/#/c/45631/
15:45 mgagne EmilienM: so we are reimplementing that logic in puppet =)
15:45 EmilienM mgagne: yes, isn't that good? I also read that code before
15:46 mgagne EmilienM: storage_availability_zone looks to be shared across the board, including cinder-api
15:47 EmilienM mgagne: also I'm trying to figure out why we have some failures in some patches, like https://review.openstack.org/#/c/82353/
15:56 anode joined #puppet-openstack
16:02 mgagne EmilienM: checking
16:02 mgagne EmilienM: do you have this issue? http://paste.openstack.org/show/82626/
16:04 mgagne EmilienM: RSpec::Puppet::ManifestMatchers::CreateGeneric implements a legacy RSpec matcher protocol.
16:05 mgagne EmilienM: rspec (3.0.0) is installed
16:05 ryanycoleman joined #puppet-openstack
16:05 mgagne EmilienM: they look to have deprecated a bunch of stuff which rspec-puppet relies on
16:05 rmoe joined #puppet-openstack
16:06 mgagne EmilienM: my gem list: http://paste.openstack.org/show/82631/
16:07 mgagne EmilienM: rspec-puppet should be fixed to support rspec 3
16:07 openstackgerrit Yanis Guenane proposed a change to stackforge/puppet-swift: Realize exported ring_*_devices resources  https://review.openstack.org/97551
16:08 mgagne EmilienM: https://github.com/rodjek/rspec-puppet/pull/201 https://github.com/rodjek/rspec-puppet/issues/200 https://github.com/rodjek/rspec-puppet/pull/199
16:08 EmilienM mgagne: come back in  5 min, in a meeting
16:08 mgagne EmilienM: grabbing lunch
16:14 abramley left #puppet-openstack
16:14 mgagne EmilienM: https://bugs.launchpad.net/​puppet-neutron/+bug/1326034
16:15 EmilienM mgagne: thank you, i guess we should submit a patch which fix tests, and then rebase all tests which are failing
16:16 mgagne EmilienM: we can either: fix rspec-puppet or enable the bunch of configs the deprecation warnings suggest
16:16 mgagne EmilienM: or cap rspec to <3.0.0
16:16 djm68 joined #puppet-openstack
16:17 EmilienM mgagne: i have no idea which is the best choice to be honest
16:17 mgagne EmilienM: I'll try to figure that out after lunch
16:17 EmilienM mgagne: enjoy it!
16:18 sbadia EmilienM: 2.14.1 ? :) https://github.com/enovance/puppet-openstack-clou​d/commit/e91fe40281207799a83ed7868ea7601e2067989f
16:19 sbadia the latest 2.14.x release on rubygem forge
16:19 * EmilienM feels like sbadia has a patch :P
16:20 sbadia yup I've fixed that with spredzy_ this morning
16:20 * sbadia submit a patch to r.o.o
16:20 sbadia ;)
16:30 EmilienM dmsimard: hey, I would like an ee on https://review.openstack.org/#/c/97551 please
16:31 dmsimard an ee ?
16:31 dmsimard I can test it in my CI see if it breaks anything but the timing is bad this week :)
16:33 hogepodge Can we fix our tests rather than pin? EmilinM?
16:33 hogepodge Deprecation failures?
16:33 ryanycoleman joined #puppet-openstack
16:34 * spredzy_ afk for lunch
16:36 xarses joined #puppet-openstack
16:36 EmilienM dmsimard: an eye, sorry
16:36 EmilienM hogepodge: that would be a good option, yes
16:36 hogepodge Is it just an api change? provider exists.
16:39 hogepodge Oh, yuck, I see. Puppet rspec problem
16:39 dtalton joined #puppet-openstack
16:40 hogepodge or rspec-puppet I mean.
16:43 sbadia also puppetlabs_spec_helper in our case (https://github.com/puppetlabs/puppetlab​s_spec_helper/blob/master/Rakefile#L32) but yep it's the same
16:44 aimon joined #puppet-openstack
16:44 dtalton joined #puppet-openstack
16:47 sbadia 621 deprecation warnings total :)
16:49 arnaud_orange left #puppet-openstack
16:51 EmilienM hogepodge: it's a lot of work like sbadia is saying, I would suggest to first ping the previous release, and then fix the tests. It blocks our reviews...
16:51 EmilienM and to be honest, it's out of my schedule this week to fix 600 tests in one puppet module
16:52 sbadia I take a look, deprecation warning are not blocking
16:54 djm68 joined #puppet-openstack
16:57 sbadia but in fact we have only one error
16:57 sbadia \o/ fixed :)
16:57 sbadia I submit a patch
16:58 hogepodge EmilienM right, it’s really an rspec-puppet problem.
16:58 EmilienM lol cool sbadia
16:58 anode joined #puppet-openstack
16:58 hogepodge Which is the continuation of the problem that we don’t pin to tools we depend upon, so our gate breaks every month or two.
16:58 sbadia hogepodge: :-)
16:58 EmilienM :(
16:59 hogepodge thanks sbadia. Can you drop a link when you get it?
16:59 hogepodge EmilienM Don’t mean to make you :(
16:59 sbadia yup, https://www.relishapp.com/rspec/rspec-ex​pectations/v/2-99/docs/built-in-matchers and https://www.relishapp.com/rspec/rspe​c-expectations/v/3-0/docs/changelog
16:59 sbadia « Rename be_true and be_false to be_truthy and be_falsey. (Sam Phippen) »
17:00 sbadia and also https://stackoverflow.com/questions/23937758/rspec​-failing-error-expected-false-to-respond-to-false
17:00 sbadia we still hava a lot of deprecation warning (621 deprecation warnings total) for neutron
17:00 sbadia but the test works now
17:03 EmilienM sighs
17:04 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-neutron: Fix puppet-neutron rspec test for RSpec >= 3.0.0  https://review.openstack.org/97559
17:05 sbadia ack-grep be_{false,true} in stackforge/puppet-* now ^^'
17:07 sbadia mgagne: can I be added to https://launchpad.net/~puppet-openstack ?
17:10 sbadia cool, only puppet-neutron and puppet-openstack use this method
17:11 sputnik13 joined #puppet-openstack
17:12 EmilienM sbadia: i'll +2 once jenkins is happy, i'm going back home now
17:12 sbadia EmilienM: yup, no problem
17:15 mgagne sbadia: Looks like I don't own the team anymore
17:16 mgagne sbadia: or I never owned it
17:16 dtalton left #puppet-openstack
17:16 mgagne sbadia: which actions do you want to do?
17:17 sbadia ah, ok never mind (indeed it's dan)
17:17 mgagne sbadia: I can make it so bug management can be done by more people
17:18 sbadia because i'm too lazy to configure a bug mail (to recieve all bug of the team) :)
17:18 sbadia yep also
17:18 mgagne come on =)
17:19 djm68 joined #puppet-openstack
17:23 mgagne sbadia: done
17:23 sbadia mgagne: thanks!
17:28 openstackgerrit Sebastien Badia proposed a change to stackforge/puppet-openstack: Fix puppet-openstack rspec test for RSpec >= 3.0.0  https://review.openstack.org/97567
17:29 hogepodge \o/
17:41 mmaglana joined #puppet-openstack
17:42 aldavud joined #puppet-openstack
17:43 djm68 joined #puppet-openstack
17:46 mgagne The gate looks to be starving for nodes... http://status.openstack.org/zuul/ We won't see Jenkins verifying the change for a couple more hours ^^'
17:53 photocyte joined #puppet-openstack
17:56 ryanycoleman joined #puppet-openstack
18:02 daneyon joined #puppet-openstack
18:10 imcsk8 joined #puppet-openstack
18:10 kitp joined #puppet-openstack
18:10 ryanycoleman joined #puppet-openstack
18:12 ryanycoleman joined #puppet-openstack
18:14 spredzy_ dmsimard, ping
18:15 dmsimard spredzy_: really busy, sorry
18:15 spredzy_ dmsimard, ack
18:15 spredzy_ np
18:19 thumpba joined #puppet-openstack
18:20 shakamunyi joined #puppet-openstack
18:23 kkirkpatrick joined #puppet-openstack
18:25 shakamunyi joined #puppet-openstack
18:32 kkirkpatrick joined #puppet-openstack
18:42 ryanycoleman joined #puppet-openstack
18:43 bodepd the openstack builder doesn't even work
18:43 bodepd or at least master doesn't
18:44 bodepd michchap: dead link: http://stacktira.aptira.com/doc/index.html
18:50 EmilienM mgagne: back. Do you want to clarify the review process for people working in the same company?
18:50 mgagne EmilienM: I guess we just have to be careful
18:51 mgagne EmilienM: some people privately asked me about our policy regarding reviews
18:52 EmilienM mgagne: to be honest, we don't have interest conflict you and me. We are not working for vendors
18:52 mgagne EmilienM: I know I haven't got much time to review stuff lately but I don't mind reviewing those stuff if asked to avoid such situation
18:52 mgagne EmilienM: people external to the project don't know
18:52 EmilienM If I need a review, I just ask to core team to have a look
18:53 mgagne EmilienM: I'm not sure how other projects handle it tbh
18:53 EmilienM mgagne: i get your point and I'll be more careful
18:53 djm68 joined #puppet-openstack
18:53 mgagne EmilienM: ie, Could VMWare (just an example, no actual known cases) forces through a change of theirs in Nova if they could and has enough core reviewers?
18:54 EmilienM i don't know
18:54 bodepd mgagne: what is the concern?
18:54 EmilienM I hope not
18:56 hogepodge Anyone know what’s up with the gate? Real backlog or just lots of tasks?
18:56 hogepodge mgagne there have been accusations of that in the neutron world for a while now.
18:56 hogepodge other vendors having reviews withheld.
18:56 mgagne hogepodge: fortunately (or not) I didn't know about it
18:57 mgagne hogepodge: gate is starving for nodes
18:57 mgagne hogepodge: http://status.openstack.org/zuul/
18:57 mgagne hogepodge: 473 changes waiting to be checked
18:57 aldavud_ joined #puppet-openstack
18:58 hogepodge I was looking at that. 10 hour backlog
18:59 ryanycoleman joined #puppet-openstack
19:00 dprince joined #puppet-openstack
19:00 hogepodge mgagne EmilienM nearly everyone is getting paid to do this by someone and we all have interests. I think that being aware of them and trying to be honest in our reviews will prevail.
19:01 mgagne hogepodge: +1
19:01 EmilienM hogepodge: I use to review cinder patches for VMware, and we don't use VMware at eNovance
19:01 EmilienM s/cinder/puppet-cinder/
19:01 mgagne EmilienM: a couple of months ago, I would be reviewing close to nothing in my case :P
19:02 EmilienM :)
19:02 derekh_ joined #puppet-openstack
19:02 hogepodge yeah, oftentimes if a vendor patch comes in I try to make sure that it’s correct as much as I can, but explicitly tell them that they’re responsible for it working and being maintained if the underlying systems aren’t freely available.
19:03 mgagne hogepodge: btw, should we continue supporting grizzly?
19:03 djm68 joined #puppet-openstack
19:03 hogepodge I’m not actively supporting it, but I’ve seen patches go back. I don’t see why we shouldn’t merge back ports, but I subscribe to the n-1 thing.
19:04 hogepodge bugs, I guess. features. no. That’s silly.
19:04 mgagne hogepodge: I'm currently working on creating series/milestones in Launchpad so we can target bugs to them
19:05 mgagne hogepodge: I have a script for that, don't worry on me losing my sanity over it =)
19:08 daneyon joined #puppet-openstack
19:11 djm68 joined #puppet-openstack
19:15 djm68 joined #puppet-openstack
19:17 * mgagne has his fingers crossed
19:24 shakamunyi joined #puppet-openstack
19:37 kitp joined #puppet-openstack
19:45 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-nova: Fix typo in README.md  https://review.openstack.org/97607
19:47 aldavud joined #puppet-openstack
19:47 djm68 joined #puppet-openstack
20:03 djm68 joined #puppet-openstack
20:04 ryanycoleman joined #puppet-openstack
20:14 kkirkpatrick joined #puppet-openstack
20:14 shakamunyi joined #puppet-openstack
20:23 aldavud joined #puppet-openstack
20:23 ryanycoleman joined #puppet-openstack
20:26 ryanycoleman joined #puppet-openstack
20:29 spredzy_ mgagne, merci pour les reviews
20:29 mgagne =)
20:49 ryanycoleman joined #puppet-openstack
20:56 gildub joined #puppet-openstack
20:56 ryanycoleman joined #puppet-openstack
20:58 gildub Hi, looking for review on puppet-vswitch patch: https://review.openstack.org/#/c/94312/4
21:00 daneyon joined #puppet-openstack
21:02 ryanycoleman joined #puppet-openstack
21:04 djm68 joined #puppet-openstack
21:17 hogepodge gildub what platforms have you tested this on?
21:29 ryanycoleman joined #puppet-openstack
21:30 djm68 joined #puppet-openstack
21:39 djm68 joined #puppet-openstack
21:41 ryanycoleman joined #puppet-openstack
21:46 djm68 joined #puppet-openstack
21:47 ryanycoleman joined #puppet-openstack
21:49 anode joined #puppet-openstack
21:49 ryanycoleman joined #puppet-openstack
21:55 ryanycoleman joined #puppet-openstack
22:07 bodepd michchap: when you get a second, I'd like to sync up again
22:15 aldavud joined #puppet-openstack
22:21 sputnik13 joined #puppet-openstack
22:35 badiane_ka joined #puppet-openstack
22:37 ryanycoleman joined #puppet-openstack
22:40 aimon michchap you online?
22:41 aimon https://forge.puppetlabs.com/aimonb/aviator
22:41 aimon note the issue I created against it
22:41 aimon its trivial and will be fixed shortly by mmaglana
22:48 aimon joined #puppet-openstack
22:49 ryanycoleman joined #puppet-openstack
22:57 djm68 joined #puppet-openstack
22:59 bodepd openstackextra or extras?
22:59 bodepd I submitted a patch for openstacklib and I'm working on openstackextras
22:59 bodepd is that spacing fine?
23:00 dvorak I like extras
23:02 bodepd extras of _extras :)
23:02 bodepd s/of/or/
23:02 dvorak I'm indifferent
23:02 bodepd one word works for me for lib, but for extras it hurts my eyes
23:02 bodepd which raises the question of if I switch to _extras, should I use _lib
23:03 bodepd I friggen hate naming thing (I know I say that a lot)
23:03 bodepd mgagne: don't you like naming things and enforcing consistency?
23:04 mmaglana sidenote: bodepd  i'm pretty bad at naming things too. BUT I freakin' love the name of the latest class I wrote! https://github.com/aviator/aviator/​blob/master/lib/aviator/hashish.rb
23:04 ryanycoleman joined #puppet-openstack
23:05 dvorak bodepd: I kind of like naming, but it's really really hard to do well
23:10 bodepd dvorak: then: extras or _extras? get on it :)
23:10 dvorak extras
23:14 aldavud joined #puppet-openstack
23:15 bodepd puppet-openstackextras
23:15 dvorak that's horrible, I was wrong :)
23:15 bodepd going once, twice, (and then a few more times while I craft the patch and wait for it to be merged)
23:16 bodepd I can't tell if you are being serious or not :)
23:16 dvorak I'm kind of serious
23:16 bodepd then: puppet-openstack_extras
23:16 dvorak both the options suck
23:16 dvorak yeah, I like that slightly better
23:16 bodepd then: is it ok to have openstacklib or does it have to be puppet-openstack_lib?
23:16 dvorak clearly it should be puppet-open_stack_extras!
23:16 dvorak nah, I think openstacklib is fine
23:16 bodepd p_u_p_p...
23:16 ric`_ joined #puppet-openstack
23:17 bodepd the lib patch is up, just waiting for jenkins to check it. I'm a littel confused by their alpa checks
23:17 bodepd it looks like if fails b/c _ comes after l
23:17 bodepd which I would not expect
23:20 toabctl_ joined #puppet-openstack
23:20 EmilienM_ joined #puppet-openstack
23:20 spredzy joined #puppet-openstack
23:24 bodepd dvorak: if you have a close look at the openstack modules you can see how awful I am at it :)
23:32 ryanycoleman joined #puppet-openstack
23:36 ryanycol_ joined #puppet-openstack
23:43 openstackgerrit A change was merged to stackforge/puppet-heat: Enable heat server to be run in SSL mode  https://review.openstack.org/94412
23:50 daneyon joined #puppet-openstack
23:51 aldavud joined #puppet-openstack
23:58 aimon joined #puppet-openstack

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