Perl 6 - the future is here, just unevenly distributed

IRC log for #puppet-openstack, 2016-03-29

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

All times shown according to UTC.

Time Nick Message
00:06 openstackgerrit Alex Krzos proposed openstack/puppet-keystone: Tune number of apache processes/threads for Keystone in apache.  https://review.openstack.org/297342
00:15 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
00:17 rmallah joined #puppet-openstack
00:30 ducttape_ joined #puppet-openstack
00:36 ducttape_ joined #puppet-openstack
00:42 xarses joined #puppet-openstack
01:03 ducttape_ joined #puppet-openstack
01:37 ducttape_ joined #puppet-openstack
02:06 ducttape_ joined #puppet-openstack
02:19 ducttape_ joined #puppet-openstack
02:27 ducttape_ joined #puppet-openstack
02:34 rmallah joined #puppet-openstack
02:35 rmallah hi is there any howto on getting keystone working with puppet-openstack on ubuntu 14.04.4 LTS ?
02:38 rmallah https://gist.github.com/rmallah/804090709379cb6b8de7
02:38 rmallah i am trying to get above working.
02:40 ducttape_ joined #puppet-openstack
02:42 rmallah openstack is being called with unsupported args --default  ,  openstack project show: error: unrecognized arguments: --domain Default
02:43 rmallah https://gist.github.com/rmallah/8292d4541ada437f5ea6
02:44 mwhahaha rmallah: what version of the puppet scripts are you using and what version of openstack packages?
02:44 rmallah firstly thanks for the response.
02:45 rmallah ├── openstack-keystone (v7.0.0)
02:45 rmallah ├── openstack-keystone (v7.0.0)
02:45 rmallah root@controller01:~# openstack --version
02:45 rmallah openstack 0.3.0
02:45 rmallah root@controller01:~# lsb_release -a
02:45 rmallah No LSB modules are available.
02:45 rmallah Distributor ID: Ubuntu
02:45 rmallah Description:    Ubuntu 14.04.4 LTS
02:45 rmallah Release:        14.04
02:45 rmallah Codename:       trusty
02:46 rmallah how does one get to know if openstack version is Juno , Kilo , Liberty , Mitaka etc ?
02:59 rmallah ok now i have added ubuntu cloud ring and upgraded openstack packages , lets see how it goes,
03:03 fedexo joined #puppet-openstack
03:04 rmallah ************  OK , now it worked !! *************************
03:04 rmallah shall i attempt to patch the module to include ubuntu-cloud archive if version is 7.0.0
03:05 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
03:27 _ody rmallah: That is handled in https://github.com/openstack/puppet-openstack_extras/blob/master/manifests/repo/debian/ubuntu.pp.
03:28 rmallah thanks
03:29 rmallah i am new to openstack can anyone pls help with setting up admin-openrc.sh , i have already run puppet successfully.
03:30 rmallah my admin role is being setup as :         class { 'keystone::roles::admin': email        => 'admin', password     => 'super_secret', }
03:30 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
03:30 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Export path when puppet 4.  https://review.openstack.org/298509
03:33 mwhahaha ah sorry got side tracked
03:33 mwhahaha rmallah: that seems like an old version of the openstack client
03:34 rmallah yes
03:34 rmallah now i included UCA manually  then got
03:34 rmallah root@controller01:~# openstack --version
03:34 rmallah openstack 1.7.0
03:35 mwhahaha yea so that should probably address your problem
03:35 rmallah yes i just now  discovered https://github.com/openstack/puppet-openstack_extras    from @_ody 's comment.
03:35 rmallah the page also answeres one of my previously asked question.
03:38 openstackgerrit Merged openstack/puppet-neutron: Set default branch to stable/liberty  https://review.openstack.org/298260
03:38 rmallah can anyone pls help with OS_USERNAME , OS_PASSWORD , OS_AUTH_URL etc if openstack admin role is being setup as: class { 'keystone::roles::admin': email => 'admin', password => 'super_secret' }
03:47 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Add barbican client class and params  https://review.openstack.org/296156
03:47 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Moved logging to barbican-api module  https://review.openstack.org/296157
03:47 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Add helper files to configure paste file and barbican.conf  https://review.openstack.org/296158
03:47 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Add barbican-api manifest  https://review.openstack.org/296159
03:47 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Added KMIP plugin settings to barbican-api manifest  https://review.openstack.org/296163
03:47 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Added keystone config to barbican api manifest  https://review.openstack.org/263461
03:47 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Added settings for cert and secret store plugins  https://review.openstack.org/296162
03:47 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
03:47 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Add keystone notification options to barbican-api manifest  https://review.openstack.org/296161
03:47 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Added Dogtag settings for barbican-api  https://review.openstack.org/258851
03:47 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Add quota and secret limit parameters to barbican-api manifest  https://review.openstack.org/296160
04:01 brandon joined #puppet-openstack
04:07 openstackgerrit Michael Chapman proposed openstack/puppet-tripleo: Add neutron profiles  https://review.openstack.org/293436
04:09 brandon joined #puppet-openstack
04:13 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
04:14 aimon joined #puppet-openstack
04:16 rmallah joined #puppet-openstack
04:23 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
04:24 rmallah joined #puppet-openstack
04:25 brandon joined #puppet-openstack
04:30 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
04:46 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
04:48 openstackgerrit Alex Schultz proposed openstack/puppet-gnocchi: Switch gnocchi to rspec-puppet-facts  https://review.openstack.org/298528
04:56 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
04:56 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Disable secure_path  https://review.openstack.org/298530
05:03 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Disable secure_path  https://review.openstack.org/298530
05:03 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
05:03 rmallah joined #puppet-openstack
05:04 _ody Nothin like spelunking...
05:08 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Disable secure_path  https://review.openstack.org/298530
05:08 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
05:12 openstackgerrit Michael Chapman proposed openstack/puppet-tripleo: Add neutron profiles  https://review.openstack.org/293436
05:17 aimon joined #puppet-openstack
05:28 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Disable secure_path  https://review.openstack.org/298530
05:28 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
05:39 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Disable secure_path  https://review.openstack.org/298530
05:39 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
05:50 rmallah joined #puppet-openstack
05:56 jmikolajczak joined #puppet-openstack
05:58 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Dynamic secure_path  https://review.openstack.org/298530
05:58 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
06:08 jaosorior joined #puppet-openstack
06:11 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Update secure_path  https://review.openstack.org/298530
06:11 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
06:12 rmallah joined #puppet-openstack
06:19 tesseract joined #puppet-openstack
06:19 yolanda joined #puppet-openstack
06:22 rmallah joined #puppet-openstack
06:22 markvoelker joined #puppet-openstack
06:23 xenolog joined #puppet-openstack
06:23 markvoelker_ joined #puppet-openstack
06:26 ikkeT left #puppet-openstack
06:44 markvoelker joined #puppet-openstack
06:46 openstackgerrit Saverio Proto proposed openstack/puppet-cinder: Add osapi_max_limit parameter  https://review.openstack.org/298556
06:50 jtomasek joined #puppet-openstack
07:10 samuelBartel joined #puppet-openstack
07:23 jpena joined #puppet-openstack
07:29 openstackgerrit Cody Herriges proposed openstack/puppet-glance: Autorequire keystone_endpoint  https://review.openstack.org/298576
07:36 rcernin joined #puppet-openstack
07:49 openstackgerrit Denis Egorenko proposed openstack/puppet-glance: Add posibility to configure Glance backend settings for Glare service  https://review.openstack.org/298586
07:50 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
07:50 gfidente joined #puppet-openstack
08:11 derekh joined #puppet-openstack
08:13 markvoelker joined #puppet-openstack
08:23 jistr joined #puppet-openstack
09:09 khappone_ joined #puppet-openstack
09:14 mjb joined #puppet-openstack
09:14 mgagne joined #puppet-openstack
09:41 openstackgerrit Merged openstack/puppet-ironic: Allow chainloading of Inspector ramdisk over UEFI  https://review.openstack.org/276187
09:47 openstackgerrit Merged openstack/puppet-openstack-integration: Export path when puppet 4.  https://review.openstack.org/298509
10:14 markvoelker joined #puppet-openstack
10:47 openstackgerrit Nick Jones proposed openstack/puppet-horizon: Make syslog logging handler optional  https://review.openstack.org/298653
11:16 markvoelker joined #puppet-openstack
11:16 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: domain backend drivers set in domain config  https://review.openstack.org/298672
11:18 openstackgerrit Nick Jones proposed openstack/puppet-horizon: Make syslog logging handler optional  https://review.openstack.org/298653
11:21 EmilienM hello
11:23 johnmilton joined #puppet-openstack
11:30 openstackgerrit Matthew J Black proposed openstack/puppet-horizon: Added ensure_resource for python-memcache package.  https://review.openstack.org/298679
11:32 mjblack EmilienM: morning
11:32 dprince joined #puppet-openstack
11:36 colonwq EmilienM, good morning
11:52 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Update notification driver option  https://review.openstack.org/298687
11:56 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Update notification driver option  https://review.openstack.org/298687
12:00 degorenko morning :)
12:10 pegmanm joined #puppet-openstack
12:12 ducttape_ joined #puppet-openstack
12:17 markvoelker joined #puppet-openstack
12:24 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Update notification driver option  https://review.openstack.org/298687
12:27 openstackgerrit Denis Egorenko proposed openstack/puppet-aodh: Update notifications options  https://review.openstack.org/298702
12:27 darkxploit joined #puppet-openstack
12:32 mjblack anyone using ceilometer with the same rabbitmq as all the other components?
12:32 openstackgerrit Denis Egorenko proposed openstack/puppet-ceilometer: Update notifications options  https://review.openstack.org/298705
12:37 EmilienM mjblack: our ci does
12:38 openstackgerrit Denis Egorenko proposed openstack/puppet-cinder: Update notifications options  https://review.openstack.org/298708
12:39 jaosorior joined #puppet-openstack
12:39 openstackgerrit Denis Egorenko proposed openstack/puppet-murano: Update notification driver option  https://review.openstack.org/298687
12:43 jaosorior joined #puppet-openstack
12:45 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario003: deploy Neutron with ML2 linuxbridge  https://review.openstack.org/296102
12:46 openstackgerrit Michael Polenchuk proposed openstack/puppet-glance: [WIP] Add image properties support to glance_image  https://review.openstack.org/298718
12:51 morazi joined #puppet-openstack
12:53 michchap joined #puppet-openstack
12:55 openstackgerrit Matthew J Black proposed openstack/puppet-keystone: domain backend drivers set in domain config  https://review.openstack.org/298672
13:03 openstackgerrit Dimitri Savineau proposed openstack/puppet-tripleo: Enable HAProxy forwardfor option for Horizon.  https://review.openstack.org/296497
13:03 dprince joined #puppet-openstack
13:03 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: WIP: multi-domain testing with ldap backend.  https://review.openstack.org/296370
13:07 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-nova: Use oslo module for messaging (rabbit) configuraion  https://review.openstack.org/292309
13:08 openstackgerrit Giulio Fidente proposed openstack/puppet-ceilometer: Add 'ceilometer-service' to collector and agent/notification  https://review.openstack.org/298733
13:09 ayoung joined #puppet-openstack
13:11 openstackgerrit Giulio Fidente proposed openstack/puppet-ceilometer: Add 'ceilometer-service' tag to collector and agent/notification  https://review.openstack.org/298733
13:18 markvoelker joined #puppet-openstack
13:19 alee joined #puppet-openstack
13:19 openstackgerrit Matthew J Black proposed openstack/puppet-horizon: Added ensure_resource for python-memcache package.  https://review.openstack.org/298679
13:23 markvoelker joined #puppet-openstack
13:28 markvoelker_ joined #puppet-openstack
13:34 openstackgerrit Athlan-Guyot sofer proposed openstack/puppet-keystone: WIP: multi-domain testing with ldap backend.  https://review.openstack.org/296370
13:36 alee EmilienM, hey - so I'm trying to figure out how to do multiopt parameters
13:37 alee EmilienM, specifically, I have some parameters that I need to set using barbican_config()
13:37 EmilienM we have a provider for it.
13:38 EmilienM let me check if it landed
13:38 EmilienM alee: https://review.openstack.org/#/c/234727/
13:39 openstackgerrit Merged openstack/puppet-barbican: Add barbican client class and params  https://review.openstack.org/296156
13:39 EmilienM alee: it's used here: https://review.openstack.org/#/c/234728/
13:40 alee EmilienM, ah cool - that helps - thanks
13:40 EmilienM alee: https://review.openstack.org/#/c/234728/10/spec/acceptance/neutron_config_spec.rb look the multiple values for thisshouldexist3
13:49 chem mjblack: hi, is that ok for you if I take over https://launchpad.net/bugs/1554555, now that I have i basic ldap test env https://review.openstack.org/296370
13:49 openstack Launchpad bug 1554555 in puppet-keystone "openstack cli provider needs to pass domain in v3 calls" [Undecided,In progress] - Assigned to Matthew J Black (mjblack)
13:49 mjblack chem: sure
13:50 chem mjblack: It would mean for you to abandon https://review.openstack.org/#/c/289995
13:50 chem mjblack: ack thanks :)
13:50 openstackgerrit Matthew J Black proposed openstack/puppet-horizon: Added ensure_resource for python-memcache package.  https://review.openstack.org/298679
13:51 mjblack chem: change has been abandoned
13:52 chem mjblack: ack, going to make a placeholder for it right now
13:53 openstackgerrit Merged openstack/puppet-barbican: Moved logging to barbican-api module  https://review.openstack.org/296157
13:54 openstackgerrit Merged openstack/puppet-barbican: Add helper files to configure paste file and barbican.conf  https://review.openstack.org/296158
13:55 ducttape_ joined #puppet-openstack
14:00 dfisher joined #puppet-openstack
14:02 dhollinger joined #puppet-openstack
14:02 richm joined #puppet-openstack
14:11 iberezovskiy EmilienM, hi. I'm working to switch some modules to puppet-oslo usage. now master is opened for N, so can I drop deprecated parameters (which has warning ?
14:11 EmilienM iberezovskiy: yes
14:11 iberezovskiy cool, thx :)
14:12 EmilienM no need of deprecation, since the module never got a release.
14:12 EmilienM iberezovskiy: puppet-oslo is not used anywhere, right?
14:12 iberezovskiy I mean I'll drop deprecated parameters from other modules which I am switching to oslo usage
14:13 iberezovskiy e.g. glance and https://github.com/openstack/puppet-glance/blob/master/manifests/notify/rabbitmq.pp#L110
14:13 EmilienM oh right
14:13 EmilienM drop it
14:14 iberezovskiy great, thx
14:14 EmilienM iberezovskiy: please start with *one* module, before doing all
14:14 EmilienM so we can first review and adjust if needed puppet-oslo
14:14 iberezovskiy EmilienM, nova is ready https://review.openstack.org/#/c/292309/
14:14 EmilienM ok will look today
14:15 iberezovskiy I didn't add release notes, but I will
14:18 iberezovskiy EmilienM, ping me please, when you add puppet-oslo to tripleo, so I'll publish new patchset
14:18 EmilienM hopefully today
14:18 iberezovskiy good
14:19 openstackgerrit Gonéri Le Bouder proposed openstack/puppet-ironic: expose the ipxe_enabled parameter  https://review.openstack.org/296774
14:37 xarses joined #puppet-openstack
14:43 openstackgerrit Lokesh Jain proposed openstack/puppet-neutron: Support for Nuage Neutron plugin in puppet-neutron  https://review.openstack.org/298792
14:44 dhollinger joined #puppet-openstack
14:54 _degorenko|afk .тшсл вупщкутлщ
14:54 _degorenko|afk oops
14:54 _degorenko|afk sorry
14:54 haplo37 joined #puppet-openstack
14:57 dmsimard degorenko: well, that's not very useful lol https://translate.google.ca/?safe=off&bav=on.2,or.r_cp.&bvm=bv.117868183,d.dmo&biw=2560&bih=1325&dpr=1&um=1&ie=UTF-8&hl=en&client=tw-ob#uk/en/%D1%82%D1%88%D1%81%D0%BB%20%D0%B2%D1%83%D0%BF%D1%89%D0%BA%D1%83%D1%82%D0%BB%D1%89
14:57 iurygregory lol
14:57 iurygregory ukrainian to english O.o
14:58 degorenko dmsimard, don't worry :) that's just letters on keyboard under english, here is no any logic :)
14:59 EmilienM meeting in 2 min!
14:59 EmilienM or in 15s
14:59 iurygregory or now
14:59 iurygregory XD
15:07 openstackgerrit joined #puppet-openstack
15:07 dgurtner joined #puppet-openstack
15:07 dgurtner joined #puppet-openstack
15:12 xingchao joined #puppet-openstack
15:25 dgurtner joined #puppet-openstack
15:33 dtantsur EmilienM, per new backport policy, should I abandon https://review.openstack.org/293366 ?
15:34 EmilienM dtantsur: not sure
15:34 EmilienM I think we can backport it, let me some time to look at it today
15:34 dtantsur last time it failed CI due to outdated centos ironic-inspector package
15:34 dtantsur dunno if it got fixed
15:35 dtantsur still old: http://mirror.centos.org/centos/7/cloud/x86_64/openstack-liberty/openstack-ironic-inspector-2.2.2-1.el7.noarch.rpm :(
15:36 EmilienM dtantsur: we'll land your patch if CI is passing
15:37 dtantsur got it, thanks
15:43 _ody EmilienM: Wasn't paying atention there are the end of the meeting. Yeah things are gong fine.  You can see I simplified the change set yesterday by a lot.
15:43 mdorman joined #puppet-openstack
15:44 EmilienM excellent
15:44 _ody I was inject more opinion than functionality so I backed off.
15:44 haplo37 joined #puppet-openstack
15:46 angdraug joined #puppet-openstack
15:46 mwhahaha openstack image list --long doens't include properties :(
15:47 dprince joined #puppet-openstack
15:51 EmilienM mwhahaha: good for you, you'll contribute to osclient :P
15:51 mwhahaha i already have and it was painful :D
15:51 mwhahaha and since it wont land until N it wouldn't be helpful in the case
15:51 mwhahaha time for terrible workaroudns!
15:53 EmilienM mwhahaha: it's fair
15:54 EmilienM mwhahaha: you want it in mitaka?
15:54 mwhahaha would be ideal but i'm trying to see how much work it is
15:55 mwhahaha basically trying to kill this thing: https://github.com/openstack/fuel-library/blob/master/deployment/puppet/osnailyfacter/modular/astute/upload_cirros.rb
15:56 mwhahaha but need to be able to support --property <key>=<value> which i could add easily to the provider
15:56 mwhahaha but i'd like to do it correctly
15:56 mwhahaha :) so we'll see
15:57 mwhahaha or at least somethings that's not completely awful
15:58 xarses joined #puppet-openstack
16:00 openstackgerrit Merged openstack/puppet-barbican: Add barbican-api manifest  https://review.openstack.org/296159
16:01 ducttape_ joined #puppet-openstack
16:04 jtsigma joined #puppet-openstack
16:05 degorenko crinkle, hey :) i've answered on your questions here: https://review.openstack.org/226862 Can you please take a look?
16:08 jtsigma Hi, this is more a puppet question than openstack but I'm trying to understand the keystone resource, as an example in the puppet-heat module I see that the only code that has keystone_user_role is in the form of Resource Reference.   I don't see the initial resource established.  How does that work?
16:08 jtsigma https://review.openstack.org/226862
16:08 jtsigma whoops ignore that review
16:08 jtsigma ubuntu@W110ER:~/Downloads/puppet-heat$ grep -Ri keystone_user_role|grep -v spec
16:09 jtsigma manifests/keystone/auth_cfn.pp:    Keystone_user_role["${auth_name}@${tenant}"] ~>
16:09 jtsigma manifests/keystone/auth.pp:    Keystone_user_role["${auth_name}@${tenant}"] ~>
16:09 jtsigma I thought that in order to use a Resource reference, you have to specify the resource somewhere first.  Using   keystone_user_role { "title": }
16:09 openstackgerrit Cody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint  https://review.openstack.org/298397
16:09 crinkle degorenko: done
16:10 degorenko crinkle, thank you :)
16:12 openstackgerrit Merged openstack/puppet-glance: Add posibility to configure Glance backend settings for Glare service  https://review.openstack.org/298586
16:12 mwhahaha jtsigma: i think we change out the title to match user@project
16:12 jtsigma yes , sorry i just put something random in there
16:14 mwhahaha jtsigma: i think we define that resource in https://github.com/openstack/puppet-heat/blob/master/manifests/keystone/auth_cfn.pp#L208-L223
16:15 openstackgerrit Cody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint  https://review.openstack.org/298397
16:15 mwhahaha so we're cheating because we know that keystone::resource::service_identity will create a keystone_user_role["user@project"]
16:15 mwhahaha https://github.com/openstack/puppet-keystone/blob/master/manifests/resource/service_identity.pp#L165-L170
16:15 jtsigma mwhahaha,   very interesting!   Wow, i would've never figured that out.  Thanks saved me a ton of time
16:16 alee EmilienM, yeehah! finally ..
16:17 jtsigma it's interesting to me tho that altho the auth_cfn manifest is in keystone folder that when you call 'keystone::resource::service_identity' it knows it's not a relative class.   I didn't think it was calling keystone module because i thought you would then have to specify it absolute like ::keystone::resource::service_identity
16:17 todin joined #puppet-openstack
16:17 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Added KMIP plugin settings to barbican-api manifest  https://review.openstack.org/296163
16:17 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Added keystone config to barbican api manifest  https://review.openstack.org/263461
16:17 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Added settings for cert and secret store plugins  https://review.openstack.org/296162
16:17 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
16:17 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Add keystone notification options to barbican-api manifest  https://review.openstack.org/296161
16:17 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Added Dogtag settings for barbican-api  https://review.openstack.org/258851
16:17 openstackgerrit Ade Lee proposed openstack/puppet-barbican: Add quota and secret limit parameters to barbican-api manifest  https://review.openstack.org/296160
16:17 mwhahaha that's a scope determination thing in puppet, it's not a hard requirement unless you have overlapping names
16:18 mwhahaha it's a best practice to use the global leading :: but we've got some places where that hasn't been adopted
16:18 jtsigma ah i see.  thanks again
16:18 alee EmilienM, degorenko -- those last ones should  have all required changes at least for the bottom two patches.
16:19 _ody keystone::resource::service_identity is a defined resources.  It probably won't accept a leading ::.
16:19 alee still working on the one that requires multi-opt
16:20 mwhahaha i think resources can have a leading ::
16:20 * mwhahaha shrugs
16:20 * _ody shurgs too
16:20 _ody Never done with with resources.
16:21 _ody We only ever had the scoping problem with classes...but maybe we just didn't notice for some reason.
16:21 mwhahaha yea i could see the class scope thing being more common
16:22 jtsigma that newest keystone module is giving me probs because on trusty i have openstack client 1.0.3 and so the new keystone module uses "openstack domain list" which apparently isn't an option for my openstack client.
16:23 dhollinger joined #puppet-openstack
16:23 mwhahaha yea you have to match the module to the client version and vice versa
16:23 jtsigma ooh ok
16:23 _ody jtsigma: Could aslo be that you are using the wrong keystone api version.
16:23 jtsigma yes i'm using api v2
16:23 _ody Yeah.  That won't work for domains.
16:24 jtsigma but even if i set the API_VERSION=3 in my openrc, openstack client v1.0.3 doesn't appear to accept that
16:24 jtsigma i tested with a newer version on a diff pc an that appears to be ok.  but on this particular system 1.0.3 it doesn't like
16:29 derekh joined #puppet-openstack
16:30 ansmith joined #puppet-openstack
16:38 mgagne joined #puppet-openstack
16:49 jtomasek joined #puppet-openstack
16:54 openstackgerrit Merged openstack/puppet-nova: Use OpenstackClient for nova providers auth  https://review.openstack.org/226862
16:59 openstackgerrit Denis Egorenko proposed openstack/puppet-oslo: Add ability to configure oslo_messaging_notifications  https://review.openstack.org/298874
17:02 delattec joined #puppet-openstack
17:04 alee EmilienM, degorenko https://review.openstack.org/#/c/296160  and the next one up ..
17:05 alee should be ready to go.
17:06 llua` joined #puppet-openstack
17:06 dmsimard1 joined #puppet-openstack
17:07 Anticime1 joined #puppet-openstack
17:07 mgagne_ joined #puppet-openstack
17:07 mihgen joined #puppet-openstack
17:07 _matthias__ joined #puppet-openstack
17:07 mfisch joined #puppet-openstack
17:07 mjb joined #puppet-openstack
17:07 mgagne_ joined #puppet-openstack
17:08 jasondotstar joined #puppet-openstack
17:08 bastelfreak joined #puppet-openstack
17:11 degorenko _ody, hey, why you abandoned your patch to glance? i don't see any alternative solutions, can you point me?
17:11 openstackgerrit Cody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint  https://review.openstack.org/298397
17:12 _ody degorenko: It is here, https://github.com/openstack/puppet-glance/blob/master/manifests/keystone/auth.pp#L184-L187.
17:13 degorenko _ody, ok. And btw, autorequire equals to collectors?
17:13 _ody They basically work the same.
17:14 _ody If they find a resource then they create the relationship, if they don't then nothing happens.
17:14 degorenko _ody, hm, i though that they have strong dependency
17:15 _ody Nah.
17:15 _ody degorenko: Can you take a loot at this, https://github.com/openstack/puppet-glance/blob/master/manifests/keystone/auth.pp#L184-L203.
17:15 _ody I think that is the reason I went down the path of fixing glance, our endpoint name endes up being different then the relationship.
17:15 degorenko _ody, correct, you're right https://docs.puppetlabs.com/puppet/latest/reference/lang_relationships.html#auto-relationships
17:15 degorenko thank you for info :)
17:16 * degorenko looking
17:16 holser_ joined #puppet-openstack
17:17 saneax_AFK joined #puppet-openstack
17:17 igorbelikov joined #puppet-openstack
17:17 degorenko _ody, so, for your link
17:17 degorenko it may be not happen, yes
17:17 degorenko when we have undefined service_type parameter
17:18 degorenko _ody, take a look here: https://github.com/openstack/puppet-keystone/blob/master/manifests/resource/service_identity.pp#L183
17:19 _ody yeah.  While confusing at first it looks like service_name is being passed through
17:19 aglarendil joined #puppet-openstack
17:19 degorenko _ody, as for glance we always have default https://github.com/openstack/puppet-glance/blob/master/manifests/keystone/auth.pp#L110
17:20 degorenko same for service_name
17:21 degorenko so, it should work on defaults
17:22 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
17:23 igorbelikov joined #puppet-openstack
17:31 ducttape_ joined #puppet-openstack
17:32 EmilienM alee: looking
17:32 alee EmilienM, thanks
17:33 guimaluf joined #puppet-openstack
17:35 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-glance: Use oslo module for messaging (rabbit) configuration  https://review.openstack.org/298893
17:37 EmilienM alee: for quotas, is it worth having quota.pp like in puppet-neutron?
17:37 EmilienM we usually try to not overload classes
17:38 alee EmilienM, looking
17:39 EmilienM degorenko, alee: I have the feeling we put everything in api.pp
17:39 EmilienM it's not a blocker for me
17:39 EmilienM it's just an highlight of an error of design
17:39 degorenko EmilienM, alee i will look in a few minutes :)
17:40 alee EmilienM, it makes for a smaller module, but I think also means you have to specify more resources to get a server
17:41 alee EmilienM, I could see going either way with this.
17:41 openstackgerrit Denis Egorenko proposed openstack/puppet-nova: Add release note for OpenStack client in nova providers  https://review.openstack.org/298898
17:41 degorenko EmilienM, sorry, missed that for my openstack client in nova patch :D ^^
17:42 alee EmilienM, can I suggest we consider this as a refinement patch after these go in?
17:44 EmilienM well, the other way is to make it clean by design
17:47 delatte joined #puppet-openstack
17:47 EmilienM alee: I commented https://review.openstack.org/#/c/296161/ for alignments
17:48 alee EmilienM, I can go either way on this - it makes it more composable, but also means you need to define more things when creating a server.
17:48 EmilienM right, but that's not a problem
17:48 EmilienM a lot of puppet modules outside our forge are using a single class for everything, we don't like it much
17:49 EmilienM degorenko: thoughts?
17:51 EmilienM in fact, I'll -1 all the patches
17:51 EmilienM every patch you do is in api.pp
17:51 openstackgerrit Giulio Fidente proposed openstack/puppet-ceilometer: Add 'ceilometer-service' tag to collector and agent/notification  https://review.openstack.org/298903
17:51 EmilienM alee: look at other modules, please, they are all splitted
17:51 EmilienM in neutron, we have quota.pp, and also notifications.pp
17:52 degorenko EmilienM, you're suggesting to have separate classes for some options?
17:52 EmilienM yeah
17:52 EmilienM for quotas, notifications, plugins, etc
17:52 openstackgerrit Giulio Fidente proposed openstack/puppet-ceilometer: Set defaultbranch for git-review to stable/liberty  https://review.openstack.org/298906
17:52 degorenko well, in fact, we can have separate classes for configuring this, yes
17:52 openstackgerrit Giulio Fidente proposed openstack/puppet-ceilometer: Add 'ceilometer-service' tag to collector and agent/notification  https://review.openstack.org/298908
17:53 degorenko if we have some complex configurations, yes, i'm +1 for this
17:53 alee EmilienM, thats because all the parameters that are added there are parameters that are used by the api server.  What do you think should be in the api.pp ?
17:53 degorenko alee, you can use include from api then
17:53 EmilienM alee: this is not the right way to think
17:53 EmilienM alee: a manifest is not a service
17:53 degorenko well, it is same situation as for logging and db classes
17:53 delattec joined #puppet-openstack
17:53 EmilienM it can also be a set of parameters
17:53 EmilienM I don't want to end-up with a api.pp of 4000 lines
17:54 EmilienM and this is happenning
17:54 degorenko +100500
17:55 degorenko we can have some basic options for api class, but yeah, i'm also against to keep all possible options in one class
17:55 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: scenario002/centos7: switch RabbitMQ and OpenStack to IPv6  https://review.openstack.org/287521
17:55 EmilienM crinkle: commit message updated ^
17:55 openstackgerrit Cody Herriges proposed openstack/puppet-cinder: Establish cinder_type after keystone_endpoint  https://review.openstack.org/298397
17:56 EmilienM degorenko: specially the plugins
17:57 alee EmilienM, so what do you think should be in api.pp ?
17:57 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
17:57 EmilienM alee: what is currently in there, is fine
17:57 EmilienM but no more, imho
17:58 _ody crinkle or EmilienM: do either of you have the openstack forge account password?
17:58 alee EmilienM, degorenko I dont mind splitting it out -- I'm just trying to understand the rationale for a separate manifest as opposed to the api manifest
17:59 EmilienM _ody: I have it
17:59 EmilienM _ody: let me fix that
17:59 degorenko alee, well, you should have for such classes only important options
18:00 degorenko alee, if you need improve configuration with many options (with one logic) - create a new class
18:00 degorenko alee, as i said above - it is exactly same situation as for logging and db classes
18:03 hjensas joined #puppet-openstack
18:03 hjensas joined #puppet-openstack
18:03 alee degorenko, EmilienM ok -- so for instance , you would say a plugins.pp with the parameters that determine which plugins are enabled, and then a dogtag.pp for the dogtag plugin, kmip.pp for the kmip plugin etc. ?
18:04 EmilienM or you can have manifests/plugins/foo.pp
18:04 EmilienM and manifests/plugins/bar.pp
18:04 degorenko +1 for ^
18:04 EmilienM https://github.com/openstack/puppet-neutron/tree/master/manifests
18:05 EmilienM alee: please look how we do in other modules
18:06 llua joined #puppet-openstack
18:07 EmilienM _ody: https://review.openstack.org/#/c/298530/7/run_tests.sh sounds like a huge hack, can't we live without that?
18:08 alee EmilienM, ok
18:08 cdelatte joined #puppet-openstack
18:09 iberezovskiy EmilienM, looks I need your advice again :) I didn't find rabbit_notification_exchange parameter in glance sample config, but we set it https://github.com/openstack/puppet-glance/blob/master/manifests/notify/rabbitmq.pp#L144. it looks like it should be 'controler_exchange'. Am I right?
18:10 EmilienM iberezovskiy: should be in oslo messaging, isn't?
18:10 iberezovskiy control_exchange is in default section
18:11 iberezovskiy but I wanna set it from oslo::messaging_rabbit class
18:12 EmilienM wait
18:12 _ody EmilienM: We could make symlinks into a directory that is in secure path, as the alternative.
18:12 EmilienM why do we have oslo::messaging_rabbit and not oslo::messaging::rabbit?
18:13 iberezovskiy https://github.com/openstack/puppet-oslo/commit/b86091eb6fa5bbca7931d6407cf3e2b1c0cf3f3d
18:13 _ody I could also maybe re-arrange the order we define variables in run_tests.sh and use the full path to puppet, which might work.
18:13 EmilienM iberezovskiy: I just realized.
18:14 EmilienM while we're building this module, it might be good to rename the class
18:14 iberezovskiy np, I'll rename
18:14 EmilienM it would be very clear to have oslo::messaing::<driver>
18:14 iberezovskiy yep, like it's done for oslo::messaging::amqp
18:14 EmilienM xingchao: ^ fyi
18:15 EmilienM iberezovskiy: ok, please rename it and update the puppet-nova patch -using depends-on
18:15 EmilienM we're very close! thanks for your efforts
18:15 iberezovskiy sure
18:16 dilyin joined #puppet-openstack
18:16 EmilienM mwhahaha: dilyin and I are working together on merging puppet-pacemaker efforts, see https://review.openstack.org/#/c/296440/
18:17 EmilienM mwhahaha: it would be great to run Fuel CI jobs on openstack/puppet-pacemaker, somehow maybe an external job
18:17 xingchao EmilienM: get it
18:17 mwhahaha yea we can probably get that up when we've got the two merged together
18:17 EmilienM mwhahaha: so we can iterate on the merging without breaking both tripleo & fuel
18:17 EmilienM xingchao: thx
18:17 dilyin i can run unit tests already
18:17 EmilienM dilyin: mhh I would prefer also have functional tests
18:17 dilyin there is a job for them
18:17 EmilienM and having fuel CI running would help us
18:18 cdelatte joined #puppet-openstack
18:18 mwhahaha hmm ok would probably need a special job for that
18:18 dilyin yes, i'll adapt beaker tests too
18:18 EmilienM mwhahaha: yes
18:18 iberezovskiy igorbelikov, ^
18:18 EmilienM mwhahaha: an HA job that use openstack/puppet-pacemaker instead of fuel-infra/puppet-pacemaker
18:18 EmilienM if I'm not wrong
18:18 mwhahaha yea
18:19 EmilienM cool. So we can iterate, and keep testing it.
18:19 dilyin do we have beaker support in openstack CI?
18:19 EmilienM dilyin: of course
18:19 dilyin then we're cool it i'll be supporting acceptance tests
18:19 EmilienM on both trusty & centos7. chem already did the work on centos7 for openstack/puppet-pacemaker -> https://review.openstack.org/#/c/281376/
18:20 EmilienM cool
18:20 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Avoid secure_path  https://review.openstack.org/298530
18:20 openstackgerrit Cody Herriges proposed openstack/puppet-openstack-integration: Add explicit dependencies to pass puppet4  https://review.openstack.org/296557
18:20 EmilienM dilyin: chem is currently the guy who actively works on the module, you might sync with him tomorrow
18:20 dilyin ok, i will
18:21 openstackgerrit Merged openstack/puppet-ceilometer: Add 'ceilometer-service' tag to collector and agent/notification  https://review.openstack.org/298733
18:21 EmilienM _ody: I like it better :)
18:22 _ody EmilienM: Lets give the full path thing a go.
18:22 openstackgerrit Ivan Berezovskiy proposed openstack/puppet-oslo: Fix define name to oslo::messaging::rabbit  https://review.openstack.org/298923
18:25 chem EmilienM: dilyin (back for a few moments) we have already sync a bit ... for the beaker jobs on ubuntu it's only expecting a failure (as pcs is not there) but we only have to remove the case and see if it works on both ubuntu and centos: https://github.com/openstack/puppet-pacemaker/blob/master/spec/acceptance/basic_one_node_spec.rb#L64-L67
18:25 chem EmilienM: dilyin but I'm using pcs command ... so :)
18:25 EmilienM we'll have to hack on XML, probably
18:26 EmilienM jayg: fyi ^
18:26 chem EmilienM: dilyin it's only from https://github.com/openstack/puppet-pacemaker/blob/master/spec/acceptance/basic_one_node_spec.rb#L34 on that I use pcs to check the resource further.  The first tests are comptatible
18:27 EmilienM jayg: tl; dr: we're about to merge both puppet-pacemaker from fuel & tripleo - blocker is pcs does not exist on ubuntu - see https://review.openstack.org/#/c/296440/
18:37 EmilienM dilyin, mwhahaha, chem: in term of actions: fuel can create a separated CI job for testing openstack/puppet-pacemaker with Fuel. What can we do in our side?
18:37 iurygregory joined #puppet-openstack
18:37 dilyin yes, but it will be possible only when i finish merging, it will require some changes in fuel too
18:46 igorbelikov EmilienM, mwhahaha, dilyin: we can enable fuel-ci for puppet-pacemaker by just adding it to the list of projects fuel-ci watches.  It doesn’t matter if puppetfile points to fuel-infra instead of openstack, CI jobs will pick up the change
18:46 EmilienM igorbelikov: how?
18:49 igorbelikov EmilienM: it will fetch the change from gerrit before firing up librarian-puppet-simple, so librarian will skip it
18:49 EmilienM mhh ok
18:49 EmilienM I'll trust you on this one :-)
18:55 EmilienM crinkle: https://github.com/openstack/puppet-neutron/commit/bb9ec5d7d30053cac232ec6626f431b4ed44c80d I think it also applis to OVS agent and all oslo messaging params
18:57 crinkle EmilienM: huh you're probably right
18:58 EmilienM I'm sending a path
18:58 EmilienM patch
18:58 angdraug joined #puppet-openstack
18:59 EmilienM mflobo: I created launchpad Newton serie
18:59 EmilienM err mfish
18:59 EmilienM he's not around
19:00 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-release-tools: fix puppet-openstack-spec-helper name  https://review.openstack.org/298939
19:01 ducttape_ joined #puppet-openstack
19:02 EmilienM Guest82454: ^
19:02 EmilienM crinkle: I'm wondering if subscribing to the whole Neutron_config is safe or not
19:02 EmilienM because it's not only rabbit options, but also logging, etc
19:02 Guest82454 joined #puppet-openstack
19:03 mfisch whats up
19:03 EmilienM mfisch: I updated launchpads
19:03 EmilienM mfisch: we have 9.0.0 serie open
19:03 EmilienM look https://launchpad.net/puppet-keystone
19:03 crinkle EmilienM: yeah idunno, i'd rather not have the agent restarting all the time but i also don't know what it needs to update on
19:04 EmilienM crinkle: at least: amqp & logging options, but maybe more.
19:04 EmilienM it's a lot of crappy code to create this list
19:05 EmilienM crinkle: I could take your linuxbridge_agent_subscribe and create an array of all the params that we want, but it's crappy to me
19:05 EmilienM we'll have to maintain that list
19:06 crinkle yeah that sounds gross
19:06 crinkle but i worry about ovs agent restarting all the time
19:06 crinkle that will make an unhappy cloud
19:06 EmilienM crinkle: since we're moving things to puppet-oslo, we can (later) subscribe on puppet-oslo changes
19:06 EmilienM that is true, restarting agent sucks
19:07 crinkle ¯\_(ツ)_/¯
19:07 EmilienM crinkle: I'll think about it again when we have puppet-oslo in place
19:07 crinkle i don't have a good suggestion
19:07 crinkle okay
19:08 clayton I got stuck on this issue recently
19:08 clayton I was working on adding hooks support for puppet-neutron, and really it should be subscribing to neutron_config
19:09 clayton but it's clearly risky given the current tstate
19:09 EmilienM crinkle: see https://review.openstack.org/#/c/292309/7/manifests/init.pp
19:09 EmilienM maybe can we subscribe to the define? I'm not sure
19:09 clayton I've got two patches in progress and hynekm has another to fix the issue
19:10 EmilienM clayton: what are the links, please?
19:10 clayton I'm expecting it'll land for newton and get backported at least to mitaka
19:10 clayton this is the bug: https://bugs.launchpad.net/neutron/+bug/1514056
19:10 openstack Launchpad bug 1514056 in neutron "Restarting OVS agent drops VMs traffic when using VLAN provider bridges" [High,In progress] - Assigned to Hynek Mlnarik (hmlnarik-s)
19:10 clayton with my two patches + hynek's patch, ovs agent restarts don't cause traffic drops
19:11 EmilienM nice
19:11 clayton this is the last core openstack service I know of that can't be gracefully restarted
19:12 EmilienM clayton: what about l3 agent?
19:12 clayton l3 agent has been safe for a while afik
19:12 clayton although there are parallelization issues with restarting them on multiple boxes until mitaka
19:13 clayton so if you're running a lot of l3 agents and restart all of them in a short period of time, you're going to have problems
19:13 clayton but that's not really a problem puppet can solve, and we're dealing with mitaka anyway
19:14 clayton well, we = people working on puppet module master branch only have to worry about mitaka+ :)(
19:14 clayton I'm still trying to get us on liberty
19:14 EmilienM clayton: you're running liberty now?
19:14 clayton cinder, nova, neutron and glance are still on liberty
19:14 clayton err, no
19:14 clayton cinder, nova, neutron and glance are still on kilo
19:14 brandon joined #puppet-openstack
19:15 clayton other services are on liberty
19:15 clayton mfisch is rolling keystone up to liberty this week
19:15 EmilienM nice
19:15 clayton we probably won't seriously look at anything on mitaka until after the summit.  probably heat or keystone first
19:16 mfisch not keystone
19:16 clayton given some of the bugs that got fixed late in the liberty stable cycle for neutron and nova, I'm not excited about rushing into mitaka
19:16 mfisch not until perf is fixed
19:17 mfisch heat after a real RC is out
19:17 mfisch rc2 rolled this morning
19:17 clayton we'll finish our liberty upgrades and finish moving neutron, glance and cinder into docker, and probably just cherry-pick fixes for a while as we need them
19:26 openstackgerrit Lokesh Jain proposed openstack/puppet-neutron: Support for Nuage Neutron plugin in puppet-neutron  https://review.openstack.org/298792
19:29 EmilienM mwhahaha: can you please look https://review.openstack.org/#/c/287521/ when you can? i'll follow up with other scenarios asap
19:29 mwhahaha k
19:38 dalees` joined #puppet-openstack
19:39 jmikolajczak joined #puppet-openstack
19:45 igorbelikov EmilienM, mwhahaha: fuel-ci now runs for puppet-pacemaker
19:46 mwhahaha cool thanks
19:48 EmilienM igorbelikov: thx a lot :-)
19:58 ducttape_ joined #puppet-openstack
19:59 alee EmilienM, I notice that in https://review.openstack.org/#/c/234728 , there are no unit tests in spec/unit/type/ and spec/unit/provider ..
19:59 alee EmilienM, is there a template for one somewhere that I can copy in/modify ?
20:00 EmilienM not afik
20:00 EmilienM but acceptance + rspec is enough, for now
20:00 alee EmilienM, ok
20:00 iurygregory joined #puppet-openstack
20:02 ducttape_ joined #puppet-openstack
20:19 ericksonsantos joined #puppet-openstack
20:19 openstackgerrit Merged openstack/puppet-openstack-integration: scenario002/centos7: switch RabbitMQ and OpenStack to IPv6  https://review.openstack.org/287521
20:19 EmilienM w00t
20:20 _ody annoying sporadic CI failures today.
20:21 xingchao joined #puppet-openstack
20:21 EmilienM _ody: which ones?
20:21 _ody p-o-i.  Package related the ones I've seen.
20:22 EmilienM _ody: can you give me details please? logs? link?
20:22 _ody Yeah.  Getting.
20:22 EmilienM I'm trying to define a list of issues
20:22 EmilienM and kill them
20:22 EmilienM one of them is: stop using EPEL
20:22 _ody heh.
20:22 EmilienM EPEL is quite often down
20:23 EmilienM dmsimard: do you have some workaround to not use EPEL in our scenario001 ?
20:23 _ody EmilienM: One example, http://logs.openstack.org/30/298530/8/check/gate-puppet-openstack-integration-3-scenario001-tempest-devstack-centos7/7876eaf/console.html#_2016-03-29_18_26_34_127
20:24 dmsimard EmilienM: For Ceph ?
20:24 EmilienM dmsimard: yeah
20:25 dmsimard EmilienM: It's not a pretty workaround.
20:25 EmilienM lol
20:25 dmsimard EmilienM: I explained the issue here: https://bugs.launchpad.net/puppet-ceph/+bug/1562998
20:25 openstack Launchpad bug 1562998 in puppet-ceph "EPEL requirement on CentOS is only because of leveldb" [Undecided,New]
20:26 dmsimard I guess you could not use ceph::repo and instead set up the CentOS Storage SIG repo instead
20:26 dmsimard The dependencies are self-contained
20:26 _ody EmilienM: Other seems to be related to the job getting killed because it has ran for over 60 minutes.
20:26 EmilienM dmsimard: _ody's issue is likely a RDO server downtime, isn't?
20:26 EmilienM _ody: ouch... where does it timeout? It takes time to investigate and finds the root issue.
20:26 dmsimard We haven't had downtime recently that I know about
20:26 EmilienM "No more mirrors to try."
20:26 _ody EmilienM: http://logs.openstack.org/57/296557/24/check/gate-puppet-openstack-integration-3-scenario001-tempest-ubuntu-trusty/c74c92b/console.html#_2016-03-29_19_43_19_232
20:27 _ody That is the other type of failure I am seeing
20:27 EmilienM wait
20:27 EmilienM python-pandas-0.17.0-2.el7.x86_64: [Errno 256] No more mirrors to try.
20:27 EmilienM it's in EPEL? no..?
20:27 dmsimard pandas is in the RDO repos
20:27 EmilienM ok
20:27 dmsimard delorean-deps for trunk
20:27 EmilienM so yeah, you don't know it but RDO has also some downtimes recently
20:27 EmilienM _ody: that's a git server issue
20:27 EmilienM _ody: look the time it takes to checkout repos
20:27 EmilienM _ody: when it happens, please jump on #openstack-infra and say it
20:28 EmilienM it's likely an issue with infra servers
20:30 jtomasek_ joined #puppet-openstack
20:50 jmikolajczak joined #puppet-openstack
20:51 EmilienM _ody: w00t, scenario003 is passing puppet4 ? https://review.openstack.org/#/c/296557/
20:53 openstackgerrit Merged openstack/puppet-cinder: Establish cinder_type after keystone_endpoint  https://review.openstack.org/298397
20:53 EmilienM dmsimard: ok so I guess we can stop using EPEL and use http://cbs.centos.org/repos/storage7-ceph-hammer-candidate/x86_64/os/Packages/
20:54 EmilienM the main motivation is to stop using EPEL
21:02 _ody EmilienM: I think other might be passing.  I've seen the whole 60 minutes thing on those jobs.
21:05 dmsimard EmilienM: it kinda sucks but yeah
21:05 dmsimard EmilienM: I believe there's a package you can install that sets up the repo
21:06 dmsimard like yum install centos-release-ceph
21:06 dmsimard or something
21:06 EmilienM yeah? that would be cool
21:06 dmsimard the ceph repo was added fairly recently yet so I don't know if it's there yet
21:06 dmsimard checking
21:08 dmsimard EmilienM: well isn't that convenient: http://paste.openstack.org/show/492362/
21:09 EmilienM dmsimard: :D
21:09 EmilienM i'll test it after cross project meeting
21:22 xingchao joined #puppet-openstack
21:23 openstackgerrit Alex Schultz proposed openstack/puppet-glance: Add ability to set properties with glance_image  https://review.openstack.org/298718
21:27 openstackgerrit Matt Fischer proposed openstack/puppet-openstacklib: Don't claim you are retrying when you are not  https://review.openstack.org/298992
21:29 mfisch EmilienM: check out that awesome bug fix ^
21:29 mfisch retryin' was lyin'
21:30 mwhahaha lol
21:30 mwhahaha WHY YOU LIE TO ME!
21:30 EmilienM I like mfisch commits
21:31 EmilienM it usually reminds me how life is beautiful
21:31 EmilienM and how much we have fun
21:31 mfisch that threw me off for a bit
21:31 mfisch debugging something else
21:32 mfisch also FYI I'm seeing some cases on VMs where liberty keystone + OSC takes > 20 seconds to make an endpoint
21:32 mfisch I'd like to bump that timeout
21:32 mfisch under no other load this box takes 12s
21:33 mwhahaha osc isn't the fastest thing in the world
21:33 mwhahaha most of it is just loading all the client libs
21:33 mfisch I think maybe somethineg like if it's a no retry we wait 60s
21:42 mfisch dilyin: I think we should change this code so that if it's a no retry code we just wait 60s total
21:42 mfisch right now we abort at 20s
21:42 mfisch retry code would do retries up to 60s, no retry would just wait 60s
21:43 mwhahaha is it not configurable?
21:43 mfisch no
21:43 dilyin can you point me to this code?
21:43 mwhahaha i think we actually increased it
21:43 mwhahaha it used to be less than what it was now
21:43 mfisch i1s
21:43 mfisch https://github.com/openstack/puppet-openstacklib/blob/master/lib/puppet/provider/openstack.rb#L99-L105
21:43 mfisch if we just indent line 103 below line 101 will it work the way we want?
21:44 mfisch basically dont abort the no-retry unless we waited the full 60
21:44 mwhahaha https://github.com/openstack/puppet-openstacklib/commit/701b6fe0acf56e9132e9b0a448f28670c714cedb
21:44 mwhahaha it used to be 10 sec :D
21:44 mfisch so its not really right
21:44 mfisch I mean 20s is the hard cap on non-retry
21:44 mfisch we should wait 60s for every command
21:44 mfisch but only retry osme
21:45 mfisch I'll post a swag on this
21:46 openstackgerrit Matt Fischer proposed openstack/puppet-openstacklib: Wait longer on non-retry calls  https://review.openstack.org/298992
21:46 dilyin look like it waits for 20 second for a command to finish
21:46 mfisch somethingl ike that ^
21:46 dilyin then if it have failed waits for 3 sec
21:46 mfisch it does not
21:47 dilyin and tries again
21:47 mfisch not for no-retry commands
21:47 mfisch thats the issue
21:47 dilyin but will fail after 60 sec if no success
21:47 mfisch for example endpoint create takes > 20 seconds it just fails
21:47 dilyin so yo want to set command timeout to 60 for non-retry comamnds&
21:47 mfisch y
21:47 mfisch see if that diff makes any sense
21:48 mfisch I have cases where endpoint create > 20 seconds :*
21:48 mfisch :(
21:48 dilyin meybe just raise command timeout
21:48 ayoung joined #puppet-openstack
21:49 mfisch will this code handle a case where something hasnt returned yet?
21:49 mfisch thats what I dont know
21:49 mfisch raising the timeout is also a solution
21:49 mfisch so the command will have already failed, my idea would need more work
21:50 mfisch I think we have to raise the timeout
21:51 dilyin # timeout the openstack command
21:51 dilyin # after this number of seconds
21:51 dilyin # retry the command until the request_timeout
21:51 dilyin def self.command_timeout(action=nil)
21:51 dilyin return 60 if no_retry_actions.include action
21:51 dilyin 20
21:51 dilyin end
21:51 dilyin # run the openstack command
21:51 dilyin # with command_timeout
21:52 dilyin def self.openstack(*args)
21:52 dilyin begin
21:52 dilyin action = args[1]
21:52 dilyin Timeout.timeout(command_timeout action) do
21:52 dilyin openstack_command *args
21:52 dilyin end
21:52 mwhahaha oh noes
21:52 dilyin rescue Timeout::Error
21:52 dilyin raise Puppet::ExecutionFailure, "Command: 'openstack #{args.inspect}' has been running for more then #{command_timeout} seconds!"
21:52 dilyin end
21:52 dilyin end
21:52 dilyin something like this?
21:53 openstackgerrit Matt Fischer proposed openstack/puppet-openstacklib: Adjust timeout for non-retry calls  https://review.openstack.org/298992
21:53 mfisch spasiba
21:53 mfisch look right? ^
21:53 dilyin set include? there
21:54 mfisch where?
21:54 dilyin and you forgot action
21:54 dilyin and command_timeout action
21:54 mfisch got that
21:54 openstackgerrit Matt Fischer proposed openstack/puppet-openstacklib: Adjust timeout for non-retry calls  https://review.openstack.org/298992
21:55 mfisch oh I see what you mean
21:55 dilyin but still maybe set comamnd timeot to 60 and request_timeout to 300
21:55 mfisch I think this is probably a good start
21:55 openstackgerrit Matt Fischer proposed openstack/puppet-openstacklib: Adjust timeout for non-retry calls  https://review.openstack.org/298992
21:55 mfisch added the include?
21:55 mfisch I need to run but thanks for the assist can you +1/-1 it?
21:56 dilyin action = args[1]
21:56 mfisch (action=nil) thats not enough?
21:56 mfisch I'm not super good at ruby
21:56 mfisch begin
21:56 mfisch action = args[1]
21:56 mfisch Timeout.timeout(command_timeout action) do
21:57 openstackgerrit Matt Fischer proposed openstack/puppet-openstacklib: Adjust timeout for non-retry calls  https://review.openstack.org/298992
21:57 mfisch I will test this later
21:57 mfisch ttyl
22:13 markvoelker joined #puppet-openstack
22:17 openstackgerrit Emilien Macchi proposed openstack/puppet-openstack-integration: repos: bump rdo to mitaka/current-passed-ci  https://review.openstack.org/299005
22:17 EmilienM dmsimard: fyi ^
22:20 zino left #puppet-openstack
22:21 markvoelker joined #puppet-openstack
22:22 markvoelker_ joined #puppet-openstack
22:24 xingchao joined #puppet-openstack
22:24 o_sutur joined #puppet-openstack
22:31 openstackgerrit Alex Schultz proposed openstack/puppet-glance: Add ability to set properties with glance_image  https://review.openstack.org/298718
22:32 mwhahaha glance_image needs reworking because v1 and v2 of the glance api have some important differences
22:33 EmilienM just to be transparent, I realized I did a mistake during the release process, on puppet-aodh only. I created 8.0.0 tag on stable/liberty, because of an error in my script. Sorry for that. I'm creating 8.0.1 right now
22:33 EmilienM mwhahaha: yeah, we were expecting that...
22:33 mwhahaha it's not too bad i'll poke at it later
22:50 _nick joined #puppet-openstack
22:57 EmilienM ok new fun error: http://logs.openstack.org/03/298903/1/gate/gate-puppet-ceilometer-puppet-beaker-rspec-ubuntu-trusty/b37eea0/console.html#_2016-03-29_22_05_42_191
23:00 openstackgerrit Emilien Macchi proposed openstack/puppet-keystone: CI test - never merge  https://review.openstack.org/299020
23:05 xarses joined #puppet-openstack
23:13 openstackgerrit Matt Fischer proposed openstack/puppet-openstacklib: Adjust timeout for non-retry calls  https://review.openstack.org/298992
23:16 EmilienM our CI is now suffering of a puppetlabs packaging issue
23:17 ducttape_ joined #puppet-openstack
23:17 EmilienM hiera is no longer a dependency, it seems, so the package is not in nodepool images (trusty) and our CI tried to purge the package, but since it does not exist anymore, job fail.
23:17 EmilienM discussion on #openstack-infra
23:19 EmilienM wrong statement, it's a nodepool issue, not puppetlabs.
23:24 ansmith joined #puppet-openstack
23:25 xingchao joined #puppet-openstack
23:29 _ody ok.  That would have been odd.
23:31 EmilienM _ody: yeah nothing on puppetlabs side.
23:31 EmilienM _ody: we think it was a typo in a script that build ubuntu images..
23:33 alee joined #puppet-openstack
23:38 openstackgerrit Matt Fischer proposed openstack/puppet-openstacklib: Adjust timeout for non-retry calls  https://review.openstack.org/298992
23:38 jtomasek joined #puppet-openstack
23:46 openstackgerrit Matt Fischer proposed openstack/puppet-openstacklib: Adjust timeout for non-retry calls  https://review.openstack.org/298992
23:49 EmilienM zuul is being restarted
23:49 EmilienM expect some NOT_REGISTERED
23:51 EmilienM topic is already there

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