Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2015-02-03

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

All times shown according to UTC.

Time Nick Message
00:12 markvoelker joined #puppet-openstack
00:22 openstackgerrit Merged stackforge/puppet-ceph: Bump default version to giant, test against giant and drop emperor  https://review.openstack.org/135668
00:37 gildub joined #puppet-openstack
00:38 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: fix osd removal to be idempotent  https://review.openstack.org/151968
00:52 markvoelker joined #puppet-openstack
00:54 aimon joined #puppet-openstack
00:55 openstackgerrit Merged stackforge/puppet-ceph: fix osd removal to be idempotent  https://review.openstack.org/151968
01:18 simonmcc joined #puppet-openstack
01:56 markvoelker joined #puppet-openstack
02:09 rmoe joined #puppet-openstack
02:14 vinsh joined #puppet-openstack
02:16 xarses joined #puppet-openstack
02:41 ericpeterson joined #puppet-openstack
02:51 xingchao joined #puppet-openstack
02:57 markvoelker joined #puppet-openstack
02:59 openstackgerrit Matt Fischer proposed stackforge/puppet-heat: Rework delegated roles  https://review.openstack.org/146734
02:59 mfisch anyone have any great ideas on why this test fails?
02:59 mfisch https://review.openstack.org/#/c/152321/
03:00 mfisch I didn't change any code related to it and it passes on my box
03:33 markvoelker joined #puppet-openstack
04:24 gildub joined #puppet-openstack
04:28 openstackgerrit Matt Fischer proposed stackforge/puppet-cinder: Don't add a new line if the rbd_user changes  https://review.openstack.org/152384
04:36 mfisch hogepodge: FYI - all my jenkins jobs are now failing
04:36 mfisch hogepodge: they seem to be on tests that expect errors but are not getting what they expect
04:36 mfisch something changed in a module, but I dont know what
04:36 mfisch I have failures in cinder, heat, nova
04:37 mfisch crinkle: ^
04:52 sanjayu joined #puppet-openstack
05:19 openstackgerrit Matt Fischer proposed stackforge/puppet-ceph: Fix incorrect email in header  https://review.openstack.org/152392
07:52 hostranger joined #puppet-openstack
07:54 hostranger left #puppet-openstack
08:00 tfz joined #puppet-openstack
08:18 dalgaaf joined #puppet-openstack
08:58 jistr joined #puppet-openstack
09:17 Madkiss EmilienM|afk: ping
09:20 jpena joined #puppet-openstack
09:21 dgurtner joined #puppet-openstack
09:21 marvin0815_home left #puppet-openstack
09:21 derekh joined #puppet-openstack
09:34 bogdando joined #puppet-openstack
09:42 stamak joined #puppet-openstack
10:12 openstackgerrit Bogdan Dobrelya proposed stackforge/puppet-neutron: Fix db-sync dependencies  https://review.openstack.org/130171
10:23 openstackgerrit Merged stackforge/puppet-ceph: Fix incorrect email in header  https://review.openstack.org/152392
11:03 openstackgerrit Sebastien Badia proposed stackforge/puppet-sahara: Migrate postgresql backend to use openstacklib::db::postgresql  https://review.openstack.org/145511
11:03 openstackgerrit Sebastien Badia proposed stackforge/puppet-sahara: fixtures: Unpin puppetlabs-mysql module  https://review.openstack.org/145510
11:03 openstackgerrit Sebastien Badia proposed stackforge/puppet-sahara: Rewrite puppet-sahara from scratch  https://review.openstack.org/145509
11:03 openstackgerrit Sebastien Badia proposed stackforge/puppet-sahara: policy.json: Allow one to manage them from the puppet module  https://review.openstack.org/152470
11:27 jistr joined #puppet-openstack
11:39 ddmitriev joined #puppet-openstack
11:55 cdent joined #puppet-openstack
12:01 openstackgerrit Lukas Bezdicka proposed stackforge/puppet-nova: Add support for nova Pci passthrough  https://review.openstack.org/145787
12:08 EmilienM good morning!
12:09 EmilienM Madkiss: pong
12:13 derekh joined #puppet-openstack
12:15 EmilienM Madkiss: your patch is reviewed, a minor thing - I'll +2 after
12:17 derekh joined #puppet-openstack
12:20 Madkiss EmilienM: excellent, thanks
12:20 EmilienM jpena: have you seen https://review.openstack.org/#/c/151609 ?
12:21 jpena EmilienM: no, hadn't seen it. Will give it a look later today
12:23 openstackgerrit Martin Gerhard Loschwitz proposed stackforge/puppet-nova: Support for heal_instance_info_cache_interval  https://review.openstack.org/105245
12:24 EmilienM Madkiss: thx, +2
12:26 Madkiss it would be sort of awesome to get this into stable/juno as well
12:30 trash How could https://github.com/stackforge/puppet-nova/commit/6f744cd4be483696de97bbf89836c6171d09f0b4 go through review? We were wondering why nova cannot spawn any VMs anymore, because the nova-scheduler "randomly" was stopped. $enable = false which is the new default explains that.
12:31 openstackgerrit Ivan Berezovskiy proposed stackforge/puppet-ceilometer: Add support for ceilometer-polling agent  https://review.openstack.org/150003
12:32 trash If one even has to explicetly load the nova::scheduler class, why would you want to also set a parameter to actually enable it?
12:32 trash :-(
12:33 trash But yeah, other classes already seem to implement it that way in puppet-nova.
12:34 EmilienM trash: I don't see something wrong in this patch
12:35 EmilienM trash: the use-case is when running Pacemaker, we may want to disable service management, but actually configure nova scheduler - and let pacemaker manage the service.
12:35 EmilienM so I think something is wrong in your setup but nothing related to this very old patch
12:36 trash Well, some default parameter changed from true to false which leads to nova-scheduler being actively stopped for us. Of course, now I've put nova::scheduler::enabled: true in hiera
12:37 trash And that behavior was not there with the stable/icehouse branch.
12:38 trash Whatever, moving on. :-)
12:38 trash More things to fix.
12:38 EmilienM trash: you're running a commit from 2012
12:38 EmilienM we are in 2015, I guess there is a reason - I can find which one if you want
12:39 trash Yep, I think you are right, refactoring in one of our wrapper modules broke that.
12:39 EmilienM trash: https://github.com/stackforge/puppet-nova/commit/c45f573882939ba66136e7b7c2196908294ee2b4
12:39 EmilienM I guess you're talking about that one
12:40 EmilienM which is a commit from one year ^^
12:40 trash It's really actually the missing parameter in our wrapper modules which call nova::scheduler
12:40 trash The parameter was not moved to the new place.
12:40 trash m(
12:40 social huh I'm getting ceph error in puppet-nova on unrelated review
12:42 EmilienM social: if you don't go ahead with a real description I don't know how to help
12:43 social EmilienM: http://logs.openstack.org/87/145787/11/check/gate-puppet-nova-puppet-unit-3.3/9b891e4/console.html at end
12:44 EmilienM social: gerrit URL?
12:44 social https://review.openstack.org/#/c/145787/
12:46 EmilienM social: do you run the tests locally before sending to gerrit?
12:46 markvoelker joined #puppet-openstack
12:46 social EmilienM: this time no, I have to rerun as the change was adding -
12:46 Madkiss EmilienM: https://review.openstack.org/#/c/105245/ hates me :(
12:46 marvin0815 joined #puppet-openstack
12:55 social EmilienM: ok same error locally, I'll look into it
12:55 openstackgerrit Ivan Berezovskiy proposed stackforge/puppet-ceilometer: Add support for ceilometer-polling agent  https://review.openstack.org/150003
12:56 markvoelker joined #puppet-openstack
12:56 jpena_ joined #puppet-openstack
12:57 xarses joined #puppet-openstack
12:58 kbyrne joined #puppet-openstack
13:04 dprince joined #puppet-openstack
13:08 openstackgerrit Matt Fischer proposed stackforge/puppet-cinder: Don't add a new line if the rbd_user changes  https://review.openstack.org/152384
13:08 mfisch EmilienM: morning
13:12 EmilienM mfisch: o/
13:12 mfisch did something odd change with a library yesterday?
13:12 mfisch I rebased a change and now some tests fail, specifically one that expect an error
13:12 mfisch they're not getting the right error
13:12 mfisch here's one: https://review.openstack.org/#/c/152321/
13:12 Madkiss EmilienM: could you lend me a hand on why the test is failing on me? :(
13:12 Madkiss pretty please
13:13 EmilienM a lot of folks have rspec errors today
13:13 * EmilienM checking
13:13 mfisch here's another one: https://review.openstack.org/#/c/152321/
13:13 mfisch Madkiss: I think a library broke
13:13 Madkiss yeah
13:13 Madkiss https://review.openstack.org/#/c/105245/
13:13 Madkiss it's giving me the finger, too
13:13 EmilienM spredzy: social FYI ^
13:13 sbadia hi
13:13 mfisch to me it was all stuff that wanted something to throw an error
13:14 EmilienM sbadia: hey
13:14 EmilienM sbadia: look the backlog
13:14 sbadia Madkiss: yep, errors are not related to your patch
13:14 mfisch all passed on my box
13:14 sbadia EmilienM: yep, I'm catching this
13:14 Madkiss sbadia: whew. I was already about to hire a shrink.
13:14 sbadia (I currently investigate gems changes)
13:14 sbadia Madkiss: :D
13:15 mfisch Madkiss: we just need a recheck no bug cron job
13:15 Madkiss alrighty
13:16 sbadia EmilienM: I've also this errors in my local env, I'll dig this
13:16 mfisch I'll see if I can diff gem versions between the pass and fail on my oldest changeset
13:16 EmilienM sbadia: it affects all modules, even our composition layer
13:17 sbadia yup, it's rspec-mocks :)
13:17 sbadia published 3day ago (x.6)
13:17 mfisch do we not have stuff pinned like rpsec-mocks?
13:17 EmilienM sbadia: you have a commit?
13:17 sbadia no it's a rspec-core no?
13:18 EmilienM which commit is breaking?
13:18 EmilienM sbadia: yes it is
13:18 spredzy sbadia, rspec-puppet 2.0.0 also came out few days ago
13:18 spredzy and it is being installed during tests
13:19 spredzy sbadia, few days ago == yesterday for rspec-puppet 2.0.0 (https://rubygems.org/gems/rspec-puppet/versions/2.0.0)
13:19 sbadia indeed…
13:19 EmilienM great
13:20 mfisch sbadia EmilienM this is what broke between old and new job
13:20 mfisch mfischer@Matts-MBP-2:~$ diff /tmp/lll /tmp/new
13:20 mfisch 21c21
13:20 mfisch < Using rspec-puppet 1.0.1
13:20 mfisch ---
13:20 mfisch > Installing rspec-puppet 2.0.0
13:20 mfisch based on comparing a job that passed vs a failure
13:21 mfisch rpsec-mocks was the same version on both
13:21 sbadia ok, yep my bad (thois gem is pinned in core), ok ok
13:22 spredzy no CHANGELOG unfortunately on the repo
13:22 sbadia mfisch, spredzy EmilienM, so… we pin to 1.0.1 or we update all specs to 2.x ?
13:23 sbadia spredzy: yep :(
13:23 EmilienM lol
13:23 EmilienM sbadia: I guess the option #2 is huge ?
13:23 sbadia (changes are trivial)
13:23 spredzy sbadia, I'll do it in two step. #1 pin to 1.0.1 so we can keep working
13:23 mfisch I dont know the scope of work but it seems like pinning and then filing a bug to upgdade
13:23 spredzy #2 see what is needed for 2.0.0
13:23 EmilienM mfisch: spredzy: sounds reasonable
13:23 sbadia ok so :) (for the bug, and pin)
13:23 crinkle +1
13:23 sbadia I'll do the patchs ?
13:24 spredzy Creating a card for investigating the needed change for 2.0.0
13:24 sbadia spredzy: :D
13:24 EmilienM sbadia: go ahead and let us know here please so we can review quickly
13:24 EmilienM spredzy: lol
13:24 sbadia EmilienM: ack, let's go
13:24 sbadia mfisch, spredzy : thanks!
13:24 EmilienM sbadia: spredzy: and also, #3 blame the guy who released 2.0.0
13:25 mfisch no thanks to you guys for fixing
13:25 mfisch EmilienM: yay for opensource! ;)
13:25 IvanBerezovskiy joined #puppet-openstack
13:26 EmilienM he/she owes us a beer
13:26 morazi joined #puppet-openstack
13:28 Madkiss how can we retrigger the verfiication tests?
13:29 EmilienM Madkiss: "recheck"  - but wait please
13:29 * Madkiss gets out the number of the shrink again
13:29 mfisch Madkiss: "recheck bug xxxxxx" but it wont help for a bit
13:30 mfisch is there a central place we can specify this pin or does it need to be done per module?
13:30 EmilienM mfisch: openstack-infra I think
13:30 EmilienM sbadia: can you confirm me?
13:31 spredzy trello updated. sbadia you've been asigned the pining card ;)
13:31 EmilienM sbadia: we have to poke o-infra team to have your fix merged today
13:32 spredzy EmilienM, why do we need to poke infra ? rspec is pinned at the Gemfile level, so if we merge it next time it will pull the proper version
13:32 spredzy or am I mistaken ?
13:32 sbadia mfisch, EmilienM yup !
13:33 sbadia in project-config
13:33 sbadia gate-{name}-puppet-unit-{puppet_version}
13:33 EmilienM yep
13:33 spredzy ok thanks for the info
13:33 sbadia (just before bundle part), I just test this workflow, (and fil a bug before)
13:33 EmilienM spredzy: https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/puppet-module-jobs.yaml
13:38 crinkle the gemfile should be where gems are pinned, why would we do this in infra?
13:38 crinkle add a line here for rspec-puppet https://github.com/stackforge/puppet-keystone/blob/master/Gemfile#L5
13:39 EmilienM it's also a solution yeah
13:39 EmilienM sbadia: ^
13:39 crinkle it'll be easier to manage this if we don't have to bother infra for it
13:40 sbadia the main problem is about rspec-core (pinned on 2.14.x)
13:40 sbadia http://paste.debian.net/plain/143742
13:40 sbadia put local gem cache is re-used by bundle
13:41 sbadia I take a look on other infra modules (in order to see if I can pin also rspec-core infra-wide)
13:41 sbadia s/put/but
14:07 nibalizer crinkle: is up on stage at configmgmt camp talking about puppet
14:07 nibalizer and openstack
14:07 * EmilienM sends kudos to crinkle
14:08 EmilienM so to give you an update, we are going to patch all our modules
14:08 EmilienM sbadia: you take puppet-a* to puppet-m included ? and I take the rest
14:08 sbadia \o/ me send some midi-chlorian to crinkle
14:09 sbadia EmilienM: hum, I'll use fungi tips
14:10 sbadia so I can manage all of thems
14:10 dvorak sbadia: what OS are you using designate on?
14:10 sbadia dvorak: yep debian
14:11 dvorak ah, ok, we're on ubuntu, so no packages available from upstream
14:11 sbadia (zigo just packaged and fixed some bugs)
14:11 dvorak trying to decide whether or not to do the packaging, or try to just run it from a virtualenv
14:11 EmilienM sbadia: ok
14:16 markvoelker joined #puppet-openstack
14:25 Madkiss alrighty
14:25 Madkiss can we now retrigger the jenkins tests?
14:27 EmilienM Madkiss: please wait, sbadia is working on it
14:28 richm joined #puppet-openstack
14:29 Madkiss ah, ok
14:30 mschmitt joined #puppet-openstack
14:33 openstackgerrit Sebastien Badia proposed stackforge/puppet-ceilometer: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152529
14:33 openstackgerrit Sebastien Badia proposed stackforge/puppet-ceph: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152530
14:33 openstackgerrit Sebastien Badia proposed stackforge/puppet-cinder: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152531
14:33 openstackgerrit Sebastien Badia proposed stackforge/puppet-designate: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152532
14:33 openstackgerrit Sebastien Badia proposed stackforge/puppet-glance: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152533
14:33 openstackgerrit Sebastien Badia proposed stackforge/puppet-gnocchi: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152534
14:33 openstackgerrit Sebastien Badia proposed stackforge/puppet-heat: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152535
14:33 openstackgerrit Sebastien Badia proposed stackforge/puppet-horizon: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152536
14:34 sbadia spam :)
14:34 openstackgerrit Sebastien Badia proposed stackforge/puppet-ironic: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152537
14:34 openstackgerrit Sebastien Badia proposed stackforge/puppet-keystone: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152538
14:34 openstackgerrit Sebastien Badia proposed stackforge/puppet-manila: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152539
14:34 openstackgerrit Sebastien Badia proposed stackforge/puppet-monasca: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152540
14:34 openstackgerrit Sebastien Badia proposed stackforge/puppet-neutron: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152541
14:34 openstackgerrit Sebastien Badia proposed stackforge/puppet-nova: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152542
14:34 openstackgerrit Sebastien Badia proposed stackforge/puppet-openstacklib: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152543
14:34 openstackgerrit Sebastien Badia proposed stackforge/puppet-sahara: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152544
14:34 openstackgerrit Sebastien Badia proposed stackforge/puppet-swift: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152545
14:34 openstackgerrit Sebastien Badia proposed stackforge/puppet-tempest: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152546
14:35 openstackgerrit Sebastien Badia proposed stackforge/puppet-trove: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152547
14:35 openstackgerrit Sebastien Badia proposed stackforge/puppet-tuskar: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152548
14:35 openstackgerrit Sebastien Badia proposed stackforge/puppet-vswitch: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152549
14:35 EmilienM boom
14:36 sbadia I'll manage the step #2 (https://github.com/rodjek/rspec-puppet/commit/8459e14807977244c00bdbcf190062c529b63474)
14:37 sbadia spredzy: thanks for the card!
14:37 Madkiss sbadia: thanks a bunch man
14:38 sbadia no problems :)
14:43 prad joined #puppet-openstack
14:43 openstackgerrit Sebastien Badia proposed stackforge/puppet-gnocchi: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152534
14:44 rcallawa joined #puppet-openstack
14:49 dalgaaf joined #puppet-openstack
14:53 EmilienM I think we have to backport everything
14:53 EmilienM sbadia: ^
14:53 EmilienM to stable/juno, even stable/icehouse
14:54 EmilienM mgagne: you around? we need core-review on https://review.openstack.org/#/q/status:open+branch:master+topic:bug/1417604,n,z
14:56 EmilienM sbadia: quick comment on https://review.openstack.org/#/c/152534/
14:57 social EmilienM: ok the tests fail because of my patch but I have no idea how :/
15:01 markvoelker joined #puppet-openstack
15:02 rcallawa_ joined #puppet-openstack
15:06 EmilienM if crinkle is around, maybe we can have it merged today
15:07 sbadia EmilienM: you are right on gnocchi, I split the commits
15:07 sbadia (re)
15:09 sbadia EmilienM: yep for backports (on juno), for icehouse we already pin to 1.0.1
15:09 EmilienM ok great
15:09 EmilienM I'll do the backports
15:09 sbadia so no need to backport this to icehouse
15:09 sbadia EmilienM: ack, thx!
15:10 sbadia EmilienM: or spredzy ?
15:10 sbadia in order to have two core to validate :)
15:11 EmilienM yeah
15:14 mdorman joined #puppet-openstack
15:14 sbadia EmilienM: héhé, for gnocchi comment, I can't separate the two changes (rspec-puppet change need puppet-lint happy (for mysql params removed), and mysql_params (lint) need rspec-puppet 1.0.1) :)
15:14 EmilienM ok
15:14 EmilienM +2 then
15:15 sbadia thx
15:23 richm with keystone v3, using JSON in keystone resource titles - how to tell puppet that keystone_user { '{"name":"admin", "domain":"services"}'} and keystone_user { '{"name":    "admin",    "domain":   "services"}'} _are the same_?
15:23 richm are there any puppet experts here that know how to make puppet do comparisons using the _normalized_ string?
15:26 larsks richm: why are we passing JSON to a puppet resource? Why not something like 'domain/name' or 'name@domain' or 'name:::domain' or whatever?
15:26 social richm: make title - name parameter just name and add username and domain parameters
15:26 richm larsks: because in keystone both "/" and "@" are valid strings in keystone user and project names
15:27 ericpeterson joined #puppet-openstack
15:27 social richm: backwards compatibility would be username undefined use :name otherwise use username and domain
15:27 larsks richm: Right. So use a multi-character delimiter like "::" and document that we won't let you use that in a keystone name.
15:27 richm social: ok - then how does puppet know the difference between keystone_user {"admin": domain => 'foo'} and keystone_user {"admin": domain => 'bar'}
15:27 larsks That seems much simpler (and more readable!) than putting JSON strings into resource titles.
15:28 richm larsks: how can we guarantee that keystone won't have a user with "::"?
15:28 social richm: taking username and domain allows you to just use whatever as name of the resource
15:28 larsks richm: Like I said, we fix that via documentation.  We don't care what users *exist* in keystone, only those created via Puppet resources, right?
15:28 richm larsks: wrong
15:28 larsks richm: ?
15:28 richm larsks: because we want to support things like read-only LDAP identity backends
15:29 richm social: ?
15:29 larsks Okay.  I'm not sure I follow there, but I still that trying to use JSON in resource titles seems craaaaaazy, man.
15:31 richm larsks: what if I want to use an admin user called "foo::bar" already defined in LDAP?  keystone_user { 'foo::bar' }
15:32 richm keystone_user {} doesn't necessarily _create_ the user, it just ensures that it exists (somehow) with all of the given attributes
15:33 larsks richm: I understand what you are saying, but I still think that fixing that through documentation ("this module does not support user names containing ":SEPERATOR:"" is the way to go rather than trying to using JSON in the title.
15:34 richm ok
15:35 social richm: I would guess keystone_user { "whatever": username => 'admin', domain => 'foo'} ?
15:36 richm social: I see - so "whatever" will _not_ be the username - it will be a unique string generated by whoever writes the puppet code
15:36 social exactly
15:37 social I'm not sure if you'll be able to make it ensurable :/
15:37 richm that will work too, but it breaks the convention (and backwards compatibility - and possibly other things that you mention) that the resource title is the name of the resource
15:38 social richm: if username is not defined use title
15:38 social that's backwards compatibility
15:38 richm yes
15:38 vinsh joined #puppet-openstack
15:39 richm so either 1) use json 2) use an "odd" delimiter string 3) make it the responsibility of the puppet code writer to use unique titles with username parameters
15:39 vinsh_ joined #puppet-openstack
15:40 social richm: my few cents on 3
15:40 richm my preference is 2) assuming we can come up with an "odd" enough string
15:41 social admin-31337-foo? :)
15:41 richm heh
15:42 openstackgerrit Merged stackforge/puppet-sahara: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152544
15:44 openstackgerrit Merged stackforge/puppet-vswitch: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152549
15:45 openstackgerrit Merged stackforge/puppet-trove: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152547
15:46 michchap_ EmilienM: I don't have merge rights on a few of them
15:46 EmilienM me neither
15:46 michchap_ ceph/monasca
15:46 openstackgerrit Merged stackforge/puppet-tuskar: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152548
15:47 openstackgerrit Merged stackforge/puppet-swift: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152545
15:47 michchap_ the rest are +A
15:47 openstackgerrit Merged stackforge/puppet-nova: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152542
15:48 openstackgerrit Merged stackforge/puppet-designate: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152532
15:48 openstackgerrit Merged stackforge/puppet-ironic: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152537
15:48 sbadia yep me too, but I preferred to commit the fix also in these projects
15:49 sbadia michchap_, EmilienM thx!
15:50 openstackgerrit Merged stackforge/puppet-glance: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152533
15:50 openstackgerrit Merged stackforge/puppet-horizon: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152536
15:50 openstackgerrit Merged stackforge/puppet-openstacklib: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152543
15:50 openstackgerrit Merged stackforge/puppet-manila: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152539
15:50 openstackgerrit Merged stackforge/puppet-keystone: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152538
15:51 openstackgerrit Merged stackforge/puppet-heat: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152535
15:51 EmilienM michchap_: thx man, good night
15:51 openstackgerrit Merged stackforge/puppet-cinder: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152531
15:51 michchap_ EmilienM: I can't sleep. Might be up for a bit longer
15:51 openstackgerrit Merged stackforge/puppet-ceilometer: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152529
15:52 openstackgerrit Merged stackforge/puppet-neutron: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152541
16:03 IvanBerezovskiy left #puppet-openstack
16:05 rcallawa joined #puppet-openstack
16:08 social EmilienM: actually the rspec patch fixed the issue for me ^_^
16:09 EmilienM great
16:14 EmilienM no need t backport it on puppet-nova
16:15 EmilienM sbadia: michchap_ : backports done
16:18 sbadia EmilienM: thx
16:18 sbadia ( https://review.openstack.org/#/q/owner:"Emilien+Macchi"+branch:stable/juno+status:open,n,z )
16:26 openstackgerrit Lukas Bezdicka proposed stackforge/puppet-nova: Add support for nova Pci passthrough  https://review.openstack.org/145787
16:31 openstackgerrit Merged stackforge/puppet-ceph: spec: pin rspec-puppet to 1.0.1  https://review.openstack.org/152530
16:32 hkumar joined #puppet-openstack
16:32 openstackgerrit Merged stackforge/puppet-nova: Support for heal_instance_info_cache_interval  https://review.openstack.org/105245
16:39 mwoodson joined #puppet-openstack
16:44 ToMiles joined #puppet-openstack
16:49 mdorman merge day  \o/
16:52 aimon joined #puppet-openstack
16:56 openstackgerrit Richard Megginson proposed stackforge/puppet-openstack-specs: Support Keystone v3 API in openstack puppet modules  https://review.openstack.org/150108
16:57 richm ok - this uses option 2 - using "::" as a delimiter - please review ^^^
17:02 rmoe joined #puppet-openstack
17:14 openstackgerrit David Moreau Simard proposed stackforge/puppet-ceph: define ceph::rgw::keystone.  https://review.openstack.org/117422
17:16 dmsimard dachary: Any chance to see https://review.openstack.org/#/c/101751/ happening ?
17:17 dachary dmsimard: thanks for the reminder :-) I abandonned for lack of time.
17:18 dmsimard dachary: Would help a lot, especially considering we're lacking CentOS7 and Ubuntu 14 tests
17:18 dmsimard Actually, I don't know, well.. it could help until we move to beaker
17:19 dachary dmsimard: I'm sure it would speed up things
17:27 dprince joined #puppet-openstack
17:29 openstackgerrit Sebastien Badia proposed stackforge/puppet-sahara: Migrate postgresql backend to use openstacklib::db::postgresql  https://review.openstack.org/145511
17:29 openstackgerrit Sebastien Badia proposed stackforge/puppet-sahara: fixtures: Unpin puppetlabs-mysql module  https://review.openstack.org/145510
17:29 openstackgerrit Sebastien Badia proposed stackforge/puppet-sahara: Rewrite puppet-sahara from scratch  https://review.openstack.org/145509
17:29 openstackgerrit Sebastien Badia proposed stackforge/puppet-sahara: policy.json: Allow one to manage them from the puppet module  https://review.openstack.org/152470
17:45 mfisch sbadia: did that new module change land?
17:46 mfisch looks like it... wonder why I dont see a rebase option
17:46 openstackgerrit Matt Fischer proposed stackforge/puppet-heat: Rework delegated roles  https://review.openstack.org/146734
17:47 openstackgerrit Matt Fischer proposed stackforge/puppet-glance: Support identity_uri  https://review.openstack.org/152321
17:47 mfisch ah force reload does the trick
17:47 openstackgerrit Matt Fischer proposed stackforge/puppet-cinder: Don't add a new line if the rbd_user changes  https://review.openstack.org/152384
17:48 sbadia (re)
17:48 sbadia hum, yep, I tried the rebase button of gerrit, but the big patch (rewrite) was conflicting with gemfile changes
17:49 sbadia so, I had to update related changes also :(
17:54 mfisch yay all 3 of my broken commits passed
17:54 sbadia \o/
17:55 * sbadia brb
18:02 hkumar joined #puppet-openstack
18:05 openstackgerrit Ryan Bak proposed stackforge/puppet-monasca: Makes agent version configurable via param  https://review.openstack.org/152639
18:07 EmilienM sbadia: https://review.openstack.org/#/c/152595/ is actually wrong
18:11 EmilienM puppet-swift is not affected
18:11 EmilienM in stable/juno
18:25 tfz joined #puppet-openstack
18:27 xarses joined #puppet-openstack
18:36 badiane_ka joined #puppet-openstack
18:49 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Adding base infrastructure for beaker CI tests  https://review.openstack.org/151970
18:59 aimon joined #puppet-openstack
19:02 openstackgerrit Merged stackforge/puppet-monasca: Makes agent version configurable via param  https://review.openstack.org/152639
19:07 mschmitt joined #puppet-openstack
19:07 ibravo joined #puppet-openstack
19:11 morazi joined #puppet-openstack
19:31 mmagr joined #puppet-openstack
19:34 stamak joined #puppet-openstack
19:51 jpena joined #puppet-openstack
19:51 jpena_ joined #puppet-openstack
20:10 imcsk8 joined #puppet-openstack
20:11 cdent joined #puppet-openstack
20:40 aimon joined #puppet-openstack
20:45 ToMiles_ joined #puppet-openstack
20:51 openstackgerrit joined #puppet-openstack
20:54 anteaya joined #puppet-openstack
21:16 xarses joined #puppet-openstack
21:41 dkaigarodsev joined #puppet-openstack
21:42 aarefiev joined #puppet-openstack
22:23 mfisch mgagne: I updated the review for tagging the cinder packages: https://review.openstack.org/#/c/151838/
22:23 mgagne mfisch: glance*
22:23 mfisch yeah one of those disk storagey things ;)
22:38 EmilienM mgagne: I'm cleaning very old patches
22:39 mgagne the one from april?
22:39 mgagne :D
22:39 EmilienM I set them as abandoned and put a great message
22:39 EmilienM mgagne: you think it's bad
22:39 EmilienM ?
22:39 mgagne just hilarious =)
22:39 EmilienM mgagne: why?
22:39 EmilienM mgagne: which one for example?
22:40 mgagne it got abandoned, cannot find the link
22:41 EmilienM mgagne: btw, would you mind to review https://review.openstack.org/#/q/owner:"Emilien+Macchi"+branch:stable/juno+status:open,n,z ?
22:48 openstackgerrit David Gurtner proposed stackforge/puppet-ceph: Adding base infrastructure for beaker CI tests  https://review.openstack.org/151970
23:09 gildub joined #puppet-openstack
23:22 simonmcc joined #puppet-openstack
23:30 ToMiles joined #puppet-openstack
23:54 openstackgerrit Bradley Klein proposed stackforge/puppet-monasca: Add support for influxdb 0.9.0  https://review.openstack.org/152733
23:59 rcallawa joined #puppet-openstack

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