Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2013-11-14

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

All times shown according to UTC.

Time Nick Message
00:13 rmoe joined #puppet-openstack
00:22 openstackgerrit A change was merged to stackforge/puppet-cinder: [bp] Add support for Cinder multi-backend  https://review.openstack.org/54591
00:42 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: Add Glance support  https://review.openstack.org/56302
01:11 xarses joined #puppet-openstack
01:18 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: Add Swift support  https://review.openstack.org/56305
01:19 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: Add Glance support  https://review.openstack.org/56302
01:21 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: Add Swift as backup backend support  https://review.openstack.org/56305
01:42 xingchao joined #puppet-openstack
01:49 rongze joined #puppet-openstack
02:34 Jean-Roger joined #puppet-openstack
02:38 odyi joined #puppet-openstack
02:40 xingchao joined #puppet-openstack
02:45 xingchao_ joined #puppet-openstack
03:12 rmoe joined #puppet-openstack
03:19 michchap joined #puppet-openstack
03:33 rmoe joined #puppet-openstack
03:50 otherwiseguy joined #puppet-openstack
03:53 michchap_ joined #puppet-openstack
04:02 michchap joined #puppet-openstack
04:09 e1mer joined #puppet-openstack
04:14 e1mer joined #puppet-openstack
04:27 tnoor joined #puppet-openstack
04:29 tnoor1 joined #puppet-openstack
04:40 marun joined #puppet-openstack
04:49 raies joined #puppet-openstack
04:49 raies hi
04:49 raies I am trying to install openstack using puppet
04:50 raies I followed link - www.cybera.ca/news-and-events/tech-radar/lets-build-cloud-using-puppet-install-openstack
04:50 raies But during apply I am getting an error "Could not autoload keystone_config: Could not autoload /etc/puppet/modules/keystone/lib/puppet/provider/keystone_config/ini_setting.rb: undefined method `provider' for nil:NilClass at /etc/puppet/modules/keystone/manifests/init.pp:78"
04:50 raies Can someone please help me on this
04:50 raies ^^
05:04 raies #openstack-qa
05:11 rongze_ joined #puppet-openstack
05:34 rongze joined #puppet-openstack
07:03 rharrison joined #puppet-openstack
07:48 mmagr joined #puppet-openstack
07:58 Krast joined #puppet-openstack
08:12 EmilienM michchap: good morning sir
08:12 michchap EmilienM: morning :)
08:12 EmilienM michchap: actually afternoon, sorry
08:13 michchap did loic ask you for the cinder patches?
08:13 EmilienM michchap: no, why ?
08:13 michchap ah...he posted an issue against the stuff bodepd was working on that needed exactly the patches you made
08:13 EmilienM i just say it missed and could not resist to do it
08:13 EmilienM any URL ? mail ?
08:15 EmilienM michchap: the patch for Glance support in Cinder ?
08:15 michchap yep
08:15 EmilienM michchap: where did he post it ?
08:16 EmilienM there is a lot of patches to review by the way
08:17 michchap yeah I know :)
08:17 EmilienM michchap: i wonder why dachary would need this patch
08:17 michchap https://github.com/CiscoSystems/openstack-installer/issues/230
08:17 EmilienM oh
08:17 * EmilienM reads
08:18 michchap he is mostly doing ceph stuff...the patch is needed to back cinder with rbd
08:18 michchap I'm really surprised it wasn't in already - that feature was introduced in folsom
08:18 EmilienM michchap: oh ok
08:18 EmilienM michchap: but i also need to do the same stuff like https://review.openstack.org/#/c/56305/ but for Ceph, (i did it for Swift) and i'm going to do the same for ceph as soon i have time
08:19 EmilienM michchap: cool, let's review it and merge it asap then
08:19 michchap I already gave a +1, I'll change it to a +2 if someone else +1's it
08:19 EmilienM michchap: ok
08:20 michchap also I think I shouldn't have moerged that multi-backend patch
08:20 michchap I didn't realise it was duplicating all the backend code
08:21 EmilienM oh i see
08:21 EmilienM we should revert it and work on current code to add multibackend support
08:21 EmilienM i don't like dupplication
08:22 EmilienM michchap: what do you think ?
08:22 michchap https://github.com/stackforge/puppet-cinder/blob/master/manifests/volume/rbd.pp
08:22 michchap https://github.com/stackforge/puppet-cinder/blob/master/manifests/backend/rbd.pp
08:22 michchap it should just be a flag, I don't know what I was thinking when I +2'ed it
08:23 michchap I'll submit another patch that removes the backend ones and just adds a flag and updates the readme, I think
08:23 EmilienM \o/ things happen no worries :)
08:23 EmilienM michchap: that would be awesome
08:23 michchap bbl
08:23 EmilienM michchap: i can help you with that if you need
08:26 rongze joined #puppet-openstack
08:35 dalgaaf_ joined #puppet-openstack
08:44 michchap joined #puppet-openstack
08:44 sfu joined #puppet-openstack
08:45 mmagr joined #puppet-openstack
08:45 dalgaaf_ joined #puppet-openstack
08:45 para__ joined #puppet-openstack
08:46 para__ joined #puppet-openstack
08:46 Krast joined #puppet-openstack
09:01 dachary EmilienM: I guess you figured out why the glance patch is needed ;-)
09:01 dvanders joined #puppet-openstack
09:02 dachary funny how things happen at exactly the same time
09:19 derekh joined #puppet-openstack
09:19 EmilienM dachary: i did it randomly yesterday
09:20 EmilienM dachary: funny
09:20 dachary :-)
09:20 EmilienM dachary: let me add RBD cinder backup support
09:20 EmilienM asap
09:20 dachary I'm not there yet, but it would be great indeed.
09:21 dachary EmilienM: do you know how to tell bundle to re-install a dependency that has been removed ?
09:21 dachary r$ bundle show puppetlabs_spec_helper
09:21 dachary The gem puppetlabs_spec_helper has been deleted. It was installed at:
09:21 dachary /var/lib/gems/1.9.1/gems/puppetlabs_spec_helper-0.4.1
09:28 dachary nevermind, just using --path and getting rid of that
09:33 rongze joined #puppet-openstack
09:37 EmilienM dachary: sorry no
09:38 rongze joined #puppet-openstack
09:41 dachary cinder/cinder/common/config.py this is actually evaluated to replace with the values from the corresponding options :'$glance_host:$glance_port' ? EmilienM <
09:44 rongze joined #puppet-openstack
09:52 dachary EmilienM: it just occurs to me. Whey cinder::glance when someone could just as easily write  cinder_config { ... } ? Is there a value to a wrapper that merely translate the arguments and transparently pass them to cinder_config ?
09:54 michchap dachary: think of how the data model works for openstack-installer - everything must be wrapped
09:54 dachary michchap: in the context of openstack-installer, is it possible to do something like cinder::cinder_config::DEFAULT/glance_api_server: foo:9292 ?
09:54 dachary michchap: :-)
09:55 dachary data mapping only works for class ? not for providers ?
09:55 michchap dachary: I see what you're saying...but I don't think so
09:55 michchap dachary: yeah exactly that
09:55 michchap wait till bodepd comes back from holiday and see if you can convince him to write it :D
09:56 dachary :-D It looks like this would allow to get rid of wrappers that have no value otherwise
09:56 michchap dachary: I just moved stack-builder into Pypi as 'scenariobuilder'
09:57 dachary cool :-)
10:53 dachary joined #puppet-openstack
12:02 michchap joined #puppet-openstack
12:03 morazi joined #puppet-openstack
12:07 rongze joined #puppet-openstack
12:37 michchap joined #puppet-openstack
12:55 morazi joined #puppet-openstack
12:59 dachary joined #puppet-openstack
13:06 EmilienM fvollero: ping
13:17 raies hi anybody
13:18 raies I wanted to install puppet 3.*
13:18 raies for this I followed "http://projects.puppetlabs.com/projects/1/wiki/Puppet_Ubuntu"
13:18 raies but I am getting error "http://pastebin.com/WsumNV76"
13:18 raies can someone help
13:19 EmilienM raies: not sure that's the best place to ask
13:19 EmilienM raies: most of people here talks about openstack modules
13:19 EmilienM raies: maybe you could find more help on #puppet
13:20 raies ok
13:31 rcrit I have one thing I want to do, be able to set the qpid_protocol in glance
13:31 rcrit this requires a change in two submodules in packstack. Do I need a separate bug for each?
13:32 rcrit I filed one already, https://bugs.launchpad.net/packstack/+bug/1251050
13:50 dprince joined #puppet-openstack
14:09 Anju joined #puppet-openstack
14:09 Anju left #puppet-openstack
14:14 otherwiseguy joined #puppet-openstack
14:32 bogdando joined #puppet-openstack
14:48 fvollero EmilienM: pong
15:11 rongze joined #puppet-openstack
15:13 dmsimard joined #puppet-openstack
15:17 dmsimard dachary: I found how to use multiple VMs for rspec-system
15:18 dmsimard dachary: Not too hard, and should allow us to do more tests (e.g, installing cuttlefish, dumpling, emperor..)
15:18 dmsimard dachary: http://rubydoc.info/gems/rspec-system
15:19 dachary dmsimard: amazing : I was doing *exactly* this right now ;-)
15:19 dachary I'm troubled though because
15:20 dmsimard dachary: You can also specify the specs of the vm
15:20 dachary GEM_HOME=~/.gems RS_SET=debian-70rc1-x64 bundle exec rake spec:system
15:20 dachary is just ignored
15:20 dachary dmsimard: yes, and multi nodes
15:21 dachary I was getting ready to work on ceph::mon ;-)
15:21 mjblack joined #puppet-openstack
15:21 dmsimard dachary: For my review ?
15:23 dachary am I behind for your review ?
15:23 dachary I am
15:23 dachary I'll get to it later today. I must run some errands now.
15:23 dachary I browsed the sources and can't figure out why RS_SET is ignored but that's most likely a technicality
15:24 dachary Using rspec-system (2.5.1)
15:24 dachary or an older version...
15:25 mjblack joined #puppet-openstack
15:25 dachary yes, the version is too old
15:26 dachary things move fast
15:26 dachary :-)
15:27 dachary 2.7.2 works better ;-)
15:28 dachary bbl
15:28 dmsimard dachary: We should probably define a set of nodes on ubuntu (with hostnames like mon01, mon02, mon03, osd01, osd02, mds01, etc..) and the same on redhat
15:28 dachary exactly
15:29 dachary this is going to be awesome !
15:29 rongze joined #puppet-openstack
15:40 dmsimard dachary: If you're willing to be the one dedicated to rspec-system tests, i'm all for it. I can do spec tests but it's depressing for me :)
15:49 rcrit I'm having a hard time figuring out how qpid vs rabbitmq is specified in the glance puppet module. On my RHOS Install it is using qpid but I can't figure out why.
15:52 otherwiseguy joined #puppet-openstack
16:25 rongze joined #puppet-openstack
16:27 xarses joined #puppet-openstack
16:34 xarses rcrit: https://github.com/stackforge/puppet-glance/tree/master/manifests/notify
16:35 xarses did you use puppet-openstack?
16:36 rcrit xarses, right, I see the code, just don't know how it is invoked
16:36 rcrit I want to pass in the qpid options from puppet-openstack
16:36 rcrit and add protocol to the qpid option list so I can set it to 'ssl'
16:38 EmilienM fvollero: https://review.openstack.org/56417
16:39 xarses rcrit: thats odd
16:39 rcrit what's odd?
16:40 xarses i don't see anything either, it looks like another module has to call for the glance::notify::[qpid|rabbitmq] class
16:40 fvollero EmilienM: i'll have to nack it
16:41 EmilienM fvollero: my patch is not good ?
16:41 fvollero EmilienM: Yeah, i'm searching for the _packstack_ way to do it, 1 sec :)
16:42 EmilienM sur
16:44 rcrit xarses, ok, so I might do that in openstack::glance then?
16:45 xarses rcrit: aparently not
16:45 rcrit yes, but *can* I do it there?
16:45 rcrit and I guess I'd need a conditional for qpid vs rabbitmq
16:46 xarses rcrit: hmm
16:47 fvollero EmilienM: you saw the comment ?
16:47 xarses rcrit: in glance-api.conf is notifier_strategy commented out?
16:48 xarses rcrit: or set to noop
16:48 EmilienM fvollero: ack
16:49 rcrit xarses, on my installed test box (via Foreman) it is set to qpid
16:49 rcrit but I can't figure out where that is getting set
16:49 openstackgerrit Francesco Vollero proposed a change to stackforge/puppet-ceilometer: Updates to params.pp for mongo and postgres packages  https://review.openstack.org/54797
16:49 xarses rcrit: its noop for me
16:50 EmilienM fvollero: the code s already good
16:50 fvollero EmilienM: Anyway could you give a look at this one ? ^^
16:50 EmilienM fvollero: so i guess i have a wrong version of packstack on my RHEL
16:50 rcrit hmm, maybe it's something specific to RHOS/Foreman that is adding it
16:51 xarses rcrit: from what im reading, you dont have to have notifications in glance
16:51 EmilienM fvollero: i abandon the patch
16:51 rmoe joined #puppet-openstack
16:51 EmilienM fvollero: do you know how packaging is managed ?
16:51 fvollero EmilienM: Uhm, ok, run yum info packstack-puppet-packages
16:52 xarses rcrit: so it doesn't have to be configured for either queue
16:52 rcrit xarses, well it seems that RHOS configures it for qpid, so I want to secure it in that case
16:52 fvollero EmilienM: sorry the package is packstack-modules-puppet.noarch
16:52 rcrit the only place in the astapor code I can find that is in the ceilometer_controller module
16:52 fvollero EmilienM: tell me which version  you hgave
16:52 fvollero s/hgave/have/
16:53 rcrit and I'm not sure why that was even called, if indeed that is where it is getting set
16:53 EmilienM fvollero: 0.11.dev806.el6
16:53 xarses rcrit: well if its calling glance::notify::qpid and you do too, it will end up with duplicated decl
16:54 fvollero EmilienM: it's way old
16:54 rcrit yeah
16:54 xarses rcrit: hmm i could imagine that ceilometer might want that
16:54 fvollero EmilienM: on my fedora 19 i have Release     : 0.28.dev677.fc19
16:54 EmilienM fvollero: i have RHEL 6.4
16:54 EmilienM omg
16:54 EmilienM ok
16:54 fvollero EmilienM: give 1 sec and i'll check
16:54 rcrit I'm not sure how to figure out what puppet module makes any given change to a file
16:55 rcrit I guess I could poke at what I did file and change it to some other value and see if things change...
16:55 xarses grep -ri notifier_strategy *
16:55 xarses rcrit: in the modules folder
16:56 xarses or grep -ri -e 'glance::notify::' *
16:56 EmilienM fvollero: i use http://rdo.fedorapeople.org/openstack-havana/rdo-release-havana.rpm
16:56 xarses rcrit: one of the two must answer
16:56 rcrit just in glance, and one call in quickstack/manifests/ceilometer_controller.pp
16:57 rcrit I'll have to double-check the hostgroup config in foreman but I'm not explicitly calling this controller type
16:57 dachary back
16:57 xarses rcrit: sounds like quickstack then, where is that from?
16:58 rcrit that is a configuration for Foreman
16:58 xarses rcrit: is it calling glance::notify::qpid then?
16:59 rcrit no, it sets that value directly
16:59 xarses rcrit: ahh, so its naughty
16:59 rcrit but I'm not clear that it is being executed
16:59 rcrit the classes I'm using shouldn't be including that, I guess unless it is getting pulled in somewhere non-obvious
17:00 rcrit aha, and so it is. Ok, I've tracked down what they are doing, and I guess it is wrong :-(
17:00 fvollero michchap: ping... it's ok now for you https://review.openstack.org/#/c/54797/ ?
17:01 rcrit xarses, ok, so the right thing to do hwere thenw ould be to call glance::notify::qpid directly then?
17:01 xarses rcrit: well the proper thing would be to call glance::notify::qpid and pass the options you need
17:01 rcrit class { 'glance::notify::qpid': args ... } ?
17:01 xarses rcrit: correct, if the glance module is a requirement, than that should be fine
17:01 rcrit ok, thanks for your time. This, in a way, may actually make my life easier :-)
17:02 rcrit oh, and the tests in the glance module were very helpful too, so kudos on that
17:03 fvollero EmilienM: the last stable package in RDO is openstack-puppet-modules-2013.2-3.el6.noarch.rpm could you check that you have the same ?
17:05 dachary dmsimard: I'm adding basic support for debian-70rc1 integration test in puppet-ceph.
17:05 dmsimard rc1 ?
17:05 dmsimard debian 7 has been out for a while :)
17:06 dmsimard dachary: but sure, we can test ubuntu, debian and rhel
17:06 dmsimard dachary: Should give us a good enough coverage for everything otu there
17:07 dachary I'm just following the example that works. And postpone being smarter about it for later.
17:07 EmilienM fvollero: 3.el6
17:07 EmilienM 2013.2.3.el6
17:08 EmilienM fvollero: but the problem is not in modules
17:08 EmilienM but in packstack itself
17:08 EmilienM afik
17:08 EmilienM fvollero: let's switch on #rdo
17:09 dachary even that is non trivial : it fails.
17:10 * dachary has an incentive to find the release instead of the rc1 ;-)
17:11 dachary http://puppet-vagrant-boxes.puppetlabs.com/ does not seem to have anything more recent
17:13 dachary hum that was a trivial error : never run integration tests while writing a file under emacs as it will fail to copy the #backup file :-)
17:17 fvollero EmilienM: ok
17:21 rmoe joined #puppet-openstack
17:21 openstackgerrit Loic Dachary proposed a change to stackforge/puppet-ceph: Add support for Debian wheezy integration tests  https://review.openstack.org/56427
17:22 angdraug joined #puppet-openstack
17:27 dachary dmsimard: https://review.openstack.org/#/c/56270/4/manifests/mon.pp,unified : is $id evaluated somehow ?
17:27 dachary if it is, where does it come from ?
17:28 dachary if it is not, it does not seem to be a particularly useful default
17:29 dachary what happens if https://review.openstack.org/#/c/56270/4/manifests/mon.pp,unified $mon_data is undef ? Is the line just ignored and not written to the config file ?
17:41 xarses dachary: if any setting passed to the ini_file provider is undef, no line will be written into the config
17:46 rongze joined #puppet-openstack
17:52 badiane_ka joined #puppet-openstack
17:53 tnoor joined #puppet-openstack
17:56 tnoor1 joined #puppet-openstack
17:57 hogepodge joined #puppet-openstack
17:59 dmsimard dachary: $id is expanded dynamically by ceph
17:59 dmsimard it's a metavariable
18:15 thumpba joined #puppet-openstack
18:16 rongze joined #puppet-openstack
18:53 fvollero bodepd: ping... if you've time can you check https://review.openstack.org/#/c/54797/
19:21 rongze joined #puppet-openstack
19:24 digicloseup joined #puppet-openstack
19:25 digicloseup joined #puppet-openstack
19:26 fvollero hogepodge: ping :) you think it's a good moment to have 54797 in puppet-ceilometer as well? :)
19:27 hogepodge Have you checked against the latest mongo release? I'd be happy to get it in to stable if you submit the cherry pick. :-)
19:29 hogepodge Is puppet-heat ready for a stable/havana release?
19:29 hogepodge Can one of the developers on it update the README file if that's the case?
20:22 rongze joined #puppet-openstack
20:25 openstackgerrit Rob Crittenden proposed a change to stackforge/puppet-glance: Add parameter qpid_protocol to notify::qpid so ssl can be enabled.  https://review.openstack.org/56460
20:36 angdraug joined #puppet-openstack
20:56 EmilienM hogepodge: i take it
20:56 EmilienM hogepodge: it is afik
20:58 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-heat: Update README  https://review.openstack.org/56463
20:58 EmilienM hogepodge: ^
21:19 dachary fc__: do you know how to create a prefab ? https://github.com/puppetlabs/rspec-system#custom-prefabs
21:19 dachary I'm new to vagrant ;-)
21:22 dmsimard dachary: Regarding defaults
21:22 rongze joined #puppet-openstack
21:23 dmsimard dachary: Pretty sure I saw somewhere in the blueprint that the module should be able to install/configure a ceph cluster with little to no effort with sensible defaults
21:23 * dachary browsing http://www.vagrantbox.es/
21:24 dachary dmsimard: yes, and I think it's important.
21:24 dachary dmsimard: what I'm saying is that there is no need for the ceph module to provide defaults when ceph already provides defaults.
21:25 dmsimard dachary: We can rely on ceph to provide defaults, I don't mind. I'm personally a fan of putting the default values in the config file so I have a better visibility of what they are
21:27 dachary dmsimard: I'm worried about the effort (significant I think) to make sure the puppet default is the same as the ceph default
21:27 dachary for the mon_data path for instance, I believe it is not
21:28 dachary and if the puppet default is different to make sure there is no negative impact
21:28 dmsimard dachary: The defaults that I put aren't necessarily the ceph defaults
21:29 dachary for instance if osd data was somewhere different than /var/lib/ceph/osd/$cluster-$id the init scripts won't parse it properly
21:29 dmsimard the init script goes through ceph.conf
21:30 dmsimard it actually parses each section
21:32 dachary dmsimard: I don't follow ?
21:33 dmsimard dachary: the init script will know the data is elsewhere if it's declared in ceph.conf
21:35 dachary does anyone know what "Debian Wheezy 7.2 amd64 (VirtualBox Guest Additions 4.3.0) (2013/10/19)" http://www.vagrantbox.es/ means ?
21:37 dachary dmsimard: I'm afraid not https://github.com/ceph/ceph/blob/master/src/upstart/ceph-osd.conf#L14
21:38 mgagne dachary: it means is an image of "Debian Wheezy 7.2" (http://www.debian.org/News/2013/20131012) based on "am64" architecture and have "VirtualBox Guest Additions 4.3.0" installed. Build was done on "2013/10/19".
21:38 mgagne dachary: pretty much self-explaining
21:39 dmsimard dachary: That was not my understanding, looking at https://github.com/ceph/ceph/blob/master/src/init-ceph.in. I definitely don't have a problem starting my OSDs/MONs in different paths currently
21:39 dachary dmsimard: using undef avoids that kind of issue for sure
21:40 dachary dmsimard: I'm not saying inti-ceph.in has this issue. I'm saying https://github.com/ceph/ceph/blob/master/src/upstart/ceph-osd.conf#L14 does ;-)
21:40 dachary mgagne: what is "Guest Additions 4.3.0" ?
21:41 dachary is it commonly known in the vagrant world ?
21:41 * dachary looking
21:41 * dachary reading http://docs-v1.vagrantup.com/v1/docs/base_boxes.html "Guest Additions"
21:41 dmsimard dachary: Guest additions are virtualbox software additions (kernel modules)
21:41 mgagne ^
21:42 dmsimard dachary: It's the equivalent of xenserver tools if you're familiar with the xen world
21:42 dachary I'm not ;-)
21:42 mgagne people using virtualbox usually know about those stuff ;)
21:42 dachary yes, that's what makes me uneasy with virtualbox : I don't know this world.
21:42 mgagne vagrantbox assumes you know a bit about virtualbox or the hypervisor you use
21:43 dachary I should have phrased my question differently : does this image have a puppet client built-in ?
21:43 dmsimard dachary: For instance, you can't have bi-directional clipboard on a virtualbox VM without guest additions.
21:43 dmsimard dachary: It probably does not, no.
21:44 dachary I have no clue how much tweaking ken barber did to the box images in order for them to be "rspec_system" ready
21:44 dmsimard dachary: But then again, it's a vagrant box, vagrant boxes usually have puppet on them
21:44 dachary dmsimard: thanks for the example, I'll remember that ;-)
21:44 mgagne dachary: I don't know, better test it yourself as there is no convention in box names as you can see.
21:44 dachary this is a rabbit hole
21:44 mgagne yes
21:44 * dachary digging
21:45 * dachary waiting
21:45 mgagne dachary: you should poke those people if you need a box for Debian 7.2 with the latest Puppet: http://puppet-vagrant-boxes.puppetlabs.com/
21:45 dachary it's probably a waste of time to do that, I should give up and file a bug
21:46 mgagne dachary: as you can see, they build and provide some boxes
21:46 dachary yes, and it will probably take them a few weeks to come up with a debian box
21:46 mgagne Debian 7.2 is just Debian 7.0rc1 with all updates applied
21:46 mgagne nothing special
21:47 dachary in the meantime it bothers me that we have only a single box to work with but I guess it's not too much of a problem
21:47 dachary a single distribution to work with
21:47 mgagne who is hosting the infra?
21:47 dachary which infra ?
21:48 mgagne "it bothers me that we have only a single box to work with" -> where?
21:49 mgagne dachary: updating the box to 7.2 is only a matter of updating those lines and rebuild the box: https://github.com/puppetlabs/puppet-vagrant-boxes/blob/master/definitions/debian-70rc1-x64-vbox4210/definition.rb#L8-10
21:52 dachary I have no clue how to rebuild the box
21:52 mgagne puppetlabs uses veewee to build those boxes
21:53 dachary https://github.com/puppetlabs/rspec-system/issues/76
21:54 mgagne dachary: https://github.com/puppetlabs/puppet-vagrant-boxes/pull/20
21:54 mgagne dachary: and now 7.2
21:54 dmsimard dachary: Not sure how lucky we will be, this dates back 23 days ago https://github.com/puppetlabs/rspec-system/issues/65
21:55 dachary I would not obsess about this, it would have been nice but there is urgency ;-)
21:56 mgagne dachary: I see... it would require updating https://github.com/puppetlabs/rspec-system/blob/master/resources/prefabs.yml too..
22:02 dachary dmsimard: I'll wait for your current patch to land before submitting https://bugs.launchpad.net/puppet-ceph/+bug/1251443
22:02 * dachary browsing https://github.com/cernceph/puppet-ceph/
22:13 bitblt joined #puppet-openstack
22:13 bitblt joined #puppet-openstack
22:14 dachary "it bothers me that we have only a single box to work with" -> where? => I meant that it bothers me that the tests run on a single "box" in the vagrant sense, i.e. a single operating system. Sorry for the confusion.
22:14 dachary mgagne: ^
22:15 xarses rcrit: ping
22:16 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-neutron: Add ML2 plugin support  https://review.openstack.org/48289
22:19 otherwiseguy joined #puppet-openstack
22:23 rongze joined #puppet-openstack
22:29 dmsimard dalgaaf_: ping
22:37 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: Add Swift as backup backend support  https://review.openstack.org/56305
22:40 Hunner Arg. I wish the puppet-vagrant-boxes were auto-generated and didn't need ken to hand-create them
22:40 xarses joined #puppet-openstack
22:42 Hunner Let me check with our eng about getting those re-rolled
22:43 Hunner dachary: You should be able to create the boxes on any mac/linux box with virtualbox, vagrant, and veewee
22:44 Hunner mgagne: You can create a .prefabs.yml file on a per-project basis... the prefabs.yml in rspec-system is just a "global" prefabs list
22:44 dachary Hunner: thanks :-)
22:44 mgagne Hunner: how about a jenkins job to build those? =)
22:45 Hunner mgagne: Yeah... there should have been a long time ago :/
22:45 dachary Hunner: I appreciate that you push for this. It's not a blocker though, to be honest.
22:45 Hunner The builds are *so* unreliable though
22:45 mgagne Hunner: =)
22:45 Hunner dachary: It's something I've wanted for months now too, since I use those templates on a daily basis
22:45 dachary :-D
22:47 dmsimard dachary, dalgaaf_: So I thought a lot about the defaults and how we want to configure ceph - I agree that we should leave defaults up to ceph
22:48 dmsimard However, how does a user configure ceph ? We put every single configuration variable as a parameter to various classes ?
22:48 dmsimard Or do we not pass any parameters (or only the strict minimum?) and have the user use ceph_config in the puppetmaster manifests ?
22:49 dachary dmsimard: I'm afraid that's what we have to do ( every single configuration variable ).
22:51 dmsimard dachary: It's crazy because every parameter can be in [global] and then these parameters can also be declared in subsections such as [mds], [mon] or [osd] (and these have higher priority over [global])
22:51 dachary If ceph_config was able to be overriden the same way classes are, it would not be necessary. But since it is not, a wrapper is necessary it seems.
22:51 dachary dmsimard: It's crazy indeed. I think we should go for a reasonable set and wait for bodepd to come back before going crazy ;-)
22:52 dachary the more exotic params can be added later and won't require a change in logic
22:52 dmsimard dachary: It looks like a ceph::conf will in fact be necessary if that is what we want ? In fact it would probably be ceph::conf::global, ceph::conf::osd, ceph::conf::mon (and so on)
22:52 dachary dmsimard: what about we stick to the params that have an impacct on th elogic of the module ?
22:53 dmsimard dachary: How do you define something that has an impact ?
22:54 dachary good question
22:54 dachary what about we stick to the params that are currently supported by ceph modules and wait for bodepd to be back before going crazy with the more exotic ones ?
22:54 dalgaaf_ We would pass arrays to ceph conf classes and let them be handled by the ruby provider code?!
22:55 Hunner dachary: The boxes have nothing different from veewees upstream templates other than the "nocm" boxes which don't include puppet/chef. Nothing rspec-system specific
22:55 dmsimard dalgaaf_: Right now what I did looks like this: https://review.openstack.org/#/c/56270/4/manifests/init.pp
22:55 dalgaaf_ key=values pairs/strings in a array ... but I can live with a sane default set
22:56 dachary dalgaaf_: resorting to an array that is injected sound reasaonble for all params for which the logic of the module is unchanged
22:57 dachary ceph_config { $mon_data => ...., injected_array } something like that ?
22:57 dalgaaf_ I'm not sure, I'm not familiar with puppet-inifiles and if this would be able to handle it
22:58 dalgaaf_ but in fact: there is a limited set of config variables you want to change via puppet
22:58 dmsimard dalgaaf_: There's no need for an array, you can call ceph_config from anywhere and set [namespace]/key => value
22:59 dalgaaf_ so we could add as base a sane and limited set
22:59 dmsimard If the module does not "support" the key/value you want, you can still configure it through puppet in the puppetmaster manifests
23:00 dmsimard i'll give it some more thought tonight, probably submit another review tomorrow
23:01 dmsimard feel free to add your input in the review
23:01 dachary dmsimard: ok
23:01 dmsimard I'm really leaning towards doing something very modular like ceph::conf::global, ceph::conf::osd
23:04 dmsimard or here
23:04 dmsimard how about this
23:04 michchap joined #puppet-openstack
23:05 dmsimard ceph::conf that has for parameters $scope, and $key = value
23:06 dmsimard hmm, wait, no - that sounded better in my head
23:06 dmsimard I'm off for now, taking a break :)
23:21 xarses_ joined #puppet-openstack
23:24 rongze joined #puppet-openstack

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