Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
23:57 rmoe joined #puppet-openstack
23:57 rongze joined #puppet-openstack
00:22 hogepodge joined #puppet-openstack
00:57 openstackgerrit Zeb Palmer proposed a change to stackforge/puppet-openstack: Typo fix and slight rewording  https://review.openstack.org/60723
01:12 tnoor joined #puppet-openstack
01:16 otherwiseguy joined #puppet-openstack
01:16 mgagne joined #puppet-openstack
01:16 mgagne joined #puppet-openstack
02:26 rongze joined #puppet-openstack
02:28 marun joined #puppet-openstack
02:37 tnoor2 joined #puppet-openstack
03:06 dsockwell joined #puppet-openstack
03:10 xingchao joined #puppet-openstack
03:23 mgagne xingchao: trying to update/refactor/modernize puppet-swift atm
04:00 mgagne xingchao: ping
04:21 xingchao mgagne:  I just come back, updating puppet-swift ?
04:22 mgagne xingchao: well, trying to work on it
04:22 mgagne xingchao: I have one question: do you use concat fragment to inject "local" (your own) configurations to swift?
04:24 xingchao mgagne: yes, we have some custom middlewares, so I add them in proxy-server.conf by concat
04:25 mgagne xingchao: ok, that's what I thought. Moving to native types would break this feature and require end users to use native type to inject his own configs.
04:26 mgagne xingchao: there is no known way to support both methods (concat and inifile provider) :-/
04:26 xingchao mgagne: do you plan to improve this feature ?  :)
04:27 mgagne xingchao: I already moved all swift-proxy.conf configs to ini. I was about to do the same with account, object and container and saw a comment: "you can now add your custom fragments at the user level"
04:28 mgagne xingchao: and then: damn... I didn't think about that.
04:29 xingchao mgagne: aha, I'm going to see what you have changed :)
04:29 otherwiseguy joined #puppet-openstack
04:30 mgagne xingchao: haven't posted yet
04:30 mgagne xingchao: I can publish on an other repo
04:30 mgagne xingchao: it's a work in progress
04:31 xingchao mgagne: ok, I would like to review it when you post this.
04:31 mgagne xingchao: sure
04:32 mgagne xingchao: here is what I did so far: https://github.com/mgagne/puppet-swift/tree/bp/swift-proxy-config-ini-settings
04:33 mgagne xingchao: so the question is: is there a way to support both methods or not? =)
04:33 mgagne xingchao: and there is also a big challenge: account, object and container share a lot of common filters, even with proxy.
04:34 michchap mgagne: what about a second parameter for all classes allowing a list of custom settings, that is undef by default?
04:35 mgagne michchap: the idea is that you can do it yourself in your private manifest without the help of puppet-swift
04:35 xingchao mgagne: I will view it after lunch
04:35 michchap mgagne: ah fair enough
04:36 mgagne xingchao: it's 11:35pm here and still sunday :)
04:36 mgagne michchap: but it would break all setups using the actual config system (concat fragments)
04:37 mgagne michchap: I expect those people to use templates or plain files to inject their configs which is a lot different than puppet native types.
04:37 michchap mgagne: Yep that is true. We would need a deprecation period.
04:37 mgagne michchap: "just update that plain text file" vs "write a bunch of puppet manifests"
04:38 michchap mgagne: Is that common? It doesn't sound like a very effective way to use puppet. Manifests are needed at some level in any case.
04:41 mgagne michchap: sure. I'm however not alone having this situation at my job where there is less knowledgeable people about puppet. Being able to say "just update this config file and save it" is awesome. They don't have to care about puppet syntax or having to purge all configs with ensure => absent.
04:42 michchap mgagne: I understand :)
04:43 mgagne michchap: so I'm not sure anymore it's a good idea to move it a native type (for swift)
04:48 michchap mgagne: Could you make the same argument about all the native types for the other projects? What makes swift different?
04:49 mgagne michchap: the plethora of filters reusable by account, object, container, proxy which each have their own native type, making it a challenge to write reusable code/manifest.
04:50 mgagne michchap: "making it right" would require a lot of rewrite/refactoring (nothing impossible of course)
05:08 mjblack joined #puppet-openstack
05:37 rongze joined #puppet-openstack
05:53 marun joined #puppet-openstack
06:39 rongze joined #puppet-openstack
06:45 hbroyon joined #puppet-openstack
06:46 hbroyon Hello, someone could review and/or approve /link https://review.openstack.org/#/c/56770/ please ?
06:59 rongze joined #puppet-openstack
07:11 blkperl hbroyon: left you some comments, looks like it was merged thouhg
07:16 hbroyon We changed it to correspond better to the requirements
08:06 hbroyon joined #puppet-openstack
08:20 EmilienM good morning here
08:22 hbroyon Hello
08:23 Jean-Roger Hi
08:35 marun joined #puppet-openstack
08:48 hbroyon Parcontre I really do not understand how the code with a verified false can be merged? Can you explain me please ? :)
08:51 hbroyon But**.
09:04 mmagr joined #puppet-openstack
09:07 bauzas joined #puppet-openstack
09:13 EmilienM fc__: o/
09:13 EmilienM fc__: do you have time today to review backports ?
09:13 dachary hbroyon: it has to be rebased
09:13 dachary because another merge happened this week-end that conflict with yours
09:14 fc__ EmilienM: maybe
09:15 Jean-Roger Bad luck hbroyon :/
09:16 LAnthony joined #puppet-openstack
09:16 hbroyon I know that
09:16 hbroyon always
09:16 dachary hbroyon: you just need to rebase, nothing complex. And since the first use case was accepted, the others will be a little more than a formality, I think.
09:16 dachary hbroyon: William had good input and I updated the use case accordingly.
09:17 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-neutron: Add metering agent support  https://review.openstack.org/60656
09:19 EmilienM michchap: ping
09:20 hbroyon dachary : ok thanks :)
09:25 derekh joined #puppet-openstack
09:31 anaboikina joined #puppet-openstack
10:15 EmilienM fc__: i wonder why nova::compute has $virtio_nic with false by default.
10:15 EmilienM fc__: any idea ?
10:15 fc__ none
10:16 EmilienM i thought virtio was a good choice
10:30 SabrinaVernalde joined #puppet-openstack
10:31 SabrinaVernalde Hello
10:31 dachary SabrinaVernalde: hi
10:32 hbroyon Re.
10:33 dachary hbroyon: could you copy/paste the new version of https://wiki.openstack.org/wiki/Puppet-openstack/ceph-blueprint#I_want_to_try_this_module.2C_heard_of_ceph.2C_want_to_see_it_in_action ?
10:34 dachary hbroyon: SabrinaVernalde : I know ahmad has permission but I don't see him around
10:35 BastienReaux joined #puppet-openstack
10:35 dachary can we do this without him ?
10:35 hbroyon he arrives
10:35 dachary or do we have to wait /
10:35 dachary ?
10:35 dachary ok cool
10:35 Jean-Roger He'll be here soon
10:35 dachary the plan is simple : copy/paste, rebase ( you've done that a few times this should be no trouble ) and git review
10:36 dachary well, it's not just copy/paste as you need to reformat the text in markdown and check how it looks on https://gist.github.com/ but that should not be too long
10:36 hbroyon He can't do it nox
10:36 hbroyon w
10:36 amazyad joined #puppet-openstack
10:37 dachary why ?
10:37 amazyad we can't do it right now, gerrit blocked by proxy in univ
10:38 Jean-Roger I believe you can use FrozenWay
10:38 BastienReaux Is this working on linux?
10:38 Jean-Roger Yes
10:39 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-swift: Add include_service_catalog support  https://review.openstack.org/60521
10:39 openstackgerrit Loic Dachary proposed a change to stackforge/puppet-ceph: ceph::mon create or destroy monitors  https://review.openstack.org/56841
10:40 amazyad either way, i don't have my pc, Jean
10:40 hbroyon We can't modifie now because
10:40 hbroyon that
10:40 Jean-Roger One does not simply go out without his computer
10:40 dachary ok
10:40 BastienReaux Lend us your then Jean-Roger
10:41 anaboikina left #puppet-openstack
10:41 dachary then you can prepare the copy paste now
10:41 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-swift: Add include_service_catalog support  https://review.openstack.org/60521
10:41 dachary copy + format to markdown + show how it looks on https://gist.github.com/
10:42 dachary can you do that now BastienReaux  hbroyon amazyad SabrinaVernalde ? I'll check this when you're done.
10:42 amazyad ok
10:42 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-swift: Add include_service_catalog support  https://review.openstack.org/60521
10:43 dachary amazyad: before sending the review make sure you git pull from the repo : you will need to rebase your change.
10:43 dachary because https://github.com/stackforge/puppet-ceph/commit/ce74ba9910b17b05d415b811fdb69faf3ea4c632 was committed this week-end
10:44 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-swift: Add include_service_catalog support  https://review.openstack.org/60521
10:45 amazyad ok
10:46 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-swift: Set include_service_catalog to False  https://review.openstack.org/60521
10:46 dachary xarses: would you have time to review https://review.openstack.org/#/c/57063/ ? or dalgaaf_  maybe ?
10:47 Jean-Roger I can review it if required
10:47 dachary Jean-Roger: that's kind of you, sure :-) I'm also fishing for +2 here.
10:48 dachary xarses: sorry, my mistake ;-)
10:49 * EmilienM loves the "fishing" expression
10:49 dachary :-)
10:49 dachary good morning EmilienM
10:49 EmilienM dachary: o/
10:51 amazyad https://gist.github.com/anonymous/7870449
10:51 Jean-Roger You need to remove useless spaces amazyad
10:51 Jean-Roger Like line 7
10:52 Jean-Roger Line 9, line 14
10:52 EmilienM fc__: i wonder if file_hack.pp is still useful in puppet-nova
10:52 EmilienM it was a hack for Essex as far i know
10:54 * dachary reading
10:54 dachary amazyad: if you want to catch my attention, just include my nick in the line and it will trigger a notification on my screen ;-)
10:55 dachary https://gist.github.com/anonymous/7870449 shows .txt
10:55 dachary if you name it .md
10:55 dachary it will show how it looks when formated
10:55 dachary amazyad: ^
10:55 BastienReaux ho ok
10:55 hbroyon oh okay
10:56 fc__ EmilienM: check if it's used somewhere but should be safe to remove
10:56 fc__ (I guess)
10:56 dachary https://gist.github.com/dachary/3617138b3b8f9c7b0c55 < for instance
10:56 EmilienM yeah
10:56 EmilienM i gonna remove it
10:56 fc__ EmilienM: if it's not used in our manifests it's harmless though
10:57 EmilienM fc__: I also clean-up nova manifests today
10:57 amazyad dachary https://gist.github.com/amazyad/7870492
10:57 EmilienM i don't like to see comments like "Delete later"
10:58 amazyad dachary https://gist.github.com/amazyad/7870520
10:58 dachary this is better
10:58 dachary amazyad: you lost the boldface though
10:59 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: Clean-up files use  https://review.openstack.org/60792
10:59 EmilienM fc__: ^
10:59 dachary amazyad: see how ceph::mon is bold in https://wiki.openstack.org/wiki/Puppet-openstack/ceph-blueprint#I_want_to_try_this_module.2C_heard_of_ceph.2C_want_to_see_it_in_action ?
11:00 amazyad yes i see
11:00 Jean-Roger Showed him
11:00 dachary Jean-Roger: kind of you :-)
11:00 amazyad hehe
11:06 amazyad dachary https://gist.github.com/amazyad/7870628
11:06 dachary amazyad: perfect
11:07 amazyad thank you
11:08 dachary hbroyon: SabrinaVernalde BastienReaux amazyad when you're back on the computer with the repository, you must a) pull --rebase, b) resolve the conflict, c) copy paste https://gist.github.com/amazyad/7870628 in USECASES.md, d) git review, e) ping me if I don't act soon enough
11:09 dachary resolving the conflict is something you've not done yet and that will give you an opportunity to learn ;-)
11:09 * dachary trying to see the positive aspect of things
11:10 Jean-Roger They were asking for new experience
11:10 amazyad no problem
11:11 dachary it may have been more fair to you if I had not approved the other patch before yours. Because you've done a *lot* of rebase since the beginning already ;-)
11:11 dachary thanks for your patience
11:11 Jean-Roger I think it is fine this way
11:11 amazyad yes, and we helped them all to make their patches.
11:12 amazyad but it's okey
11:15 dachary :-)
11:20 hbroyon bye bye
11:20 hbroyon left #puppet-openstack
11:20 BastienReaux left #puppet-openstack
11:26 rongze joined #puppet-openstack
11:44 hbroyon joined #puppet-openstack
12:08 EmilienM fc__: puppet-nova bug: https://bugs.launchpad.net/puppet-nova/+bug/1259137
12:09 EmilienM not a big deal, but really need to be cleaned
12:12 EmilienM wow, another bug with vif driver
12:12 EmilienM it's clean-up day \o/
12:32 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: Fix wrong vif driver by default  https://review.openstack.org/60799
12:40 rongze joined #puppet-openstack
13:02 openstackgerrit A change was merged to stackforge/puppet-keystone: Create memcache_servers option  https://review.openstack.org/60472
13:17 openstackgerrit François Charlier proposed a change to stackforge/puppet-keystone: Updated endpoints to support paths and ssl  https://review.openstack.org/55303
13:21 fc___ joined #puppet-openstack
13:21 openstackgerrit A change was merged to stackforge/puppet-heat: Removes Unneeded mysql::server Dependency  https://review.openstack.org/60036
13:23 openstackgerrit François Charlier proposed a change to stackforge/puppet-keystone: Updated endpoints to support paths and ssl  https://review.openstack.org/55303
13:23 mjblack joined #puppet-openstack
13:31 openstackgerrit Matthew Mosesohn proposed a change to stackforge/puppet-cinder: Add more granular logging support  https://review.openstack.org/59338
13:40 mattymo hey EmilienM want to look at my changes ^?
13:41 EmilienM mattymo: sure, i add it on my todo :)
13:41 rongze joined #puppet-openstack
13:42 mattymo EmilienM, we need to improve spec tests for sure. Feel free to -1 me but I want to make sure logging.pp proposal is feasible
13:43 EmilienM mattymo: ok, i will
13:43 EmilienM thank you
13:44 EmilienM mattymo: but you still use a template, not sure it will pass
13:45 mattymo the template location is definable, not even enabled by default, and to make a provider for a python logging file would be overkill
13:45 EmilienM ok
13:45 mattymo mgagne said a user could provide his own template from his myopenstack class to override
13:45 EmilienM mattymo: i'll review it asap
13:46 mattymo no asap needed) just looking for some ideas of where else to improve
13:46 mattymo then I can move on to limits, and then nova virt tweaks. There's a lot of tweaks needed for nova compute when running vm in a vm
13:48 openstackgerrit François Charlier proposed a change to stackforge/puppet-keystone: Updated endpoints to support paths and ssl  https://review.openstack.org/55303
13:59 dprince joined #puppet-openstack
14:07 rharrison joined #puppet-openstack
14:08 markvoelker joined #puppet-openstack
14:14 michchap EmilienM: pong
14:15 EmilienM w00t
14:15 EmilienM michchap: it's quite late afik for you
14:15 michchap EmilienM: its 1am
14:15 EmilienM or maybe it's your ghost ?
14:16 EmilienM michchap: i was asking for help
14:16 EmilienM michchap: i need reviews
14:16 michchap you have so many patches!
14:16 EmilienM sorry for that
14:17 michchap it's good :)
14:17 mattymo michchap, where are you located?
14:17 michchap mattymo: Sydney
14:17 mattymo nice
14:18 michchap it's very nice but the time difference is pretty bad
14:18 mattymo I suffer enough moving from the US to Russia
14:22 michchap so the US West coast starts to come online in your evening?
14:23 mattymo yeah
14:23 mattymo at Mirantis our management is in mountain view, 12 hours behind us
14:23 mattymo so if they want to meet with us they have to wake up early and do conf calls from home
14:27 EmilienM michchap: reviews are not urgent, it could wait tomorrow of course
14:30 michchap EmilienM: yeah we're looking at doing a release kinda soon so I might not do many this week.
14:32 EmilienM michchap: oh i see
14:32 EmilienM no worries at all
14:32 EmilienM do what you can of course
14:32 EmilienM i just try to shake puppet folks here
14:32 EmilienM i like doing patches, but i also like reviews from peers :)
14:35 michchap mattymo: Are you guys at mirantis interested in the scenario_node_terminus and puppet_openstack_builder stuff?
14:35 mattymo michchap, what is that?
14:35 EmilienM it's awesome ^
14:36 michchap mattymo: It's an attempt by bodepd to replace the top level manifests in site.pp and in puppet-openstack with a yaml based data model
14:36 michchap so you can say things like..."These are the classes for a control node, these for a compute node, an all-in-one is compute+control" and it actually works.
14:37 michchap and you can swap databases for the whole project between pgsql/mysql with one line of yaml
14:38 michchap I'm writing some detailed tutorials on using it at the moment, but it's slow.
14:43 mattymo michchap, oh we have a huge yaml in fuel also
14:43 mattymo except not classes.. we have a top level class with roles
14:43 mattymo based on roles defined in yaml, we do certain classes
14:48 michchap mattymo: I should look at that in more detail - it would be nice if we had a common system for composing roles
14:49 mattymo our code is really a mess because it combines ruby and python
14:50 michchap which bits are using python?
14:55 mattymo https://github.com/stackforge/fuel-web/tree/master/nailgun/nailgun/fixtures
14:55 mattymo this specifies values
14:57 otherwiseguy joined #puppet-openstack
14:57 EmilienM fc__: fyi, with oslo.messaging stuff, we will have a lot of refactorization to do on puppet modules :)
14:57 mattymo and I believe we generate and upload yaml to node here https://github.com/stackforge/fuel-astute/blob/master/lib/astute/deployment_engine.rb
15:01 otherwiseguy joined #puppet-openstack
15:03 rongze joined #puppet-openstack
15:03 jkyle joined #puppet-openstack
15:05 fvollero EmilienM: https://review.openstack.org/#/c/60449/2 but I didn't already do those changes? My memory is kinda broken
15:06 EmilienM fvollero: it's a backport amigo
15:07 fvollero EmilienM: Oooh, merci :)
15:07 EmilienM fvollero: :)
15:07 EmilienM fvollero: this WE, i did a bunch of backports
15:07 decede joined #puppet-openstack
15:07 comptona joined #puppet-openstack
15:07 EmilienM fvollero: https://review.openstack.org/#/dashboard/3153 if you want to have a look
15:09 EmilienM fc__: do I dream or we need to add Keystone user manually when using swift backend in Glance ?
15:09 tavasti joined #puppet-openstack
15:09 fc__ EmilienM: don't know
15:10 fvollero EmilienM: o_O what I should see ? it's an humongous list of changes :)
15:10 EmilienM fvollero: yeah, busy december
15:11 fvollero EmilienM: look mine https://review.openstack.org/#/dashboard/8083
15:11 EmilienM fc__: it's the case. I think it deserves do a patch, isn't it ?
15:11 fvollero EmilienM: :)
15:11 EmilienM fvollero: seems like you are ready for christmas holidays
15:11 EmilienM ahah
15:12 fc__ EmilienM: does using swift backend in glance require a user ?
15:12 EmilienM fc__: I guess Sir.
15:12 EmilienM fc__: glance uses a specific user to get/put images, which is a keystone user
15:13 EmilienM fc__: it's good to specify an user, but how could it work if we don't actually create the user :)
15:14 EmilienM fc__: https://bugs.launchpad.net/puppet-glance/+bug/1259205
15:15 fc__ EmilienM: the ability to store images in each user tenant is absent too
15:15 hbroyon joined #puppet-openstack
15:17 nightfly joined #puppet-openstack
15:20 EmilienM fc__: i did not know this feature
15:20 EmilienM could you extend or point me some doc?
15:22 fc__ EmilienM: https://github.com/openstack/glance/blob/master/etc/glance-api.conf#L285-L358
15:23 fc__ in particular swift_store_milti_tenant
15:23 EmilienM fc__: thanks
15:23 EmilienM i'll do it asap
15:24 EmilienM fc__: a lot of work, when do we sleep by the way ?
15:25 EmilienM maybe fvollero has some time to make it :P
15:25 fc__ Wen do *WE* sleep ? Didn't we agree tha I would sleep, you would work ?
15:26 EmilienM \o/
15:29 openstackgerrit Matthew Mosesohn proposed a change to stackforge/puppet-cinder: Add more granular logging support  https://review.openstack.org/59338
15:29 mattymo ^ - fixed spec tests (but not happy about it)
15:30 EmilienM mattymo: happiness is important
15:30 rongze joined #puppet-openstack
15:34 mattymo EmilienM, ini_setting provider allows ensurable resources for each config file entry
15:35 mattymo and you can do cinder_config { '[DEFAULT]value': ensure => absent } to make sure this value isn't in the config
15:35 fvollero EmilienM: hahahahahahaahahahhaahhaa
15:35 mattymo but spec tests fail to identify this as should_not_contain
15:35 fvollero EmilienM: You hate me so much? :) I'm going in PTO until January 2014 mon ami
15:37 EmilienM fvollero: PTO ?
15:38 fvollero EmilienM: Yep, Paid Time Off :)
15:38 fvollero EmilienM: a.k.a. vacations \o/
15:38 EmilienM fvollero: i was joking
15:38 EmilienM fvollero: i took all the christmas week off
15:38 fvollero EmilienM: I knew it, but was just to make the discussion more serious :)
15:39 fvollero EmilienM: Wow... you're a slacker my friend
15:40 EmilienM fvollero: really? I'll continue to put -1 to you while my vacations :P
15:41 mjblack joined #puppet-openstack
15:43 mjblack joined #puppet-openstack
15:48 tnoor joined #puppet-openstack
15:50 tnoor2 joined #puppet-openstack
15:52 dmsimard joined #puppet-openstack
16:01 fvollero EmilienM: that's blackmailing :) But, you've a point :)
16:01 EmilienM fvollero: :P
16:08 mgagne joined #puppet-openstack
16:13 openstackgerrit A change was merged to stackforge/puppet-ceilometer: changed rabbit_virtualhost(wrong) to rabbit_virtual_host  https://review.openstack.org/60447
16:13 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Alarm: record history support  https://review.openstack.org/60448
16:14 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Updates to params.pp for mongo and postgres packages  https://review.openstack.org/60449
16:16 openstackgerrit A change was merged to stackforge/puppet-neutron: Add missing vpnaas class in example manifest  https://review.openstack.org/60655
16:27 EmilienM mgagne: fc__ : thx for reviews on backports, i appreciate
16:38 badiane_ka joined #puppet-openstack
16:43 hbroyon joined #puppet-openstack
16:50 openstackgerrit ahmad mazyad proposed a change to stackforge/puppet-ceph: Adds use case to USECASES.md  https://review.openstack.org/56770
16:51 LAnthony joined #puppet-openstack
16:53 EmilienM fc__: could you update https://blueprints.launchpad.net/puppet-keystone/+spec/serve-keystone-from-wsgi please ?
16:53 hbroyon Hello, someone could review please https://review.openstack.org/#/c/56770/ ?
16:57 sputnik13net joined #puppet-openstack
16:59 EmilienM fc__: mgagne: what do you think about this one ? > https://blueprints.launchpad.net/puppet-keystone/+spec/create-ressources
17:00 fc__ EmilienM: it is already possible
17:00 mgagne EmilienM: yep...
17:00 EmilienM oops
17:00 amazyad joined #puppet-openstack
17:00 mgagne EmilienM: using it to add our system users
17:00 EmilienM mgagne: could you delete the blueprint ? i can't
17:00 EmilienM please
17:00 mgagne EmilienM: checking
17:01 EmilienM fc__: mgagne : so i just have to use keystone::roles::admin class by replacing params ?
17:01 mgagne EmilienM: see nova::keystone::auth
17:01 mgagne EmilienM: for an example
17:01 EmilienM ok thanks
17:02 EmilienM indeeed
17:02 EmilienM mgagne: my bad :(
17:02 hbroyon Can you explain me what it s bad ? :s
17:03 hbroyon :/
17:03 amazyad https://review.openstack.org/#/c/56770/
17:03 hbroyon amazyad : (1 inline comment)
17:04 hbroyon I don't really understant :/
17:04 mgagne hbroyon: you can read inline comments by clicking on "Side-by-Side" next to file with the "1 comment" note.
17:04 amazyad I would prefer spaces to be used just like we use spaces in manifests for consistency.
17:05 mattymo mgagne, I'm on my way out, but I updated my logging for cinder change request
17:06 mgagne mattymo: I'll review as soon as I find the time thanks
17:06 mattymo I should pop online in 1-2 hrs
17:07 Jean-Roger amazyad did you understand what's wrong ?
17:09 mgagne btw guys, I updated all the openstack puppet modules to include the icehouse series.
17:10 amazyad not really
17:10 Jean-Roger You used tabulations
17:10 angdraug joined #puppet-openstack
17:10 Jean-Roger You need to use spaces
17:10 amazyad !!
17:11 mgagne amazyad: tabulations are marked as red arrows in the diff view
17:11 amazyad yes, i saw it, thank you
17:14 openstackgerrit ahmad mazyad proposed a change to stackforge/puppet-ceph: Adds use case to USECASES.md  https://review.openstack.org/56770
17:14 EmilienM mgagne: sorry for spams, did a lof of patches over the last days, i use to add you as reviewer :)
17:14 mgagne EmilienM: all fine. I had time this weekend to work on puppet so =)
17:14 EmilienM yeah i saw
17:14 mgagne EmilienM: mainly swift but still
17:14 EmilienM mgagne: good to know
17:14 Jean-Roger Looks good now amazyad
17:15 EmilienM mgagne: any blueprint/bug ?
17:15 mgagne EmilienM: https://blueprints.launchpad.net/puppet-swift/+spec/swift-proxy-config-ini-settings
17:15 mgagne EmilienM: WIP here: https://github.com/mgagne/puppet-swift/commits/bp/swift-proxy-config-ini-settings
17:15 EmilienM nice
17:15 EmilienM bbl
17:16 mgagne EmilienM: I had questions/concerns about the migration path for people using concat fragments to inject their own configs
17:16 amazyad thank you jean-roger, you are really kind :)
17:17 rongze joined #puppet-openstack
17:17 Jean-Roger Just see what the reviewers think about it
17:18 amazyad Well, you may be able to review it, maybe give us +!
17:18 amazyad Well, you may be able to review it, maybe give us +1
17:18 hbroyon :)
17:19 hbroyon review and approve I hope :D
17:20 comptona joined #puppet-openstack
17:23 Jean-Roger dachary https://review.openstack.org/56770
17:23 Jean-Roger And you guys could also take a look if you have time, would be nice ^^
17:29 xarses joined #puppet-openstack
17:34 fvollero mgagne: ping
17:35 rmoe joined #puppet-openstack
17:36 fvollero mgagne: if i would like to set multiple keystone endpoints for v1 and v2 cinder volume_version, it's possible to do it with the actual version of puppet-cinder ? https://github.com/stackforge/puppet-cinder/blob/master/manifests/keystone/auth.pp
17:43 amazyad left #puppet-openstack
17:48 bauzas joined #puppet-openstack
17:50 rmoe_ joined #puppet-openstack
17:53 rongze joined #puppet-openstack
17:55 mgagne fvollero: yes and no. You can't use puppet-cinder to do it. puppet-cinder won't however prevent/block you from doing it in your own private manifest
17:56 fvollero mgagne: ok, since i'm gonna do it in packstack (so private manifests) i'll have to define a function or you've a more 'clean' way ?
17:58 mgagne fvollero: the current design does not allow to manage multiple versions of the endpoints. It would require a refactor.
17:58 fvollero mgagne: ok, so puppet-cinder need a patch :)
17:58 fvollero mgagne: we can work together on the 'how to refactor' ?
17:59 mgagne fvollero: cinder::keystone::auth suffers from its design. it's managing too much stuff in the same manifest
17:59 mgagne fvollero: it's managing user/role, service and endpoint in the same manifest
18:00 sputnik13net joined #puppet-openstack
18:05 dtalton joined #puppet-openstack
18:06 dtalton has anyone seen any class inheritance issues with puppet_openstack_builder
18:07 dtalton it looks like my includes aren't getting included
18:08 thumpba joined #puppet-openstack
18:13 brdude joined #puppet-openstack
18:19 dtalton2 joined #puppet-openstack
18:40 mattymo|home joined #puppet-openstack
18:40 mattymo|home mgagne, hi
18:43 dtalton3 joined #puppet-openstack
18:44 prad joined #puppet-openstack
18:48 hbroyon joined #puppet-openstack
18:50 hbroyon joined #puppet-openstack
19:12 mmagr joined #puppet-openstack
19:16 * dachary looking at https://review.openstack.org/#/c/56770/
19:18 openstackgerrit A change was merged to stackforge/puppet-ceph: Adds use case to USECASES.md  https://review.openstack.org/56770
19:33 LAnthony *applause*
19:34 mgagne mattymo|home: hi, sorry I don't have much time today ^^'
19:37 mgagne fc__: can you take a look at the other backports EmilienM did? I +2 most of them.
19:39 openstackgerrit A change was merged to stackforge/puppet-cinder: Add Glance support  https://review.openstack.org/60457
19:40 openstackgerrit A change was merged to stackforge/puppet-cinder: Add Swift as backup backend support  https://review.openstack.org/60459
19:41 bauzas joined #puppet-openstack
19:42 fc__ mgagne: doing it right now !
19:42 openstackgerrit A change was merged to stackforge/puppet-cinder: Improve Cinder Backup support  https://review.openstack.org/60657
19:42 openstackgerrit A change was merged to stackforge/puppet-cinder: Fix default Swift URL in Backup service  https://review.openstack.org/60658
19:42 openstackgerrit A change was merged to stackforge/puppet-cinder: Add Ceph as backup backend support  https://review.openstack.org/60659
19:43 mattymo|home mgagne, it's ok. If you find time tomorrow it would be great
19:43 hbroyon OMG Thanks !!! =D
19:44 openstackgerrit A change was merged to stackforge/puppet-glance: Add Cinder backend to image store  https://review.openstack.org/60451
19:44 fc__ mgagne: could you eventually have a look at this one, at least to get your opinion if I need to fix something https://review.openstack.org/#/c/55303/ ?
19:46 fc__ good night all !
19:46 LAnthony Have sweet dreams ;)
19:56 dtalton2 joined #puppet-openstack
20:04 rongze joined #puppet-openstack
20:06 dtalton2 joined #puppet-openstack
20:16 LAnthony left #puppet-openstack
20:23 dtalton3 joined #puppet-openstack
20:30 sputnik13net joined #puppet-openstack
20:45 dprince joined #puppet-openstack
20:46 dtalton joined #puppet-openstack
20:51 xarses joined #puppet-openstack
20:55 marun joined #puppet-openstack
21:01 sputnik13net joined #puppet-openstack
21:09 otherwiseguy joined #puppet-openstack
21:29 EmilienM mgagne: i think https://bugs.launchpad.net/puppet-glance/+bug/1259205 is invalid
21:29 EmilienM because we could use the service user already created by Glance
21:29 EmilienM what do you think ?
21:29 mgagne EmilienM: what's the best practice?
21:30 EmilienM really, we use service account
21:30 EmilienM so...
21:30 mgagne EmilienM: still: what's the best practice?
21:31 EmilienM mgagne: i'm not sure about that one. I check with Glance guys
21:36 EmilienM mgagne: no worries for https://review.openstack.org/#/c/60799/1, it's a WIP
21:36 EmilienM but your -2 makes sense, I love your -2 :)
21:37 EmilienM mgagne: for the glance stuff, a dev says me that it makes sense to use service account
21:37 mgagne EmilienM: great
21:37 openstackgerrit joined #puppet-openstack
21:51 EmilienM mgagne: another one to backport: https://review.openstack.org/#/c/60656/
21:52 EmilienM mgagne: what do we wait to merge https://review.openstack.org/#/c/60137/ ?
21:52 mgagne EmilienM: what are you waiting for? =)
21:52 EmilienM mgagne: can I merge it ?
21:52 mgagne EmilienM: go
21:53 openstackgerrit A change was merged to stackforge/puppet-ironic: Add Modulefile  https://review.openstack.org/60137
21:53 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-ironic: Delete travis QA service  https://review.openstack.org/60204
21:54 EmilienM mgagne: and another one regarding swift optimization > https://review.openstack.org/#/c/60521/
21:55 mgagne EmilienM: I don't have much time to deal with puppet stuff atm, sorry
21:56 EmilienM mgagne: no worries, good luck :)
21:57 openstackgerrit joined #puppet-openstack
21:58 openstackgerrit A change was merged to stackforge/puppet-ironic: Delete travis QA service  https://review.openstack.org/60204
22:18 rongze joined #puppet-openstack
22:24 xarses joined #puppet-openstack
22:40 dtalton2 joined #puppet-openstack
22:41 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-nova: Fix wrong vif driver by default  https://review.openstack.org/60799
22:44 EmilienM i need reviews on this one ^
22:58 dtalton3 joined #puppet-openstack

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