Perl 6 - the future is here, just unevenly distributed

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

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

All times shown according to UTC.

Time Nick Message
00:01 zhangjn joined #puppet-openstack
00:01 gfidente joined #puppet-openstack
00:08 ntpttr joined #puppet-openstack
00:29 aimon joined #puppet-openstack
01:00 markvoelker joined #puppet-openstack
01:01 zhangjn joined #puppet-openstack
01:29 gildub joined #puppet-openstack
01:32 aimon joined #puppet-openstack
01:36 ntpttr joined #puppet-openstack
01:38 jerrygb joined #puppet-openstack
01:41 ntpttr left #puppet-openstack
01:54 rahulait joined #puppet-openstack
01:56 zacdev joined #puppet-openstack
02:01 markvoelker joined #puppet-openstack
02:13 sanjayu joined #puppet-openstack
02:23 gildub_ joined #puppet-openstack
02:31 ducttape_ joined #puppet-openstack
02:36 nlahouti joined #puppet-openstack
02:38 nlahouti joined #puppet-openstack
02:45 ducttape_ joined #puppet-openstack
03:00 rahulait joined #puppet-openstack
03:02 markvoelker joined #puppet-openstack
03:51 gildub joined #puppet-openstack
03:51 nlahouti joined #puppet-openstack
03:55 rahulait left #puppet-openstack
03:56 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Add composite namevar for tenant, user, user_role.  https://review.openstack.org/226919
05:03 markvoelker joined #puppet-openstack
05:05 sanjayu joined #puppet-openstack
05:19 guessi joined #puppet-openstack
05:23 guessi left #puppet-openstack
05:25 mwoodson joined #puppet-openstack
05:36 guessi joined #puppet-openstack
05:36 guessi left #puppet-openstack
05:37 nlahouti joined #puppet-openstack
06:15 khushbu_ joined #puppet-openstack
06:36 nlahouti joined #puppet-openstack
06:50 markvoelker joined #puppet-openstack
07:04 rahulait joined #puppet-openstack
07:11 fedexo joined #puppet-openstack
07:15 jerrygb joined #puppet-openstack
07:45 dgurtner joined #puppet-openstack
07:45 dgurtner joined #puppet-openstack
07:46 sleinen-AS559 joined #puppet-openstack
07:50 xarses joined #puppet-openstack
07:55 zigo joined #puppet-openstack
07:58 xarses joined #puppet-openstack
08:02 paramite joined #puppet-openstack
08:40 jpena joined #puppet-openstack
08:52 markvoelker joined #puppet-openstack
09:00 jistr joined #puppet-openstack
09:02 dgurtner joined #puppet-openstack
09:16 jerrygb joined #puppet-openstack
09:22 karimb joined #puppet-openstack
09:22 jtomasek joined #puppet-openstack
09:36 dgurtner joined #puppet-openstack
09:45 derekh joined #puppet-openstack
09:52 jtomasek joined #puppet-openstack
09:53 markvoelker joined #puppet-openstack
10:00 IvanBerezovskiy joined #puppet-openstack
10:03 samuelBartel joined #puppet-openstack
10:06 openstackgerrit Mark Chappell proposed openstack/puppet-tripleo: Add hiera calls to allow management of ports and protocols  https://review.openstack.org/240208
10:13 zhangjn joined #puppet-openstack
10:31 openstackgerrit Maksym Yatsenko proposed openstack/puppet-nova: [WIP]  https://review.openstack.org/233011
10:38 zhangjn joined #puppet-openstack
10:40 zhangjn joined #puppet-openstack
11:09 markvoelker joined #puppet-openstack
11:19 khushbu_ joined #puppet-openstack
11:19 openstackgerrit Maksym Yatsenko proposed openstack/puppet-nova: [WIP]  https://review.openstack.org/233011
11:32 khushbu_ joined #puppet-openstack
11:38 skolekonov joined #puppet-openstack
11:41 khushbu_ joined #puppet-openstack
11:49 khushbu_ joined #puppet-openstack
11:54 khushbu joined #puppet-openstack
11:58 khushbu joined #puppet-openstack
12:03 jerrygb joined #puppet-openstack
12:03 xarses joined #puppet-openstack
12:09 markvoelker joined #puppet-openstack
12:09 dgurtner joined #puppet-openstack
12:12 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Add composite namevar for tenant, user, user_role.  https://review.openstack.org/226919
12:26 chem joined #puppet-openstack
12:29 chem` joined #puppet-openstack
12:34 zhangjn joined #puppet-openstack
12:36 EmilienM joined #puppet-openstack
12:43 ducttape_ joined #puppet-openstack
12:43 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Client and dashboard class implementation  https://review.openstack.org/218333
12:47 xarses joined #puppet-openstack
12:47 khushbu_ joined #puppet-openstack
12:53 dgurtner joined #puppet-openstack
12:53 khushbu_ joined #puppet-openstack
12:56 dprince joined #puppet-openstack
13:01 jerrygb joined #puppet-openstack
13:05 khushbu_ joined #puppet-openstack
13:16 EmilienM good morning
13:17 samuelBartel joined #puppet-openstack
13:19 asilenkov joined #puppet-openstack
13:20 degorenko joined #puppet-openstack
13:22 abramley_ joined #puppet-openstack
13:22 vinsh heyooo
13:22 mkarpin1 Hey!
13:22 LimorStotland joined #puppet-openstack
13:22 gildub joined #puppet-openstack
13:23 mkarpin joined #puppet-openstack
13:29 richm joined #puppet-openstack
13:40 sleinen-AS559 joined #puppet-openstack
13:43 LimorStotland left #puppet-openstack
13:55 khushbu_ joined #puppet-openstack
13:59 sleinen-AS559 joined #puppet-openstack
14:02 EmilienM mwhahaha: everyone is trying to use $::os_service_default
14:02 EmilienM I sent an email to stop it
14:03 EmilienM we should only use puppet-cinder for now
14:04 EmilienM mwhahaha: do you know which bug spredzy was talking about?
14:12 markvoelker joined #puppet-openstack
14:15 morazi joined #puppet-openstack
14:15 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-cinder: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/241214
14:16 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-cinder: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/241214
14:16 degorenko EmilienM, +1 :)
14:22 openstackgerrit Thomas Goirand proposed openstack/puppet-horizon: Do not run collectstatic or compress in Debian  https://review.openstack.org/241218
14:22 EmilienM degorenko: I replied on https://review.openstack.org/238623
14:22 vinsh_ joined #puppet-openstack
14:22 EmilienM I'm not sure we have an alternative
14:23 degorenko EmilienM, erm, it is for one project. Patch dependency?
14:23 EmilienM degorenko: yes but the first one will fail CI and will never merge
14:23 degorenko EmilienM, oh, crap
14:23 EmilienM the second one will pass, ok but what about the first? :)
14:23 degorenko EmilienM, you're right, sorry
14:23 degorenko my bad
14:23 EmilienM I'm not sure it's a big deal
14:24 degorenko yeah, i agree
14:24 nlahouti joined #puppet-openstack
14:27 nlahouti1 joined #puppet-openstack
14:32 openstackgerrit Thomas Goirand proposed openstack/puppet-horizon: Do not run collectstatic or compress in Debian  https://review.openstack.org/241218
14:32 zigo EmilienM: Could you give me comments on my patch just above?
14:32 zigo That's my first puppet patch, so it's probably very bad... :P
14:34 mwhahaha it's not so bad :D
14:34 openstackgerrit Thomas Goirand proposed openstack/puppet-horizon: Do not run collectstatic or compress in Debian  https://review.openstack.org/241218
14:35 zigo I'm gessing I should have written "Debian" and not "debian", as facter returns "Ubuntu" (with upper case U), so here comes iteration 3 of the patch...
14:35 degorenko zigo, what is package_type?
14:35 degorenko i mean, where patch for this?
14:36 zigo A variable set if you install this package: http://anonscm.debian.org/cgit/openstack​/puppet-openstack-debian-fact.git/tree/
14:36 zigo :)
14:40 delattec joined #puppet-openstack
14:41 EmilienM oh nice
14:41 degorenko zigo, yea, but also we are discussed about adding patch to openstacklib
14:41 degorenko which will be override operating system if we need
14:41 EmilienM mwhahaha: any thoughts about os_service_default?
14:41 degorenko will*
14:42 degorenko and you don't need any additional if's in params
14:42 mwhahaha EmilienM: i think it's fine provided we properly check the variables being used have a sane default or if we have previously set something within puppet that we do not convert those (re: log dirs, log levels, etc)
14:43 mwhahaha EmilienM: I think the logging problem was that we converted it to os_service_default which the code interprets as no logging, so we shouldn't have converted those.
14:43 EmilienM mwhahaha: I don't see where is the bug, since we still have log files in our jobs
14:43 mwhahaha EmilienM: in my patches for the cinder backends I did not convert the items where we had previously specified something that was different than the default
14:46 EmilienM mwhahaha: do you think we could go ahead with all other openstack modules?
14:47 EmilienM mwhahaha: do you have an example or link about the logging problem?
14:55 mwhahaha yea i haven't seen it either i'll take some time today to poke at it
14:55 mwhahaha yes we just need to be diligent to make sure the settings don't incur any regressions
14:55 mwhahaha i scrolled through my history and spredzy had mentioned two things
14:55 mwhahaha spredzy: EmilienM: gchamoul mwhahaha the issue are those two lines https://github.com/openstack/puppet-cinder​/blob/master/manifests/logging.pp#L96-L97
14:55 mwhahaha spredzy: Default in oslo.log is set to None, so by default it doesn't log anything in any log file, while it uses to be /var/log/cinder.log, so it breaks backward compat
14:55 mwhahaha so we just need to revert those two parameters and when we convert things we need to make sure that what we're converting is currently set to undef or the actual default value
14:55 mwhahaha if it's not the actual default value we'll want to come up with some sort of migration strategy or we just leave it
14:55 dfisher joined #puppet-openstack
15:03 skolekonov EmilienM, hi, are you going to hold today's meeting?
15:03 EmilienM skolekonov: why not?
15:03 skolekonov I see nobody at #openstack-meeting-4
15:04 skolekonov Is it later?
15:04 EmilienM I do
15:04 EmilienM it's in one hour
15:04 EmilienM I think?
15:05 skolekonov Hm, I thought it's 15-00 UTC now
15:05 IvanBerezovskiy me2
15:07 vinsh joined #puppet-openstack
15:11 skolekonov hobana.freenode.net :Tuesday November 3 2015 -- 15:10:48 +00:00
15:12 bogdando joined #puppet-openstack
15:13 openstackgerrit Maksym Yatsenko proposed openstack/puppet-nova: [WIP]  https://review.openstack.org/233011
15:14 EmilienM I think you're right, damn
15:15 EmilienM clayton, crinkle, mfisch, degorenko: around?
15:15 mwhahaha silly clocks
15:16 EmilienM skolekonov, mwhahaha : let's do it
15:25 skolekonov EmilienM, please look at this patch https://review.openstack.org/#/c/235356/, are you ok with it?
15:27 EmilienM skolekonov: I'm reviewing it
15:28 markvoelker joined #puppet-openstack
15:28 vinsh_ joined #puppet-openstack
15:28 IvanBerezovskiy EmilienM: why did you miss 2 last topics? :-(
15:28 skolekonov ok, thanks
15:28 EmilienM IvanBerezovskiy: ?
15:28 EmilienM IvanBerezovskiy: the agenda was on https://etherpad.openstack.org/p/pupp​et-openstack-weekly-meeting-20151103
15:28 EmilienM I only see 2 topics
15:28 EmilienM for open discussion, it's up to people to come up and talk
15:29 IvanBerezovskiy I've just waited my turn to start discussion about pymysql
15:29 openstackgerrit Alexey Deryugin proposed openstack/puppet-murano: Add basic acceptance test  https://review.openstack.org/233591
15:29 IvanBerezovskiy https://review.openstack.org/#/c/241214/
15:30 EmilienM IvanBerezovskiy: don't worry, we can talk here if you want
15:30 EmilienM it's in my review list for today
15:31 IvanBerezovskiy EmilienM: thx. I just wanna know you thoughts about it. If you are ok, I'll do this for all modules
15:31 EmilienM IvanBerezovskiy: ok, looking now
15:32 ducttape_ joined #puppet-openstack
15:34 openstackgerrit Nate Potter proposed openstack/puppet-neutron: Allow customizations of db sync command line  https://review.openstack.org/241012
15:36 EmilienM skolekonov: I reviewed https://review.openstack.org/#/c/235356/ - the patch is almost good for me, please read comments and tell me if that's fine for you
15:37 sleinen-AS559 joined #puppet-openstack
15:38 skolekonov EmilienM, if we leave 'password' value without default value, we will break backward compatibility, won't we? Because somebody can still want to use only deprecated nova_admin_password and related parameters
15:38 EmilienM IvanBerezovskiy: why no package on centos? is it MySQL-python-1.2.3-11.el7.x86_64 ?
15:39 EmilienM ah
15:39 EmilienM I see
15:39 EmilienM nice catch
15:39 EmilienM skolekonov: just the test case then
15:40 EmilienM IvanBerezovskiy: +2, lgtm
15:40 IvanBerezovskiy EmilienM: no, it's not. latest version of pymysql is 0.6.6 and as I didn't find this package for centos
15:41 IvanBerezovskiy thx again
15:41 skolekonov EmilienM, I modified the existing test, now it's called 'should configure neutron server with deprecated parameters'. Should I add an additional one?
15:41 EmilienM IvanBerezovskiy: oh wait
15:41 EmilienM IvanBerezovskiy: can you patch acceptance? :)
15:41 skolekonov EmilienM, and added new tests for new parameters. Diff looks a bit confusing I guess
15:42 IvanBerezovskiy EmilienM: hm, I guess I can, but I don't exactly know where to patch it :)
15:43 EmilienM IvanBerezovskiy: in spec/acceptance/
15:43 IvanBerezovskiy looks like here https://github.com/openstack/puppet-cinder/blob/​master/spec/acceptance/basic_cinder_spec.rb#L100
15:43 EmilienM IvanBerezovskiy: yes, make sure we're using the new string
15:43 IvanBerezovskiy ok, will do
15:45 sergmelikyan joined #puppet-openstack
15:46 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-cinder: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/241214
15:46 IvanBerezovskiy let's see how it will work on centos
15:46 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Evaluator service  https://review.openstack.org/237662
15:47 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Client class  https://review.openstack.org/237706
15:47 openstackgerrit Emilien Macchi proposed openstack/puppet-aodh: Implement Notifier service  https://review.openstack.org/237708
15:49 rahulait joined #puppet-openstack
15:53 openstackgerrit Nate Potter proposed openstack/puppet-glance: Allow customization of db sync command line  https://review.openstack.org/241003
15:56 ntpttr joined #puppet-openstack
16:00 mfisch meeting?
16:01 mfisch oh damnit time changes!
16:01 vinsh_ oh yeah!
16:01 chem joined #puppet-openstack
16:01 mfisch sorry EmilienM
16:01 mfisch damn you benjamin franklin!
16:01 mfisch and your terrible ideas
16:01 jistr joined #puppet-openstack
16:01 vinsh_ I'm sure he is still eating his morning crepes
16:02 vinsh_ le crepes
16:02 EmilienM lol
16:02 EmilienM we did the meeting
16:02 mfisch note to PTL, remind stupid americans about time changes!!!
16:02 EmilienM I missed it too
16:02 EmilienM I'm stupid :)
16:02 EmilienM hey mfisch, can you look https://review.openstack.org/#/q/to​pic:puppet/dropfixtures+status:open,n,z also please?
16:04 angdraug joined #puppet-openstack
16:05 mfisch oui mon ami
16:05 EmilienM mfisch: petit? :)
16:05 mfisch petit chu
16:18 fedexo joined #puppet-openstack
16:21 _ody yaaaawwn...
16:22 _ody bummer slept through the meeting.
16:22 IvanBerezovskiy joined #puppet-openstack
16:23 openstackgerrit Nate Potter proposed openstack/puppet-ceilometer: Allow customization of db sync command line  https://review.openstack.org/241276
16:24 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: DNM - CI test  https://review.openstack.org/241277
16:24 social mfisch: i think the isdue isnt people but daylight saving. makes no sense
16:25 mfisch EmilienM: this patch says to rely on git to pull stuff with r10k?
16:27 EmilienM mfisch: yes
16:27 EmilienM we merged it in master a few time ago
16:27 EmilienM mfisch: but we need to backport it
16:27 EmilienM the stable branch CI is currently broken
16:31 jistr does someone have bandwidth to give a second core review on this? https://review.openstack.org/#/c/239680/
16:32 jistr it should fix undefined ordering on the resources in heat::keystone::domain class and make it defined
16:32 EmilienM jistr: do you think we can merge https://review.openstack.org/237612 ? ooo job is red
16:32 jistr the red tripleo job shouldn't be a big issue as we have a patch ready to make it work https://review.openstack.org/#/c/239707/
16:33 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Remove occurrence of @@default_domain_id.  https://review.openstack.org/241284
16:33 skolekonov joined #puppet-openstack
16:33 jistr EmilienM: ^^ that was for the patch i posted, not for the one you posted
16:33 mfisch EmilienM: dumb questions: where's the file that r10k uses to pull from?
16:33 jistr EmilienM: i'll look at the other one
16:34 EmilienM mfisch: the Puppetfile in puppet-openstack-integration
16:34 EmilienM mfisch: those patches are backports, master already has this code BTW
16:34 EmilienM mwhahaha: I +A https://review.openstack.org/236004 so we can make progress on this
16:35 mwhahaha ok
16:35 mwhahaha i'll take a look at the comments left on the backend review today
16:37 jistr EmilienM: the tripleo jobs should be mostly stable right now. I see from the CI log that the error on that heat patch is Error: Could not find resource 'Anchor[heat::db::begin]' for relationship on 'Class[Heat::Db::Mysql]'
16:38 jistr doesn't seem related to the patch itself though... i'll trigger a recheck and we'll see
16:38 EmilienM jistr: mhhk
16:39 jistr a lot of green lately http://tripleo.org/cistatus.html
16:41 mfisch EmilienM: ah ok
16:42 jistr EmilienM: hmm that puppet-heat patch says Parent(s): 3b9cdd01bcccee0f24edfcbf1f50fa81d0cd8352Merge "Prepare 6.1.0 release."
16:42 openstackgerrit Nate Potter proposed openstack/puppet-sahara: Allow customization of db sync command line  https://review.openstack.org/241290
16:43 jistr that might be a reason why it failed too... i think we had a pin of puppet-heat because of a different breakage recently
16:43 jistr maybe rebasing that change to the latest master could make it pass
16:44 nlahouti joined #puppet-openstack
16:44 jistr can this be done with the "rebase change" button in gerrit?
16:44 * jistr has never used it yet
16:44 mfisch EmilienM: why is the horizon one of these reverting?
16:45 EmilienM mfisch: have you looked at the commit msg?
16:45 markvoelker joined #puppet-openstack
16:45 mfisch not yet just saw it on the main list
16:45 mfisch getting there
16:46 EmilienM mfisch: let me look
16:46 EmilienM I think denis did the patch
16:46 mfisch ah I see its a redo of a CP
16:46 EmilienM mfisch: so the initial patch was wrong, denis did a mistake in the backprt
16:47 EmilienM mfisch: when he backported my patch, he did an error in rebase
16:47 EmilienM I want to revert the patch and re-do the backport
16:47 EmilienM so it's cleaner
16:47 chem joined #puppet-openstack
16:47 openstackgerrit Merged openstack/puppet-cinder: add kombu_reconnect_delay parameter to cinder  https://review.openstack.org/239907
16:48 openstackgerrit Nate Potter proposed openstack/puppet-cinder: Allow customization of db sync command line  https://review.openstack.org/240963
16:48 _skolekonov EmilienM, I've answered to your comment in https://review.openstack.org/#/c/235356, please see if it's what you mean
16:49 openstackgerrit Nate Potter proposed openstack/puppet-nova: Allow customization of db sync command line  https://review.openstack.org/241009
16:50 EmilienM _skolekonov: I'm dumb
16:50 EmilienM _skolekonov: +a
16:50 openstackgerrit Nate Potter proposed openstack/puppet-ceilometer: Allow customization of db sync command line  https://review.openstack.org/241276
16:51 _skolekonov EmilienM, :) thank you
16:51 EmilienM jistr: I would monitor https://review.openstack.org/#/c/235356/ and see if tripleo survives with it
16:51 EmilienM _skolekonov: please take care of updating puppet-openstack-integration. We don't like warnings
16:51 openstackgerrit Merged openstack/puppet-designate: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/238627
16:51 _skolekonov EmilienM, I will
16:52 cwolferh joined #puppet-openstack
16:53 openstackgerrit Merged openstack/puppet-cinder: Move default values to $::os_service_default  https://review.openstack.org/236004
16:53 IvanBerezovskiy EmilienM: do you know folks who are working on packaging for cbs.centos.org ? I wanna ask them to build python-pymysql. for now I've found it only in fedora http://pkgs.fedoraproject.org/​cgit/python-PyMySQL.git/tree/
16:54 EmilienM IvanBerezovskiy: go on #rdo and ask
16:54 EmilienM ping number80 or apevec
16:54 IvanBerezovskiy EmilienM: thx!
16:58 nlahouti joined #puppet-openstack
16:59 jistr EmilienM: well... :D i'm hoping for it
16:59 openstackgerrit Merged openstack/puppet-glance: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/238616
17:00 nlahouti joined #puppet-openstack
17:02 openstackgerrit Merged openstack/puppet-neutron: Deprecate nova_admin_ options  https://review.openstack.org/235356
17:03 EmilienM jistr: yeah.. me too
17:03 myatsenko EmilienM: hi Emilien! can you review it again https://review.openstack.org/#/c/233339/ and set +1 for Workflow to merge it.
17:04 EmilienM myatsenko: so its only for nova network in puppet-nova?
17:05 myatsenko EmilienM:  yes, its needed for nova  , for this one: https://review.openstack.org/#/c/233011/
17:05 EmilienM myatsenko: why not patching puppet-nova then?
17:05 EmilienM the Gem is not used/useful for puppet-openstack-integration AFIK
17:08 openstackgerrit Nate Potter proposed openstack/puppet-heat: Allow customization of db sync command line  https://review.openstack.org/241299
17:13 chem richm: EmilienM should be good to be merged, but lacks another core https://review.openstack.org/#/c/219289/
17:13 myatsenko EmilienM:  yeh, to pass tests I updated Gemfile in openstack/puppet-nova patch , but I was told that correct way  - > add this new  requerement to openstack/puppet-openstack-integration project.  Thus this patch was created https://review.openstack.org/#/c/233339/,  but if correct way to solve this problem - add changes only to Gemfile of puppet-nova project, I will abondone patch for "openstack/puppet-openstack-integration"
17:14 karimb hello buddies, shouldnt the neutron::agents::ml2::ovs class take care of creating br-tun bridge ?
17:14 karimb provided i m setting enable_tunneling => true
17:14 clayton I put the time for the meeting on my calendar as UTC, but I had a doctor's appointment at the same time
17:14 EmilienM br-tun is created by neutron AFIK
17:14 EmilienM clayton: np, hope all is fine
17:15 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Add composite namevar for tenant, user, user_role.  https://review.openstack.org/226919
17:15 ducttape_ joined #puppet-openstack
17:19 openstackgerrit Merged openstack/puppet-openstack-integration: gemfile: Added netaddr package  https://review.openstack.org/233339
17:20 karimb EmilienM what do you mean by neutron, neutron class ?
17:20 EmilienM karimb: no, neutron ovs agent already take care of this
17:20 EmilienM not puppet
17:20 EmilienM neutron
17:20 karimb ah ok
17:21 karimb not happening here
17:23 _ody I had this issue when I was setting things up from scrath a couple months ago using a new composite layer...sorry really can't remember
17:23 _ody The two major problems I had was trying to use the master branch on kilo just doesn't work so I rolled back to 6.x and iptables.
17:24 _ody OpenStack doesn't add the iptables rules for gre or vxlan and neither do the puppet modules.
17:24 _ody Not a problem in our production deployment since we use midonet.
17:25 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Add composite namevar for tenant, user, user_role.  https://review.openstack.org/226919
17:27 vinsh joined #puppet-openstack
17:29 tdb I'm having an issue with the puppet-ceph module. Mon resources keep reapplying. It looks like the problem this patch is trying to fix: https://review.openstack.org/#/c/151969/ . But that's months old, so makes me wonder if I'm missing something?
17:31 mwhahaha since that patch was never merged, you're probably not missing something
17:32 tdb so either other people are ignoring the fact the resources reapply on every Puppet run, or I'm doing something differently?
17:33 mwhahaha or people aren't doing it via puppet or that module
17:33 * mwhahaha shrugs
17:33 tdb of course; it won't affect people not using the module :)
17:34 mwhahaha was going to take a look at fixing up that patch, seems the puppet-ceph module is a bit neglected
17:35 openstackgerrit Emilien Macchi proposed openstack/puppet-trove: Fix rabbit_userid parameter  https://review.openstack.org/241308
17:35 EmilienM jpena: ^
17:36 tdb mwhahaha: I'm just building a new Ceph cluster, and since all our other stuff is Puppet driven I thought it'd make sense to use the module rather than make a new one internally.
17:37 mwhahaha no you're right in trying to use the puppet-ceph module rather then writing your own
17:37 mwhahaha it seems it needs some love
17:37 tdb but not if the module isn't getting a lot of love :)
17:38 tdb I suppose I'm just raising it here to see if the module is still alive, or if everyone has moved on to something else I haven't seen
17:38 mwhahaha well it seems that no one bothered to pay attention to the ci failures or merge conflicts
17:39 mwhahaha I know we want to move to the puppet-ceph module so it'll get more attention in the upcoming future
17:39 ntpttr joined #puppet-openstack
17:40 dgurtner joined #puppet-openstack
17:41 tdb I'm happy to give it a try and report issues or give patches where I can
17:41 tdb currently the patch I linked above is the only problem I'm having
17:42 openstackgerrit Bradley Klein proposed openstack/puppet-monasca: Change default monasca api port to 8072 to avoid murano conflict  https://review.openstack.org/241313
17:42 mwhahaha yea i'm taking a look at rebasing that
17:42 mwhahaha to see if it can be salvaged
17:44 tdb if you do I'll give it a try
17:46 EmilienM jayg: I remember you mentionned to use Vagrant to build an openstack cloud with our modules, have you seen our outcome of a summit session? we want to help newcomers about that
17:51 dprince joined #puppet-openstack
17:54 openstackgerrit Alex Schultz proposed openstack/puppet-ceph: fix mon creation to be more idempotent  https://review.openstack.org/151969
17:55 mwhahaha tdb: try -^ and let me know if that doesn't work
17:56 tdb k, will do
17:57 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: Remove occurrence of @@default_domain_id.  https://review.openstack.org/241284
17:59 bklei joined #puppet-openstack
18:02 openstackgerrit Merged openstack/puppet-ironic: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/238596
18:02 openstackgerrit Merged openstack/puppet-tuskar: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/238140
18:03 aimon joined #puppet-openstack
18:04 openstackgerrit Merged openstack/puppet-neutron: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/238615
18:11 IvanBerezovskiy left #puppet-openstack
18:16 rahulait joined #puppet-openstack
18:16 openstackgerrit Merged openstack/puppet-glance: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/238619
18:18 EmilienM clayton, mfisch: can you look https://review.openstack.org/#/c/241308/ ? it's fixing a bug in puppet-trove, I'm trying to bring back our integration testing alive
18:19 openstackgerrit Merged openstack/puppet-horizon: Revert "Try to use zuul-cloner to prepare fixtures"  https://review.openstack.org/240714
18:20 tdb mwhahaha: looks like you missed a bit of the original patch
18:20 tdb but even with that, it still does these each time:
18:20 tdb Notice: /Stage[main]/Main/Node[death]/Ceph::Mon[deat​h]/File[/tmp/ceph-mon-keyring-death]/ensure: defined content as '{md5}b1b65a0cb1950831065b0a70bc9ed7ca'
18:20 tdb Notice: /Stage[main]/Main/Node[death]/Ceph::Mon​[death]/Exec[rm-keyring-death]/returns: executed successfully
18:21 tdb but the original patch didn't address that either
18:21 mwhahaha k i'll take a look at that as well, i'm trying to get the CI working as it appears to be broken
18:21 EmilienM so integration testing is now failing because of second puppet run that is not idempotent on centos: http://logs.openstack.org/08/241308/1/check/gate-p​uppet-openstack-integration-scenario001-dsvm-cento​s7/f0dc430/console.html#_2015-11-03_17_57_05_804
18:22 openstackgerrit Merged openstack/puppet-tempest: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/238154
18:23 EmilienM Hunner, _ody: ^
18:23 EmilienM what is managing /var/lib/puppet/state/* files?
18:23 mwhahaha tdb: most of the other items from that review were addressed in other patches, what's left is the stuff that isn't already in the code base i think
18:23 EmilienM it sounds like the SElinux context is wrong
18:24 openstackgerrit Merged openstack/puppet-keystone: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/238605
18:25 tdb mwhahaha: it's the bit at line 156 of mon.pp in patch set 5 that you were missing, and isn't already there
18:26 mwhahaha ah got lost in the move
18:26 tdb mwhahaha: let me know how it goes. there's not much point me looking at it if you're doing it, but if you give up I'll take a stab.
18:26 mwhahaha i'll get that unless added back in
18:28 EmilienM oh that's weird, puppet is installed in our integration jobs and it should not
18:31 jayg EmilienM: just got back from lunch, that is good news about the making things easier for newcomers!
18:32 EmilienM jayg: I was wondering if you're interested to contribute to it
18:33 openstackgerrit Merged openstack/puppet-openstacklib: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/238610
18:33 EmilienM crinkle: should not we stop installing puppet with spec/acceptance_helper and rely on Gems ?
18:33 EmilienM http://logs.openstack.org/08/241308/1/check/gat​e-puppet-trove-puppet-beaker-rspec-dsvm-centos7​/88abff6/console.html#_2015-11-03_17_48_48_756
18:34 dmsimard dgurtner: so I haven't had time to figure out why those centos tests are failing - I'll guess no one has had any either? :p
18:34 jayg EmilienM: sure, if I have anything of use, I would be happy to contribute
18:34 openstackgerrit Alex Schultz proposed openstack/puppet-ceph: fix mon creation to be more idempotent  https://review.openstack.org/151969
18:36 dmsimard mwhahaha: oh, please let me know if you figure it out :)
18:36 mwhahaha :D
18:36 mwhahaha my money is on no since beaker comes up with different errors locally than what the CI does
18:36 * mwhahaha flips tables
18:38 _ody EmilienM: Best to my knowledge only the puppet agent should be managing /var/lib/puppet/state/* files.
18:39 EmilienM _ody: we don't run agent
18:39 EmilienM but I think I found the bug
18:40 EmilienM it's the version of puppet that we use
18:40 _ody EmilienM: Sorry I use the words "puppet agent" and "puppet apply" interchangeably.
18:41 EmilienM mhh
18:41 bnemec joined #puppet-openstack
18:42 EmilienM that's weird we did not have this pb before
18:42 EmilienM _ody: regarding the logs, can you confirm that's why second puppet run is not idempotent?
18:43 EmilienM I'm not sure
18:44 EmilienM _ody: I can't find why the second puppet run is not idempotent
18:46 _ody looking
18:52 mwhahaha dmsimard: are the ceph packages used centos7 compatible?
18:53 openstackgerrit Bradley Klein proposed openstack/puppet-monasca: Change default monasca api port to 8070 to avoid murano conflict  https://review.openstack.org/241313
18:53 dmsimard mwhahaha: I'm not familiar with the CentOS7 installation of Ceph :(
18:54 mwhahaha i think that's why the ci is broken
18:54 mwhahaha the systemd stuff looks broken
18:55 dmsimard mwhahaha: you're unable to reproduce locally ?
18:56 mwhahaha no i got it
18:57 mwhahaha BEAKER_destroy=no BEAKER_set=centos-70-x64 bundle exec rake beaker
18:59 openstackgerrit Maksym Yatsenko proposed openstack/puppet-nova: [WIP]  https://review.openstack.org/233011
19:01 openstackgerrit Merged openstack/puppet-sahara: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/238599
19:04 openstackgerrit Merged openstack/puppet-heat: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/238609
19:08 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: CI test - do not merge  https://review.openstack.org/241334
19:08 angdraug joined #puppet-openstack
19:09 _ody EmilienM: I don't have beaker setup right now but I dug through our internal ticketing for PE customers and found the selinux contexts flip around if you don't have selinux bindings installed for ruby but according to the puppet package dependencies in epel, they should be being installed.
19:09 _ody wait...we use gems don't we in testing
19:11 EmilienM I thought we did
19:11 EmilienM but no
19:12 EmilienM so for beaker jobs, we're purging puppet packaging but our acceptance helper is installing it again: http://logs.openstack.org/08/241308/1/check/gat​e-puppet-trove-puppet-beaker-rspec-dsvm-centos7​/88abff6/console.html#_2015-11-03_17_48_48_756 cc crinkle
19:12 EmilienM and for integration jobs, we even don't run the macro that purge puppet package: https://review.openstack.org/241329
19:13 EmilienM crinkle: should we patch our beaker config to stop installing puppet ?
19:13 crinkle EmilienM: no? it should be installing puppet
19:13 crinkle why is that a problem?
19:14 sergmelikyan joined #puppet-openstack
19:14 _ody Yeah.  If uninstall it, it should be fine.
19:14 EmilienM crinkle: I've been investigating a bug for ~3 hours and trying to figure out why our second puppet run (integration testing on centos7) is failing
19:14 EmilienM I mean, not failing, but not idempotent
19:14 EmilienM I managed to reproduced it and it seems it's something in the version of Puppet installed by packaging
19:15 EmilienM http://logs.openstack.org/08/241308/1/check​/gate-puppet-openstack-integration-scenario​001-dsvm-centos7/f0dc430/logs/rpm-qa.txt.gz
19:15 EmilienM puppet-3.6.2-3.el7.noarch
19:15 EmilienM and http://logs.openstack.org/08/241308/1/che​ck/gate-puppet-trove-puppet-beaker-rspec-​dsvm-centos7/88abff6/logs/rpm-qa.txt.gz
19:15 EmilienM puppet-3.8.3-1.el7.noarch
19:16 EmilienM the bug is not in puppet-3.8.3-1.el7.noarch because beaker jobs are green
19:16 EmilienM but integration is red, which is using 3.6
19:16 crinkle the integration tests should install puppet, since the beaker jobs are installing puppet
19:16 EmilienM crinkle: so you can -1 my patch
19:16 EmilienM or no
19:16 EmilienM you can still +1
19:16 EmilienM but we need to handle the puppet package in our integration testing
19:17 crinkle yes
19:17 EmilienM and make sure we do it the same way as beaker
19:17 EmilienM crinkle: ok?
19:17 EmilienM great, I'm doing it
19:17 crinkle yes
19:17 EmilienM thx crinkle
19:17 EmilienM we still have a bug in puppet 3.6 though
19:17 EmilienM I'll put it on my todo and trying to report it upstream
19:23 nlahouti joined #puppet-openstack
19:24 _ody EmilienM: The difference between integration and beaker looks to be puppetlabs-release-7-11 vs. puppetlabs-release-7-10.
19:24 dgurtner dmsimard: correct ;)
19:24 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install puppet from puppetlabs.com repo  https://review.openstack.org/241340
19:25 EmilienM _ody, crinkle ^
19:25 dgurtner dmsimard: i think the issue is that 1) there's not enough resources (of some kind) 2) there's a timing issue 3) there's already something on that port
19:26 dgurtner dmsimard: but it seems that apache fails to start and then the request to test fails
19:26 dgurtner dmsimard: maybe the easiest is to just remove rgw from the test set and instead concentrate on the base module functionality to work
19:27 mwhahaha dgurtner: what are you referring to?
19:29 EmilienM _ody: stupid copy paste, thx
19:29 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install puppet from puppetlabs.com repo  https://review.openstack.org/241340
19:31 dgurtner mwhahaha: http://logs.openstack.org/62/230362/1/c​heck/gate-puppet-ceph-puppet-beaker-rsp​ec-dsvm-centos7/bfb76f1/console.html.gz
19:31 mwhahaha oh i've traced it down to radosgw won't start
19:31 vinsh gah... learning how to run spec tests now that they are all r10k.. not as simple as before
19:31 vinsh keep getting some script dir or puppet dir or something wrong
19:31 dgurtner mwhahaha: right... but why?
19:31 mwhahaha no idea trying to figure it out :D
19:32 dgurtner mwhahaha: because it does start _sometimes_
19:32 mwhahaha it's a config with that test on centos7
19:32 mwhahaha specifically the 'should install one monitor/osd with a rgw and keystone integration'
19:32 mwhahaha from ceph_rgw_spec.rb
19:32 dgurtner mwhahaha: also clearly it works on the external CI: http://paste.ubuntu.com/12637643/
19:32 mwhahaha centos7 vs ubuntu packaging?
19:33 mwhahaha systemd of fail?
19:33 mwhahaha :D
19:33 dgurtner mwhahaha: the exact same code has 100% success on the external CI system
19:34 mwhahaha same packages?
19:35 mwhahaha no it's not
19:35 dgurtner which package differs? apache?
19:35 mwhahaha so it's failing with 0.94.5
19:35 mwhahaha [00;00m[00;00m  Notice: /Stage[main]/Main/Package[ceph-radosgw]/ensure: ensure changed '0.94.3-0.el7.centos' to 'purged'
19:35 mwhahaha ceph-radosgw-0.94.5-0.el7.centos.x86_64
19:36 dgurtner theoretically should/could be the same... I was lazy to search through successful/failing tests
19:37 dgurtner mwhahaha: here: http://logs.openstack.org/62/230362/1/c​heck/gate-puppet-ceph-puppet-beaker-rsp​ec-dsvm-centos7/8537272/console.html.gz
19:38 dgurtner failing with 0.94.3-0.el7.centos
19:41 dgurtner mwhahaha: are you using the rgw code anywhere? or do you see any added value in testing it?
19:41 mwhahaha fuel uses rgw but we're not currently using puppet-ceph as we have our own
19:41 mwhahaha the problem is we want to migrate to puppet-ceph so we'll need it
19:42 mwhahaha the errors the ci is currently throwing is the radosgw not running
19:42 mwhahaha so i'm not sure the historical ci issues match what is currently happening
19:43 dgurtner mwhahaha: I can trigger a rerun with current code (you can as well, just "recheck")
19:43 mwhahaha i know, i'm able to reproduce the radosgw not starting in a local vm and i'm looking into it now
19:46 dgurtner mwhahaha: one other issue is that rgw can urn with apache and civetweb
19:46 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install puppet from puppetlabs.com repo  https://review.openstack.org/241340
19:46 mwhahaha yea i think we run it with apache
19:46 dgurtner mwhahaha: and we dont provide civetweb config yet, so maybe the rgw module is outdated anyways?
19:54 mwhahaha oh what the, i downgraded all the 0.94.5 packages to .3 and then updated them back to .5 and now it's working
19:55 dgurtner mwhahaha: it's working _sometimes_ as mentioned before :P
19:55 openstackgerrit Nate Potter proposed openstack/puppet-designate: Allow customization of db sync command line  https://review.openstack.org/241351
19:55 mwhahaha (╯°□°)╯︵ ɥdǝɔ
19:56 dgurtner mwhahaha: there's a lot of pain involved with this particular puppet module :P
19:57 jerrygb joined #puppet-openstack
19:59 mwhahaha i wonder if there's an action in the packaging that has changed
20:02 openstackgerrit Javier Peña proposed openstack/puppet-vswitch: Set NAME as well as DEVICE in ifcfg-*  https://review.openstack.org/241354
20:02 openstackgerrit Rohit Pagedar proposed openstack/puppet-nova: Making instance_name_template configurable.  https://review.openstack.org/234972
20:03 openstackgerrit Javier Peña proposed openstack/puppet-vswitch: Set NAME as well as DEVICE in ifcfg-*  https://review.openstack.org/241354
20:11 jtomasek joined #puppet-openstack
20:19 openstackgerrit Rohit Pagedar proposed openstack/puppet-nova: Making instance_name_template configurable.  https://review.openstack.org/234972
20:20 paramite joined #puppet-openstack
20:22 vinsh keystone module changes have broken puppet-swift spec tests.
20:22 vinsh I'll put up a review to fix this
20:23 mwhahaha dgurtner: i think it's the systemd script, manually running it seems to work but trying to start it via systemd seems like it's not working correctly
20:27 vinsh Author: Gilles Dubreuil <gilles@redhat.com>
20:27 vinsh Date:   Tue Sep 29 17:19:49 2015 +1000
20:27 vinsh broke it. :)
20:29 nlahouti joined #puppet-openstack
20:30 mwhahaha dgurtner: i think it's because the initscript attempts to start radosgw using a su command to 'apache' which is disabled on centos7
20:33 mwhahaha dgurtner: https://github.com/ceph/ceph/pull/6271/
20:37 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Remove tenant parameter from keystone_user  https://review.openstack.org/241360
20:37 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Remove tenant parameter from keystone_user  https://review.openstack.org/241360
20:39 dgurtner mwhahaha: so the answer is to wait until chuanhong, dachary and sage get this merged?
20:40 mwhahaha probably
20:40 mwhahaha seems to be a systemd thing since centos7 is the problematic test. we could switch beaker centos7 to  non-voting for now maybe?
20:43 dgurtner it would be nice to make individual test scenarios optional
20:44 dgurtner I'm somewhat torn between making centos7 non-voting or disabling the rgw patch
20:44 dgurtner s/patch/test/
20:47 mwhahaha it'd probably be best to comment out that scenario for now so we're still testing some things or switch it to pending maybe?
20:48 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install puppet from puppetlabs.com repo  https://review.openstack.org/241340
21:00 _ody joined #puppet-openstack
21:02 cwolferh joined #puppet-openstack
21:03 EmilienM _ody: what is the diff between https://yum.puppetlabs.com/puppet​labs-release-pc1-el-7.noarch.rpm and https://yum.puppetlabs.com/puppet​labs-release-pc1-el-7.noarch.rpm ?
21:05 EmilienM I wonder what is Puppet Collections
21:06 dgurtner joined #puppet-openstack
21:09 crinkle it's how puppet-agent is distributed for puppet 4
21:09 crinkle so puppet, facter, hiera, etc are bundled together
21:10 EmilienM crinkle: got it, thx
21:10 EmilienM crinkle: if you have little time, do you see something wrong in https://jenkins04.openstack.org/job/g​ate-puppet-openstack-integration-scen​ario001-dsvm-centos7/90/consoleFull ?
21:11 EmilienM the job is still failing even with puppet 3.8
21:11 _ody EmilienM: 1. you posted the same link twice.  2. What crinkle said.
21:11 EmilienM _ody: oops :)
21:12 crinkle EmilienM: idk it's super hard to find anything with the debug output on
21:12 _ody Puppet Collections makes distro packaging hard but everything else easier.
21:12 dgurtner_ joined #puppet-openstack
21:12 EmilienM _ody: I thought the job would not fail but it does again. I don't see anything wrong in the puppet run
21:12 _ody EmilienM: Well it was a bug that needed to be fixed anyway...
21:12 EmilienM crinkle: could it be at 21:04:49.545 ?
21:12 EmilienM _ody: yeah
21:13 EmilienM crinkle: it looks like the 3 seluser things make the puppet run not idempotent
21:14 crinkle EmilienM: if something changed it would be a Notice: not a Debug:
21:14 EmilienM I did not know that, ool
21:14 EmilienM cool*
21:15 EmilienM crinkle: maybe 2015-11-03 21:04:57.616 Notice: /Stage[main]/Nova/Package[python-greenlet]/ensure: created
21:16 EmilienM in puppet-nova
21:16 crinkle EmilienM: yep
21:16 crinkle so it's a packaging issue
21:16 EmilienM I would like to cry
21:17 EmilienM it sounds like it's installed in the first puppet run though
21:17 _ody ha! Good eye crinkle, I didn't eve think about that.
21:18 crinkle sometimes apt behaves weird with puppet
21:18 EmilienM yeah, I did not know I had to look at "Notice" for second puppet run
21:18 crinkle is python-greenlet a virtual package maybe?
21:18 EmilienM crinkle: it's yum
21:18 crinkle oh
21:18 crinkle then idk
21:18 _ody Yeah.  If this was apt I'd say python-greenlet was a virtual package.
21:18 EmilienM I'm investigating
21:19 openstackgerrit David Gurtner proposed openstack/puppet-ceph: Update .gitreview for new namespace  https://review.openstack.org/236539
21:20 EmilienM oh I think I got it
21:20 EmilienM http://logs.openstack.org/40/241340/4/check​/gate-puppet-openstack-integration-scenario​001-dsvm-centos7/9ba9d77/logs/syslog.txt.gz
21:20 EmilienM Nov 03 20:54:36 template-devstack-centos7-1446560094.novalocal yum[3578]: Installed: python2-greenlet-0.4.9-1.el7.x86_64
21:20 EmilienM it's python2-greenlet
21:20 dgurtner_ mwhahaha: lets see if this helps... https://review.openstack.org/#/c/236​539/2/spec/acceptance/ceph_rgw_spec.rb
21:20 EmilienM crinkle, _ody : so puppet think python-greenlet is not installed while it is, with python2-greenlet name
21:20 EmilienM maube that's it
21:22 EmilienM _ody: it looks like a bug in Package resource with Yum provider
21:22 EmilienM I'm trying to reproduce
21:23 dgurtner_ mwhahaha: I worry we still suffer from https://review.openstack.org/#/c/230362/ or duplicate https://review.openstack.org/#/c/233066/
21:24 cwolferh joined #puppet-openstack
21:24 mwhahaha i did't run into those errors
21:24 mwhahaha so who knows
21:27 dgurtner_ mwhahaha: also additionally painful that I need to squeeze this into the stackforge->openstack ns change, as I can't submit it otherwise
21:27 mwhahaha not true but we need that done anyway :D
21:27 openstackgerrit Merged openstack/puppet-trove: Fix rabbit_userid parameter  https://review.openstack.org/241308
21:28 nlahouti joined #puppet-openstack
21:29 EmilienM _ody: http://projects.puppetlabs.com/issues/5831
21:29 EmilienM that's the bug we're hitting I think
21:31 dgurtner_ mwhahaha: it's just ugly
21:31 mwhahaha i know
21:33 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: do not manage python-greenlet anymore  https://review.openstack.org/241373
21:34 nibalizer anyone feeling up to reviewing a report processor? https://review.openstack.org/#/c/240459/3/modules/​openstack_project/lib/puppet/reports/puppetdb_file.rb hunner spun that up at the infra hack day
21:34 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install puppet from puppetlabs.com repo  https://review.openstack.org/241340
21:36 EmilienM nibalizer: wow, that's a nice feature
21:36 EmilienM nibalizer: why not pushing this kind of code in an upstream module/lib/repo?
21:38 Hunner EmilienM: It's actually kind of specialized, I think
21:38 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/203220
21:38 Hunner EmilienM: The puppetdb module tries to support masterless but assumes nodes can contact the puppetdb service
21:38 Hunner EmilienM: It would be complicated to try and give a generic solution where the nodes can't talk to it though
21:38 vinsh EmilienM: Need to merge: https://review.openstack.org/#/c/241360/  swift unit tests broken without it.
21:40 EmilienM oh I missed that one?
21:40 EmilienM I thought we fixed all modules
21:40 EmilienM mfisch: ^
21:42 openstackgerrit David Gurtner proposed openstack/puppet-ceph: Update .gitreview for new namespace  https://review.openstack.org/236539
21:42 dgurtner_ mwhahaha: ok, it's betternow except you can't purge what you never added, so ^
21:54 cwolferh joined #puppet-openstack
21:58 nlahouti joined #puppet-openstack
21:59 dgurtner_ mwhahaha: look here :) https://review.openstack.org/#/c/236539/
21:59 _ody EmilienM: Looks like it.  TBH I am still pretty new to EL and didn't even think it had virtual packages.  I run into this all the time on debian, where it is also not fixed.
21:59 mwhahaha hey it's green, woo
22:01 nlahouti joined #puppet-openstack
22:04 EmilienM ok, integration jobs should be back with https://review.openstack.org/#/c/241340/ and its dependencies
22:05 dtalton joined #puppet-openstack
22:10 openstackgerrit Alex Schultz proposed openstack/puppet-cinder: Update cinder backends to use os_service_default  https://review.openstack.org/239800
22:16 EmilienM crinkle: if you don't mind to review https://review.openstack.org/#/c/241340/
22:17 EmilienM and the dependency in puppet-nova
22:17 jerrygb joined #puppet-openstack
22:22 nibalizer EmilienM: so it might eventually end up in a module/lib/repo but for now incubating in system-config makes sense
22:23 nibalizer since it might need unforseen changes
22:27 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install puppet from puppetlabs.com repo  https://review.openstack.org/241340
22:29 openstackgerrit Merged openstack/puppet-ceilometer: Try to use zuul-cloner to prepare fixtures  https://review.openstack.org/238613
22:32 jerrygb joined #puppet-openstack
22:40 dgurtner joined #puppet-openstack
22:43 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: loadbalancer: add Aodh API support  https://review.openstack.org/241395
22:45 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/203220
22:47 _ducttape_ joined #puppet-openstack
22:48 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: run_tests: install puppet from puppetlabs.com repo  https://review.openstack.org/241340
22:49 patrickeast hi, i'm trying to get a all-in-one liberty setup going with the puppet modules and am seeing errors like: http://paste.openstack.org/show/477914/
22:49 patrickeast anyone know what might be causing that?
22:51 patrickeast for reference this is what my manifest looks like for the neutron part (where that error came from) http://paste.openstack.org/show/477915/
22:52 dgurtner joined #puppet-openstack
22:53 ducttape_ joined #puppet-openstack
22:58 rahulait joined #puppet-openstack
23:06 openstackgerrit Merged openstack/puppet-swift: Remove tenant parameter from keystone_user  https://review.openstack.org/241360
23:10 nlahouti joined #puppet-openstack
23:23 cwolferh joined #puppet-openstack
23:43 nlahouti1 joined #puppet-openstack
23:51 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Manage swift with swiftinit service provider  https://review.openstack.org/203220
23:52 guessi joined #puppet-openstack
23:54 patrickeast hmm so if i'm reading this right you need to have keystone v3 available for this to work https://github.com/openstack/puppet-ke​ystone/blob/master/lib/puppet/provider​/keystone_user/openstack.rb#L89-L126
23:55 angdraug joined #puppet-openstack
23:56 patrickeast which is maybe a problem since it appears following this as an example sets me up for v2 endpoints https://wiki.openstack.org/​wiki/Puppet/Deploy#Keystone
23:57 guessi left #puppet-openstack
23:57 patrickeast the first time applying the manifest things were ok, the service user/tenant are there and things are working, but subsequent apply's fail

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