Friday, 2018-02-09

*** alee has joined #puppet-openstack00:03
*** xarses_ has quit IRC00:06
openstackgerritJames E. Blair proposed openstack/puppet-nova stable/ocata: Zuul: Remove project name  https://review.openstack.org/54250200:07
openstackgerritzhongshengping proposed openstack/puppet-designate master: Designate/Neutron communicaton failure  https://review.openstack.org/53421400:46
openstackgerritJames E. Blair proposed openstack/puppet-vswitch stable/newton: Zuul: Remove project name  https://review.openstack.org/54251901:07
*** vnogin has joined #puppet-openstack01:15
*** vnogin has quit IRC01:20
*** xarses has joined #puppet-openstack01:21
mnasersai_: doyou have more of the error?01:38
sai_yes01:38
openstackgerritMerged openstack/puppet-nova master: Add CORS configuration support  https://review.openstack.org/54184601:42
sai_https://www.irccloud.com/pastebin/4IeHNa9b/01:43
mnasersai_: we dont support puppet 5 yet01:47
sai_mnaser: ok, is there a other way around to solve this?01:50
sai_mnaser: to use puppet <5?01:50
mnaseruse puppet 4 and you should be good :)01:51
sai_mnaser: thank you, after using puppet 4.10.1, the tests are running fine02:44
*** ilbot3 has quit IRC02:46
sai_mnaser: if puppet 5 is not supported why is it being picked up when running the tests? It should be using puppet 4 right?02:47
mnaserIs the gemfile installing puppet 4?02:47
sai_no gem installing puppet 502:48
sai_sorry small correction gem is using puppet 502:49
openstackgerritMerged openstack/puppet-neutron master: Corrections in comments  https://review.openstack.org/54199302:57
*** ilbot3 has joined #puppet-openstack02:57
openstackgerritSai Ram Peesapati proposed openstack/puppet-neutron master: Optionally set device_driver to $::os_service_default  https://review.openstack.org/54255202:59
openstackgerritSai Ram Peesapati proposed openstack/puppet-neutron stable/pike: Corrections in comments  https://review.openstack.org/54255703:08
openstackgerritSai Ram Peesapati proposed openstack/puppet-neutron stable/ocata: Corrections in comments  https://review.openstack.org/54255903:08
openstackgerritSai Ram Peesapati proposed openstack/puppet-neutron stable/newton: Corrections in comments  https://review.openstack.org/54256003:08
*** ykarel|away has joined #puppet-openstack03:44
*** ykarel|away is now known as ykarel03:45
*** skramaja has joined #puppet-openstack03:50
openstackgerritMerged openstack/puppet-manila stable/ocata: Zuul: Remove project name  https://review.openstack.org/54247504:05
openstackgerritMerged openstack/puppet-aodh stable/ocata: Zuul: Remove project name  https://review.openstack.org/54238004:12
openstackgerritMerged openstack/puppet-heat stable/newton: Make heat parameter plugin_dirs configurable  https://review.openstack.org/54140204:19
openstackgerritMerged openstack/puppet-horizon stable/newton: Zuul: Remove project name  https://review.openstack.org/54242104:19
openstackgerritMerged openstack/puppet-nova stable/ocata: Zuul: Remove project name  https://review.openstack.org/54250204:26
openstackgerritMerged openstack/puppet-vswitch stable/newton: Zuul: Remove project name  https://review.openstack.org/54251904:26
openstackgerritMerged openstack/puppet-ceilometer master: Remove ceilometer api class  https://review.openstack.org/54156204:26
openstackgerritMerged openstack/puppet-aodh stable/pike: Zuul: Remove project name  https://review.openstack.org/54248604:26
openstackgerritMerged openstack/puppet-oslo master: Revert "Update olso.log parameters"  https://review.openstack.org/53462904:32
openstackgerritMerged openstack/puppet-oslo stable/newton: Zuul: Remove project name  https://review.openstack.org/54242904:32
openstackgerritMerged openstack/puppet-cinder master: Remove deprecated enable_v1/v2_api options  https://review.openstack.org/54158704:32
openstackgerritMerged openstack/puppet-openstack-integration master: Remove deprecated backend_driver options  https://review.openstack.org/54209904:32
openstackgerritMerged openstack/puppet-glance master: Remove deprecated sync_db options  https://review.openstack.org/54206604:37
openstackgerritJames E. Blair proposed openstack/puppet-trove stable/newton: Zuul: Remove project name  https://review.openstack.org/54258804:48
*** vnogin has joined #puppet-openstack05:15
*** vnogin has quit IRC05:20
openstackgerritzhongshengping proposed openstack/puppet-aodh master: Add some kombu options  https://review.openstack.org/54259905:47
openstackgerritMerged openstack/puppet-openstack-integration master: Updated from Puppet OpenStack modules constraints  https://review.openstack.org/54155205:56
openstackgerritJames E. Blair proposed openstack/puppet-nova stable/newton: Zuul: Remove project name  https://review.openstack.org/54260706:09
openstackgerritzhongshengping proposed openstack/puppet-cinder master: Add some kombu options  https://review.openstack.org/54261506:20
openstackgerritzhongshengping proposed openstack/puppet-ceilometer master: Add some kombu options  https://review.openstack.org/54261606:20
openstackgerritzhongshengping proposed openstack/puppet-congress master: Add some kombu options  https://review.openstack.org/54261706:20
openstackgerritzhongshengping proposed openstack/puppet-barbican master: Add some kombu options  https://review.openstack.org/54261806:20
openstackgerritzhongshengping proposed openstack/puppet-designate master: Add kombu_failover_strategy option  https://review.openstack.org/54262206:29
*** xarses has quit IRC07:01
*** yprokule has joined #puppet-openstack07:04
tobascoEmilienM: we can merged that given the spec_helper change is merged aswell, the spec_helper patch is blocked by zuul for the unit test failure which is not caused by that change07:05
tobascoi started some troubleshooting yesterday but didn't have time to check it out properly07:05
tobascoseems like its a puppet change causing it or something with gem_home paths messing it up07:06
*** rcernin has quit IRC07:17
*** rmart04 has joined #puppet-openstack07:17
openstackgerritzhongshengping proposed openstack/puppet-ironic master: Add kombu_failover_strategy option  https://review.openstack.org/54266407:21
openstackgerritzhongshengping proposed openstack/puppet-vitrage master: Add kombu_failover_strategy option  https://review.openstack.org/54266507:21
openstackgerritzhongshengping proposed openstack/puppet-manila master: Add kombu_failover_strategy option  https://review.openstack.org/54266607:21
openstackgerritzhongshengping proposed openstack/puppet-magnum master: Add kombu_failover_strategy option  https://review.openstack.org/54266707:21
openstackgerritzhongshengping proposed openstack/puppet-glance master: Add kombu_failover_strategy option  https://review.openstack.org/54266807:21
openstackgerritzhongshengping proposed openstack/puppet-mistral master: Add kombu_failover_strategy option  https://review.openstack.org/54266907:21
openstackgerritzhongshengping proposed openstack/puppet-tacker master: Add kombu_failover_strategy option  https://review.openstack.org/54267007:21
openstackgerritzhongshengping proposed openstack/puppet-trove master: Add kombu_failover_strategy option  https://review.openstack.org/54267107:21
openstackgerritzhongshengping proposed openstack/puppet-octavia master: Add kombu_failover_strategy option  https://review.openstack.org/54267207:21
openstackgerritzhongshengping proposed openstack/puppet-nova master: Add kombu_failover_strategy option  https://review.openstack.org/54267307:21
*** rmart04 has quit IRC07:22
*** etingof has quit IRC07:22
*** etingof has joined #puppet-openstack07:34
*** piepmatz has joined #puppet-openstack07:56
*** dtantsur|afk is now known as dtantsur08:08
*** jpena|off is now known as jpena08:10
*** amoralej|off is now known as amoralej08:16
*** makowals has quit IRC08:23
*** tesseract has joined #puppet-openstack08:26
*** colonwq has quit IRC08:27
*** colonwq has joined #puppet-openstack08:33
*** etingof has quit IRC08:33
*** etingof has joined #puppet-openstack08:47
*** sai_ has quit IRC08:51
*** gfidente has joined #puppet-openstack08:56
*** gfidente has quit IRC08:56
*** gfidente has joined #puppet-openstack08:56
*** ykarel is now known as ykarel|lunch09:08
openstackgerritJames E. Blair proposed openstack/puppet-congress stable/pike: Zuul: Remove project name  https://review.openstack.org/54273709:10
tobascocentos failures in ci, anybody know if it's tempest or actually centos related?09:22
tobascotempest fails some neutron tests09:22
*** hjensas|afk is now known as hjensas09:26
openstackgerritJames E. Blair proposed openstack/puppet-swift stable/pike: Zuul: Remove project name  https://review.openstack.org/54274409:30
*** dprince has joined #puppet-openstack09:41
openstackgerritMerged openstack/puppet-designate master: Remove deprecated backend_driver options  https://review.openstack.org/54206409:42
*** shardy has joined #puppet-openstack09:49
openstackgerritBogdan Dobrelya proposed openstack/puppet-oslo master: Revert "Update olso.log parameters"  https://review.openstack.org/54275209:59
*** derekh has joined #puppet-openstack10:01
openstackgerritJames E. Blair proposed openstack/puppet-designate stable/ocata: Zuul: Remove project name  https://review.openstack.org/54276110:10
*** ykarel|lunch is now known as ykarel10:11
*** dprince has quit IRC10:20
*** salmankhan has joined #puppet-openstack10:23
*** vnogin has joined #puppet-openstack10:25
*** mkarpin has joined #puppet-openstack10:27
openstackgerritJames E. Blair proposed openstack/puppet-manila stable/newton: Zuul: Remove project name  https://review.openstack.org/54276810:30
*** dtantsur is now known as dtantsur|bbl10:43
*** chandankumar has quit IRC10:54
*** vnogin has quit IRC10:59
*** vnogin has joined #puppet-openstack11:01
*** jaganathan has quit IRC11:06
*** chandankumar has joined #puppet-openstack11:17
*** vnogin has quit IRC11:25
*** hjensas has quit IRC11:44
*** hjensas has joined #puppet-openstack11:45
*** vnogin has joined #puppet-openstack11:47
*** etingof has quit IRC11:51
*** salmankhan has quit IRC12:04
*** etingof has joined #puppet-openstack12:04
*** salmankhan has joined #puppet-openstack12:05
*** skramaja has quit IRC12:14
*** simonmcc has quit IRC12:34
*** simonmcc has joined #puppet-openstack12:35
*** dtantsur|bbl is now known as dtantsur12:40
*** salmankhan has quit IRC12:50
*** salmankhan has joined #puppet-openstack12:50
openstackgerritHarald JensÃ¥s proposed openstack/puppet-neutron master: Fix dependency cycle issue  https://review.openstack.org/54279913:00
*** tesseract has quit IRC13:12
*** jpena is now known as jpena|lunch13:15
*** jaganathan has joined #puppet-openstack13:31
*** rrutkowski has joined #puppet-openstack13:44
*** rrutkowski has quit IRC13:47
*** rrutkowski has joined #puppet-openstack13:47
*** trown|outtypewww is now known as trown13:49
openstackgerritJames E. Blair proposed openstack/puppet-neutron stable/newton: Zuul: Remove project name  https://review.openstack.org/54283513:51
*** rrutkowski has quit IRC13:59
*** salmankhan has quit IRC14:00
*** salmankhan has joined #puppet-openstack14:06
*** vnogin has quit IRC14:12
*** jpena|lunch is now known as jpena14:18
*** vnogin has joined #puppet-openstack14:18
*** amoralej is now known as amoralej|lunch14:20
*** tesseract has joined #puppet-openstack14:28
*** vabada has joined #puppet-openstack14:52
mnasertobasco: got a log or a failing change?14:52
hjensasI see these errors when running unit tests: http://logs.openstack.org/99/542799/1/check/puppet-openstack-unit-4.8-centos-7/0309ab4/job-output.txt.gz#_2018-02-09_13_30_47_51325114:52
*** vabada has quit IRC14:52
hjensasI get them with and without my change there locally.14:53
*** vabada has joined #puppet-openstack14:53
*** vabada has quit IRC14:53
*** vabada has joined #puppet-openstack14:54
hjensasmnaser: tobasco: Are you seeing the sys-log related issues as well?14:55
mnaserhjensas: hmm i think i know what this is related to14:55
*** ykarel is now known as ykarel|away14:56
mnaserhjensas: so it looks like we merged this https://review.openstack.org/#/c/444345/14:56
mnaserand reverted it here https://review.openstack.org/#/c/534629/14:56
mnaserwhich means that any unit tests which were depending on this have changed now14:57
hjensasmnaser: yup, so there is a need to revert unittest changes as well. I see.14:57
mnaserhjensas: either revert the unit test changes or revert that change14:57
mnaserhjensas: ill check with oslo team14:58
hjensasmnaser: thanks.15:00
*** hjensas is now known as hjensas|afk15:00
*** ykarel|away has quit IRC15:01
*** amoralej|lunch is now known as amoralej15:07
tobascomnaser: hjensas|afk i was more referring to the integration http://logs.openstack.org/96/541296/1/check/puppet-openstack-integration-4-scenario001-tempest-centos-7/80bf998/job-output.txt.gz#_2018-02-09_08_02_58_32798015:11
openstackgerritJames E. Blair proposed openstack/puppet-sahara stable/newton: Zuul: Remove project name  https://review.openstack.org/54287415:11
*** salmankhan has quit IRC15:11
*** salmankhan has joined #puppet-openstack15:12
mnaseri saw that yesterday tobasco and recheck'd to see what ends up happening15:17
mnasermwhahaha: EmilienM what do you think about dropping the unit tests for $openstack_service::logging because all they do is.. unit test the oslo::log ?15:18
mnaserconsidering we probably already have coverage in oslo::log .. then could we just rely on it?15:18
mnaserotherwise we either a) revert that patch again to get things to pass again or b) change all unit tests to use underscore again15:18
mnaseri haven't gotten anything from the oslo team regarding why/whats going on.. so i'd like to get your feedback before deciding15:19
*** salmankhan has quit IRC15:24
EmilienMmnaser: yeah, we can remove15:32
*** salmankhan has joined #puppet-openstack15:42
openstackgerritJames E. Blair proposed openstack/puppet-neutron stable/ocata: Zuul: Remove project name  https://review.openstack.org/54290115:52
mwhahahaWe don't need to drop the tests15:58
mwhahahaThey should not be checking the config setting from Oslo::log15:58
openstackgerritMohammed Naser proposed openstack/puppet-neutron master: Drop neutron::logging testing  https://review.openstack.org/54290415:59
openstackgerritMohammed Naser proposed openstack/puppet-neutron master: Fix dependency cycle issue  https://review.openstack.org/54279916:02
*** sai_ has joined #puppet-openstack16:02
mnasermwhahaha: oops didnt see you message, i can change my commit, so we should drop unit testing for anything that oslo::log sets?16:04
*** prometheanfire has joined #puppet-openstack16:08
prometheanfirewhen requirements branches/unfreezes you will start getting rocky requirements updates to your master(queens) branch, either don't merge them or get a +1 from a requirements core (please)16:08
prometheanfirecan I get an ack from ptl(preferably) or a core?16:09
*** jaosorior has quit IRC16:10
*** vnogin has quit IRC16:20
mnaserfound our main issue .. https://github.com/openstack/oslo.config/commit/0c6040ef6b85a77abd277a9e63be614877ca7c6a16:22
mnaserprometheanfire: so just to confirm that we will not merge any requirements updates until we branched stable/queens ?16:23
*** vnogin has joined #puppet-openstack16:23
mnaserthat way we'll make sure we dont break the world? :>16:23
prometheanfiremore or less16:24
openstackgerritMerged openstack/puppet-manila stable/newton: Zuul: Remove project name  https://review.openstack.org/54276816:24
prometheanfireit's more so that you don't move on to rocky, then when queens is branched you get downgrades16:25
prometheanfireand your tags would have bad reqs16:25
prometheanfirebad all around16:25
mnaserprometheanfire: sounds good16:26
mnaserthanks for the heads up16:26
*** yprokule has quit IRC16:27
prometheanfire:D16:27
*** prometheanfire has left #puppet-openstack16:27
openstackgerritJames E. Blair proposed openstack/puppet-zaqar stable/ocata: Zuul: Remove project name  https://review.openstack.org/54291216:32
hjensas|afkmnaser: Need to pull out L54:L58 in spec/classes/neutron_init_spec.rb as well?16:38
mnaserhjensas|afk: im not sure, i'm waiting to hear from mwhahaha about what he has to say about it, i wasnt sure16:39
mnaseri'll make sure to rebase your change once its fixed16:39
*** hjensas|afk is now known as hjensas16:39
hjensasmnaser: ok, thanks.16:41
*** jaganathan has quit IRC16:49
openstackgerritMerged openstack/puppet-designate stable/ocata: Zuul: Remove project name  https://review.openstack.org/54276116:53
*** danpawlik has quit IRC16:59
openstackgerritMerged openstack/puppet-swift stable/pike: Zuul: Remove project name  https://review.openstack.org/54274417:04
openstackgerritMerged openstack/puppet-neutron stable/newton: Zuul: Remove project name  https://review.openstack.org/54283517:09
*** vnogin has quit IRC17:15
*** vnogin has joined #puppet-openstack17:16
*** piepmatz has quit IRC17:16
*** vnogin has quit IRC17:25
openstackgerritJames E. Blair proposed openstack/puppet-designate stable/newton: Zuul: Remove project name  https://review.openstack.org/54295517:32
mwhahahamnaser: id prefer we just until test the call to Oslo::log gets the right perms. But we can drop it if it's preferable17:36
openstackgerritMerged openstack/puppet-sahara stable/newton: Zuul: Remove project name  https://review.openstack.org/54287417:51
*** shardy has quit IRC17:51
openstackgerritJames E. Blair proposed openstack/puppet-congress stable/ocata: Zuul: Remove project name  https://review.openstack.org/54296117:52
*** derekh has quit IRC18:00
*** vnogin has joined #puppet-openstack18:05
*** vnogin has quit IRC18:09
*** etingof has quit IRC18:11
*** colonwq has quit IRC18:14
*** etingof has joined #puppet-openstack18:25
*** trown is now known as trown|lunch18:38
*** dtantsur is now known as dtantsur|afk18:45
*** rrutkowski has joined #puppet-openstack18:52
openstackgerritJames E. Blair proposed openstack/puppet-horizon stable/ocata: Zuul: Remove project name  https://review.openstack.org/54297818:52
*** rrutkowski has quit IRC18:54
*** rrutkowski has joined #puppet-openstack18:55
*** jpena is now known as jpena|off19:05
openstackgerritMerged openstack/puppet-zaqar stable/ocata: Zuul: Remove project name  https://review.openstack.org/54291219:08
*** vnogin has joined #puppet-openstack19:24
openstackgerritJames E. Blair proposed openstack/puppet-ovn stable/newton: Zuul: Remove project name  https://review.openstack.org/54299119:33
*** etingof has quit IRC19:33
openstackgerritMerged openstack/puppet-neutron stable/ocata: Zuul: Remove project name  https://review.openstack.org/54290119:36
*** etingof has joined #puppet-openstack19:37
*** gfidente has quit IRC19:41
*** vnogin has quit IRC19:43
*** tesseract has quit IRC20:00
*** salmankhan has quit IRC20:11
*** trown|lunch is now known as trown20:14
*** amoralej is now known as amoralej|off20:30
*** vnogin has joined #puppet-openstack20:48
openstackgerritJames E. Blair proposed openstack/puppet-gnocchi stable/ocata: Zuul: Remove project name  https://review.openstack.org/54302521:13
*** olasd has quit IRC21:17
*** vnogin has quit IRC21:21
*** vnogin has joined #puppet-openstack21:23
*** trown is now known as trown|outtypewww21:39
*** vnogin has quit IRC21:45
*** rrutkowski has quit IRC21:52
*** rrutkowski has joined #puppet-openstack21:52
openstackgerritJames E. Blair proposed openstack/puppet-designate stable/pike: Zuul: Remove project name  https://review.openstack.org/54303321:53
*** dprince has joined #puppet-openstack22:03
*** patrickeast has quit IRC22:26
*** patrickeast has joined #puppet-openstack22:26
*** pradk has quit IRC22:35
*** fultonj has quit IRC22:40
*** pradk has joined #puppet-openstack22:47
*** pradk has quit IRC22:48
*** dprince has quit IRC22:54
*** AndyWojo has quit IRC23:04
*** AndyWojo has joined #puppet-openstack23:04
openstackgerritMerged openstack/puppet-ovn stable/newton: Zuul: Remove project name  https://review.openstack.org/54299123:08
*** jtomasek has quit IRC23:11
openstackgerritMerged openstack/puppet-congress stable/pike: Zuul: Remove project name  https://review.openstack.org/54273723:13
openstackgerritMerged openstack/puppet-congress stable/ocata: Zuul: Remove project name  https://review.openstack.org/54296123:13
*** vnogin has joined #puppet-openstack23:31
*** salmankhan has joined #puppet-openstack23:42
*** vnogin has quit IRC23:46
openstackgerritMerged openstack/puppet-designate stable/newton: Zuul: Remove project name  https://review.openstack.org/54295523:48
openstackgerritMerged openstack/puppet-gnocchi stable/ocata: Zuul: Remove project name  https://review.openstack.org/54302523:54
openstackgerritMerged openstack/puppet-ironic master: Add kombu_failover_strategy option  https://review.openstack.org/54266423:59
openstackgerritMerged openstack/puppet-horizon stable/ocata: Zuul: Remove project name  https://review.openstack.org/54297823:59

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