Friday, 2016-01-22

« Thursday, 2016-01-21 Index Saturday, 2016-01-23 »
*** ducttape_ has quit IRC00:01
*** baker has joined #puppet-openstack00:06
*** jfluhmann has quit IRC00:06
*** markvoelker has quit IRC00:11
*** gildub has joined #puppet-openstack00:15
*** zhangjn has joined #puppet-openstack00:17
*** baker has quit IRC00:18
*** markvoelker has joined #puppet-openstack00:18
*** angdraug has joined #puppet-openstack00:21
*** gildub has quit IRC00:32
*** xarses has quit IRC00:35
*** gildub has joined #puppet-openstack00:37
*** zhangjn has quit IRC00:39
*** zhangjn has joined #puppet-openstack00:41
*** zhangjn has quit IRC00:41
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: spec: Enable webmock connect to IPv4 link-local  https://review.openstack.org/27109200:47
*** baker has joined #puppet-openstack00:47
*** baker has quit IRC00:49
*** joshsamuelson has quit IRC00:50
*** jprs has joined #puppet-openstack00:51
*** beddari has quit IRC00:55
*** xarses has joined #puppet-openstack00:56
*** beddari has joined #puppet-openstack00:57
*** rahulait has quit IRC00:58
*** aimon has quit IRC01:02
*** karimb has quit IRC01:04
*** markvoelker has quit IRC01:09
*** xarses has quit IRC01:20
*** morazi has quit IRC01:22
*** jfluhmann has joined #puppet-openstack01:29
*** xarses has joined #puppet-openstack01:48
*** mdorman has joined #puppet-openstack01:52
*** angdraug has quit IRC01:58
*** baker has joined #puppet-openstack02:02
*** jaosorior has quit IRC02:12
*** jaosorior has joined #puppet-openstack02:12
*** mdorman has quit IRC02:18
*** rahulait has joined #puppet-openstack02:21
*** rahulait has quit IRC02:32
*** xingchao has joined #puppet-openstack02:41
*** mdorman has joined #puppet-openstack02:53
*** richm has quit IRC02:53
openstackgerritMichael Chapman proposed openstack/puppet-aodh: Rename type on expirer service name  https://review.openstack.org/27112302:59
openstackgerritMichael Chapman proposed openstack/puppet-aodh: Rename typo on expirer service name  https://review.openstack.org/27112303:01
michchap^ delicious irony03:01
mdorman:)03:02
*** v1k0d3n has quit IRC03:16
*** severion has joined #puppet-openstack03:16
*** mdorman has quit IRC03:45
*** v1k0d3n has joined #puppet-openstack03:46
*** severion has quit IRC03:48
*** saneax_AFK is now known as saneax04:16
*** saneax is now known as Guest267204:17
*** aimon has joined #puppet-openstack04:28
*** Guest2672 is now known as saneax_AFK04:30
*** baker has quit IRC04:35
*** darkxploit has quit IRC04:53
openstackgerritAndy Botting proposed openstack/puppet-rally: Add settings for rally benchmarking  https://review.openstack.org/27113804:57
*** darkxploit has joined #puppet-openstack05:07
openstackgerritLiao Penghui proposed openstack/puppet-nova: Add warning for deprecated network_device_mtu  https://review.openstack.org/26878305:25
*** zhangjn has joined #puppet-openstack05:27
*** zhangjn has quit IRC05:28
*** gildub has quit IRC05:43
*** markvoelker has joined #puppet-openstack05:44
*** markvoelker_ has joined #puppet-openstack06:04
*** markvoelker has quit IRC06:07
*** narasimhasv has joined #puppet-openstack06:23
*** jaosorior has quit IRC06:32
*** jaosorior has joined #puppet-openstack06:33
openstackgerritIvan Chavero proposed openstack/puppet-ironic: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/27115106:36
*** fedexo has joined #puppet-openstack06:39
*** kjelly has joined #puppet-openstack06:45
openstackgerritJuan Antonio Osorio Robles proposed openstack/puppet-tripleo: SSL/Cinder: enable ssl_header_handler filter  https://review.openstack.org/26865306:47
openstackgerritAndrew Spiers proposed openstack/puppet-designate: Make keystone memcached servers configurable  https://review.openstack.org/27001806:52
*** magicboiz has joined #puppet-openstack07:05
openstackgerritnarasimha18sv proposed openstack/puppet-openstack_extras: Adding domain parameters to openrc file  https://review.openstack.org/27115607:05
*** magicboiz has quit IRC07:06
*** magicboiz has joined #puppet-openstack07:08
*** fedexo has quit IRC07:13
*** rcernin has joined #puppet-openstack07:31
*** psanchez has joined #puppet-openstack08:02
*** _matthias__ is now known as _matthias_08:04
*** dgurtner has joined #puppet-openstack08:17
*** dgurtner has quit IRC08:17
*** dgurtner has joined #puppet-openstack08:17
*** foxhunt_away is now known as foxhunt7208:26
*** markvoelker_ has quit IRC08:38
*** karimb has joined #puppet-openstack08:44
*** _degorenko|afk is now known as degorenko08:48
*** magicboiz has quit IRC08:53
*** yolanda has joined #puppet-openstack09:00
openstackgerritKarim Boumedhel proposed openstack/puppet-designate: Set correct options path for bind9 agent  https://review.openstack.org/26785609:05
*** magicboiz has joined #puppet-openstack09:08
*** rochaporto has joined #puppet-openstack09:15
*** magicboiz has quit IRC09:15
*** jistr has joined #puppet-openstack09:15
*** magicboiz has joined #puppet-openstack09:15
*** paramite has joined #puppet-openstack09:22
*** gfidente has joined #puppet-openstack09:26
*** markvoelker has joined #puppet-openstack09:34
*** markvoelker has quit IRC09:39
*** derekh has joined #puppet-openstack09:39
*** markvoelker has joined #puppet-openstack09:39
*** IvanBerezovskiy has joined #puppet-openstack09:42
*** markvoelker has quit IRC09:44
*** markvoelker has joined #puppet-openstack09:45
*** markvoelker has quit IRC09:50
openstackgerritnarasimha18sv proposed openstack/puppet-openstack_extras: Adding domain parameters to openrc file  https://review.openstack.org/27115609:55
*** igajsin has left #puppet-openstack10:01
*** chem has joined #puppet-openstack10:06
*** darkxploit has quit IRC10:12
*** jaosorior has quit IRC10:14
*** jaosorior has joined #puppet-openstack10:15
*** darkxploit has joined #puppet-openstack10:25
*** saneax_AFK is now known as saneax10:25
*** saneax is now known as Guest3228910:26
*** markvoelker has joined #puppet-openstack10:35
*** markvoelker has quit IRC10:39
*** bastelfreak is now known as hrnz10:50
*** hrnz has quit IRC10:50
*** bastelfreak1 has joined #puppet-openstack10:51
*** bastelfreak1 is now known as bastelfreak10:52
openstackgerrityuan.lu proposed openstack/puppet-glance: Add warning for deprecated options  https://review.openstack.org/26722110:57
*** markvoelker has joined #puppet-openstack11:12
*** rochaporto has quit IRC11:14
*** markvoelker has quit IRC11:18
*** chem has quit IRC11:25
*** narasimhasv has quit IRC11:31
*** chem has joined #puppet-openstack11:37
*** trown|outttypeww is now known as trown11:39
*** xingchao has quit IRC11:47
*** openstackgerrit has quit IRC11:47
*** openstackgerrit has joined #puppet-openstack11:47
*** jaosorior has quit IRC12:03
*** jaosorior has joined #puppet-openstack12:03
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Add is_ipv6_address function  https://review.openstack.org/27126012:06
*** markvoelker has joined #puppet-openstack12:07
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Add is_ipv6_address function  https://review.openstack.org/27126012:11
*** markvoelker has quit IRC12:12
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Add is_ipv6_address function  https://review.openstack.org/27126012:25
*** dprince has joined #puppet-openstack12:25
EmilienMhello12:29
*** openstackgerrit has quit IRC12:33
*** openstackgerrit has joined #puppet-openstack12:33
michchapEmilienM: morning12:36
openstackgerritGiulio Fidente proposed openstack/puppet-tripleo: Add is_ipv6_address function  https://review.openstack.org/27126012:38
mattymoEmilienM, hey12:41
mattymoEmilienM, I see you contributed to puppet-sysctl. I just realized that it doesn't auto-apply changes (sysctl -p)12:41
mattymodo you think it should be adjusted to apply? (optionally or automatically)12:42
*** markvoelker has joined #puppet-openstack12:42
EmilienMmattymo: me ???12:45
EmilienMdegorenko: I don't know what happens with puppet-keystone12:47
mattymohttps://github.com/duritong/puppet-sysctl/graphs/contributors well 1 commit12:47
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: CI test - never merge  https://review.openstack.org/27127412:47
EmilienMah https://github.com/duritong/puppet-sysctl/commit/c97e1469d648c5bc16d09a606e1d6f987b48ada612:48
EmilienMmattymo: I don't remembr it did not apply12:48
mattymoI read this note though: https://github.com/duritong/puppet-sysctl/blob/master/lib/puppet/type/sysctl.rb#L3-L1012:49
mattymoactually nevermind EmilienM: https://github.com/duritong/puppet-sysctl/blob/master/manifests/value.pp it's fine12:51
mattymosorry to bug you12:52
*** chem has quit IRC12:53
*** chem has joined #puppet-openstack12:54
*** markvoelker has quit IRC12:54
EmilienMwe're having CI issues12:54
EmilienMWebMock::NetConnectNotAllowedError everywhere12:54
EmilienMafter puppet 3.412:55
mattymoto what host?12:55
EmilienMall unit test jobs are failing on 3.6 3.7 3.8 4.x12:55
EmilienMsbadia: you already saw that in the patst12:56
EmilienMpast12:56
EmilienMit was https://launchpad.net/bugs/149263612:56
openstackLaunchpad bug 1492636 in puppet-keystone "Unit tests (puppet 3.3.x or 3.4.x) random erros : WebMock::NetConnectNotAllowedError" [Undecided,In progress] - Assigned to Emilien Macchi (emilienm)12:56
EmilienMI tried to fix it with keystone: https://review.openstack.org/#/c/271092/112:56
EmilienMit looks like it works12:56
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: beaker: stop testing eventlet and enforce wsgi  https://review.openstack.org/27081312:57
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: Use puppet-openstack_spec_helper for Rakefile & spec_helper_acceptance  https://review.openstack.org/26908712:57
EmilienMdprince: fyi it also affects puppet-tripleo12:57
sbadiayes sire! (indeed for 1492636)12:58
sbadiahello here12:58
degorenkoEmilienM, hey, sorry, i was on meeting12:59
EmilienMI'm not sure but I noticed it now affects puppet-glance13:01
EmilienMand puppet-tripleo13:01
EmilienMmaybe more, I don't know13:01
degorenkoEmilienM, really strange errors with keystone13:02
degorenkoEmilienM, is your changes for keystone looking on latests patch set for webmock? Looks like they are on 1 patch set13:04
*** jayg|g0n3 is now known as jayg13:10
*** karimb has quit IRC13:11
EmilienMdegorenko: I don't get the question13:11
EmilienMit's really affecting all modules13:12
degorenkoEmilienM, your message above: what's wrong with keystone. Is it about https://review.openstack.org/269087 and https://review.openstack.org/270813 ?13:12
EmilienMI see the error everywhere13:12
degorenkofor keystone spec right?13:12
EmilienMdegorenko: not at all13:12
EmilienMdegorenko: the 2 patches are on top of the mock fix13:12
degorenkoEmilienM, http://logs.openstack.org/87/269087/7/check/gate-puppet-keystone-puppet-unit-3.4-dsvm-centos7/5c2da2b/console.html#_2016-01-22_00_05_31_28013:12
degorenkoit is for rake13:13
degorenkolooks like it looking on 1 patch set for mock13:13
*** karimb has joined #puppet-openstack13:13
degorenkoare these changes rebased on 2 patch set with mock?13:13
EmilienMno !13:14
degorenkothen it can be reason13:14
degorenkobecause mock patch is good13:14
EmilienMit's affecting all modules, for the first test that is run13:14
EmilienMso we need to patch the lib13:15
degorenkoall modules?13:15
degorenkocrap13:15
EmilienMdegorenko: see comment on https://review.openstack.org/#/c/271092/13:16
degorenkonow i get it13:17
degorenkowe can add require for puppet-openstack_spec_helper/some-file in project spec helper13:18
degorenkoEmilienM, and in puppet-openstack_spec_helper add what you'va added in keystone13:18
degorenkois it your vision?13:18
EmilienMthat's a lot of patches again13:18
degorenkowell13:19
EmilienMI'm looking at another approach if possible13:19
degorenkook13:19
degorenkoanother way13:19
EmilienMI think we could ship spec/spec_helper.rb in the lib13:20
beddariso working on openid connect for keystone, there is a config setting where section is [federation] but the key "trusted_dashboard" is an array ... and they've implemented that so that you just repeat the key to specify the array. Is there any way you currently represent such settings? (first code block here http://docs.openstack.org/developer/keystone/federation/websso.html)13:20
degorenkoso this is needed for all spec tests? classes/defines?13:20
EmilienMmaybe not13:20
degorenkoi don't think, that openstacklib can help, because no one requires this in tests classes13:21
openstackgerritliaonanhai proposed openstack/puppet-ceilometer: Consistent with other core module, auth_uri,identity_uri  https://review.openstack.org/26951313:21
EmilienMdegorenko: I'm not talking about openstacklib13:21
EmilienMbut puppet-openstack_spec_helper13:21
degorenkoah, i though that lib = openstacklib13:21
EmilienMsorry13:22
degorenkono problem13:22
degorenkolooking13:22
EmilienMdegorenko: I have an idea13:23
degorenkowhat is it?13:23
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: spec: Enable webmock connect to IPv4 link-local  https://review.openstack.org/27128213:25
*** xingchao has joined #puppet-openstack13:25
EmilienMdegorenko: that ^13:25
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: defaults: Enable webmock connect to IPv4 link-local  https://review.openstack.org/27128213:25
degorenkoit's ugly to mix it with os_service_default13:26
EmilienMdegorenko: is defaults.rb only for os_service_default ?13:26
degorenkoEmilienM, module named as OSDefaults13:26
degorenkoso i think yes13:26
degorenkoit will work13:26
degorenkobut :)13:27
EmilienMyeah13:27
EmilienMactually the doc says:13:27
EmilienM# This file contains a module to return a default set of facts and supported13:27
EmilienM# operating systems for the tests in this module.13:27
EmilienMmaybe we can just change the doc13:27
degorenko:D13:27
EmilienMand say we also support commont bits for our unit tests13:27
EmilienM:)13:28
degorenkoyeah13:28
EmilienMI really want to avoid patching our 26 modules again13:28
*** xingchao has quit IRC13:28
degorenkoagree13:28
EmilienMand that's not so ugly, defaults.rb is not a bad name13:28
degorenkoi just though that this file only for os_defaults13:28
degorenkoi saw him, but skipped13:28
degorenkowell, yeah, it works for me13:29
degorenkobut change doc section :) smtg like: common file for default set of function/facts/etc13:29
EmilienMok13:29
degorenkoin some day probably we will refactor this13:29
*** ducttape_ has joined #puppet-openstack13:30
beddariasking shorter, any advice as to how to handle this config format for keystone.conf -> http://pastie.org/1069671813:32
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: defaults: Enable webmock connect to IPv4 link-local  https://review.openstack.org/27128213:33
*** xingchao has joined #puppet-openstack13:33
EmilienMdegorenko: ^13:33
EmilienMbeddari: Error 522 on your url13:33
beddari:~(13:33
degorenkobeddari, with current scheme of our config providers - there is no way13:34
degorenkoi can open your link :(13:34
degorenko:)13:34
beddarihuh, well, wrapped defines allowed?13:34
beddariI could do it with a private define I think, but ... :-o13:34
beddarior inline_template13:35
degorenkoour current conf provider is looking for some key under section and updates it13:35
degorenkobut you can use inline yes13:35
beddariyes I've read that13:35
beddariok thnx, I'll try and see if any other module already does13:36
beddariremind me to yell at the ones that wrote that config format13:36
degorenkowe already faced this problem in fuel, some ceilo config values, but we don't have good workaround for that13:37
*** markvoelker has joined #puppet-openstack13:38
degorenkobeddari, you can propose patch to puppet-openstacklib, which will fix this problem13:38
degorenkoall another modules has openstack config from openstacklib module as parent13:38
beddaridegorenko: yup, thanks for suggesting13:38
degorenkobeddari, yw13:38
degorenkoEmilienM, looks good13:39
degorenkowaiting for tests :)13:39
EmilienMdegorenko: tests are not working yet13:39
EmilienMI'm working on it13:40
degorenkook, ping me if some another help is needed13:40
*** markvoelker has quit IRC13:43
EmilienMdegorenko: maybe should I require 'webmock/rspec' in the defaults.rb ?13:43
*** ducttape_ has quit IRC13:43
*** xingchao has quit IRC13:44
degorenkoEmilienM, well, for modules no13:45
degorenkobecause of https://github.com/openstack/puppet-sahara/blob/master/spec/spec_helper.rb#L313:45
EmilienMyeah13:45
EmilienMok13:45
degorenkobut may be we can get failed tests for spec_helper project13:45
degorenkowe can also ask for review crinkle, may be she has some another ideas13:46
EmilienMyeah13:46
degorenkoi'm ok with current scheme13:46
EmilienMor sbadia ?13:46
degorenkosure :)13:46
degorenkoany core review is welcome, sure13:47
degorenko_ody :)13:47
sbadiadegorenko, EmilienM ok for me13:49
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: spec: Enable webmock connect to IPv4 link-local  https://review.openstack.org/27130013:49
*** xarses_ has joined #puppet-openstack13:49
EmilienMsbadia, degorenko: another bad news, it's affecting stable branches13:50
EmilienMand stable branches do not use the gem13:50
degorenkofacepalm :(13:50
sbadiaoh??13:50
degorenkoand they do not use require on spec_helper repo13:50
sbadiaand the gem isn't source by another?13:50
degorenkono13:51
degorenkofor stable13:51
EmilienMsbadia: is it from https://github.com/bblimke/webmock/ ?13:51
*** xarses has quit IRC13:52
EmilienMI don't see any very recent release in webmock13:52
EmilienMdegorenko, sbadia: if it's ok for you, can you review https://review.openstack.org/#/c/271282/ ?13:54
EmilienMso we can fix master first13:54
EmilienMwe will revert it if crinkle or _ody find a better way13:54
*** jfluhmann has quit IRC13:56
sbadiahum not sure but we don't use webmock anywhere in our modules ($ ack-grep stub_request)13:57
EmilienMsbadia: I grepped stub_request and found nothing?13:58
sbadiayep, so I wonder if webmock is still used in our modules13:59
*** markvoelker has joined #puppet-openstack13:59
EmilienMsbadia: isn't done by compile and compile.with ?13:59
sbadiayep but if we dont have the require?14:00
sbadiabut anyway ok with you patch/fix14:01
EmilienMsbadia: wait14:01
*** markvoelker_ has joined #puppet-openstack14:01
EmilienMsbadia: I would rather try to drop the dependency14:01
EmilienMif we don't use it14:02
sbadiaindeed, it would be better :)14:02
EmilienMyes14:02
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Drop webmock dependency  https://review.openstack.org/27130014:03
*** markvoelker has quit IRC14:04
EmilienMsbadia: so we have a fix to a problem we don't understand14:05
EmilienMsbadia: which commit in mock broke our ci14:06
EmilienMthat's the question (or it can be a dependency)14:06
sbadiayes I think it's a depdendency14:06
*** morazi has joined #puppet-openstack14:10
EmilienMdegorenko: so this approach will be 25 patches for each branch but at least we'll drop a dependency14:11
EmilienMit's worth it I think14:12
*** xingchao has joined #puppet-openstack14:14
degorenkowe have this problem for all branches? liberty,kilo,etc?14:15
*** richm has joined #puppet-openstack14:18
EmilienMdegorenko: all branches that have mock in Gemfile14:18
degorenkoomg :(14:20
degorenkothat's definitely problem14:20
EmilienMI don't find anyway to have a central Gemfile14:20
degorenkoyeah14:21
openstackgerritEmilien Macchi proposed openstack/puppet-aodh: drop mock dependency  https://review.openstack.org/27131714:27
openstackgerritEmilien Macchi proposed openstack/puppet-barbican: drop mock dependency  https://review.openstack.org/27132014:29
EmilienMdegorenko: only keystone is using it I thikn14:29
degorenkook then :)14:30
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: drop mock dependency  https://review.openstack.org/27132214:31
openstackgerritEmilien Macchi proposed openstack/puppet-ceph: drop mock dependency  https://review.openstack.org/27132314:32
openstackgerritEmilien Macchi proposed openstack/puppet-cinder: drop mock dependency  https://review.openstack.org/27132414:33
openstackgerritEmilien Macchi proposed openstack/puppet-designate: drop mock dependency  https://review.openstack.org/27132514:34
openstackgerritMerged openstack/puppet-tripleo: Drop webmock dependency  https://review.openstack.org/27130014:34
EmilienMdegorenko: glance too14:36
EmilienMin fact there is mock and webmock, and glance use mock14:37
*** xingchao has quit IRC14:37
openstackgerritEmilien Macchi proposed openstack/puppet-glance: drop mock dependency  https://review.openstack.org/27132914:39
openstackgerritEmilien Macchi proposed openstack/puppet-gnocchi: drop mock dependency  https://review.openstack.org/27133014:41
openstackgerritEmilien Macchi proposed openstack/puppet-heat: drop mock dependency  https://review.openstack.org/27133314:45
openstackgerritEmilien Macchi proposed openstack/puppet-horizon: drop mock dependency  https://review.openstack.org/27133514:47
*** xingchao has joined #puppet-openstack14:47
openstackgerritEmilien Macchi proposed openstack/puppet-ironic: drop mock dependency  https://review.openstack.org/27133614:48
*** mwoodson has left #puppet-openstack14:48
openstackgerritEmilien Macchi proposed openstack/puppet-keystone: spec: Enable webmock connect to IPv4 link-local  https://review.openstack.org/27109214:50
openstackgerritEmilien Macchi proposed openstack/puppet-manila: drop mock dependency  https://review.openstack.org/27133814:50
openstackgerritEmilien Macchi proposed openstack/puppet-mistral: drop mock dependency  https://review.openstack.org/27133914:51
openstackgerritEmilien Macchi proposed openstack/puppet-modulesync-configs: drop mock dependency  https://review.openstack.org/27134114:52
openstackgerritEmilien Macchi proposed openstack/puppet-monasca: drop mock dependency  https://review.openstack.org/27134214:53
openstackgerritEmilien Macchi proposed openstack/puppet-murano: drop mock dependency  https://review.openstack.org/27134314:54
openstackgerritEmilien Macchi proposed openstack/puppet-neutron: drop mock dependency  https://review.openstack.org/27134414:54
openstackgerritEmilien Macchi proposed openstack/puppet-nova: drop mock dependency  https://review.openstack.org/27134514:56
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_extras: drop mock dependency  https://review.openstack.org/27134714:57
*** markvoelker has joined #puppet-openstack14:57
*** xarses_ has quit IRC14:58
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: drop mock dependency  https://review.openstack.org/27134814:58
openstackgerritEmilien Macchi proposed openstack/puppet-openstacklib: drop mock dependency  https://review.openstack.org/27134914:58
openstackgerritEmilien Macchi proposed openstack/puppet-rally: drop mock dependency  https://review.openstack.org/27135114:59
openstackgerritEmilien Macchi proposed openstack/puppet-sahara: drop mock dependency  https://review.openstack.org/27135315:00
*** markvoelker_ has quit IRC15:01
*** ducttape_ has joined #puppet-openstack15:09
openstackgerritEmilien Macchi proposed openstack/puppet-tempest: drop mock dependency  https://review.openstack.org/27136215:11
*** markvoelker has quit IRC15:12
openstackgerritEmilien Macchi proposed openstack/puppet-trove: drop mock dependency  https://review.openstack.org/27136315:12
openstackgerritEmilien Macchi proposed openstack/puppet-vitrage: drop mock dependency  https://review.openstack.org/27136415:13
openstackgerritEmilien Macchi proposed openstack/puppet-vswitch: drop mock dependency  https://review.openstack.org/27136715:14
openstackgerritEmilien Macchi proposed openstack/puppet-zaqar: drop mock dependency  https://review.openstack.org/27136815:14
*** xingchao_ has joined #puppet-openstack15:14
*** baker has joined #puppet-openstack15:16
*** xingchao has quit IRC15:17
*** xingchao has joined #puppet-openstack15:17
*** jistr has quit IRC15:18
EmilienMdprince, degorenko: we need to land https://review.openstack.org/26912415:19
EmilienMbecause puppet-aodh & puppet-gnocchi beaker CI jobs are broken15:20
EmilienMsbadia: ^15:20
dprince+2 from me15:21
*** xingchao_ has quit IRC15:21
EmilienMthanks15:21
EmilienMthanks sbadia and degorenko too, it helps :)15:22
EmilienMdegorenko: you can 'recheck' your gnocchi patch ;-)15:22
EmilienMwith this bump, you should have no issue15:22
degorenko\o/15:23
degorenkonice15:23
openstackgerritMerged openstack/puppet-openstack-integration: Bump RDO to 21/01/2016 URL repo  https://review.openstack.org/26912415:24
*** dfisher has joined #puppet-openstack15:33
*** xarses_ has joined #puppet-openstack15:39
*** dhollinger has joined #puppet-openstack15:44
*** markvoelker has joined #puppet-openstack15:44
*** dgurtner has quit IRC16:00
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos/rdo: use current-passed-ci URL  https://review.openstack.org/27140916:05
EmilienMdmsimard: ^16:05
dmsimardty16:05
openstackgerritBradley Klein proposed openstack/puppet-monasca: Refactory keystone service and endpoint to be more like other puppet modules  https://review.openstack.org/27141016:07
*** mdorman has joined #puppet-openstack16:07
openstackgerritKarim Boumedhel proposed openstack/puppet-designate: Set correct options path for bind9 agent  https://review.openstack.org/26785616:12
EmilienMdegorenko, sbadia: https://review.openstack.org/#/c/271092/16:15
*** rcernin has quit IRC16:15
degorenkoEmilienM, why we need it in master now? We already merged that to spec_helper repo16:16
EmilienMand https://review.openstack.org/#/q/topic:puppet/drop-mock16:16
EmilienMdegorenko: no we didn't16:16
EmilienMwait16:16
EmilienMI abandonned it16:16
EmilienMdegorenko: https://review.openstack.org/#/c/271282/16:17
degorenkoah16:17
degorenkoi see16:17
degorenkoso, this problem only for keystone right?16:17
EmilienMdegorenko: for now, yes but also glance16:17
EmilienMand maye more16:17
degorenkook16:17
EmilienMdegorenko: this serie of patches aims to drop useless gem we had16:18
EmilienMand keep it only where it's actually needed16:18
degorenkonot all of them have +1 from ci16:18
EmilienMyep16:18
EmilienMI'm investigating16:18
EmilienMbut any help is welcome16:18
degorenkolooks like some jenkins slave failures16:19
EmilienMfor ironic:16:19
EmilienMhttp://logs.openstack.org/36/271336/1/check/gate-puppet-ironic-puppet-syntax-3-dsvm-centos7/cb55a08/console.html#_2016-01-22_15_03_04_91516:19
*** rahulait has joined #puppet-openstack16:19
degorenkobut why rake is aborted?16:19
EmilienMI don't know but there is something new16:20
EmilienMI have the same error elsewhere16:20
degorenkohm16:20
EmilienMin puppet-murano16:20
EmilienMsome jenkins failure are also EPEL downtimes16:22
degorenkofor some of them recheck should help16:23
EmilienMyes16:23
EmilienMthis one is new: http://logs.openstack.org/33/271333/1/check/gate-puppet-heat-puppet-beaker-rspec-dsvm-centos7/ce7d558/console.html#_2016-01-22_15_11_54_92816:23
EmilienMdoing recheck to see if it's recurrent16:26
EmilienMdegorenko: look https://review.openstack.org/#/c/271343/16:26
EmilienMdegorenko: only murano & ironic have syntax jobs issues16:27
EmilienMdoing recheck also to see if it comes back16:28
degorenkowill hope it will help :D16:28
*** IvanBerezovskiy has left #puppet-openstack16:29
EmilienMwill see16:29
*** karimb has quit IRC16:30
*** foxhunt72 is now known as foxhunt_away16:31
*** jaosorior has quit IRC16:35
openstackgerritBradley Klein proposed openstack/puppet-monasca: Refactory keystone service and endpoint to be more like other puppet modules  https://review.openstack.org/27141016:39
*** degorenko is now known as _degorenko|afk16:41
*** jistr has joined #puppet-openstack16:43
*** rcernin has joined #puppet-openstack16:54
gfidenteEmilienM, are pull requets the right way to go for puppet-stdlib ?16:57
openstackgerritEmilien Macchi proposed openstack/puppet-ceilometer: Implement auth_endpoint_type parameter  https://review.openstack.org/27142616:58
EmilienMgfidente: yes, I think so16:58
EmilienMgfidente: ping us here if you have a PR on it, we have some folks from puppetlabs who can quickly help16:58
gfidenteEmilienM, yeah but this is definitely out of my territory16:59
gfidentehttps://github.com/puppetlabs/puppetlabs-stdlib/pull/57016:59
gfidenteI just had to do a bunch of copy paste16:59
gfidenteand i've already spotted an error :P16:59
EmilienMgfidente: nice contribution !16:59
openstackgerritMaksym Yatsenko proposed openstack/puppet-horizon: Use keystone v3 with horizon by default  https://review.openstack.org/26900017:02
*** derekh has quit IRC17:02
*** jistr has quit IRC17:08
*** jistr has joined #puppet-openstack17:09
*** jistr has quit IRC17:14
*** trown is now known as trown|lunch17:15
*** jistr has joined #puppet-openstack17:17
openstackgerritBradley Klein proposed openstack/puppet-monasca: Refactor keystone service and endpoint to be more like other puppet modules  https://review.openstack.org/27141017:24
dmsimardgfidente: looks to me like I could pass an ipv4 to the is_ipv6 function and it'd return true17:28
dmsimardand vice-versa17:28
dmsimardunless I read that wrong17:28
dmsimardok yeah I read that wrong :P17:28
gfidentedmsimard, it'd have been nice to do that with is_ip_address though17:29
*** ducttape_ has quit IRC17:31
*** ducttape_ has joined #puppet-openstack17:32
*** ducttape_ has quit IRC17:33
*** ducttape_ has joined #puppet-openstack17:34
*** ducttape_ has quit IRC17:34
EmilienMgfidente: why not using the other function and add a boolean if we want the catalog failing or not17:34
*** ducttape_ has joined #puppet-openstack17:35
gfidenteEmilienM, is_ip_address you mean?17:35
EmilienMgfidente: yeah17:36
gfidenteEmilienM, yeah I was thinking about doing that too, but saw the two distinguished validate_ipv? functions17:36
gfidentecan you comment that in the PR? I want to try it as well17:36
gfidentedmsimard, where does is_ipv6 come from?17:37
dmsimardgfidente: I meant is_ipv6_address17:38
EmilienMgfidente: commented https://github.com/puppetlabs/puppetlabs-stdlib/pull/570#issuecomment-17398914817:42
gfidenteEmilienM, on make validate return true/false17:43
gfidenteis that acceptable or should validate_ raise by definition?17:43
EmilienMgfidente: like I said in my comment, to be backward compatible you'll need to raise by default17:44
gfidenteah I see17:44
gfidenteso you mean use a parameter which makes those return instead of raise17:44
gfidenteI'll try to put up another submission next week17:49
*** mdavidson has quit IRC17:50
EmilienMgfidente: I'll also look later if I can help !17:50
*** rahulait has quit IRC17:51
-openstackstatus- NOTICE: Restarting zuul due to a memory leak17:51
openstackgerritDan Prince proposed openstack/puppet-mistral: The workflow service type should be workflowv2  https://review.openstack.org/27144917:52
*** jistr has quit IRC17:57
*** gfidente is now known as gfidente|afk17:57
*** aimon has quit IRC18:05
mwhahaha_ody: you around today? we're running into some weird issue with facts & rspec under the latest version of facter18:09
mwhahahafacter 2.4.5 that is18:10
*** xingchao has quit IRC18:10
*** trown|lunch is now known as trown|mostlyafk18:26
openstackgerritBradley Klein proposed openstack/puppet-monasca: Refactor keystone service and endpoint to be more like other puppet modules  https://review.openstack.org/27141018:26
*** yolanda has quit IRC18:26
nibalizermwhahaha: is it 'all rspec unit tests fail' ?18:27
mwhahahano18:27
nibalizerhttps://tickets.puppetlabs.com/browse/PUP-5743 is a thing18:27
*** angdraug has joined #puppet-openstack18:27
mwhahahait's related to a fact18:27
nibalizeroh okay18:28
nibalizerI have no information for you then18:28
*** bklei has joined #puppet-openstack18:29
mwhahahahttp://logs.openstack.org/13/270313/5/gate/gate-fuel-library-puppet-unit-3.8-dsvm-centos7/95defa4/console.html#_2016-01-22_12_10_04_20018:29
mwhahahafacter 2.4.4 works but 2.4.5 doesn'18:29
mwhahahadoesn't but i was looking at the code diffs and i can't see anything that would explain why18:30
mwhahahamaybe it's a puppet 3.8.5 with 2.4.5 thing18:30
* mwhahaha shrugs18:30
*** baker has quit IRC18:36
*** gfidente|afk has quit IRC18:38
*** aimon has joined #puppet-openstack18:39
*** aimon has quit IRC18:42
*** baker has joined #puppet-openstack18:42
*** aimon has joined #puppet-openstack18:44
*** xingchao has joined #puppet-openstack18:45
*** xingchao has quit IRC18:50
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Enable X-Forwarded-Proto header for Heat and Nova  https://review.openstack.org/27148119:00
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Enable X-Forwarded-Proto header for keystone_public  https://review.openstack.org/27148219:02
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Add tripleo_ipv6_interfaces fact  https://review.openstack.org/27148319:03
mwhahahafound it, nibalizer they changed up how they generate ipaddress facts in facter and it's breaking everything, https://github.com/puppetlabs/facter/commit/ec73549be0c26aab40a8968330726b369ab080f719:03
mwhahahai reverted the ipaddress.rb to one from 2.4.4 and everything is golden19:04
mwhahahai'm going to try and write up a bug in a bit19:04
*** dhollinger has quit IRC19:05
EmilienMmwhahaha: can you look https://review.openstack.org/#/q/topic:puppet/drop-mock when you got time please?19:07
EmilienMand https://review.openstack.org/27109219:07
EmilienMit would help to unblock our CI19:07
EmilienMcurrent unit test jobs are failing on some modules19:07
*** rcernin has quit IRC19:12
*** markvoelker has quit IRC19:27
openstackgerritEmilien Macchi proposed openstack/puppet-gnocchi: Change db-sync command for gnocchi and drop mock  https://review.openstack.org/26921419:36
openstackgerritBradley Klein proposed openstack/puppet-monasca: Use puppet-openstack_spec_helper for Rakefile & spec_helper_acceptance  https://review.openstack.org/27149319:36
*** rahulait has joined #puppet-openstack19:36
mwhahahaYea when I get back to my computer19:36
bkleii'm seeing a bunch of gate jobs fail with 'NOT_REGISTERED' -- anyone else seeing the same?19:42
bkleisimple jobs like puppet-syntax..19:42
iurygregorywe have a problem in your CI XD19:42
openstackgerritEmilien Macchi proposed openstack/puppet-oslo: drop mock dependency  https://review.openstack.org/27149719:42
iurygregorybut i don't think this have some to NOT REGISTRED... maybe the infra is down19:43
EmilienMbklei: this one is not relate to what I've seen19:43
EmilienMlet me check19:43
bkleithat's what it seems like, i haven't seen that one before19:43
bkleiexample https://review.openstack.org/#/c/271410/19:43
EmilienMmaybe a change in zuul19:43
EmilienMnibalizer: ^ any hint?19:44
EmilienMI don't see any recent change in project-config19:44
bkleisomething seems to have changed earlier today i had some jobs succeed, then in the last few hours started getting the not registered19:45
*** EmilienM sets mode: +o crinkle19:46
*** EmilienM sets mode: +o mwhahaha19:46
*** EmilienM sets mode: +o mfisch19:46
*** EmilienM sets mode: +o clayton19:46
*** EmilienM sets mode: +o chem19:46
*** EmilienM sets mode: +o richm19:46
*** EmilienM sets mode: +o spredzy19:46
*** EmilienM sets mode: +o sbadia19:46
*** xingchao has joined #puppet-openstack19:47
bkleiactually, i see a whole bunch of jobs failing like that, puppet-ceilometer, heat, etc19:50
EmilienMmhh19:50
*** xingchao has quit IRC19:51
*** dhollinger has joined #puppet-openstack20:07
*** dprince has quit IRC20:18
*** dprince has joined #puppet-openstack20:18
EmilienMbklei: I see the bug too20:32
EmilienMI have no idea why20:32
EmilienMwe need to 'recheck'20:32
bkleicool, will try20:32
EmilienMyeah sorry for that20:33
bkleinp20:33
EmilienMbklei: reading a bit of infra backport, it seems like we're not alone20:35
bkleimisery loves company :)20:35
EmilienMour CI is really in bad shape today20:35
EmilienMsome beaker jobs are broken20:35
EmilienMmost of unit tests jobs are broken20:36
bkleimiller time20:36
EmilienMthe good news is all is under review20:36
EmilienMthe bad news is, it's not going to merge today for sure20:36
EmilienMwe also need to backport all the fixes in stable20:36
EmilienMcrinkle: I started to work on testing our new library https://review.openstack.org/269901 but I have a question. bundle is installing the gem already (because it's in Gemfile). I was wondering how I could replace it by the current repo. So I'm building a new gem, but when I install it, it does not override the one installed by bundler. Any advise?20:45
openstackgerritMerged openstack/puppet-barbican: drop mock dependency  https://review.openstack.org/27132020:45
openstackgerritMerged openstack/puppet-monasca: drop mock dependency  https://review.openstack.org/27134220:47
*** xingchao has joined #puppet-openstack20:48
*** chem has quit IRC20:49
openstackgerritMerged openstack/puppet-designate: drop mock dependency  https://review.openstack.org/27132520:51
crinkleEmilienM: you can't override it, 'bundle *' means only use things in the gemfile20:52
crinkleEmilienM: you could change the gemfile to use :path instead of :git20:52
EmilienMcrinkle: oh I did not think about it :-)20:53
EmilienMthank you!20:53
*** xingchao has quit IRC20:53
openstackgerritBradley Klein proposed openstack/puppet-monasca: Refactor keystone service and endpoint to be more like other puppet modules  https://review.openstack.org/27141020:54
*** dprince has quit IRC20:55
openstackgerritMerged openstack/puppet-modulesync-configs: drop mock dependency  https://review.openstack.org/27134120:56
*** jayg is now known as jayg|g0n321:02
openstackgerritMerged openstack/puppet-openstack-integration: drop mock dependency  https://review.openstack.org/27134821:08
openstackgerritMerged openstack/puppet-rally: drop mock dependency  https://review.openstack.org/27135121:10
openstackgerritMerged openstack/puppet-vitrage: drop mock dependency  https://review.openstack.org/27136421:12
openstackgerritMerged openstack/puppet-mistral: drop mock dependency  https://review.openstack.org/27133921:13
openstackgerritMerged openstack/puppet-zaqar: drop mock dependency  https://review.openstack.org/27136821:13
openstackgerritMerged openstack/puppet-openstacklib: drop mock dependency  https://review.openstack.org/27134921:15
openstackgerritMerged openstack/puppet-glance: drop mock dependency  https://review.openstack.org/27132921:16
openstackgerritMerged openstack/puppet-ceilometer: drop mock dependency  https://review.openstack.org/27132221:16
openstackgerritMerged openstack/puppet-horizon: drop mock dependency  https://review.openstack.org/27133521:18
*** jtomasek has quit IRC21:18
openstackgerritMerged openstack/puppet-neutron: drop mock dependency  https://review.openstack.org/27134421:20
openstackgerritMerged openstack/puppet-nova: drop mock dependency  https://review.openstack.org/27134521:20
*** markvoelker has joined #puppet-openstack21:40
*** markvoelker has quit IRC21:40
*** markvoelker has joined #puppet-openstack21:41
openstackgerritBradley Klein proposed openstack/puppet-monasca: Use puppet-openstack_spec_helper for Rakefile & spec_helper_acceptance  https://review.openstack.org/27149321:45
*** imcsk8 is now known as imcsk8|afk21:47
openstackgerritMerged openstack/puppet-monasca: Refactor keystone service and endpoint to be more like other puppet modules  https://review.openstack.org/27141021:51
*** rcernin has joined #puppet-openstack21:52
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: First implementation of run_tests.sh  https://review.openstack.org/26990122:07
EmilienMcrinkle: first implementation, that should work, any feedback is welcome :-)22:08
*** dhollinger has quit IRC22:17
*** cwolferh has quit IRC22:21
*** RichardRaseley has joined #puppet-openstack22:22
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: First implementation of run_tests.sh  https://review.openstack.org/26990122:29
*** markvoelker has quit IRC22:32
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper: CI test - do not merge  https://review.openstack.org/27060022:44
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration: repos/rdo: use current-passed-ci URL  https://review.openstack.org/27140922:47
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo: Drop webmock dependency  https://review.openstack.org/27155322:48
*** xingchao has joined #puppet-openstack22:50
*** baker has quit IRC22:52
*** rcernin has quit IRC22:53
*** aimon has quit IRC22:54
*** xingchao has quit IRC22:54
*** aimon has joined #puppet-openstack22:55
*** derekh has joined #puppet-openstack22:58
EmilienMxarses_: can you look https://review.openstack.org/#/c/271323/ please ?22:59
*** aimon has quit IRC23:00
*** bklei has quit IRC23:07
*** bklei has joined #puppet-openstack23:08
*** bklei_ has joined #puppet-openstack23:09
EmilienMthis new error is funny: https://review.openstack.org/#/c/271336/23:09
EmilienMhttp://logs.openstack.org/36/271336/1/check/gate-puppet-ironic-puppet-syntax-3-dsvm-centos7/f321f19/console.html#_2016-01-22_20_48_37_94123:09
EmilienMit sounds like related to https://github.com/puppetlabs/puppet/commit/191ccfcfb23dfca3c6affe6c98cf3154c8d9e78923:11
*** ducttape_ has quit IRC23:11
*** bklei has quit IRC23:12
*** rahulait has quit IRC23:12
*** vishpat has joined #puppet-openstack23:13
*** bklei_ has quit IRC23:13
EmilienM_ody: ^23:14
EmilienMI think puppet 3.8.5 contains something that breaks some of our syntax jobs23:16
openstackgerritMerged openstack/puppet-cinder: drop mock dependency  https://review.openstack.org/27132423:18
* EmilienM reading https://github.com/puppetlabs/puppet/compare/3.8.4...3.8.523:19
EmilienMgot it https://github.com/puppetlabs/puppet/commit/8df197b7046bedad928c648dfcbf434c386a09df23:20
openstackgerritMerged openstack/puppet-keystone: spec: Enable webmock connect to IPv4 link-local  https://review.openstack.org/27109223:35
*** bklei has joined #puppet-openstack23:39
*** derekh has quit IRC23:46
*** xingchao has joined #puppet-openstack23:51
*** bklei has quit IRC23:54
*** bklei has joined #puppet-openstack23:54
*** xingchao has quit IRC23:55
*** mdorman has quit IRC23:57
*** bklei has quit IRC23:58
« Thursday, 2016-01-21 Index Saturday, 2016-01-23 »

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