Wednesday, 2016-01-13

_odygchamoul: Just got word on the 4.3.x issue.  It missed 4.3.2 and there will likely not be a 4.3.3.00:04
*** dgurtner has quit IRC00:07
*** dfisher has quit IRC00:18
*** xingchao has joined #puppet-openstack00:24
*** ducttape_ has joined #puppet-openstack00:33
*** foxhunt72 is now known as foxhunt_away00:44
*** mdorman has quit IRC00:48
*** zhangjn has joined #puppet-openstack00:51
*** xarses has quit IRC00:58
*** xingchao has quit IRC01:14
*** xingchao has joined #puppet-openstack01:18
*** ducttape_ has quit IRC01:22
*** xingchao has quit IRC01:23
*** ducttape_ has joined #puppet-openstack01:29
*** ducttape_ has quit IRC01:30
*** baker_ has quit IRC01:37
*** ducttape_ has joined #puppet-openstack01:44
*** xingchao has joined #puppet-openstack01:49
*** ducttape_ has quit IRC01:54
*** ntpttr has quit IRC01:58
*** ducttape_ has joined #puppet-openstack02:12
*** ducttape_ has quit IRC02:13
*** zhangjn has quit IRC02:15
*** ccie6747 has joined #puppet-openstack02:22
*** xarses has joined #puppet-openstack02:36
*** zhangjn has joined #puppet-openstack02:39
*** ilbot3 has quit IRC02:44
*** ilbot3 has joined #puppet-openstack02:47
*** gildub has joined #puppet-openstack03:10
*** xingchao has quit IRC03:19
*** xingchao has joined #puppet-openstack03:20
*** baker has joined #puppet-openstack03:28
*** baker_ has joined #puppet-openstack03:36
*** baker has quit IRC03:39
*** jfluhmann has joined #puppet-openstack03:43
*** zhangjn has quit IRC03:46
*** ccie6747 has quit IRC03:51
*** severion has joined #puppet-openstack04:06
*** v1k0d3n has quit IRC04:09
*** v1k0d3n has joined #puppet-openstack04:10
*** baker_ has quit IRC04:12
*** severion has quit IRC04:14
*** zhangjn has joined #puppet-openstack04:16
*** v1k0d3n has quit IRC04:17
*** v1k0d3n has joined #puppet-openstack04:17
*** rahulait has joined #puppet-openstack04:19
*** rahulait has quit IRC04:19
*** richm has quit IRC04:19
*** v1k0d3n has quit IRC04:21
*** dh64 has quit IRC04:25
*** zhangjn has quit IRC04:29
openstackgerritXingchao Yu proposed openstack/puppet-designate: Add designate::mdns class  https://review.openstack.org/21497604:32
*** xingchao has quit IRC04:33
mfischmdorman looking04:41
openstackgerritMerged openstack/puppet-nova: Use auth_uri to get nova credentials  https://review.openstack.org/25828504:45
*** xingchao has joined #puppet-openstack04:50
*** ferest has joined #puppet-openstack04:53
*** ferest_ has joined #puppet-openstack04:56
openstackgerritMatt Fischer proposed openstack/puppet-keystone: Hash domains by name  https://review.openstack.org/26667704:56
*** ferest has quit IRC04:57
openstackgerrityuan.lu proposed openstack/puppet-horizon: Improve format of the documentations.  https://review.openstack.org/26650405:08
*** Liao has joined #puppet-openstack05:11
*** zhangjn has joined #puppet-openstack05:15
*** xingchao_ has joined #puppet-openstack05:19
*** xingchao has quit IRC05:22
*** ferest_ has quit IRC05:33
*** Liao has quit IRC05:33
*** Liao has joined #puppet-openstack05:34
*** jaosorior has joined #puppet-openstack05:41
*** Liao has quit IRC05:45
*** zhangjn has quit IRC05:46
*** Liao has joined #puppet-openstack05:46
openstackgerritLiao Penghui proposed openstack/puppet-nova: Fix inconsistent type/provider for paste configs.  https://review.openstack.org/26669205:46
*** Liao_ has joined #puppet-openstack05:54
*** xingchao_ has quit IRC05:54
*** Liao has quit IRC05:57
*** zhangjn has joined #puppet-openstack06:07
*** zhangjn has quit IRC06:07
*** zhangjn has joined #puppet-openstack06:08
*** xarses_ has joined #puppet-openstack06:09
*** narasimhasv has joined #puppet-openstack06:09
*** xarses has quit IRC06:12
*** xingchao has joined #puppet-openstack06:14
*** xingchao_ has joined #puppet-openstack06:15
openstackgerritXiaohua Yuan proposed openstack/puppet-designate: Add designate::pool_manager class  https://review.openstack.org/26671106:17
*** xingchao has quit IRC06:19
*** zhangjn has quit IRC06:23
*** zhangjn has joined #puppet-openstack06:25
*** markvoelker has quit IRC06:27
openstackgerritXingchao Yu proposed openstack/puppet-designate: Add designate::mdns class  https://review.openstack.org/21497606:33
*** xingchao has joined #puppet-openstack06:37
*** zhangjn has quit IRC06:40
*** xingchao_ has quit IRC06:40
*** ferest has joined #puppet-openstack06:41
openstackgerritXingchao Yu proposed openstack/puppet-keystone: (WIP) Update keystone::resource::authtoken  https://review.openstack.org/26672306:44
*** ferest has quit IRC06:48
*** zhangjn has joined #puppet-openstack06:50
*** jaosorior has quit IRC06:57
*** cwolferh has quit IRC07:18
*** gildub has quit IRC07:19
*** markvoelker has joined #puppet-openstack07:28
*** jtomasek has joined #puppet-openstack07:28
*** markvoelker has quit IRC07:33
*** jpena has joined #puppet-openstack07:47
*** foxhunt_away is now known as foxhunt7208:00
*** rochaporto has joined #puppet-openstack08:03
*** jistr has joined #puppet-openstack08:07
*** jpena is now known as jpena|mtg08:07
*** jistr is now known as jistr|doc08:07
*** mkarpin has joined #puppet-openstack08:07
*** zhangjn has quit IRC08:11
*** Guest71407 is now known as saneax_AFK08:22
*** zhangjn has joined #puppet-openstack08:23
*** dgurtner has joined #puppet-openstack08:29
*** dgurtner has quit IRC08:29
*** dgurtner has joined #puppet-openstack08:29
*** samuelBartel has joined #puppet-openstack08:32
*** igajsin has quit IRC08:33
*** skolekonov has joined #puppet-openstack08:50
*** Liao has joined #puppet-openstack08:52
*** Liao_ has quit IRC08:52
gchamoul_ody: thx ... then we will wait for the 4.4.008:54
*** markvoelker has joined #puppet-openstack08:54
*** markvoelker has quit IRC08:59
*** zhangjn has quit IRC09:03
openstackgerritGael Chamoulaud proposed openstack/puppet-swift: [WIP] Fix unit tests failing against Puppet 4.3.0  https://review.openstack.org/25023309:07
*** Liao has quit IRC09:13
*** Liao has joined #puppet-openstack09:14
*** Liao has quit IRC09:15
*** Liao has joined #puppet-openstack09:16
*** Liao has quit IRC09:16
*** Liao has joined #puppet-openstack09:17
openstackgerritGael Chamoulaud proposed openstack/puppet-swift: Fix unit tests failing against Puppet 4.3.0  https://review.openstack.org/25023309:17
*** jaosorior has joined #puppet-openstack09:18
*** Liao has quit IRC09:18
*** Liao has joined #puppet-openstack09:19
*** Liao has quit IRC09:20
*** rmart04 has joined #puppet-openstack09:21
*** zhangjn has joined #puppet-openstack09:21
*** Liao has joined #puppet-openstack09:23
*** Liao has quit IRC09:23
*** Liao has joined #puppet-openstack09:29
*** gfidente has joined #puppet-openstack09:29
*** ferest has joined #puppet-openstack09:33
*** Liao has quit IRC09:34
*** Liao has joined #puppet-openstack09:35
*** derekh has joined #puppet-openstack09:35
*** zhangjn has quit IRC09:36
*** ferest_ has joined #puppet-openstack09:39
openstackgerritLiao Penghui proposed openstack/puppet-horizon: Add List/None type value in the local_settings.  https://review.openstack.org/26681409:41
*** _degorenko|afk is now known as degorenko09:41
*** ferest has quit IRC09:42
*** Liao_ has joined #puppet-openstack09:49
*** Liao has quit IRC09:51
openstackgerritDenis Egorenko proposed openstack/puppet-openstack-integration: Remove unneeded var aodh_enabled from aodh manifest  https://review.openstack.org/26681709:53
*** Liao_ has quit IRC10:12
*** igajsin has joined #puppet-openstack10:12
openstackgerritXingchao Yu proposed openstack/puppet-designate: Add designate::mdns class  https://review.openstack.org/21497610:12
*** Liao has joined #puppet-openstack10:13
*** karimb has joined #puppet-openstack10:17
*** jistr|doc is now known as jistr10:19
*** jistr has quit IRC10:26
*** gildub has joined #puppet-openstack10:30
*** igajsin has left #puppet-openstack10:45
*** saneax_AFK is now known as saneax10:48
*** saneax is now known as Guest9401710:49
TheBallPIis puppet-ceph supposed to work with puppet >=4.0.0 ?10:49
*** jistr has joined #puppet-openstack10:51
*** markvoelker has joined #puppet-openstack10:55
*** markvoelker has quit IRC11:00
*** jaosorior has quit IRC11:05
*** jaosorior has joined #puppet-openstack11:05
*** jaosorior has quit IRC11:06
*** jaosorior has joined #puppet-openstack11:07
*** xingchao has quit IRC11:09
*** xingchao has joined #puppet-openstack11:09
openstackgerritMerged openstack/puppet-rally: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26455711:13
*** xingchao has quit IRC11:15
*** gildub has quit IRC11:15
openstackgerritMerged openstack/puppet-aodh: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26453811:18
*** mjblack has joined #puppet-openstack11:19
*** aarefiev has quit IRC11:20
openstackgerritLukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/20087711:20
*** aarefiev has joined #puppet-openstack11:21
openstackgerritMerged openstack/puppet-murano: Update to pull in rspec-puppet-facts  https://review.openstack.org/26455211:23
*** Liao has quit IRC11:25
*** cgfbee has quit IRC11:32
*** ferest has joined #puppet-openstack11:34
openstackgerritAlexey Deryugin proposed openstack/puppet-openstack-integration: [WIP] scenario002: enable application catalog service  https://review.openstack.org/25043611:35
*** ferest_ has quit IRC11:37
openstackgerritMerged openstack/puppet-cinder: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26454111:39
*** gchamoul is now known as gchamoul|afk11:43
*** pma_ has quit IRC11:50
*** skolekonov has quit IRC11:51
*** markvoelker has joined #puppet-openstack11:56
openstackgerritLukas Bezdicka proposed openstack/puppet-ceilometer: Support identity_uri and auth_uri properly and add auth_version option  https://review.openstack.org/18228111:58
*** pma_ has joined #puppet-openstack11:59
openstackgerritOleksiy Molchanov proposed openstack/puppet-ceph: Add possibility to use mod_proxy for radosgw  https://review.openstack.org/26602712:00
*** markvoelker has quit IRC12:00
openstackgerritOleksiy Molchanov proposed openstack/puppet-ceph: Add possibility to use mod_proxy for radosgw  https://review.openstack.org/26602712:02
*** cgfbee has joined #puppet-openstack12:05
*** cgfbee has quit IRC12:10
*** xingchao has joined #puppet-openstack12:11
*** rahulait has joined #puppet-openstack12:13
*** mlrn has left #puppet-openstack12:15
*** cgfbee has joined #puppet-openstack12:17
*** xingchao has quit IRC12:17
*** morazi has quit IRC12:35
openstackgerritOleksiy Molchanov proposed openstack/puppet-ceph: Add possibility to use mod_proxy for radosgw  https://review.openstack.org/26602712:35
EmilienMgood morning12:43
*** paramite has joined #puppet-openstack12:52
degorenkomorning :)12:52
iurygregorymorning =)12:53
*** markvoelker has joined #puppet-openstack12:57
openstackgerritOleksiy Molchanov proposed openstack/puppet-ceph: Add possibility to use mod_proxy for radosgw  https://review.openstack.org/26602712:59
EmilienMdegorenko: ok so aodh is current broken in debian13:01
EmilienMhttps://bugs.launchpad.net/cloud-archive/+bug/1533014/comments/413:01
openstackEmilienM: Error: Could not gather data from Launchpad for bug #1533014 (https://launchpad.net/bugs/1533014). The error has been logged13:02
EmilienMzigo is currently working on it13:02
*** markvoelker has quit IRC13:02
degorenkoack :)13:02
EmilienMbut after that we'll have to wait for ubuntu to upgrade their package13:02
EmilienMwhich can take 1 day or 1 week I have no idea13:02
zigoI'm not, I'm currently out of work, because git.debian.org is down... :(13:02
zigoWill do as soon as it's up.13:02
EmilienMso I suggest we disable aodh testing on ubuntu13:02
EmilienMso we can make progress: bump our CI and re-enable aodh on ubuntu later this week13:03
degorenkoi'm ok with it13:03
*** Guest94017 is now known as saneax_AFK13:04
openstackgerritMerged openstack/puppet-openstack_extras: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26455613:05
*** rcernin has joined #puppet-openstack13:09
openstackgerritMerged openstack/puppet-glance: Update to pull in rspec-puppet-facts  https://review.openstack.org/26454313:13
openstackgerritMerged openstack/puppet-nova: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26455413:15
*** ducttape_ has joined #puppet-openstack13:15
*** jayg|g0n3 is now known as jayg13:15
*** pbelamge_ has joined #puppet-openstack13:18
*** igajsin has joined #puppet-openstack13:22
*** trown|outttypeww is now known as trown13:22
*** igajsin has left #puppet-openstack13:22
*** morazi has joined #puppet-openstack13:26
*** markvoelker has joined #puppet-openstack13:31
*** ferest has quit IRC13:34
*** ducttape_ has quit IRC13:35
openstackgerritAlexey Deryugin proposed openstack/puppet-tempest: [WIP] Add required parameters for Sahara  https://review.openstack.org/26691213:36
openstackgerritAlexey Deryugin proposed openstack/puppet-openstack-integration: [WIP] scenario002: enable application catalog service  https://review.openstack.org/25043613:37
*** gchamoul|afk is now known as gchamoul13:40
*** ccie6747 has joined #puppet-openstack13:47
*** jistr has quit IRC13:48
*** jistr has joined #puppet-openstack13:49
*** rmart04_ has joined #puppet-openstack14:03
*** rmart04 has quit IRC14:04
*** rmart04_ is now known as rmart0414:04
openstackgerritDmitry Tantsur proposed openstack/puppet-ironic: Enable changing list of inspection collectors  https://review.openstack.org/26692614:04
*** richm has joined #puppet-openstack14:13
*** ccie6747 has quit IRC14:17
*** ferest has joined #puppet-openstack14:18
*** dprince has joined #puppet-openstack14:24
openstackgerritLukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/20087714:34
*** priteau has joined #puppet-openstack14:41
EmilienMdegorenko: ubuntu is actually fixing it right now14:43
EmilienMwhich will avoid us to disable testing14:45
degorenkoEmilienM, are you talking about aodh?14:45
EmilienMyeah14:45
degorenkonice14:46
degorenkowe with mkarpin trying to get working heat with os_service_default https://review.gerrithub.io/#/c/259613/14:47
degorenkoEmilienM, ^14:47
*** dfisher has joined #puppet-openstack14:47
*** jpena has joined #puppet-openstack14:55
*** rahulait has quit IRC14:57
*** bklei has joined #puppet-openstack14:58
michchapEmilienM: is the puppet-openstack core group responsible for puppet-tripleo, or is that a different group?15:00
EmilienMmichchap: we can +2 on it, but it's more tripleo folks who review it15:00
EmilienMthey are core on it too15:00
michchapEmilienM: cool, thanks. The ODL patch is ready for review so I thought I'd see who I can hassle :)15:01
EmilienMmichchap: I can look at it today15:01
michchapEmilienM: cheers.15:02
*** ducttape_ has joined #puppet-openstack15:05
*** narasimhasv has quit IRC15:09
*** xarses_ has quit IRC15:13
*** baker has joined #puppet-openstack15:19
openstackgerritLukas Bezdicka proposed openstack/puppet-neutron: Install service packages on api nodes  https://review.openstack.org/20087715:22
*** Liao has joined #puppet-openstack15:26
openstackgerritLiao Penghui proposed openstack/puppet-horizon: Add List/None type value in the local_settings.  https://review.openstack.org/26681415:26
*** pbelamge_ has quit IRC15:26
*** paramite is now known as paramite|afk15:27
*** Liao has quit IRC15:30
*** dfisher has quit IRC15:34
*** mdorman has joined #puppet-openstack15:39
*** jpena|mtg has quit IRC15:42
*** xarses_ has joined #puppet-openstack15:45
openstackgerritMykyta Karpin proposed openstack/puppet-heat: Switch Heat to $::os_service_default - part 2  https://review.openstack.org/24971115:48
*** dhollinger has joined #puppet-openstack15:49
*** paramite|afk is now known as paramite15:49
*** jfluhmann has quit IRC15:50
openstackgerritXingchao Yu proposed openstack/puppet-designate: Add designate::mdns class  https://review.openstack.org/21497615:51
*** Liao has joined #puppet-openstack15:52
*** Liao_ has joined #puppet-openstack15:54
*** Liao has quit IRC15:54
*** Liao has joined #puppet-openstack15:57
*** Liao_ has quit IRC15:57
openstackgerritLiao Penghui proposed openstack/puppet-horizon: Add List/None type value in the local_settings.  https://review.openstack.org/26681415:57
*** Liao has quit IRC15:59
*** Liao has joined #puppet-openstack15:59
*** Liao has quit IRC15:59
*** Liao has joined #puppet-openstack16:00
*** Liao is now known as LiaoPenghui16:00
*** LiaoPenghui is now known as Liao16:01
*** bklei has quit IRC16:03
*** dgurtner_ has joined #puppet-openstack16:07
*** dgurtner_ has quit IRC16:07
*** dgurtner_ has joined #puppet-openstack16:07
*** Liao_ has joined #puppet-openstack16:07
*** Liao has quit IRC16:07
*** dgurtner has quit IRC16:09
*** rmart04 has quit IRC16:10
*** Liao has joined #puppet-openstack16:10
*** Liao_ has quit IRC16:11
openstackgerritLiao Penghui proposed openstack/puppet-ceilometer: Move workers options into specific sections.  https://review.openstack.org/26652316:12
*** rahulait has joined #puppet-openstack16:12
openstackgerritAlexey Deryugin proposed openstack/puppet-openstack-integration: [WIP] scenario002: enable application catalog service  https://review.openstack.org/25043616:14
*** Liao has quit IRC16:16
*** trown is now known as trown|brb16:17
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Add api_paste type/provider for Sahara  https://review.openstack.org/26700716:17
degorenkomgagne, hey, take a look please ^^16:17
*** trown|brb is now known as trown16:18
*** angdraug has joined #puppet-openstack16:23
*** ferest has quit IRC16:24
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: Enable X-Forwarded-Proto header for cinder  https://review.openstack.org/26701216:26
*** skippy has joined #puppet-openstack16:33
skippyCould someone help me run spec tests on openstack-ceph?  I'm getting "Could not parse for environment rp_env: No file(s) found for import of '.../puppet-ceph/spec/fixtures/manifests/site.pp' at line 3"16:33
skippyI see at https://github.com/openstack/puppet-ceph/tree/master/spec/fixtures that there is no "manifests" directory.  I had one in my local clone, which I've removed to make it look like the repo.16:33
*** foxhunt72 is now known as foxhunt_away16:37
EmilienMxarses_ ^16:45
*** xarses_ is now known as xarses16:45
xarsesthanks EmilienM16:46
*** samuelBartel has quit IRC16:47
*** cdelatte has joined #puppet-openstack16:47
xarsesskippy: should be `bundle install && bundle exec rake spec`, it looks like manifests/site.pp should be there16:49
*** rcernin has quit IRC16:51
skippyrake aborted!16:54
openstackgerritXingchao Yu proposed openstack/puppet-keystone: (WIP) Update keystone::resource::authtoken  https://review.openstack.org/26672316:54
skippyDon't know how to build task 'rspec'16:54
xarsesif you haven't used your bundle cache directory in a while, upgrade it, if that doesn't work delete it16:55
*** cwolferh has joined #puppet-openstack16:56
skippyremoved .bundle.  ran `bundle install`.  `bundle exec rake rspec` still fails with "Don't know how to build task 'rspec'"16:56
xarsesspec, not rspec16:57
skippyjust figured that out, too!16:57
*** jfluhmann has joined #puppet-openstack16:57
skippyok.  that ran.  Thanks!  all of the /define/ tests appear to have failed with Puppet syntax errors.16:58
xarseswhich version?16:58
xarsesof puppet?16:58
skippyoh, wait, syntax errors in the manifest I edited.  False alarm.  This is what I was trying to identify.  Thank!16:58
xarsesok16:59
*** jpena has quit IRC17:00
*** dgurtner_ has quit IRC17:01
openstackgerritDenis Egorenko proposed openstack/puppet-keystone: Refactor of domain_name_from_id  https://review.openstack.org/26703217:01
openstackgerritDenis Egorenko proposed openstack/puppet-neutron: Add api_paste type/provider for Neutron  https://review.openstack.org/26703517:03
*** rochaporto has quit IRC17:08
*** jistr has quit IRC17:08
*** karimb has quit IRC17:09
*** aimon_ has quit IRC17:10
openstackgerritMerged openstack/puppet-ironic: Enable changing list of inspection collectors  https://review.openstack.org/26692617:11
openstackgerritXingchao Yu proposed openstack/puppet-designate: Add designate::mdns class  https://review.openstack.org/21497617:11
*** karimb has joined #puppet-openstack17:15
openstackgerritDenis Egorenko proposed openstack/puppet-sahara: Add api_paste type/provider for Sahara  https://review.openstack.org/26700717:16
*** IvanBerezovskiy has left #puppet-openstack17:16
openstackgerritDenis Egorenko proposed openstack/puppet-neutron: Add api_paste type/provider for Neutron  https://review.openstack.org/26703517:17
*** ntpttr has joined #puppet-openstack17:20
ntpttrDoes anyone know what's causing no tests to run against this patch? https://review.openstack.org/#/c/254392/17:22
ntpttrit was rebased days ago but no checks happened17:22
openstackgerritDenis Egorenko proposed openstack/puppet-heat: Add type/provider for paste configs  https://review.openstack.org/21182017:23
xarsesntpttr: no clue, thats odd17:23
xarsesyou might want to check with operators17:23
ntpttrxarses: who should I reach out to?17:25
ntpttrnot sure who the operators are :)17:25
xarses#openstack-operators17:25
ntpttrsounds good, thanks17:25
*** karimb has quit IRC17:39
openstackgerritXingchao Yu proposed openstack/puppet-designate: Remove pick for managing package name.  https://review.openstack.org/26705817:39
*** rmart04 has joined #puppet-openstack17:43
colonwqdmsimard, what do you think of changing frontend_type to rgw_frontend_type? I would rather keep rgw_frontends as that is the name of the parameter in ceph.conf.17:48
dmsimardcolonwq: yeah I saw dgurtner's comment - I didn't realize it came from what was the parameter name in cpeh.conf17:48
dmsimardcolonwq: in that respect, the argument name is fine :)17:48
skippycolonwq: is that for choosing fastcgi versus fcgi?17:49
dmsimardskippy: versus civetweb17:49
openstackgerritNate Potter proposed openstack/puppet-magnum: Initial commit of api.pp  https://review.openstack.org/25439217:49
skippydmsimard: link please to work in progres?17:49
skippyI'm reworking https://review.openstack.org/#/c/238526/3/manifests/rgw/apache24.pp17:49
dmsimardhttps://review.openstack.org/#/c/254548/17:49
dmsimardskippy: ^17:49
dmsimardcolonwq: rgw_frontend_type is fair17:49
colonwqdmsimard, cool. I'll finish up.17:51
ntpttrWhen any cores have a moment would you mind giving this patch a +A, it has two +2s right now https://review.openstack.org/#/c/253216/17:52
*** rmart04 has left #puppet-openstack17:56
_odyntpttr: I'll take a looke.17:56
ntpttr_ody: thanks!17:58
openstackgerritDenis Egorenko proposed openstack/puppet-designate: Remove using pick function  https://review.openstack.org/26706918:01
degorenko_ody, are you there?18:01
degorenkoplease remove +A18:01
degorenkohttps://review.openstack.org/#/c/25321618:01
*** derekh has quit IRC18:01
degorenkoi've found problem :(18:01
degorenkoohj no18:01
degorenkosorry18:01
ntpttrdegorenko: uh oh what is it?18:01
degorenkocrazy now :( so many review today18:01
ntpttrdegorenko: something I can fix now?18:01
mwhahahayou can't inherit two classes can you?18:02
degorenkontpttr, no problem :) sorry for disturbing18:02
degorenkojust misread18:02
ntpttrdegorenko: ah all right no problem hah18:03
degorenkomwhahaha, take a look please https://review.openstack.org/#/c/267069/18:03
mwhahahayea that's what i was asking about, can you actually inherit from two classes?18:03
degorenkohm, good question18:04
mwhahahaalso why are we inheriting from designate18:04
degorenkoactually, i saw this construction somewhere18:04
degorenkoi don't know :)18:04
mwhahahapretty sure those should just be inherit ::designate::params18:05
degorenkoyeah, probably18:06
degorenkomwhahaha, btw, rake-lint doesn't see problems with double inheritance18:06
openstackgerritEmile Snyder proposed openstack/puppet-ceph: Adds public_addr setting.  https://review.openstack.org/26233018:06
*** kgalanov has quit IRC18:08
*** mwhahaha has quit IRC18:08
*** patrickeast has quit IRC18:08
*** aglarendil has quit IRC18:08
*** rhefner has quit IRC18:08
*** simonmcc has quit IRC18:08
*** holser_ has quit IRC18:08
*** mwhahaha has joined #puppet-openstack18:10
mwhahahathat was weird18:10
trownhmm... is there a reason that qos_notification_drivers was defaulted differently in this backport https://github.com/openstack/puppet-neutron/commit/e81bc875fd2053ba6b82f8d1e87e115d81f957d9 than in the original https://github.com/openstack/puppet-neutron/commit/eaf2f39b904fba951452997deede6319d9ebcebe18:11
trownthe default in the backport is not valid, and neutron-server consequently does not start18:11
trownI am a bit confused how that backport could pass CI18:12
*** mwhahaha has quit IRC18:14
*** mwhahaha has joined #puppet-openstack18:15
degorenkotrown, you're right http://docs.openstack.org/liberty/config-reference/content/neutron-conf-changes-liberty.html18:15
degorenkofeel free update patch18:15
trowndegorenko: cool will do18:16
*** rmart04 has joined #puppet-openstack18:16
*** mwhahaha_ has joined #puppet-openstack18:16
trowndegorenko: was trying to determine if it was intentional18:16
*** mwhahaha has quit IRC18:16
*** mwhahaha_ is now known as mwhahaha18:16
degorenkohm, really strange how this patch passed ci then18:18
*** simonmcc has joined #puppet-openstack18:19
*** rmart04 has quit IRC18:20
*** holser_ has joined #puppet-openstack18:20
trowndegorenko: is $::os_service_default only available on master?18:21
degorenkoyes18:22
degorenkofor neutron18:22
degorenkoit is for liberty only in cinder18:22
trownah, ok that then makes a bit more sense, so for liberty I should hardcode that to upstream default18:22
degorenkoall another project don't have this feture in liberty18:22
mwhahahadid only cinder get it? i thought we got the starting point for it in all modules18:22
degorenkoprojects*18:22
degorenkono, i meant, for liberty only cinder got it18:23
degorenkoall another projects got it in mitaka18:23
mwhahahaah18:23
*** patrickeast has joined #puppet-openstack18:23
mwhahahayea i thought we at least got the starting point in for all the other modules, that's unfortunate18:24
mwhahahabut it mostly relies on the openstacklib stuff so there's no reason you can't use it if we backport @default_params stuff18:24
degorenkoit is possible18:25
*** ericksonsantos has quit IRC18:25
*** rhefner has joined #puppet-openstack18:25
*** aglarendil has joined #puppet-openstack18:25
degorenkoi'm off for today :)18:26
degorenko_degorenko|afk18:26
*** degorenko is now known as _degorenko|afk18:27
openstackgerritMerged openstack/puppet-magnum: Initial commit of init.pp  https://review.openstack.org/25321618:27
openstackgerritskippy proposed openstack/puppet-ceph: Add Apache 2.4 proxy_fastcgi support for radosgw  https://review.openstack.org/23852618:27
*** iurygregory has quit IRC18:27
*** priteau has quit IRC18:28
*** rcernin has joined #puppet-openstack18:30
openstackgerritJohn Trowbridge proposed openstack/puppet-neutron: Default qos_notification_drivers to upstream  https://review.openstack.org/26708118:30
*** mgagne has quit IRC18:30
*** trown is now known as trown|lunch18:31
*** mgagne has joined #puppet-openstack18:31
*** kgalanov has joined #puppet-openstack18:31
*** mgagne is now known as Guest8280318:31
*** Guest82803 is now known as mgagne18:32
*** mgagne has quit IRC18:33
*** mgagne has joined #puppet-openstack18:33
xarsesntpttr: maybe because it says "cant merge"18:33
xarsesit can't even start the tests18:33
openstackgerritNate Potter proposed openstack/puppet-magnum: Initial commit of api.pp  https://review.openstack.org/25439218:38
openstackgerritNate Potter proposed openstack/puppet-magnum: Initial commit of api.pp  https://review.openstack.org/25439218:39
xarsesthere we go, that started now18:39
xarsesntpttr: ^18:39
xarseshttp://status.openstack.org/zuul/18:40
ntpttrxarses: yep, after the depenancy was merged and I fixed the merge conflict it got it going18:40
ntpttrxarses: thanks18:40
skippyi'm new to Gerrit.  I assume the "Conflics with" at the top right here means that this can't be tested?  https://review.openstack.org/#/c/23852618:44
mwhahahano it can be tested, that just tells you if your change will conflict with another18:44
mwhahahameaning if yours gets merged, it'll break those changes or if their's gets merged yours will break18:45
skippyok. thanks.18:45
mwhahahaand by break those changes I mean it'll put it into a merge conflict state where they'll have to be updated18:46
mwhahahaand merged manually18:46
mwhahahaer srolved manually18:46
mwhahahas/srolved/resolved18:46
*** ntpttr has quit IRC18:48
*** ntpttr has joined #puppet-openstack18:50
*** imcsk8 is now known as imcsk8|bank18:54
openstackgerritOleksiy Molchanov proposed openstack/puppet-ceph: Add possibility to use mod_proxy for radosgw  https://review.openstack.org/26602719:00
*** ericksonsantos has joined #puppet-openstack19:01
*** iurygregory has joined #puppet-openstack19:03
*** jaosorior has quit IRC19:05
*** jaosorior has joined #puppet-openstack19:05
*** paramite is now known as paramite|afk19:07
*** paramite|afk is now known as paramite19:16
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Use HAProxy 'transparent' bind option for compat with IPv6  https://review.openstack.org/26530919:18
*** paramite has quit IRC19:18
*** ccie6747 has joined #puppet-openstack19:32
mwhahahaEmilienM: You said the UCA mitaka packages were working right?19:37
*** trown|lunch is now known as trown19:43
EmilienMmwhahaha: except aodh19:45
mwhahahaok19:45
EmilienMmwhahaha: and with chem's patches on puppet-neutron19:45
mwhahahathanks19:45
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Use HAProxy 'transparent' bind option for compat with IPv6  https://review.openstack.org/26530919:48
*** ntpttr has quit IRC20:02
*** ccie6747 has quit IRC20:05
*** foxhunt_away is now known as foxhunt7220:08
*** rochaporto has joined #puppet-openstack20:09
*** rochaporto has quit IRC20:09
*** rochaporto has joined #puppet-openstack20:10
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Disable ip_nonlocal_bind (rely on the HAProxy 'transparent' option)  https://review.openstack.org/26713820:14
*** ccie6747 has joined #puppet-openstack20:17
*** rcernin has quit IRC20:20
*** ericksonsantos has quit IRC20:27
EmilienMzigo: repo is up http://anonscm.debian.org/cgit/openstack/aodh20:29
*** jaosorior has quit IRC20:30
*** jaosorior has joined #puppet-openstack20:30
*** jaosorior has quit IRC20:34
*** rochaporto has quit IRC20:36
*** ccie6747 has quit IRC20:47
*** rcernin has joined #puppet-openstack20:49
openstackgerritOleksiy Molchanov proposed openstack/puppet-ceph: Add possibility to use mod_proxy for radosgw  https://review.openstack.org/26602720:54
openstackgerritskippy proposed openstack/puppet-ceph: Add Apache 2.4 proxy_fastcgi support for radosgw  https://review.openstack.org/23852620:56
skippyheh.  we seem to be working on the same solution via different implementations20:56
openstackgerritAlex Schultz proposed openstack/puppet-ceilometer: Update to pull in rspec-puppet-facts  https://review.openstack.org/26454021:02
*** gildub has joined #puppet-openstack21:03
mwhahahagoing to apologize now for the mass review updates as I'm going to fix some of these rspec-puppet-facts reviews21:03
openstackgerritAlex Schultz proposed openstack/puppet-designate: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26454221:04
*** rcernin has quit IRC21:04
openstackgerritAlex Schultz proposed openstack/puppet-gnocchi: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26454421:04
EmilienMmwhahaha: cool np21:04
openstackgerritAlex Schultz proposed openstack/puppet-heat: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26454521:05
openstackgerritAlex Schultz proposed openstack/puppet-horizon: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26454621:05
openstackgerritAlex Schultz proposed openstack/puppet-ironic: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26454721:05
openstackgerritAlex Schultz proposed openstack/puppet-keystone: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26454821:07
openstackgerritAlex Schultz proposed openstack/puppet-magnum: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26454921:07
openstackgerritAlex Schultz proposed openstack/puppet-mistral: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26455121:08
openstackgerritAlex Schultz proposed openstack/puppet-neutron: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26455321:08
openstackgerritAlex Schultz proposed openstack/puppet-openstacklib: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26455521:09
openstackgerritAlex Schultz proposed openstack/puppet-sahara: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26455821:09
*** ntpttr has joined #puppet-openstack21:11
openstackgerritAlex Schultz proposed openstack/puppet-ironic: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26454721:11
*** imcsk8|bank is now known as imcsk821:13
openstackgerritAlex Schultz proposed openstack/puppet-swift: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26455921:14
openstackgerritMerged openstack/puppet-barbican: Update to pull in rspec-puppet-facts  https://review.openstack.org/26453921:14
openstackgerritAlex Schultz proposed openstack/puppet-tempest: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26456021:14
openstackgerritAlex Schultz proposed openstack/puppet-trove: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26456221:15
openstackgerritAlex Schultz proposed openstack/puppet-vitrage: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26456421:15
openstackgerritAlex Schultz proposed openstack/puppet-vswitch: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26456521:15
openstackgerritAlex Schultz proposed openstack/puppet-zaqar: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26456621:16
*** rahulait has quit IRC21:17
openstackgerritAlex Schultz proposed openstack/puppet-manila: Add rspec-puppet-facts to the spec_helper  https://review.openstack.org/26455021:18
EmilienMmwhahaha: why not using https://github.com/openstack/puppet-openstack_spec_helper?21:19
mwhahahawhat is that21:20
EmilienMso we have a single place to patch21:20
EmilienMcrinkle had this great idea a few months ago21:20
EmilienMto have a gem for utilities for running rspec tests21:20
mwhahahawell so thats why i put it in the module sync config21:20
mwhahahaso in theory when we have to adjust it again we can just module sync it21:20
mwhahahathe problem is i should have gotten that one committed first :D21:21
EmilienMor use this gem21:21
EmilienMso we only have one patch to do21:21
mwhahahaso we can probably use that for the default.rb21:21
crinkleyeah sorry i haven't finished that yet21:21
EmilienM(this gem is not used anywhere, we would have to add a patch everywhere to use it first)21:21
mwhahahabut the spec_helpers are not 100% the same21:21
*** rochaporto has joined #puppet-openstack21:21
EmilienMcrinkle: no problem, you already helped with the idea and the basic structure21:21
openstackgerritMerged openstack/puppet-cinder: Fix wrong location for lock_path of params  https://review.openstack.org/26630821:21
openstackgerritAlex Schultz proposed openstack/puppet-aodh: Update tests to leverage rspec-puppet-facts  https://review.openstack.org/26614221:22
EmilienMmwhahaha: we could have the default.rb indeed21:22
mwhahahaso i agree doing it in one place would be idea21:22
mwhahahaand actually puppet-rspec-facts will use the metadata.json for the OSes21:22
EmilienMI just realized that21:23
EmilienMsorry if I did not think about it before :(21:23
mwhahahaso if we fixed those in all the modules, the get_supported_os would no longer need to be used21:23
EmilienMit would prevent us for lot of patche21:23
mwhahahaindeed21:23
EmilienMmwhahaha: do you want to kil me?21:24
mwhahahaperhaps we can tackle getting the spec helper in place as part of the mid-cycle21:24
EmilienMsure21:24
mwhahahano i don't mind most of these changes were done via msync anyway :)21:25
mwhahahasome of them got merged without the fedora support anyway so they need to be retouched nayway21:26
EmilienMok21:26
EmilienMI really want to have this gem in place21:27
EmilienMso it's less patches each time we change a thing21:27
EmilienMthat's imho the best way to scale the number of modules21:27
HunnerEmilienM: Your mitaka email links [2] and [3] are duplicates21:27
EmilienMHunner: because I'm so dumb21:27
EmilienMlet me fix that21:27
*** angdraug has quit IRC21:28
*** xarses has quit IRC21:30
*** rochaporto has quit IRC21:30
openstackgerritAlex Schultz proposed openstack/puppet-openstack_spec_helper: Move defaults.rb to openstack spec helper  https://review.openstack.org/26718521:30
mwhahahathere's the default.rb for the spec helper -^21:32
*** jtomasek has quit IRC21:32
HunnerAh, the parent patch21:32
EmilienMyeah :)21:33
*** RichardRaseley has joined #puppet-openstack21:34
openstackgerritAlex Schultz proposed openstack/puppet-openstack_spec_helper: Move defaults.rb to openstack spec helper  https://review.openstack.org/26718521:34
EmilienMubuntu team is very responsive, aodh is going to be fixed very soon. Which will make our bump to mitaka possible this week eventually21:34
mwhahahaso i'm not sure we can convert over to the puppet-openstack_spec_helper until that change gets merged since I don't think depends-on will work for the gemfile21:35
mwhahahait seems that there isn't a good way to test that21:36
EmilienMmwhahaha: it won't21:37
EmilienMlooking at the patch21:38
EmilienMmwhahaha: +221:38
EmilienMonce merged, I guess we'll need to publish the gem21:38
mwhahahaor we can just refer to it via the git repo21:39
mwhahahaas indicated by the readme :)21:40
EmilienMmwhahaha: I prefer your idea lol21:43
EmilienMmwhahaha: do you like the plan?21:43
EmilienMwe can try for one module first21:43
mwhahahaindeed21:43
EmilienMand if it works, we can patch all modules to use this gem21:43
mwhahahaif it works i can just quickly update all my patches to use it instead21:44
EmilienMyeah21:44
EmilienMand no worries for the merged patches, we can still change it21:44
mwhahahayup21:44
*** michchap has quit IRC21:47
*** michchap has joined #puppet-openstack21:47
*** aimon has joined #puppet-openstack21:50
*** dprince has quit IRC21:51
EmilienMcrinkle: can you review https://review.openstack.org/#/c/267185/ please ?21:52
EmilienMso we can start using it with mwhahaha21:52
crinkleEmilienM: sure21:53
EmilienMthanks!21:53
openstackgerritLiao Penghui proposed openstack/puppet-horizon: Add List/None type value in the local_settings.  https://review.openstack.org/26681421:54
*** xarses has joined #puppet-openstack21:54
*** rcernin has joined #puppet-openstack21:54
*** ccie6747 has joined #puppet-openstack21:55
*** aimon has quit IRC22:02
*** aimon has joined #puppet-openstack22:03
*** RichardRaseley has quit IRC22:09
*** rcernin has quit IRC22:10
*** jayg is now known as jayg|g0n322:18
*** trown is now known as trown|outttypeww22:21
*** ccie6747 has quit IRC22:26
*** gildub has quit IRC22:30
openstackgerritMerged openstack/puppet-openstack_spec_helper: Move defaults.rb to openstack spec helper  https://review.openstack.org/26718522:33
EmilienMmwhahaha: ^22:37
EmilienMthanks crinkle!22:37
mfischEmilienM: in your email you have 2 of the same link22:38
mfischah nm I see the follow up22:38
EmilienMmfisch: and I fixed it22:38
EmilienMmfisch: sorry I was sleeping22:38
EmilienMmwhahaha: if you want I can start with one module22:38
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Make haproxy balancer default options configurable  https://review.openstack.org/26721122:41
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Use HAProxy 'transparent' bind option for compat with IPv6  https://review.openstack.org/26721222:43
*** aimon has quit IRC22:44
mwhahahaEmilienM: you can or I can, i'm going to be out the next few days so if i don't get to it today feel free to start with one of the modules22:45
EmilienMmwhahaha: I'm playing with puppet-aodh now22:45
mwhahahasure22:45
*** dhollinger has quit IRC23:11
openstackgerrityuan.lu proposed openstack/puppet-glance: Delete deprecated options in Liberty  https://review.openstack.org/26722123:15
*** thumpba has joined #puppet-openstack23:17
*** thumpba has quit IRC23:21
*** ccie6747 has joined #puppet-openstack23:28
*** baker has quit IRC23:29
openstackgerritEmilien Macchi proposed openstack/puppet-aodh: Use openstack spec helper for spec defaults  https://review.openstack.org/26722423:30
*** baker has joined #puppet-openstack23:32
*** baker has quit IRC23:33
*** gildub has joined #puppet-openstack23:34
*** baker has joined #puppet-openstack23:37
*** ntpttr has quit IRC23:40
*** a7ndrew has quit IRC23:45
*** ducttape_ has quit IRC23:50
*** baker has quit IRC23:52
*** ntpttr has joined #puppet-openstack23:54
*** ntpttr has left #puppet-openstack23:57

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!