Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-05-13

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

All times shown according to UTC.

Time Nick Message
00:00 openstackgerrit Merged openstack/puppet-magnum: Use oslo module for messaging (rabbit) configuration  https://review.openstack.org/313024
00:19 openstackgerrit zhongshengping proposed openstack/puppet-neutron: Add the capability to configure neutron_vpnaas.conf  https://review.openstack.org/315353
00:20 ayoung joined #puppet-openstack
00:27 markvoelker joined #puppet-openstack
00:31 openstackgerrit zhongshengping proposed openstack/puppet-aodh: Add spec test for aodh::config  https://review.openstack.org/315819
00:37 ntpttr_ joined #puppet-openstack
01:01 openstackgerrit liaonanhai proposed openstack/puppet-manila: Add support for db_max_retries param  https://review.openstack.org/315297
01:03 openstackgerrit joined #puppet-openstack
01:08 openstackgerrit zhongshengping proposed openstack/puppet-mistral: Define db related parameters in class ::mistral  https://review.openstack.org/315830
01:12 openstackgerrit zhongshengping proposed openstack/puppet-mistral: Define db related parameters in class ::mistral  https://review.openstack.org/315830
01:15 ntpttr_ joined #puppet-openstack
01:21 openstackgerrit zhongshengping proposed openstack/puppet-mistral: Define db related parameters in class ::mistral  https://review.openstack.org/315830
01:21 xiaohua joined #puppet-openstack
01:23 fedexo joined #puppet-openstack
01:25 openstackgerrit zhongshengping proposed openstack/puppet-mistral: Define db related parameters in class ::mistral  https://review.openstack.org/315830
01:29 openstackgerrit zhongshengping proposed openstack/puppet-neutron: Add the capability to configure neutron_vpnaas.conf  https://review.openstack.org/315353
01:29 markvoelker joined #puppet-openstack
01:30 openstackgerrit zhongshengping proposed openstack/puppet-manila: Fix markdown format typo  https://review.openstack.org/315833
01:53 ducttape_ joined #puppet-openstack
02:15 imcsk8 joined #puppet-openstack
02:15 xiaohua_ joined #puppet-openstack
02:16 openstackgerrit zhongshengping proposed openstack/puppet-cinder: Add spec test for cinder::config  https://review.openstack.org/315841
02:18 openstackgerrit zhongshengping proposed openstack/puppet-aodh: Add spec test for aodh::config  https://review.openstack.org/315819
02:26 ducttape_ joined #puppet-openstack
02:28 ducttape_ joined #puppet-openstack
02:30 markvoelker joined #puppet-openstack
02:31 xiaohua joined #puppet-openstack
02:33 openstackgerrit Matt Fischer proposed openstack/puppet-ceilometer: metering_secret was renamed to telemetry_secret  https://review.openstack.org/315846
02:33 xiaohua_ joined #puppet-openstack
02:37 mfisch EmilienM: see I love ceilometer
02:37 mfisch ^
02:38 openstackgerrit Matt Fischer proposed openstack/puppet-ceilometer: metering_secret was renamed to telemetry_secret  https://review.openstack.org/315846
02:39 _ducttape_ joined #puppet-openstack
02:40 openstackgerrit Matt Fischer proposed openstack/puppet-openstack-integration: Rename metering_secret to telemetry_secret  https://review.openstack.org/315848
02:44 mfisch crinkle: 3 min
02:46 crinkle mfisch: k
02:57 xiaohua joined #puppet-openstack
02:59 xiaohua_ joined #puppet-openstack
03:24 ducttape_ joined #puppet-openstack
03:24 ducttape_ joined #puppet-openstack
03:31 markvoelker joined #puppet-openstack
03:40 openstackgerrit Merged openstack/puppet-keystone: Add some kombu options  https://review.openstack.org/315169
03:50 chandankumar joined #puppet-openstack
04:01 ducttape_ joined #puppet-openstack
04:31 markvoelker joined #puppet-openstack
04:48 xiaohua joined #puppet-openstack
05:21 jaosorior joined #puppet-openstack
05:26 xiaohua joined #puppet-openstack
05:32 markvoelker joined #puppet-openstack
05:50 rcernin joined #puppet-openstack
05:57 fangzhen joined #puppet-openstack
05:59 fedexo joined #puppet-openstack
06:04 fedexo joined #puppet-openstack
06:06 jmikolajczak joined #puppet-openstack
06:24 chandankumar joined #puppet-openstack
06:25 EmilienM mfisch: so glad
06:26 EmilienM good morning
06:33 markvoelker joined #puppet-openstack
06:49 xiaohua joined #puppet-openstack
06:51 Mosibi joined #puppet-openstack
06:51 xiaohua joined #puppet-openstack
06:52 EmilienM a lot of beaker jobs are failing, I think we need to land https://review.openstack.org/315770 asap
06:56 tesseract joined #puppet-openstack
06:58 Mosibi joined #puppet-openstack
07:20 fawadkhaliq joined #puppet-openstack
07:20 paramite joined #puppet-openstack
07:27 openstackgerrit Xiaohua Yuan proposed openstack/puppet-trove: Add Users/DBs related parameters to guestagent class  https://review.openstack.org/305717
07:27 xiaohua_ joined #puppet-openstack
07:31 openstackgerrit Xiaohua Yuan proposed openstack/puppet-trove: Add datastore_registry_ext option to trove-guestagent.conf  https://review.openstack.org/305591
07:31 xiaohua joined #puppet-openstack
07:34 markvoelker joined #puppet-openstack
07:34 hjensas joined #puppet-openstack
07:35 openstackgerrit liaonanhai proposed openstack/puppet-manila: Add support for db_max_retries param and  https://review.openstack.org/315297
07:36 shardy joined #puppet-openstack
07:37 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: deploy keystone at step 3 and 4  https://review.openstack.org/315943
07:37 danpawlik joined #puppet-openstack
07:39 karimb joined #puppet-openstack
08:10 xiaohua joined #puppet-openstack
08:13 evidex joined #puppet-openstack
08:13 xiaohua joined #puppet-openstack
08:17 openstackgerrit Javier Peña proposed openstack/puppet-tempest: Rename deprecated configuration options  https://review.openstack.org/314594
08:22 rmart04 joined #puppet-openstack
08:22 gfidente joined #puppet-openstack
08:25 leanderthal|afk joined #puppet-openstack
08:35 markvoelker joined #puppet-openstack
08:35 openstackgerrit Michal Adamczyk proposed openstack/puppet-magnum: Various fixes  https://review.openstack.org/313293
08:46 jtomasek_ joined #puppet-openstack
08:46 timrc joined #puppet-openstack
08:46 jistr joined #puppet-openstack
09:05 dgurtner joined #puppet-openstack
09:05 dgurtner joined #puppet-openstack
09:09 dmk0202 joined #puppet-openstack
09:18 openstackgerrit Jerzy Mikolajczak proposed openstack/puppet-neutron: Deprecate prevent_arp_spoofin option  https://review.openstack.org/315980
09:36 markvoelker joined #puppet-openstack
09:43 openstackgerrit Babu Shanmugam proposed openstack/puppet-tripleo: Add OVN support for neutron profile  https://review.openstack.org/314875
09:45 chem joined #puppet-openstack
09:45 openstackgerrit Babu Shanmugam proposed openstack/puppet-tripleo: Add OVN support for neutron profile  https://review.openstack.org/314875
09:49 mkarpin joined #puppet-openstack
10:04 openstackgerrit Merged openstack/puppet-ironic: Implement ironic::drivers::deploy class  https://review.openstack.org/315233
10:05 openstackgerrit Merged openstack/puppet-cinder: Introduce support for oslo.messaging amqp driver configuration  https://review.openstack.org/315195
10:33 jtomasek_ joined #puppet-openstack
10:36 markvoelker joined #puppet-openstack
10:45 dgurtner joined #puppet-openstack
10:49 beddari joined #puppet-openstack
11:02 openstackgerrit Xiaohua Yuan proposed openstack/puppet-trove: Add Users/DBs related parameters to guestagent class  https://review.openstack.org/305717
11:02 xiaohua_ joined #puppet-openstack
11:02 openstackgerrit Xiaohua Yuan proposed openstack/puppet-trove: Add datastore_registry_ext option to trove-guestagent.conf  https://review.openstack.org/305591
11:03 xiaohua__ joined #puppet-openstack
11:03 leanderthal joined #puppet-openstack
11:07 openstackgerrit zhongshengping proposed openstack/puppet-cinder: Add spec test for cinder::config  https://review.openstack.org/315841
11:08 openstackgerrit Merged openstack/puppet-neutron: Introduce support for oslo.messaging amqp driver configuration  https://review.openstack.org/315192
11:10 johnmilton joined #puppet-openstack
11:13 openstackgerrit zhongshengping proposed openstack/puppet-neutron: Add the capability to configure neutron_vpnaas.conf  https://review.openstack.org/315353
11:14 mgould joined #puppet-openstack
11:14 openstackgerrit Michal Adamczyk proposed openstack/puppet-magnum: Various fixes  https://review.openstack.org/313293
11:14 openstackgerrit zhongshengping proposed openstack/puppet-nova: Deprecate listen parameter  https://review.openstack.org/315524
11:18 jaosorior EmilienM: Still online?
11:19 fangzhen_ joined #puppet-openstack
11:30 openstackgerrit James Slagle proposed openstack/puppet-ironic: Revert "allow to configure my_ip parameter"  https://review.openstack.org/316039
11:30 jaosorior joined #puppet-openstack
11:31 dgurtner joined #puppet-openstack
11:31 openstackgerrit James Slagle proposed openstack/puppet-ironic: Revert "allow to configure my_ip parameter"  https://review.openstack.org/316039
11:32 morazi joined #puppet-openstack
11:37 markvoelker joined #puppet-openstack
11:40 openstackgerrit Matthew Mosesohn proposed openstack/puppet-nova: Add nova_service type for purging nova services from offline hosts  https://review.openstack.org/309103
11:42 derekh joined #puppet-openstack
11:45 EmilienM jaosorior: yes
11:46 ansmith joined #puppet-openstack
11:48 EmilienM jaosorior: we reverted the puppet-ironic patch
11:48 EmilienM mwhahaha, degorenko: fyi
11:52 xb_ joined #puppet-openstack
12:05 openstackgerrit Jiri Stransky proposed openstack/puppet-tripleo: Add a service manifest  https://review.openstack.org/313543
12:10 openstackgerrit Jiri Stransky proposed openstack/puppet-tripleo: Add a service manifest  https://review.openstack.org/313543
12:12 openstackgerrit Brad P. Crochet proposed openstack/puppet-tripleo: Add Heat profiles  https://review.openstack.org/313019
12:13 openstackgerrit Andy Smith proposed openstack/puppet-ironic: Introduce support for oslo.messaging amqp driver configuration  https://review.openstack.org/315242
12:14 ducttape_ joined #puppet-openstack
12:16 leanderthal joined #puppet-openstack
12:17 mwhahaha K
12:17 degorenko ok
12:21 aarefiev joined #puppet-openstack
12:23 chem joined #puppet-openstack
12:33 EmilienM mwhahaha: can we get https://review.openstack.org/#/c/315770/ please?
12:33 EmilienM I'm afraid a lot of beaker jobs are failing without it
12:34 dprince joined #puppet-openstack
12:38 markvoelker joined #puppet-openstack
12:41 iurygregory joined #puppet-openstack
12:42 rcernin_ joined #puppet-openstack
12:43 iurygregory morning
12:45 mwhahaha OK +A
13:00 ayoung joined #puppet-openstack
13:02 openstackgerrit Denis Egorenko proposed openstack/puppet-swift: Add systemd service provider for Xenial  https://review.openstack.org/316080
13:02 degorenko vinsh, ping and take a look ^
13:03 openstackgerrit Merged openstack/puppet-barbican: Added keystone config to barbican api manifest  https://review.openstack.org/263461
13:06 fedexo joined #puppet-openstack
13:11 dprince joined #puppet-openstack
13:16 ducttape_ joined #puppet-openstack
13:20 fawadkhaliq joined #puppet-openstack
13:27 rcernin joined #puppet-openstack
13:27 * mjblack yawns
13:28 openstackgerrit Merged openstack/puppet-keystone: stop managing eventlet service on RedHat  https://review.openstack.org/315770
13:28 EmilienM ok if you still see some beaker jobs errors on centos7 because of keystone service, please ping me
13:37 openstackgerrit Denis Egorenko proposed openstack/puppet-openstack-integration: WIP: Support of Ubuntu 16.04 LTS Xenial deployments  https://review.openstack.org/308530
13:39 markvoelker joined #puppet-openstack
13:39 xiaohua joined #puppet-openstack
13:43 openstackgerrit Pradeep Kilambi proposed openstack/puppet-tripleo: loadbalancer: add Aodh API support  https://review.openstack.org/288122
13:45 vinsh degorenko: I wondered.. couldn't we just remove lines 30-36 entirely?
13:45 vinsh puppet should choose those providers as default anyway
13:46 EmilienM I think I tried
13:46 openstack joined #puppet-openstack
13:47 EmilienM https://review.openstack.org/#/c/312770/
13:47 vinsh It worked for you.... just that you needed to update all the spec tests to match that I think
13:47 degorenko vinsh, do not use any hardcoded providers at all?
13:47 vinsh right
13:47 vinsh puppet will choose upstart for 14.04
13:47 EmilienM yes but AFIK there is a bug _ody found
13:47 vinsh oh https://github.com/puppetlabs/puppet/blob/master/lib/puppet/provider/service/upstart.rb
13:47 EmilienM https://review.openstack.org/#/c/312284/
13:48 EmilienM https://tickets.puppetlabs.com/browse/PUP-5353
13:48 EmilienM I'm not sure it's related
13:54 dfisher joined #puppet-openstack
13:54 openstack joined #puppet-openstack
13:55 markvoelker joined #puppet-openstack
14:07 ducttape_ joined #puppet-openstack
14:07 clayton _ody: did you guys ever get a vmpooler equivalent for openstack?
14:08 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module  https://review.openstack.org/263462
14:08 openstackgerrit Merged openstack/puppet-ironic: Revert "allow to configure my_ip parameter"  https://review.openstack.org/316039
14:10 alee degorenko, good catch on the missing manifest.  I was still trying to wrack my brains and figure out why it wasn't failing for me locally.
14:11 degorenko alee, it happens :) no problem
14:12 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: Revert "Revert "allow to configure my_ip parameter""  https://review.openstack.org/316112
14:12 mwhahaha oh the good ol' revert revert
14:13 degorenko ha ha
14:13 iurygregory revert can always solve a problem =P
14:13 iurygregory but you can't revert your life =( ahsuahsuahsuash
14:13 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: Revert "Revert "allow to configure my_ip parameter""  https://review.openstack.org/316112
14:14 degorenko iurygregory, what does mean 'ahsuahsuahsuash' ?
14:15 iurygregory degorenko, ''ahsuahsuahsuash'' == laugh in Brazil XD sorry
14:16 degorenko ah, ok :)
14:16 EmilienM or just say 'lol'
14:18 iurygregory lol its common in game i think XD sorry
14:18 vinsh It's fun to try and laugh saying "ahsuahsuahsuash" :p
14:18 degorenko :D
14:19 iurygregory i can do this \o/ people say that when i laugh it's funny
14:23 vinsh degorenko: Looks like CI timeout/failure in 316080 scenario002
14:24 degorenko recheck is coming
14:26 mfisch EmilienM: is there a way to undef a parameter in rspec?
14:26 mfisch like I want all the defaults but want to remove 1
14:28 iurygregory nil will work i think
14:30 mfisch lemme try
14:30 iurygregory good luck o/
14:34 mfisch adding a test so I get to pick the string
14:34 mfisch 'broncos' it is
14:35 openstackgerrit Matt Fischer proposed openstack/puppet-ceilometer: metering_secret was renamed to telemetry_secret  https://review.openstack.org/315846
14:35 mfisch degorenko: can you look at https://review.openstack.org/#/c/314408/ today
14:36 degorenko mfisch, ack
14:36 mfisch spasiba
14:37 degorenko looks good, +A
14:37 mfisch degorenko: I updated this hot mess too
14:37 mfisch https://review.openstack.org/#/c/315846/
14:37 mfisch ceilometer devs /me shakes fist
14:38 mfisch oh dang whitespace, lemme fix
14:38 openstackgerrit Matt Fischer proposed openstack/puppet-ceilometer: metering_secret was renamed to telemetry_secret  https://review.openstack.org/315846
14:39 shardy joined #puppet-openstack
14:43 openstackgerrit zhongshengping proposed openstack/puppet-aodh: Add spec test for aodh::config  https://review.openstack.org/315819
14:43 openstackgerrit zhongshengping proposed openstack/puppet-cinder: Add spec test for cinder::config  https://review.openstack.org/315841
14:44 alee left #puppet-openstack
14:44 alee joined #puppet-openstack
14:45 alee degorenko, EmilienM, mwhahaha  -- jenkins is happy again -- https://review.openstack.org/#/c/263462/
14:47 degorenko alee, may be then you can also back listening with('tcp') ? :)
14:47 alee degorenko, I ended up taking that out before because the test failed with tcp
14:48 gfidente joined #puppet-openstack
14:48 gfidente joined #puppet-openstack
14:48 alee degorenko, I dont think that was related to the missing manifest
14:48 alee degorenko, I'll try locally again - but I believe it fails in this case.
14:50 openstackgerrit zhongshengping proposed openstack/puppet-barbican: Add unit test for barbican::config  https://review.openstack.org/316134
14:58 mattymo` joined #puppet-openstack
14:58 mattymo` looking for reviewers: https://review.openstack.org/#/c/309103/ finally passes CI
15:01 openstackgerrit Marios Andreou proposed openstack/puppet-tripleo: Adds the puppet-tripleo manifests for manila  https://review.openstack.org/313527
15:07 yo_ joined #puppet-openstack
15:07 openstackgerrit Brad P. Crochet proposed openstack/puppet-tripleo: Add Heat profiles  https://review.openstack.org/313019
15:10 degorenko alee_afk, ok
15:12 vinsh degorenko: Crazy.. when I apply your patch 316080.. almost all of the spec tests fail for latest puppet with "     Evaluation Error: Error while evaluating a Function Call, 'versioncmp' parameter 'a' expects a String value, got Undef at /root/puppet-swift/spec/fixtures/modules/swift/manifests/params.pp:30:12 on node puppet-xenial.openstacklocal"
15:13 degorenko vinsh, what?
15:13 degorenko how?
15:13 vinsh see http://logs.openstack.org/80/316080/1/check/gate-puppet-swift-puppet-unit-latest-centos-7/926256a/console.html
15:14 karimb joined #puppet-openstack
15:14 haplo37 joined #puppet-openstack
15:15 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add 3 api classes and params  https://review.openstack.org/314596
15:16 openstackgerrit Merged openstack/puppet-trove: default swift_url to SERVICE_DEFAULT  https://review.openstack.org/314408
15:18 openstackgerrit joined #puppet-openstack
15:20 vinsh degorenko: Ah, maybe centos doesn't have a fact for operatingsystemmajrelease
15:21 degorenko vinsh, it is case for debian based os
15:21 vinsh But, I am running it on 16.04 and when I debug operatingsystemmajrelease indeed is "16.04"
15:21 vinsh oh yeah
15:21 degorenko when i tested it on my 16.04 - everything was fine
15:21 vinsh So the value is populated.. but needs to be stubbed out for the tests I think
15:21 degorenko probably yes
15:21 degorenko it is not in tests
15:21 degorenko you're right
15:22 degorenko but then - its strange that it passed ci
15:22 degorenko oh, of course it is passed on 14.04
15:22 vinsh well.. it's the unit tests that fail with puppet >4.0
15:22 vinsh Something about latest puppet and that fact
15:23 degorenko so, fact is missed in tests
15:23 degorenko how is puppet4 here?
15:23 vinsh http://logs.openstack.org/80/316080/1/check/gate-puppet-swift-puppet-unit-latest-centos-7/926256a/
15:23 vinsh runs latest puppet.. like I have on my test node
15:24 vinsh puppet apply shows a value for that fact.. but running spec tests it fails
15:24 degorenko got undef - right
15:24 degorenko because tests dont know about this :D
15:24 vinsh why does it pass with puppet 4.0 then?
15:25 vinsh gate-puppet-swift-puppet-unit-4.0-centos-7  PASS
15:25 degorenko good question
15:25 degorenko may be here some changes in versioncmp function?
15:25 degorenko between 4.0 and latest?
15:25 vinsh Yeah.. that sounds right
15:26 degorenko anyway, we need update tests as well, because it will fail on xenial - i mean unit tests
15:26 degorenko and on centos
15:26 vinsh found it
15:26 degorenko and? :) where is problem?
15:26 vinsh need quotes
15:26 vinsh if versioncmp('$::operatingsystemmajrelease', '16') >= 0 {
15:26 iurygregory O.o
15:26 vinsh otherwise it doesn't evaluate
15:26 iurygregory lol
15:26 mwhahaha the fact is missing
15:26 degorenko quotes
15:26 mwhahaha in the test
15:26 mwhahaha that's the fix
15:27 vinsh no. the fact is there
15:27 degorenko really?
15:27 vinsh Just versioncomp with puppet 4.4 is different
15:27 degorenko ah
15:27 mwhahaha no, it's missing
15:27 mwhahaha http://logs.openstack.org/80/316080/1/check/gate-puppet-swift-puppet-unit-latest-centos-7/926256a/console.html#_2016-05-13_13_20_05_971
15:28 mwhahaha Undef
15:28 vinsh that's because its missing quotes
15:28 mwhahaha no
15:28 mwhahaha that's missing
15:28 mwhahaha quoting it just turns it to an empty string
15:28 mwhahaha the problem is that the fact is not there
15:28 degorenko ^ yes, not undef
15:28 degorenko but fact is missing
15:28 mwhahaha the correct fix is to provide the fact
15:28 degorenko puppet 4.4 cant work with undef
15:28 mwhahaha wrapping in quotes hides it
15:28 vinsh ok.. so puppet 4.4 alerts on this when 4.0 would let an empty string pass.
15:29 degorenko and older version undef == ''
15:29 degorenko sounds like that
15:29 mwhahaha yea we've hit this oodles of times
15:29 mwhahaha if you switch to rspec-puppet-facts this info gets provided
15:29 vinsh Good to know, thanks for the detail
15:29 mwhahaha other wise you need to provide the fact
15:30 vinsh is that a module-wide switch that needs to be done?
15:30 mwhahaha no you can do it on a per test basis
15:30 degorenko so, for debian based for now we can hardcode 14.04 version i guess
15:30 degorenko or i can implement for xenial too
15:31 vinsh well. centos needs a version too
15:31 degorenko no
15:31 degorenko its only for debian
15:31 chandankumar joined #puppet-openstack
15:32 degorenko at least case for debian :D
15:32 ducttape_ joined #puppet-openstack
15:32 mwhahaha unfortunately because you put that in params, it needs to be defined for all tests that will consume params
15:32 vinsh Yes. sorry
15:32 mwhahaha so you'd have to provide the fact in all of the swift tests probably
15:32 degorenko oh, crap
15:33 ericksonsantos joined #puppet-openstack
15:33 mwhahaha or wrap it in quotes with a TODO to remove once the tests get migrated to rspec-puppet-facts
15:33 mwhahaha aka dirty hack
15:33 vinsh For debian.. I think we can remove any hard code of provider in params
15:34 degorenko i think we can ask _ody about bug in service providers
15:34 vinsh When I look at the nova case EmilienM provided from _ody.. it was only problems with puppet latest on redhat based
15:34 vinsh https://review.openstack.org/#/c/312284/4/manifests/params.pp
15:35 degorenko yeah, i saw
15:35 degorenko may be here some another issue for debian
15:35 vinsh I guess that would apply to 16.04 though..
15:36 vinsh 16.04 + puppet 4.0-4.4 maybe
15:36 degorenko well, undef for service provider for debian
15:36 degorenko looks like it can work
15:36 degorenko vinsh, i can reupload my patch without specific provider name
15:37 vinsh Yeah, will need to clean the unit tests too to match I think.
15:37 degorenko ok, i ll do that
15:38 vinsh cool.. I believe that is a better design for this part.
15:39 vinsh I'm fixing the swift-init provider now to handle 16.04
15:39 vinsh Folks should just use that provider anyway for all OS's and swift :) says me.
15:40 vinsh It turns out it works pretty well with docker+puppet for swift also.
15:40 degorenko sounds good :)
15:40 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add db related classes  https://review.openstack.org/314597
15:40 vinsh I have a change soon to review on the os_docker hooks for puppet-swift. very similar to the nova/keystone/desginate stuff.
15:41 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add db related classes  https://review.openstack.org/314597
15:41 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add 3 api classes and params  https://review.openstack.org/314596
15:42 cwolferh joined #puppet-openstack
15:43 openstackgerrit Alexey Deryugin proposed openstack/puppet-trove: Use oslo module for logging  https://review.openstack.org/315548
15:48 openstackgerrit Dmitry Ilyin proposed openstack/puppet-ec2api: Add all other classes  https://review.openstack.org/314598
15:49 mgould joined #puppet-openstack
15:55 aimon joined #puppet-openstack
15:55 openstackgerrit James Slagle proposed openstack/puppet-keystone: Revert "stop managing eventlet service on RedHat"  https://review.openstack.org/316167
15:55 openstackgerrit James Slagle proposed openstack/puppet-keystone: Revert "stop managing eventlet service on RedHat"  https://review.openstack.org/316167
15:56 iurygregory wow O.o
15:57 mwhahaha hmm seems like tripleo should setup that dependency differently
15:58 mwhahaha as you shouldn't be requiring Service['keystone'] when running under httpd
16:01 openstackgerrit Denis Egorenko proposed openstack/puppet-swift: Do not use any specific service provider for ubuntu  https://review.openstack.org/316080
16:01 degorenko vinsh, that should be fine now ^
16:02 vinsh Very good.  be back in an hour. LUNCH TIME.
16:11 openstackgerrit Alex Schultz proposed openstack/puppet-tripleo: Update keystone service name for signing keys  https://review.openstack.org/316177
16:12 iurygregory this make more sense i think ^
16:12 mwhahaha :D
16:13 iurygregory nice catch mwhahaha o/
16:16 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Add mongodb profiles  https://review.openstack.org/316179
16:17 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Add mongodb profiles  https://review.openstack.org/316179
16:22 agrebennikov joined #puppet-openstack
16:22 markvoelker joined #puppet-openstack
16:23 openstackgerrit Carlos Camacho proposed openstack/puppet-tripleo: Add mongodb profiles  https://review.openstack.org/304780
16:29 iurygregory EmilienM, https://review.openstack.org/316177
16:29 EmilienM yeah I saw it
16:29 iurygregory maybe will help i think =)
16:29 EmilienM and yeah nice catch alex
16:33 EmilienM mfisch: yeah, nil
16:33 EmilienM sorry for lag
16:34 EmilienM vinsh: yeah, we had issues with redhat, nevermind about this one
16:41 ducttape_ joined #puppet-openstack
16:42 mfisch EmilienM: merci
16:53 aimon joined #puppet-openstack
17:09 alee _degorenko|afk, yeah - it still fails with tcp - not sure why
17:10 alee _degorenko|afk, but it passes as is now .. :p
17:10 alee mwhahaha, thanks!
17:11 alee EmilienM, finally!  all the patches are in.  time to start thinking about doing tempest tests.
17:12 alee (and focusing on the triple-o patches)
17:12 EmilienM alee: congrats
17:12 EmilienM alee: and thanks for your hard work on that
17:12 EmilienM alee: I'll help you for the integration
17:13 EmilienM alee: I was thinking at using a scneario where cinder is running so we can test cinder volume encryption
17:13 EmilienM and also later, maybe a new scenario with magnum so we test magnum + barbican + lbaas (they depend each others)
17:13 alee EmilienM, cool - I need to head out to the garage right now - but I'll check back in with you later today
17:14 alee EmilienM, but yeah encrypted volumes for sure, maybe magnum, and swift just did encrypted objects
17:15 EmilienM mwhahaha: in https://review.openstack.org/#/c/316177/1/manifests/profile/base/keystone.pp -- should we include ::apache::params?
17:15 EmilienM well, maybe it's not needed
17:16 mwhahaha it's possible
17:16 EmilienM I'm watching CI now I'll let you know
17:16 mwhahaha but i think that's included in ::keystone::wsgi::apache no?
17:16 EmilienM yeah
17:16 mwhahaha i also noticed the file permissions
17:17 mwhahaha since they are set to keystone
17:17 mwhahaha but since i've never actually used tripleo, i have no idea what i'm doing :)
17:17 holser joined #puppet-openstack
17:20 kgalanov joined #puppet-openstack
17:21 openstackgerrit Andy Smith proposed openstack/puppet-sahara: Introduce support for oslo.messaging amqp driver configuration  https://review.openstack.org/315186
17:21 iurygregory congrats mwhahaha jenkis +1 XD
17:21 mwhahaha well that's cause there are no actual tests for those ;)
17:22 mwhahaha at least not in rspec
17:22 mwhahaha so +1 for not creating a syntax error
17:22 iurygregory only syntax? O.o
17:23 openstackgerrit Merged openstack/puppet-barbican: Added apache component Add test and mods for deployment as an apache module  https://review.openstack.org/263462
17:36 igorbelikov joined #puppet-openstack
17:46 ansmith joined #puppet-openstack
17:48 EmilienM mwhahaha: jobs are still running, finger crosses
17:48 EmilienM crossed*
17:52 alee joined #puppet-openstack
17:54 ducttape_ joined #puppet-openstack
17:56 ducttape_ joined #puppet-openstack
18:04 xiaohua joined #puppet-openstack
18:27 ansmith joined #puppet-openstack
18:31 openstackgerrit Merged openstack/puppet-nova: Adding a purge_config option for nova_config  https://review.openstack.org/312734
18:32 EmilienM mwhahaha: it worked!
18:33 mwhahaha Yay
18:33 EmilienM mwhahaha: you fixed tripleo CI, but we won't tell anyone.
18:36 openstackgerrit Merged openstack/puppet-tripleo: Update keystone service name for signing keys  https://review.openstack.org/316177
18:36 jtomasek_ joined #puppet-openstack
18:37 cwolferh joined #puppet-openstack
18:39 jtomasek_ joined #puppet-openstack
18:48 openstackgerrit Emilien Macchi proposed openstack/puppet-tripleo: deploy keystone at step 3 and 4  https://review.openstack.org/315943
18:48 openstackgerrit Emilien Macchi proposed openstack/puppet-ironic: Revert "Revert "allow to configure my_ip parameter""  https://review.openstack.org/316112
18:54 ducttape_ joined #puppet-openstack
18:55 darkxploit joined #puppet-openstack
18:58 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Swift defaults Nova endpoints to 2.1  https://review.openstack.org/316241
19:04 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Swift defaults Nova endpoints to 2.1  https://review.openstack.org/316241
19:13 openstackgerrit Nathan Taylor proposed openstack/puppet-keystone: Adding a purge_config option for keystone_config  https://review.openstack.org/316245
19:30 ayoung joined #puppet-openstack
19:47 openstackgerrit Emilien Macchi proposed openstack/puppet-nova: Switch API to 2.1 and deprecate 3  https://review.openstack.org/316241
19:48 EmilienM we need to be super careful with ^
19:49 mjblack EmilienM: could you take a quick look at https://review.openstack.org/#/c/313621/? I'm having trouble with the unit tests with going from ini setting to openstackconfig, its expecting a string but getting back an array.
19:50 mjblack I'm at a lost of where the issue is
19:50 EmilienM well, beaker jobs are failing too
19:50 pabelanger EmilienM: http://mirror.dfw.rax.openstack.org/centos/7/cloud/ online now, so you can updated your yum repolist to use it
19:51 EmilienM mjblack: error is pretty obvious http://logs.openstack.org/21/313621/2/check/gate-puppet-neutron-puppet-unit-4.0-centos-7/6bcf936/console.html.gz#_2016-05-06_17_03_27_605
19:51 EmilienM it expect a string and it got an array
19:51 EmilienM pabelanger: that is really awesome, I'm going to prepare that right now
19:51 mjblack EmilienM: right but I'm not sure where to fix it, unless you're saying update it to expect an array :D
19:52 pabelanger EmilienM: Ya, you can see how we do it in https://github.com/openstack-infra/project-config/blob/master/nodepool/scripts/configure_mirror.sh for CentOS-Base.repo
19:54 EmilienM pabelanger: I do it with puppet :)
19:55 EmilienM patch is coming
19:55 pabelanger EmilienM: right, /etc/nodepool/provider will give you the nodepool cloud details
20:02 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: switch centos mirror to use OpenStack mirror  https://review.openstack.org/316266
20:02 EmilienM pabelanger: ^ for master (for stable branches will come later)
20:02 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack_extras: Allow to change CentOS mirror URL  https://review.openstack.org/316267
20:02 EmilienM pabelanger: and that will be for stable branches ^
20:03 dmk0202 joined #puppet-openstack
20:03 EmilienM mjblack: https://review.openstack.org/#/c/313621/2/lib/puppet/type/neutron_config.rb
20:03 pabelanger EmilienM: that won't work well, as each cloud provider has a local mirror: EG: http://mirror.nyj01.internap.openstack.org/centos/7/
20:03 EmilienM it sounds like you're returning an array, isn't?
20:04 pabelanger EmilienM: you'll have to check which cloud / region you are in and configure accordingly
20:04 EmilienM o_O
20:04 EmilienM we can't do it with puppet
20:05 EmilienM well, actually we might be able with some tricks
20:05 EmilienM pabelanger: can you show me /etc/nodepool/provider example?
20:06 pabelanger EmilienM: http://docs.openstack.org/infra/nodepool/scripts.html#ready-script
20:06 pabelanger the configure_mirror.sh script I liked before will show how to build up the URL
20:07 EmilienM oh ok so we need to use it
20:07 pabelanger linked*
20:07 EmilienM pabelanger: configure_mirror.sh is automatically executed ?
20:08 pabelanger EmilienM: yes, each time we launch a node for the first time
20:08 pabelanger EmilienM: but we won't add the cloud repo into it, since no other job uses it
20:08 EmilienM mhh ok but I need to find how configure the cloud repo
20:08 EmilienM yeah, and it's versionned
20:08 EmilienM mitaka/liberty, etc
20:08 EmilienM I guess for now, I can use http://mirror.dfw.rax.openstack.org/centos/7/cloud/
20:09 EmilienM it will probably be a good start, isn't?
20:09 pabelanger give me a few minutes to think about it
20:10 EmilienM I'm going to sleep
20:14 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: WIP swift-init provider xenial support  https://review.openstack.org/316270
20:17 EmilienM pabelanger: if you have some idea, please let me know if Gerrit, thanks!
20:18 pabelanger EmilienM: will do
20:18 EmilienM thanks! have a nice week-end folks!
20:20 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Swift-ring-builder skip line 'Ring up-to-date..'  https://review.openstack.org/316274
20:24 morazi joined #puppet-openstack
20:31 xiaohua joined #puppet-openstack
20:47 chem` joined #puppet-openstack
20:59 openstackgerrit Merged openstack/puppet-trove: Use oslo module for logging  https://review.openstack.org/315548
21:02 johnmilton joined #puppet-openstack
21:07 iurygregory bye people, have a good weekend o/
21:17 vinsh Puppet 4.4.2... when applying file resources.. for a file that doesn't exist.  Causes an md5sum error.
21:17 vinsh it makes me sad.
21:18 vinsh *when applying a file resource from within a provider
21:20 vinsh I think this feature introduced some bug here https://tickets.puppetlabs.com/browse/PUP-5695
21:23 ducttape_ joined #puppet-openstack
21:24 vinsh only hitting this when running puppet apply locally.  Not seeing the same error in gate jobs.
21:24 vinsh hrm.
21:24 johnmilton joined #puppet-openstack
21:31 openstackgerrit Cody Herriges proposed openstack/puppet-neutron: Change MidoNet Plugin package name  https://review.openstack.org/294486
21:36 ansmith joined #puppet-openstack
21:38 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Support xenial with swift-init service provider  https://review.openstack.org/316270
21:40 openstackgerrit Adam Vinsh proposed openstack/puppet-swift: Support xenial with swift-init service provider  https://review.openstack.org/316270
21:52 dmk0202 joined #puppet-openstack
22:26 markvoelker joined #puppet-openstack
22:36 ducttape_ joined #puppet-openstack
22:42 mfisch EmilienM: je t'aime ceilometer: https://bugs.launchpad.net/puppet-ceilometer/+bug/1581691
22:42 openstack Launchpad bug 1581691 in puppet-ceilometer "ceilometer config options for agents are broken, polling agents are broken" [Undecided,New]
22:43 mfisch I will work on that this weekend
22:43 crinkle everything is broken \o/
22:43 mfisch we inherited that mess
22:43 mfisch its not puppet's fault
22:44 mfisch crinkle: https://twitter.com/openmfisch/status/730591623004848128
22:44 crinkle hahaha
22:59 xiaohua joined #puppet-openstack
23:28 markvoelker joined #puppet-openstack
23:33 cwolferh joined #puppet-openstack

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