Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2014-09-29

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

All times shown according to UTC.

Time Nick Message
00:17 britthouser joined #puppet-openstack
00:24 gaudenz joined #puppet-openstack
00:56 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: EMC VNX backend support  https://review.openstack.org/124561
00:57 openstackgerrit Emilien Macchi proposed a change to stackforge/puppet-cinder: EMC VNX backend support  https://review.openstack.org/124561
02:32 xingchao joined #puppet-openstack
02:39 badiane_ka left #puppet-openstack
03:10 underyx joined #puppet-openstack
04:08 sputnik13 joined #puppet-openstack
04:10 sputnik13 joined #puppet-openstack
04:13 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Add manage_service feature  https://review.openstack.org/115652
04:14 daneyon joined #puppet-openstack
04:19 openstackgerrit A change was merged to stackforge/puppet-glance: add command_options to glance::cache::[cleaner|pruner]  https://review.openstack.org/123076
04:19 alexiz joined #puppet-openstack
04:25 openstackgerrit A change was merged to stackforge/puppet-glance: Add manage_service feature  https://review.openstack.org/116610
04:27 gildub joined #puppet-openstack
04:27 openstackgerrit A change was merged to stackforge/puppet-heat: Add manage_service feature  https://review.openstack.org/115694
04:27 alexiz joined #puppet-openstack
04:28 openstackgerrit A change was merged to stackforge/puppet-heat: Hide secrets from puppet logs  https://review.openstack.org/124218
04:30 openstackgerrit A change was merged to stackforge/puppet-keystone: Make user creation optional when creating service.  https://review.openstack.org/120525
04:39 sanjayu joined #puppet-openstack
04:54 walkleyn joined #puppet-openstack
05:13 openstackgerrit A change was merged to stackforge/puppet-glance: New parameter for swift backend  https://review.openstack.org/124234
05:57 openstackgerrit Colleen Murphy proposed a change to stackforge/puppet-openstacklib: Implement base aviator provider  https://review.openstack.org/116753
05:58 openstackgerrit Colleen Murphy proposed a change to stackforge/puppet-keystone: (WIP) Use aviator as a REST interface for keystone_tenant provider  https://review.openstack.org/116754
06:02 dgurtner joined #puppet-openstack
06:34 stamak joined #puppet-openstack
06:34 berendt joined #puppet-openstack
06:39 sputnik13 joined #puppet-openstack
06:48 francois1 joined #puppet-openstack
06:55 dgurtner joined #puppet-openstack
07:10 TorLaedre joined #puppet-openstack
07:20 arnaud_orange joined #puppet-openstack
07:36 alokj joined #puppet-openstack
07:37 f10d4 joined #puppet-openstack
07:47 mmagr joined #puppet-openstack
07:49 dgurtner joined #puppet-openstack
07:55 stamak joined #puppet-openstack
07:58 alexiz joined #puppet-openstack
07:58 f10d4 joined #puppet-openstack
07:58 fc__ joined #puppet-openstack
07:58 dkaigarodsev joined #puppet-openstack
08:13 ddmitriev joined #puppet-openstack
08:15 dgurtner joined #puppet-openstack
08:17 derekh joined #puppet-openstack
08:41 igajsin1 joined #puppet-openstack
08:55 f10d4 joined #puppet-openstack
08:58 alexiz joined #puppet-openstack
09:12 Dafna joined #puppet-openstack
09:23 qba73 joined #puppet-openstack
09:53 sputnik13 joined #puppet-openstack
09:57 qba73 joined #puppet-openstack
09:59 alexiz joined #puppet-openstack
10:00 qba73 joined #puppet-openstack
10:25 gildub joined #puppet-openstack
10:25 qba73 joined #puppet-openstack
10:33 openstackgerrit Benedikt Trefzer proposed a change to stackforge/puppet-nova: add parameter for libvirt_service_name  https://review.openstack.org/124705
10:42 qba73 joined #puppet-openstack
10:51 qba73 joined #puppet-openstack
10:58 qba73 joined #puppet-openstack
11:00 alexiz joined #puppet-openstack
11:05 sanjayu joined #puppet-openstack
11:07 dachary joined #puppet-openstack
11:41 EmilienM Good morning
11:43 michchap EmilienM:  Morning :)
11:54 openstackgerrit Martin Mágr proposed a change to stackforge/puppet-keystone: Add manage_service feature  https://review.openstack.org/116604
12:01 shakamunyi joined #puppet-openstack
12:09 Madkiss how do I submit a bugfix against the stable/icehouse tree oh puppet-cei
12:09 Madkiss lometer?
12:12 michchap Madkiss: https://wiki.openstack.org/wiki/StableBranch#Workflow
12:12 Madkiss thanks
12:12 EmilienM Madkiss: you have a cherry-pick button on the original patch
12:12 EmilienM you select stable/icehouse, and that's it
12:13 Madkiss there is no original patch. HEAD was changed massively compared to stable/icehouse, and stable/icehouse only broke due to a recent change in puppetlabs-mysql :)
12:14 EmilienM Madkiss: oh, then you can't push a patch directly in stable/icehouse
12:14 EmilienM you have to submit against master and then backport.
12:14 EmilienM it's the same way in OpenStack
12:15 Madkiss err?
12:15 michchap EmilienM: I have submitted new commits to stable before. Sometimes there is no other option.
12:15 Madkiss So you're telling me we will have leave stable/icehouse broken, as the affected code is not present in HEAD anymore?
12:15 michchap Madkiss: if possible, apply to master and the backport
12:15 michchap Madkiss: if it's not then put something up for review so we can at least look at it.
12:16 openstackgerrit Martin Gerhard Loschwitz proposed a change to stackforge/puppet-ceilometer: Fix the mysql_grant call to not fail with puppetlabs-mysql 2.3  https://review.openstack.org/124723
12:16 Madkiss there you go
12:16 michchap aha
12:16 michchap before of the refactor this isn't even in ceilometer anymore?
12:16 michchap it's all in openstacklib.
12:17 Madkiss that's true.
12:17 Madkiss but it breaks stable/icehouse as of today.
12:17 Madkiss i.e. if I checkout puppetlabs-mysql 2.3 and puppetlabs-ceilometer (stable/icehouse) and try to run that combination, it'll fail.
12:17 michchap Madkiss: is it the same with 2.2?
12:18 Madkiss https://bugs.launchpad.net/puppet-ceilometer/+bug/1375233
12:18 Madkiss the affected commit in puppetlabs-mysql was on Jun 3rd.
12:18 Madkiss it's not in 2.2.
12:19 michchap So 2.2 behaves correctly, or it just doesn't hit the fail() and then does the wrong thing?
12:19 Madkiss If the goal is to have people not use anything newer than mysql-2.2 with stable/icehouse, we should adopt the Modulefile, shouldn't we?
12:20 michchap Madkiss: yep, that's what I'm thinking. I don't want to chase master of dependencies with our stable branches
12:20 Madkiss sure. that'd definitely work, but we need to tell people about it.
12:23 michchap Madkiss: I think the modulefile should be sufficient, is there something else you have in mind?
12:23 Madkiss no, changing the Module file should be good.
12:24 michchap what about the other openstack modules?
12:24 michchap do they have fixed mysql_grant resources, or pinned dependencies?
12:24 michchap if the former, let's just merge your patch that's already there.
12:25 Madkiss No patch necessary. All puppet modules except for ceilometer do the right thing out of the box.
12:25 Madkiss i.e. use ".*"
12:26 Madkiss mysql_grant { "${user}@${name}/${database}.*": <= from neutron
12:27 michchap done. thanks for the patch.
12:27 Madkiss thanks a bunch!
12:28 spredzy joined #puppet-openstack
12:29 EmilienM Madkiss: michchap ok for me too. Merging.
12:30 Madkiss awesome
12:36 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Fix the mysql_grant call to not fail with puppetlabs-mysql 2.3  https://review.openstack.org/124723
12:46 azemlyanov joined #puppet-openstack
13:01 shakamunyi joined #puppet-openstack
13:04 tfhartmann joined #puppet-openstack
13:11 rcallawa joined #puppet-openstack
13:11 Mr_D joined #puppet-openstack
13:16 morazi joined #puppet-openstack
13:30 sanjayu joined #puppet-openstack
13:33 admin0 joined #puppet-openstack
13:36 [1]Mr_D joined #puppet-openstack
13:43 bogdando joined #puppet-openstack
13:54 Dafna joined #puppet-openstack
13:55 openstackgerrit Martin Gerhard Loschwitz proposed a change to stackforge/puppet-neutron: Make api_extensions_path a parameter for "neutron"  https://review.openstack.org/124324
13:59 snk joined #puppet-openstack
14:00 nkinder joined #puppet-openstack
14:01 Madkiss for a unit test, with a parameter that has undef as default, what would be the correct entry?
14:01 Madkiss should contain_neutron_config('DEFAULT/api_extensions_path').with_value(undef) <= doesn't work
14:01 Madkiss "nil"?=
14:01 openstackgerrit Martin Gerhard Loschwitz proposed a change to stackforge/puppet-neutron: Make api_extensions_path a parameter for "neutron"  https://review.openstack.org/124324
14:02 shakamunyi joined #puppet-openstack
14:27 rcallawa joined #puppet-openstack
14:30 openstackgerrit Martin Mágr proposed a change to stackforge/puppet-horizon: Add possibility to manage httpd service  https://review.openstack.org/115690
14:31 rharrison joined #puppet-openstack
14:32 shakamunyi joined #puppet-openstack
14:33 rwsu joined #puppet-openstack
14:41 ericpeterson joined #puppet-openstack
14:46 openstackgerrit Martin Mágr proposed a change to stackforge/puppet-horizon: Add possibility to manage httpd service  https://review.openstack.org/115690
14:47 [1]Mr_D joined #puppet-openstack
14:50 EmilienM crinkle: sbadia: do we have a meeting today?
14:51 sbadia hi
14:52 sbadia EmilienM: hum I've a couple of info about release (stackforge), metadata.json and other things, but maybe not enough for a meeting :/
14:54 EmilienM sbadia: I have a quick update about TripleO puppet elements
14:54 sbadia oh, great!
14:55 sbadia https://etherpad.openstack.org/p/puppet-openstack-2014-09-29
14:55 sbadia (for the meeting in 5min :-))
14:55 social is anyone working on this for glance? https://bugs.launchpad.net/puppet-neutron/+bug/1356083 ?
14:56 mdorman joined #puppet-openstack
15:00 EmilienM social: don't think so: https://review.openstack.org/#/q/project:stackforge/puppet-glance+status:open,n,z
15:03 crinkle morning
15:03 crinkle yes, sorry I didn't send an agenda earlier
15:03 shakamunyi joined #puppet-openstack
15:03 sbadia np
15:03 EmilienM crinkle: back from puppetconf?
15:03 crinkle EmilienM: yep
15:04 sbadia don't hesitate if you have any other subjects (on the pad)
15:06 EmilienM sbadia: let's start?
15:06 sbadia EmilienM: go
15:06 EmilienM sbadia: so
15:06 EmilienM #startmeeting
15:06 EmilienM #topic TripleO puppet elements
15:06 sbadia #chair EmilienM
15:07 EmilienM so there is a patch in openstack/governance
15:07 EmilienM #link https://review.openstack.org/#/c/123826/
15:07 EmilienM that means TC has to vote to have this work in openstack namespace
15:07 EmilienM that means also TripleO puppet elements will not be in stackforge but in openstack!
15:08 EmilienM the patch in openstack-infra is ready
15:08 EmilienM #link https://review.openstack.org/#/c/124560/
15:08 EmilienM so puppet team will be core on this project
15:08 sbadia oh! great
15:08 EmilienM and Dan Prince is core on both TripleO & Puppet. So he will be part of reviewing process
15:09 EmilienM we have a lot of work coming, but it's very positive to see our modules consumed by tripleo...
15:09 crinkle \o/
15:09 EmilienM sbadia: I'm done, thanks
15:10 crinkle EmilienM: do you have a link for progress on tripleo-puppet-elements itself and how the modules are getting integrated?
15:10 sbadia ok, and the repo is on enovance github for the moment?
15:10 sbadia ok yes
15:10 sbadia #link https://github.com/enovance/tripleo-puppet-elements
15:10 EmilienM sbadia: the repo on enovance github will be the upstream of the module created in github/enovance
15:11 sbadia yep yep
15:11 EmilienM crinkle: there is no puppet code yet.
15:11 crinkle ah okay
15:11 EmilienM crinkle: just an empty openstack project for now.
15:11 EmilienM crinkle: I wanted this landed before the summit
15:11 EmilienM crinkle: then, we can start blueprints, etc etc
15:11 crinkle EmilienM: sounds good
15:11 EmilienM but first, I wanted to be sure that will land somewhere.
15:12 EmilienM sbadia: go ahead!
15:12 sbadia #topic Release managment
15:13 sbadia the release 4.0.0 is a bit old and i wonder to know if we planned to make a new release for 2014.1.3
15:13 sbadia #link https://wiki.openstack.org/wiki/ReleaseNotes/2014.1.3
15:14 EmilienM +1
15:14 sbadia (and in the same time, ship the new release under stackforge namespace (on puppet forge), and switch to metadata.json)
15:14 sbadia I've a patch on infra for metadata.json files https://review.openstack.org/124569
15:14 crinkle I think the plan was to migrate to the stackforge namespace on the next major release?
15:16 sbadia ok then, it's not a problem :)
15:16 crinkle I think I would prefer to do that migration on a major release to avoid dependency issues
15:16 mdorman is there any particular drawback to doing that?  do Puppetfiles reference modules via their namespace, too?  like if we change the namespace out from under people, will that break things?
15:17 mdorman i just use git urls in Puppetfile, so not sure how it works with the forge.
15:17 mdorman tl;dr - +1 on doing it at a major release
15:18 crinkle I don't know how Puppetfiles work, but the Modulefile/metadata.json refers to modules by their namespace so that can cause breakage
15:18 crinkle sbadia++ for the metadata.json patches
15:19 sbadia oh then =)
15:19 sbadia anything else?
15:20 sbadia #topic Problems with rspec tests on puppet-swift module ?
15:20 crinkle sbadia: so https://review.openstack.org/124569 will fix the puppet-lint problem?
15:20 crinkle oops
15:20 sbadia crinkle: yep (tested on my laptop)
15:20 sbadia (all releated on module layout)
15:21 crinkle cool
15:21 sbadia (and ack, thx for pe/puppet support, i'll update my patchs)
15:21 crinkle was just about to mention :)
15:22 crinkle okay, we can move on to the puppet-swift problem
15:22 sbadia yep
15:22 sbadia nobody looked for problems with puppe-swift?
15:22 sbadia it seems a bit weired…
15:23 crinkle I'm not familiar with the problem, are tests failing?
15:23 hogepodge Puppet isn't strongly namespaced, so changing to a new one shouldn't be a problem. I don't know what the plan for Puppet 4 is, though.
15:24 sbadia crinkle: no, releated to facts problems
15:24 sbadia but anyway, no need to fix this during a meeting
15:24 hogepodge Forge does depend heavily on semver, though, so switching to OpenStack release style numbering will cause problems for the Puppet Module Tool (sorry to have missed that topic discussion)
15:24 sbadia #action sbadia take a look, and fix puppet-swift problems
15:24 crinkle hogepodge: the puppet module install tool will have conflicts if you try to install the same module from two different namespaces
15:26 sbadia hogepodge: np, thanks for you comments!
15:26 crinkle hogepodge: hmm, are release notes still documented on wiki.openstack.org for semver releases?
15:26 hogepodge it shouldn't matter if dependencies are consistent, though. which can be validated during a major release.
15:27 rmoe joined #puppet-openstack
15:27 cjellick joined #puppet-openstack
15:27 hogepodge crinkle I wasn't maintaining the wiki
15:28 crinkle ah
15:28 hogepodge mgagne was maintaining that page
15:29 Guest73730 hogepodge: which page?
15:29 mgagne joined #puppet-openstack
15:29 mgagne hogepodge: me ^
15:29 crinkle mgagne: https://wiki.openstack.org/wiki/ReleaseNotes/2014.1.3
15:29 sbadia https://wiki.openstack.org/wiki/Puppet-openstack ?
15:29 hogepodge https://wiki.openstack.org/wiki/Puppet-openstack/Development
15:29 mgagne hogepodge: yes, that was me. not really maintained tbh ^^'
15:32 sbadia ok, then, next topic?
15:32 crinkle sure
15:32 sbadia puppetconf ? :)
15:33 crinkle integration testing was next?
15:34 sbadia it's linked to tripleO no?
15:34 sbadia and nothing has changed on ML :-/
15:34 crinkle okay
15:35 crinkle so re: puppetconf, I wanted to say that I had a lot of fun at the openstack table at the contributors summit, it was really nice to meet other developers and people new to openstack
15:36 crinkle one thing that came out of that was that not everyone can attend this meeting time, and while I know it's not possible to find a time that suits everyone, it might be good to reevaluate the meeting time
15:37 crinkle how was this time chosen originally?
15:37 sbadia I don't remember ^^
15:37 sbadia hogepodge mgagne EmilienM ^ ?
15:38 EmilienM sbadia: me neither
15:38 mgagne we used a multi timezone calendar to determine the best time
15:38 EmilienM I just remeber michchap was very flexible
15:38 EmilienM remember*
15:38 mgagne and found it was the time where most people weren't sleeping
15:38 crinkle it is about 1am in australia
15:40 cjellick_ joined #puppet-openstack
15:40 EmilienM yeah, we found out it was impossible to have everyone
15:40 crinkle yeah :/
15:40 EmilienM (my name is mister obvious I know :-P)
15:40 crinkle okay I'll leave that be for now
15:40 crinkle anothr couple of things about puppetconf
15:41 crinkle hogepodge did a great job giving the openstack talk
15:41 crinkle and reid vanderwiele gave a really interesting talk on puppetizing multitiered architectures that's highly applicable to us, I recommend watching the video when it's posted
15:41 crinkle that's all from me
15:42 mgagne this is relevant to my interests ^
15:44 cjellick joined #puppet-openstack
15:44 crinkle sbadia: next?
15:44 * sbadia todo++ (watch puppetconf talks) :)
15:45 sbadia hum nothing else on the pad
15:45 sbadia we have a couple of review stalled
15:45 sbadia https://bugs.launchpad.net/puppet-neutron/+bug/1356083 (backport)
15:46 sbadia crinkle: you can test your new super powers :)
15:46 sbadia and on aviator Aviator:  https://review.openstack.org/#/c/116753/ and https://review.openstack.org/#/c/116754/
15:46 crinkle please glance at 116754 even though it's marked as WIP
15:47 sbadia I said that I would look, but I didn't have time :/
15:48 crinkle sbadia: no worries
15:49 sbadia anything else?
15:49 crinkle nothing from me
15:50 sbadia thanks everyone
15:50 sbadia #endmeeting
15:51 crinkle thanks sbadia
15:52 mdorman crinkle:  is the aviator stuff theoretically in a state now that it should be workable?   was thinking about pulling down your reviews and just see how it works in the wild
15:53 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-neutron: Add ability to manage Neutron ML2 plugin package state  https://review.openstack.org/115367
15:53 daneyon joined #puppet-openstack
16:01 crinkle mdorman: yes, the base provider in openstacklib is basically done and I've rewritten the keystone_tenant provider to use the openstacklib base provider
16:01 crinkle the keystone_tenant tests don't pass yet because the openstacklib part isn't merged, but it's otherwise workable
16:02 crinkle you should be able to drop it in without seeing a difference
16:04 mdorman cool
16:04 mdorman hopingto get some time this week to check that out
16:05 crinkle thanks
16:06 social joined #puppet-openstack
16:09 doo joined #puppet-openstack
16:17 dgurtner joined #puppet-openstack
16:22 sputnik13 joined #puppet-openstack
16:27 aimon joined #puppet-openstack
16:35 thumpba_ joined #puppet-openstack
16:35 mmaglana joined #puppet-openstack
16:36 thumpba_ joined #puppet-openstack
16:37 openstackgerrit Gaudenz Steinlin proposed a change to stackforge/puppet-vswitch: Suppport for bonded interfaces  https://review.openstack.org/124789
16:38 tfhartmann joined #puppet-openstack
16:42 aimon Re Aviator : I released a new version today at request of mmaglana (Aviator creator) to update license to MIT: https://forgeapi.puppetlabs.com/v3/files/aimonb-aviator-0.4.2.tar.gz
16:42 aimon no other change was made
16:44 dgurtner_ joined #puppet-openstack
16:51 stamak joined #puppet-openstack
16:56 openstackgerrit Gaudenz Steinlin proposed a change to stackforge/puppet-vswitch: Add interface_type parameter  https://review.openstack.org/124794
17:20 [1]Mr_D joined #puppet-openstack
17:29 mgagne can't use openstack::db::mysql to manage 2 databases with the same user =(
17:30 mgagne openstacklib*
17:32 crinkle :(
17:32 rcallawa_ joined #puppet-openstack
17:32 crinkle we should've used ensure_resource there
17:32 crinkle I guess I thought we wouldn't want two databases with the same user
17:33 mgagne crinkle: I guess the "problem" is that mysql_user isn't tied to the database unlike mysql_database and mysql_grant
17:33 hogepodge crinkle This PR is worth merging. https://github.com/puppetlabs/puppetlabs-openstack/pull/96
17:34 mgagne crinkle: let me see what I can do on my side. I guess I could create an other user for my purpose
17:34 crinkle hogepodge: oops, haven't been keeping up on prs :/
17:34 mdorman mgagne / crinkle, i can confirm that use case.  our databases aren’t controlled by us, and we’re just given a single service account to access all of them.
17:35 mgagne mdorman: in fact, your use case includes the fact that you can't manage the user creation but can you create databases and grants?
17:36 hogepodge crinkle, I'm not sure about 95. 6 is very old. I'm going to create a stable branch locally so I can start targeting Juno
17:36 mgagne mdorman: I mean, in your case, I don't think you can use *::db::mysql at all
17:36 mdorman mgagne:   yes, that’s a good point.  for us, we can’t create users nor databases, but can create schema.  not sure on grants.  but yeah, in our case we wouldn’t be using that anyway
17:37 crinkle hogepodge: I'm in orientation all day today, do you want to set up a time this week to talk about releases and puppetlabs-openstack?
17:38 hogepodge crinkle maybe Friday. I'm in Austin until Thursday preparing a Juno demo.
17:38 crinkle hogepodge: I'm in meetings all day Friday, so maybe next week
17:39 rcallawa joined #puppet-openstack
17:45 hogepodge crinkle sounds good. I'll be in town until Paris.
18:05 zeus joined #puppet-openstack
18:11 snguye2 joined #puppet-openstack
18:15 berendt joined #puppet-openstack
18:31 snguye2 joined #puppet-openstack
18:54 stamak joined #puppet-openstack
18:57 rharrison joined #puppet-openstack
19:02 snguye2 joined #puppet-openstack
19:26 ericpeterson joined #puppet-openstack
19:39 openstackgerrit A change was merged to stackforge/puppet-ceilometer: Make user creation optional when creating service.  https://review.openstack.org/120477
19:51 openstackgerrit A change was merged to stackforge/puppet-heat: Make user creation optional when creating service.  https://review.openstack.org/120466
19:54 hogepodge joined #puppet-openstack
20:04 snguye2 joined #puppet-openstack
20:21 snguye2 joined #puppet-openstack
20:25 rcallawa_ joined #puppet-openstack
20:41 mdorman could someone please look at doing workflow on https://review.openstack.org/116262 and https://review.openstack.org/116265 to wrap up https://bugs.launchpad.net/puppet-nova/+bug/1360232 (Should not configure a service user in all cases where a service is configured)?  thanks
20:49 openstackgerrit Mike Dorman proposed a change to stackforge/puppet-keystone: Change keystone class to inherit keystone::params  https://review.openstack.org/124866
20:49 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstacklib: Use host_access to provision mysql_user and mysql_grant  https://review.openstack.org/124867
20:49 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstacklib: Greatly reduce code duplication in rspec tests of db::mysql  https://review.openstack.org/124868
20:49 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstacklib: Improve openstacklib::db::mysql test coverage  https://review.openstack.org/124869
20:50 EmilienM mgagne: you masterize dependencies :P
20:51 mgagne haha
20:51 mgagne EmilienM: oh true, you are in the same timezone now =)
20:52 EmilienM mgagne: yep, and we haven't see each other IRL, that's an issue we should fix
20:55 mgagne EmilienM: =)
20:57 mgagne EmilienM: btw, why are we using this syntax to pass the host? https://github.com/stackforge/puppet-openstacklib/blob/master/manifests/db/mysql/host_access.pp#L26
20:57 mgagne EmilienM: why not a host parameter? =)
20:58 EmilienM mgagne: ask the author, crinkle :)
20:58 mgagne EmilienM: well, it was copied from the previous implementation
20:58 EmilienM ah
20:58 EmilienM indeed
20:58 mgagne EmilienM: oh I'm wrong
20:59 EmilienM though
20:59 EmilienM you confuse me here
20:59 mgagne haha
20:59 mgagne it's a trap!
20:59 EmilienM mgagne: I was about +2 your patches
20:59 EmilienM in fact, I'll re-consider it
21:00 crinkle I think it needs to be $dbname_$host so that it's sure to be unique
21:00 * crinkle stares at it for a bit
21:00 mgagne crinkle: we can argue that puppet already does an excellent job in that aspect
21:01 mgagne crinkle: we don't need to reinforce it twice as it will fail catalog compilation before it reaches this validation
21:01 crinkle mgagne: you mean here https://github.com/stackforge/puppet-openstacklib/blob/master/manifests/db/mysql.pp#L75 ?
21:02 mgagne crinkle: no, puppet already breaks the world if you declare twice a resource with the same name
21:02 crinkle ah yes
21:02 mgagne crinkle: but I think I see why
21:03 mgagne crinkle: we wish to declare a bunch of host_access in a single declaration: https://github.com/stackforge/puppet-openstacklib/blob/master/manifests/db/mysql.pp#L82
21:03 mgagne crinkle: and we can only do that if the specific details of each host is passed through the title
21:03 crinkle oh I think I remember writing this now
21:04 crinkle yes, I wasn't sure how else to do that
21:06 mgagne me neither =)
21:06 openstackgerrit A change was merged to stackforge/puppet-neutron: Hide secrets from puppet logs  https://review.openstack.org/124219
21:07 rcallawa joined #puppet-openstack
21:13 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstacklib: Improve openstacklib::db::mysql test coverage  https://review.openstack.org/124869
21:15 openstackgerrit Gaudenz Steinlin proposed a change to stackforge/puppet-vswitch: Add VLAN related parameters  https://review.openstack.org/124888
21:18 openstackgerrit A change was merged to stackforge/puppet-keystone: Change keystone class to inherit keystone::params  https://review.openstack.org/124866
21:24 openstackgerrit A change was merged to stackforge/puppet-openstacklib: Use host_access to provision mysql_user and mysql_grant  https://review.openstack.org/124867
21:26 mgagne I should have changed it to WIP
21:28 snguye2 joined #puppet-openstack
21:29 openstackgerrit Mathieu Gagné proposed a change to stackforge/puppet-openstacklib: Fix support for puppetlabs-stdlib<4.2.0  https://review.openstack.org/124895
21:29 mgagne EmilienM: https://review.openstack.org/#/c/124895/
21:30 mgagne EmilienM: fix for pre-4.2.0 stdlib
21:30 EmilienM a good one.
21:31 crinkle mgagne: we should just change the Modulefile/metadata.json to use stdlib >4.2
21:32 mgagne crinkle: what's supported by Puppet Enterprise? Latest stdlib?
21:32 crinkle I believe so
21:33 crinkle and actually the modulefile already says stdlib >=4 <5
21:33 rcallawa joined #puppet-openstack
21:38 mgagne crinkle: https://github.com/puppetlabs/puppetlabs-stdlib/blob/da11903285aca54ac399d66cb453f5a8db3ac355/CHANGELOG.md#2014-06-27---supported-release-430
21:38 mgagne "This release is the first supported release of the stdlib 4 series."
21:39 crinkle okay
21:39 crinkle so change modulefile to stdlib >4.3.x?
21:41 mgagne I guess it's the right thing to do
21:55 dgurtner joined #puppet-openstack
22:05 openstackgerrit Mike Dorman proposed a change to stackforge/puppet-ceilometer: Pin gem versions for stable branch  https://review.openstack.org/124904
22:05 openstackgerrit Mike Dorman proposed a change to stackforge/puppet-ceilometer: Release 4.1.0  https://review.openstack.org/124905
22:07 hogepodge crinkle on my centos-6 vagrant cloud image for some reason vmware-tools crashes in the guest after boot. It makes the system unavailable to vagrant ssh after boot. FYI
22:08 crinkle hogepodge: I haven't experience that, is this a new image?
22:10 hogepodge Not really. I think there's an incompatability with Fusion 7
22:11 hogepodge It's been biting me pretty hard over the last couple of weeks, and I just sorted out the issue.
22:12 crinkle hogepodge: ah, I haven't upgraded to 7 yet
22:14 openstackgerrit A change was merged to stackforge/puppet-keystone: Use secret parameter for rabbit_password  https://review.openstack.org/124217
22:14 hogepodge So much coffee BS. "You must use a swirling pattern or the coffee will taste terrible"
22:15 openstackgerrit Mike Dorman proposed a change to stackforge/puppet-ceilometer: Makes kombu_ssl_* parameters optional when rabbit_use_ssl => true  https://review.openstack.org/124524
22:15 openstackgerrit Mike Dorman proposed a change to stackforge/puppet-ceilometer: Fail when ssl parameters are missing and rabbit_use_ssl is set to true  https://review.openstack.org/124523
22:15 openstackgerrit Mike Dorman proposed a change to stackforge/puppet-ceilometer: Implement rspec function it_raises  https://review.openstack.org/124909
22:30 JayJ joined #puppet-openstack
22:57 snguye2 joined #puppet-openstack
23:05 ric` joined #puppet-openstack
23:05 mgagne interesting... if you don't install nova::api on your controller, you won't get database initialization (migration) =)
23:09 nkinder joined #puppet-openstack
23:38 dgurtner joined #puppet-openstack
23:49 rmoe joined #puppet-openstack

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