Tuesday, 2018-02-06

*** mtreinish has quit IRC00:12
openstackgerritMerged openstack/puppet-openstack-integration master: Add Zaqar debugging and websocket endpoint  https://review.openstack.org/54047200:22
*** xarses_ has quit IRC00:28
mwhahahahttp://logs.openstack.org/98/535698/16/gate/puppet-openstack-unit-4.8-centos-7/bd7dedf/job-output.txt.gz#_2018-02-05_23_19_03_26172300:28
mwhahahathat's a ne wone00:28
* mwhahaha will look at it tomorrow00:28
*** vnogin has joined #puppet-openstack00:36
*** mtreinish has joined #puppet-openstack00:37
*** vnogin has quit IRC00:38
openstackgerritJames E. Blair proposed openstack/puppet-cloudkitty master: Zuul: Remove project name  https://review.openstack.org/54108200:39
openstackgerritMerged openstack/puppet-neutron stable/pike: New configuration being added for Cisco UCSM ML2 driver  https://review.openstack.org/53696201:24
openstackgerritMerged openstack/puppet-openstack_extras stable/newton: Zuul: Remove project name  https://review.openstack.org/53854601:24
openstackgerritMerged openstack/puppet-openstack_extras stable/ocata: Zuul: Remove project name  https://review.openstack.org/53854701:29
*** etingof has quit IRC01:39
*** etingof has joined #puppet-openstack01:52
*** markvoelker has joined #puppet-openstack02:22
*** markvoelker has quit IRC02:24
*** markvoelker has joined #puppet-openstack02:31
openstackgerritTony Breeds proposed openstack/puppet-ironic master: Add support for configuring ppc64le nodes  https://review.openstack.org/53562202:46
*** ilbot3 has quit IRC02:46
*** markvoelker has quit IRC02:56
*** ilbot3 has joined #puppet-openstack02:58
*** markvoelker has joined #puppet-openstack02:59
*** markvoelker has quit IRC03:11
*** markvoelker has joined #puppet-openstack03:14
*** skramaja has joined #puppet-openstack03:23
*** markvoelker has quit IRC03:25
*** markvoelker has joined #puppet-openstack03:30
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration master: Revert "Remove r10k env vars with proper args"  https://review.openstack.org/54111403:34
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration master: Revert "Remove r10k env vars with proper args"  https://review.openstack.org/54111403:34
EmilienMmwhahaha: yeah, I proposed a revert https://review.openstack.org/#/c/541114/03:34
*** markvoelker has quit IRC03:42
EmilienMmnaser: ^ fyi03:43
mnaseri guess its because we run revoke-sudo in unit tests03:44
EmilienMmnaser: ah03:49
EmilienMmnaser: should we stop that?03:49
EmilienMI'm fine to not revert and propose an actual fix03:49
mnaserEmilienM: lets clear the gate with the revert and then we can test the actual fix which i think should be install the modules before revoking sudo03:49
mnaseri think what is happening is that we install puppet, revoke sudo, install modules (which tries to install in /etc/puppet which cant be done under non priviledged user)03:50
EmilienMyeah03:50
EmilienMweird before it worked with old args03:50
EmilienMmnaser: it would be so cool to run a puppet unit job in p-o-i03:50
EmilienMbut not sure if we can03:50
mnaserold args probably didn't install in /etc ?03:50
EmilienMwe could, by creating a new job but we couldn't reuse another job, or maybe03:51
EmilienMyeah probably, dunno tbh03:51
EmilienManyway, signing off03:51
mnaseryou could do depends-on: <p-o-i> change in puppet-nova i guess but i see what you mean03:51
mnasergn :)03:51
*** morazi has joined #puppet-openstack04:01
openstackgerritMerged openstack/puppet-zaqar master: Remove MongoDB integration and transform to using POI resources  https://review.openstack.org/54047404:17
openstackgerritRabi Mishra proposed openstack/puppet-heat master: Remove support for heat-api-cloudwatch service  https://review.openstack.org/54074104:22
openstackgerritJames E. Blair proposed openstack/puppet-nova master: Zuul: Remove project name  https://review.openstack.org/54113105:22
openstackgerritMerged openstack/puppet-openstack-integration master: Revert "Remove r10k env vars with proper args"  https://review.openstack.org/54111405:43
*** morazi has quit IRC06:13
*** agurenko has joined #puppet-openstack06:38
openstackgerritJames E. Blair proposed openstack/puppet-ovn master: Zuul: Remove project name  https://review.openstack.org/54114706:42
*** rwsu has joined #puppet-openstack07:01
*** yprokule has joined #puppet-openstack07:04
*** pma has joined #puppet-openstack07:09
*** khappone has quit IRC07:17
*** rcernin has quit IRC07:37
*** khappone has joined #puppet-openstack07:39
*** vnogin has joined #puppet-openstack07:51
*** jpena|off is now known as jpena07:53
*** vnogin has quit IRC07:55
*** vnogin has joined #puppet-openstack08:06
openstackgerritMerged openstack/puppet-zaqar master: Zuul: Remove project name  https://review.openstack.org/53973408:14
*** tesseract has joined #puppet-openstack08:22
*** amoralej|off is now known as amoralej08:56
*** rpittau has joined #puppet-openstack09:01
*** gfidente has joined #puppet-openstack09:04
*** gfidente has quit IRC09:04
*** gfidente has joined #puppet-openstack09:04
*** jaosorior has quit IRC09:06
openstackgerritHarald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal ml2 plug-in  https://review.openstack.org/53782609:18
openstackgerritHarald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal - ironic-neutron-agent  https://review.openstack.org/53940509:18
*** yprokule_ has joined #puppet-openstack09:19
*** yprokule has quit IRC09:19
*** yprokule_ is now known as yprokule09:19
*** jaosorior has joined #puppet-openstack09:29
tobascoregarding the ci breakage earlier, seems like the PUPPET_MAJ_VERSION variable is used in unit module installation but not set in zuul run-unit-tests task09:43
tobascohttp://logs.openstack.org/98/535698/16/gate/puppet-openstack-unit-4.8-centos-7/bd7dedf/job-output.txt.gz#_2018-02-05_23_19_05_37002609:43
tobascohttp://logs.openstack.org/98/535698/16/gate/puppet-openstack-unit-4.8-centos-7/bd7dedf/job-output.txt.gz#_2018-02-05_23_19_03_24311009:43
tobascohttps://github.com/openstack/puppet-openstack-integration/blob/master/install_modules_unit.sh#L1409:43
tobascoand no PUPPET_MAJ_VERSION set here https://github.com/openstack/puppet-openstack-integration/blob/master/playbooks/run-unit-tests.yaml09:43
tobascoso the reason it failed was simply it used the /etc/puppet path for a puppet 4.x run09:44
*** derekh has joined #puppet-openstack09:45
tobascoalso modules testing will fail since this was not merged https://review.openstack.org/#/c/534196/09:45
tobascowas pretty much a <--> dependency, they both depended on each other09:46
openstackgerritDougal Matthews proposed openstack/puppet-mistral master: Add execution_interval to the Mistral manifest  https://review.openstack.org/54088209:52
*** salmankhan has joined #puppet-openstack09:53
*** rmart04 has joined #puppet-openstack09:58
*** salmankhan has quit IRC10:00
*** salmankhan has joined #puppet-openstack10:01
*** nmathew has joined #puppet-openstack10:19
openstackgerritJames E. Blair proposed openstack/puppet-aodh master: Zuul: Remove project name  https://review.openstack.org/54123110:24
*** dtantsur|afk is now known as dtantsur10:25
-openstackstatus- NOTICE: Our Zuul infrastructure is currently experiencing some problems and processing jobs very slowly, we're investigating. Please do not approve or recheck changes for now.10:31
openstackgerritJames E. Blair proposed openstack/puppet-neutron master: Zuul: Remove project name  https://review.openstack.org/54123810:45
*** gfidente has quit IRC11:02
*** gfidente has joined #puppet-openstack11:05
*** gfidente has quit IRC11:05
*** gfidente has joined #puppet-openstack11:05
*** nmathew has quit IRC11:38
*** salmankhan has quit IRC11:38
*** salmankhan has joined #puppet-openstack11:40
*** salmankhan has quit IRC11:48
*** salmankhan has joined #puppet-openstack11:51
*** jpena is now known as jpena|lunch12:51
*** salmankhan has quit IRC12:54
*** salmankhan has joined #puppet-openstack13:04
*** jaganathan has quit IRC13:10
*** piepmatz has joined #puppet-openstack13:14
*** amoralej is now known as amoralej|lunch13:19
*** jaganathan has joined #puppet-openstack13:23
*** jaosorior has quit IRC13:23
piepmatzpuppet-nova has tests in spec/type. the unit test pipelines do not run them:13:37
piepmatzrspec --pattern spec/\{aliases,classes,defines,unit,functions,hosts,integration,type_aliases,types\}/\*\*/\*_spec.rb13:37
piepmatzthis explains why syntax errors are not found: https://github.com/openstack/puppet-nova/blob/master/spec/type/nova_aggregate_spec.rb#L7213:37
piepmatzI am now wondering: should the pipelines cover tests from spec/type as well or are those tests actually worthless after being broken for 2 years? should tests for custom types rather be located in spec/unit/type?13:37
*** jpena|lunch is now known as jpena13:47
*** morazi has joined #puppet-openstack13:52
mnaserpiepmatz: wow that's quite teh catch13:54
EmilienMtobasco: ok, let's merge https://review.openstack.org/#/c/534196/ first13:57
EmilienMtobasco: we need to revert the revert13:57
EmilienMand update https://review.openstack.org/#/c/534196/ then13:57
*** atoth has joined #puppet-openstack13:59
*** skramaja has quit IRC14:02
*** oidgar has joined #puppet-openstack14:03
openstackgerritEmilien Macchi proposed openstack/puppet-openstack-integration master: Revert "Revert "Remove r10k env vars with proper args""  https://review.openstack.org/54127814:03
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper master: Replace r10k env vars with proper args  https://review.openstack.org/53419614:04
openstackgerritEmilien Macchi proposed openstack/puppet-openstack_spec_helper master: Replace r10k env vars with proper args  https://review.openstack.org/53419614:04
*** jaosorior has joined #puppet-openstack14:04
tobascoEmilienM: y should we push a fix setting PUPPET_MAJ_VERSION for the unit jobs playbook?14:13
EmilienMtobasco: probably, feel free to push over my patches ^ I'm in a mtg now14:14
*** pradk has joined #puppet-openstack14:16
*** jaganathan has quit IRC14:17
*** oidgar has quit IRC14:22
*** amoralej|lunch is now known as amoralej14:24
*** jaganathan has joined #puppet-openstack14:31
*** dtantsur is now known as dtantsur|bbl14:37
*** salmankhan has quit IRC14:46
tobascoEmilienM: wonder if we can't just simply drop the puppet_maj_version check for version 414:47
tobascowe dont test puppet 3 anyways14:47
tobascoin install_modules_unit that is, since integration uses that variable the unit tests doesn'y really care since /etc/puppetlabs/code is valid for both v4 and v514:50
*** salmankhan has joined #puppet-openstack14:54
openstackgerritTobias Urdin proposed openstack/puppet-openstack-integration master: Remove PUPPET_MAJ_VERSION check for unit modules  https://review.openstack.org/54129614:55
openstackgerritHarald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal - ironic-neutron-agent  https://review.openstack.org/53940515:00
openstackgerritHarald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal ml2 plug-in  https://review.openstack.org/53782615:06
openstackgerritHarald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal - ironic-neutron-agent  https://review.openstack.org/53940515:06
*** mnaser has quit IRC15:11
*** mnaser has joined #puppet-openstack15:12
*** dtantsur|bbl is now known as dtantsur15:14
*** trown is now known as trown|outtypewww15:23
*** med_ has quit IRC15:38
*** agurenko has quit IRC15:40
openstackgerritJames E. Blair proposed openstack/puppet-gnocchi stable/newton: Zuul: Remove project name  https://review.openstack.org/54131415:46
*** dtantsur is now known as dtantsur|bbl15:49
*** xarses has joined #puppet-openstack15:54
*** xarses_ has joined #puppet-openstack15:55
*** xarses has quit IRC15:58
*** zioproto has quit IRC16:06
*** jaosorior has quit IRC16:06
*** zioproto has joined #puppet-openstack16:07
*** agurenko has joined #puppet-openstack16:28
*** vnogin has quit IRC16:28
*** jaganathan has quit IRC16:30
*** yprokule has quit IRC16:34
mnaserman16:48
mnaserqueue is really backed up16:48
*** gfidente has quit IRC16:55
*** gfidente has joined #puppet-openstack17:00
*** gfidente has quit IRC17:00
*** gfidente has joined #puppet-openstack17:00
*** piepmatz has quit IRC17:05
*** jpena is now known as jpena|off17:18
*** rwsu has quit IRC17:19
*** rwsu has joined #puppet-openstack17:20
*** agurenko has quit IRC17:21
*** yprokule has joined #puppet-openstack17:27
openstackgerritHarald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal - ironic-neutron-agent  https://review.openstack.org/53940517:29
*** gfidente is now known as gfidente|afk17:38
*** rmart04 has quit IRC17:39
*** mtreinish has quit IRC17:43
*** rrutkowski has joined #puppet-openstack17:48
*** mtreinish has joined #puppet-openstack17:50
*** yprokule has quit IRC17:50
*** tesseract has quit IRC18:00
*** derekh has quit IRC18:02
*** rrutkowski has quit IRC18:02
*** morazi has quit IRC18:03
*** rrutkowski has joined #puppet-openstack18:03
*** rrutkowski has quit IRC18:04
*** rrutkowski has joined #puppet-openstack18:04
*** xarses_ has quit IRC18:06
*** rrutkowski has quit IRC18:06
*** xarses has joined #puppet-openstack18:06
*** rrutkowski has joined #puppet-openstack18:06
*** rrutkowski has quit IRC18:09
*** rrutkowski has joined #puppet-openstack18:09
*** rrutkowski has quit IRC18:10
*** rrutkowski has joined #puppet-openstack18:10
*** rrutkowski has quit IRC18:11
*** rrutkowski has joined #puppet-openstack18:11
*** dtantsur|bbl is now known as dtantsur18:22
openstackgerritHarald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal ml2 plug-in  https://review.openstack.org/53782618:28
openstackgerritHarald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal - ironic-neutron-agent  https://review.openstack.org/53940518:28
*** social has joined #puppet-openstack18:39
openstackgerritLokesh Jain proposed openstack/puppet-heat stable/pike: Make heat parameter plugin_dirs configurable  https://review.openstack.org/54139918:56
openstackgerritLokesh Jain proposed openstack/puppet-heat stable/ocata: Make heat parameter plugin_dirs configurable  https://review.openstack.org/54140018:59
openstackgerritLokesh Jain proposed openstack/puppet-heat stable/newton: Make heat parameter plugin_dirs configurable  https://review.openstack.org/54140219:03
*** morazi has joined #puppet-openstack19:07
*** rrutkowski has quit IRC19:08
*** salmankhan has quit IRC19:53
*** rrutkowski has joined #puppet-openstack20:00
*** jtomasek has quit IRC20:00
*** owalsh has quit IRC20:02
*** owalsh has joined #puppet-openstack20:02
*** amoralej is now known as amoralej|off20:02
*** dtantsur is now known as dtantsur|afk20:03
*** rmart04 has joined #puppet-openstack20:09
*** rmart04 has quit IRC20:10
openstackgerritAlex Schultz proposed openstack/puppet-nova master: Load puppet-openstack_spec_helper locally during tests  https://review.openstack.org/50796720:32
openstackgerritMerged openstack/puppet-nova master: Add support for VNC TLS  https://review.openstack.org/53569820:38
openstackgerritHarald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal - ironic-neutron-agent  https://review.openstack.org/53940520:50
*** gfidente|afk has quit IRC20:51
openstackgerritMerged openstack/puppet-glare master: Add glare client install support  https://review.openstack.org/53690320:53
*** rrutkowski has quit IRC21:06
openstackgerritMerged openstack/puppet-swift master: Add facts needed by memcached to avoid failing.  https://review.openstack.org/54102221:20
openstackgerritMohammed Naser proposed openstack/puppet-swift master: Adding barbican configuration to swift  https://review.openstack.org/52532121:40
openstackgerritMohammed Naser proposed openstack/puppet-swift master: Zuul: Remove project name  https://review.openstack.org/54092121:40
*** Lokesh_Jain__ has quit IRC21:47
*** etingof has quit IRC21:52
*** etingof has joined #puppet-openstack22:06
*** openstackgerrit has quit IRC22:16
*** rcernin has joined #puppet-openstack22:28
*** openstackgerrit has joined #puppet-openstack22:29
openstackgerritMerged openstack/puppet-panko master: Add Panko expirer class  https://review.openstack.org/53959122:29
openstackgerritMerged openstack/puppet-ovn master: Add OVS Hardware Offloading to OVN  https://review.openstack.org/53670222:39
openstackgerritMerged openstack/puppet-murano master: Add new image_filter_project_id param for dashboard  https://review.openstack.org/53874022:48
hjensasmwhahaha: EmilienM: Hey, I am struggling with the tests on my two patches. Part of the problem is that I cannot get tests running locally. http://paste.openstack.org/show/664107/ <- Any tips?22:48
mwhahahahjensas: yea you'll want to specify puppet 422:48
mwhahahahjensas: sec22:48
openstackgerritMerged openstack/puppet-aodh master: Zuul: Remove project name  https://review.openstack.org/54123122:49
mwhahahashould still work but try..22:49
mwhahahahjensas: rm -rf Gemfile.lock ; export PUPPET_GEM_VERSION='~> 4.8.0'; bundle install; bundle exec rake spec22:50
mwhahahahjensas: see if that fixes it for you22:50
hjensasmwhahaha: thanks, trying.22:50
hjensasmwhahaha: not immideatly, let me try to make a fresh clone of master and try again with PUPPET_GEM_VERSION set.22:53
hjensasmwhahaha: nope, this now : http://paste.openstack.org/show/664119/22:55
* mwhahaha tries22:55
mwhahahathat's an older version of bundler22:56
mwhahahawonder if it's bad22:56
mwhahahai've got 1.14.6 in my env and it works22:57
hjensashm, updated with 'gem update' - got bundler-1.16.1 . still fails.23:05
openstackgerritMerged openstack/puppet-gnocchi stable/newton: Zuul: Remove project name  https://review.openstack.org/54131423:11
mwhahahahjensas: yea not sure, if you post your last version i can pull it down and see if I can help. unfortunately testing envs is always hard for the rspec stuff because of the various versions23:17
hjensasmwhahaha: yeah, I've tried in vms using fedora27 and centos 7. On centos I don't have a new enough ruby to get the gems installed.23:19
mwhahahaso we have a script for centos7 to get you a workable env23:20
hjensasmwhahaha: oh, I'll try it! :)23:20
mwhahahathe ci stuff should work because we're using stock c723:21
mwhahahalet me see23:21
mwhahahawith zuulv3 it's actually ansible23:22
mwhahahaso we do https://github.com/openstack/puppet-openstack-integration/blob/master/playbooks/run-unit-tests.yaml23:22
mwhahahahttps://github.com/openstack/puppet-openstack-integration/blob/master/playbooks/prepare-node-common.yaml is the dep setup23:23
hjensasmwhahaha: thanks, I'll try to see if I can get something working from that.23:29
mwhahahai just use rvm with ruby 2.1.x on my laptop if you don't want to do the vm route23:29
openstackgerritMerged openstack/puppet-keystone stable/newton: Zuul: Remove project name  https://review.openstack.org/53857923:56

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