*** mtreinish has quit IRC | 00:12 | |
openstackgerrit | Merged openstack/puppet-openstack-integration master: Add Zaqar debugging and websocket endpoint https://review.openstack.org/540472 | 00:22 |
---|---|---|
*** xarses_ has quit IRC | 00:28 | |
mwhahaha | http://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_261723 | 00:28 |
mwhahaha | that's a ne wone | 00:28 |
* mwhahaha will look at it tomorrow | 00:28 | |
*** vnogin has joined #puppet-openstack | 00:36 | |
*** mtreinish has joined #puppet-openstack | 00:37 | |
*** vnogin has quit IRC | 00:38 | |
openstackgerrit | James E. Blair proposed openstack/puppet-cloudkitty master: Zuul: Remove project name https://review.openstack.org/541082 | 00:39 |
openstackgerrit | Merged openstack/puppet-neutron stable/pike: New configuration being added for Cisco UCSM ML2 driver https://review.openstack.org/536962 | 01:24 |
openstackgerrit | Merged openstack/puppet-openstack_extras stable/newton: Zuul: Remove project name https://review.openstack.org/538546 | 01:24 |
openstackgerrit | Merged openstack/puppet-openstack_extras stable/ocata: Zuul: Remove project name https://review.openstack.org/538547 | 01:29 |
*** etingof has quit IRC | 01:39 | |
*** etingof has joined #puppet-openstack | 01:52 | |
*** markvoelker has joined #puppet-openstack | 02:22 | |
*** markvoelker has quit IRC | 02:24 | |
*** markvoelker has joined #puppet-openstack | 02:31 | |
openstackgerrit | Tony Breeds proposed openstack/puppet-ironic master: Add support for configuring ppc64le nodes https://review.openstack.org/535622 | 02:46 |
*** ilbot3 has quit IRC | 02:46 | |
*** markvoelker has quit IRC | 02:56 | |
*** ilbot3 has joined #puppet-openstack | 02:58 | |
*** markvoelker has joined #puppet-openstack | 02:59 | |
*** markvoelker has quit IRC | 03:11 | |
*** markvoelker has joined #puppet-openstack | 03:14 | |
*** skramaja has joined #puppet-openstack | 03:23 | |
*** markvoelker has quit IRC | 03:25 | |
*** markvoelker has joined #puppet-openstack | 03:30 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration master: Revert "Remove r10k env vars with proper args" https://review.openstack.org/541114 | 03:34 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration master: Revert "Remove r10k env vars with proper args" https://review.openstack.org/541114 | 03:34 |
EmilienM | mwhahaha: yeah, I proposed a revert https://review.openstack.org/#/c/541114/ | 03:34 |
*** markvoelker has quit IRC | 03:42 | |
EmilienM | mnaser: ^ fyi | 03:43 |
mnaser | i guess its because we run revoke-sudo in unit tests | 03:44 |
EmilienM | mnaser: ah | 03:49 |
EmilienM | mnaser: should we stop that? | 03:49 |
EmilienM | I'm fine to not revert and propose an actual fix | 03:49 |
mnaser | EmilienM: 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 sudo | 03:49 |
mnaser | i 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 |
EmilienM | yeah | 03:50 |
EmilienM | weird before it worked with old args | 03:50 |
EmilienM | mnaser: it would be so cool to run a puppet unit job in p-o-i | 03:50 |
EmilienM | but not sure if we can | 03:50 |
mnaser | old args probably didn't install in /etc ? | 03:50 |
EmilienM | we could, by creating a new job but we couldn't reuse another job, or maybe | 03:51 |
EmilienM | yeah probably, dunno tbh | 03:51 |
EmilienM | anyway, signing off | 03:51 |
mnaser | you could do depends-on: <p-o-i> change in puppet-nova i guess but i see what you mean | 03:51 |
mnaser | gn :) | 03:51 |
*** morazi has joined #puppet-openstack | 04:01 | |
openstackgerrit | Merged openstack/puppet-zaqar master: Remove MongoDB integration and transform to using POI resources https://review.openstack.org/540474 | 04:17 |
openstackgerrit | Rabi Mishra proposed openstack/puppet-heat master: Remove support for heat-api-cloudwatch service https://review.openstack.org/540741 | 04:22 |
openstackgerrit | James E. Blair proposed openstack/puppet-nova master: Zuul: Remove project name https://review.openstack.org/541131 | 05:22 |
openstackgerrit | Merged openstack/puppet-openstack-integration master: Revert "Remove r10k env vars with proper args" https://review.openstack.org/541114 | 05:43 |
*** morazi has quit IRC | 06:13 | |
*** agurenko has joined #puppet-openstack | 06:38 | |
openstackgerrit | James E. Blair proposed openstack/puppet-ovn master: Zuul: Remove project name https://review.openstack.org/541147 | 06:42 |
*** rwsu has joined #puppet-openstack | 07:01 | |
*** yprokule has joined #puppet-openstack | 07:04 | |
*** pma has joined #puppet-openstack | 07:09 | |
*** khappone has quit IRC | 07:17 | |
*** rcernin has quit IRC | 07:37 | |
*** khappone has joined #puppet-openstack | 07:39 | |
*** vnogin has joined #puppet-openstack | 07:51 | |
*** jpena|off is now known as jpena | 07:53 | |
*** vnogin has quit IRC | 07:55 | |
*** vnogin has joined #puppet-openstack | 08:06 | |
openstackgerrit | Merged openstack/puppet-zaqar master: Zuul: Remove project name https://review.openstack.org/539734 | 08:14 |
*** tesseract has joined #puppet-openstack | 08:22 | |
*** amoralej|off is now known as amoralej | 08:56 | |
*** rpittau has joined #puppet-openstack | 09:01 | |
*** gfidente has joined #puppet-openstack | 09:04 | |
*** gfidente has quit IRC | 09:04 | |
*** gfidente has joined #puppet-openstack | 09:04 | |
*** jaosorior has quit IRC | 09:06 | |
openstackgerrit | Harald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal ml2 plug-in https://review.openstack.org/537826 | 09:18 |
openstackgerrit | Harald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal - ironic-neutron-agent https://review.openstack.org/539405 | 09:18 |
*** yprokule_ has joined #puppet-openstack | 09:19 | |
*** yprokule has quit IRC | 09:19 | |
*** yprokule_ is now known as yprokule | 09:19 | |
*** jaosorior has joined #puppet-openstack | 09:29 | |
tobasco | regarding 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 task | 09:43 |
tobasco | http://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_370026 | 09:43 |
tobasco | http://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_243110 | 09:43 |
tobasco | https://github.com/openstack/puppet-openstack-integration/blob/master/install_modules_unit.sh#L14 | 09:43 |
tobasco | and no PUPPET_MAJ_VERSION set here https://github.com/openstack/puppet-openstack-integration/blob/master/playbooks/run-unit-tests.yaml | 09:43 |
tobasco | so the reason it failed was simply it used the /etc/puppet path for a puppet 4.x run | 09:44 |
*** derekh has joined #puppet-openstack | 09:45 | |
tobasco | also modules testing will fail since this was not merged https://review.openstack.org/#/c/534196/ | 09:45 |
tobasco | was pretty much a <--> dependency, they both depended on each other | 09:46 |
openstackgerrit | Dougal Matthews proposed openstack/puppet-mistral master: Add execution_interval to the Mistral manifest https://review.openstack.org/540882 | 09:52 |
*** salmankhan has joined #puppet-openstack | 09:53 | |
*** rmart04 has joined #puppet-openstack | 09:58 | |
*** salmankhan has quit IRC | 10:00 | |
*** salmankhan has joined #puppet-openstack | 10:01 | |
*** nmathew has joined #puppet-openstack | 10:19 | |
openstackgerrit | James E. Blair proposed openstack/puppet-aodh master: Zuul: Remove project name https://review.openstack.org/541231 | 10:24 |
*** dtantsur|afk is now known as dtantsur | 10: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 | |
openstackgerrit | James E. Blair proposed openstack/puppet-neutron master: Zuul: Remove project name https://review.openstack.org/541238 | 10:45 |
*** gfidente has quit IRC | 11:02 | |
*** gfidente has joined #puppet-openstack | 11:05 | |
*** gfidente has quit IRC | 11:05 | |
*** gfidente has joined #puppet-openstack | 11:05 | |
*** nmathew has quit IRC | 11:38 | |
*** salmankhan has quit IRC | 11:38 | |
*** salmankhan has joined #puppet-openstack | 11:40 | |
*** salmankhan has quit IRC | 11:48 | |
*** salmankhan has joined #puppet-openstack | 11:51 | |
*** jpena is now known as jpena|lunch | 12:51 | |
*** salmankhan has quit IRC | 12:54 | |
*** salmankhan has joined #puppet-openstack | 13:04 | |
*** jaganathan has quit IRC | 13:10 | |
*** piepmatz has joined #puppet-openstack | 13:14 | |
*** amoralej is now known as amoralej|lunch | 13:19 | |
*** jaganathan has joined #puppet-openstack | 13:23 | |
*** jaosorior has quit IRC | 13:23 | |
piepmatz | puppet-nova has tests in spec/type. the unit test pipelines do not run them: | 13:37 |
piepmatz | rspec --pattern spec/\{aliases,classes,defines,unit,functions,hosts,integration,type_aliases,types\}/\*\*/\*_spec.rb | 13:37 |
piepmatz | this explains why syntax errors are not found: https://github.com/openstack/puppet-nova/blob/master/spec/type/nova_aggregate_spec.rb#L72 | 13:37 |
piepmatz | I 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 jpena | 13:47 | |
*** morazi has joined #puppet-openstack | 13:52 | |
mnaser | piepmatz: wow that's quite teh catch | 13:54 |
EmilienM | tobasco: ok, let's merge https://review.openstack.org/#/c/534196/ first | 13:57 |
EmilienM | tobasco: we need to revert the revert | 13:57 |
EmilienM | and update https://review.openstack.org/#/c/534196/ then | 13:57 |
*** atoth has joined #puppet-openstack | 13:59 | |
*** skramaja has quit IRC | 14:02 | |
*** oidgar has joined #puppet-openstack | 14:03 | |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack-integration master: Revert "Revert "Remove r10k env vars with proper args"" https://review.openstack.org/541278 | 14:03 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_spec_helper master: Replace r10k env vars with proper args https://review.openstack.org/534196 | 14:04 |
openstackgerrit | Emilien Macchi proposed openstack/puppet-openstack_spec_helper master: Replace r10k env vars with proper args https://review.openstack.org/534196 | 14:04 |
*** jaosorior has joined #puppet-openstack | 14:04 | |
tobasco | EmilienM: y should we push a fix setting PUPPET_MAJ_VERSION for the unit jobs playbook? | 14:13 |
EmilienM | tobasco: probably, feel free to push over my patches ^ I'm in a mtg now | 14:14 |
*** pradk has joined #puppet-openstack | 14:16 | |
*** jaganathan has quit IRC | 14:17 | |
*** oidgar has quit IRC | 14:22 | |
*** amoralej|lunch is now known as amoralej | 14:24 | |
*** jaganathan has joined #puppet-openstack | 14:31 | |
*** dtantsur is now known as dtantsur|bbl | 14:37 | |
*** salmankhan has quit IRC | 14:46 | |
tobasco | EmilienM: wonder if we can't just simply drop the puppet_maj_version check for version 4 | 14:47 |
tobasco | we dont test puppet 3 anyways | 14:47 |
tobasco | in 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 v5 | 14:50 |
*** salmankhan has joined #puppet-openstack | 14:54 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-openstack-integration master: Remove PUPPET_MAJ_VERSION check for unit modules https://review.openstack.org/541296 | 14:55 |
openstackgerrit | Harald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal - ironic-neutron-agent https://review.openstack.org/539405 | 15:00 |
openstackgerrit | Harald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal ml2 plug-in https://review.openstack.org/537826 | 15:06 |
openstackgerrit | Harald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal - ironic-neutron-agent https://review.openstack.org/539405 | 15:06 |
*** mnaser has quit IRC | 15:11 | |
*** mnaser has joined #puppet-openstack | 15:12 | |
*** dtantsur|bbl is now known as dtantsur | 15:14 | |
*** trown is now known as trown|outtypewww | 15:23 | |
*** med_ has quit IRC | 15:38 | |
*** agurenko has quit IRC | 15:40 | |
openstackgerrit | James E. Blair proposed openstack/puppet-gnocchi stable/newton: Zuul: Remove project name https://review.openstack.org/541314 | 15:46 |
*** dtantsur is now known as dtantsur|bbl | 15:49 | |
*** xarses has joined #puppet-openstack | 15:54 | |
*** xarses_ has joined #puppet-openstack | 15:55 | |
*** xarses has quit IRC | 15:58 | |
*** zioproto has quit IRC | 16:06 | |
*** jaosorior has quit IRC | 16:06 | |
*** zioproto has joined #puppet-openstack | 16:07 | |
*** agurenko has joined #puppet-openstack | 16:28 | |
*** vnogin has quit IRC | 16:28 | |
*** jaganathan has quit IRC | 16:30 | |
*** yprokule has quit IRC | 16:34 | |
mnaser | man | 16:48 |
mnaser | queue is really backed up | 16:48 |
*** gfidente has quit IRC | 16:55 | |
*** gfidente has joined #puppet-openstack | 17:00 | |
*** gfidente has quit IRC | 17:00 | |
*** gfidente has joined #puppet-openstack | 17:00 | |
*** piepmatz has quit IRC | 17:05 | |
*** jpena is now known as jpena|off | 17:18 | |
*** rwsu has quit IRC | 17:19 | |
*** rwsu has joined #puppet-openstack | 17:20 | |
*** agurenko has quit IRC | 17:21 | |
*** yprokule has joined #puppet-openstack | 17:27 | |
openstackgerrit | Harald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal - ironic-neutron-agent https://review.openstack.org/539405 | 17:29 |
*** gfidente is now known as gfidente|afk | 17:38 | |
*** rmart04 has quit IRC | 17:39 | |
*** mtreinish has quit IRC | 17:43 | |
*** rrutkowski has joined #puppet-openstack | 17:48 | |
*** mtreinish has joined #puppet-openstack | 17:50 | |
*** yprokule has quit IRC | 17:50 | |
*** tesseract has quit IRC | 18:00 | |
*** derekh has quit IRC | 18:02 | |
*** rrutkowski has quit IRC | 18:02 | |
*** morazi has quit IRC | 18:03 | |
*** rrutkowski has joined #puppet-openstack | 18:03 | |
*** rrutkowski has quit IRC | 18:04 | |
*** rrutkowski has joined #puppet-openstack | 18:04 | |
*** xarses_ has quit IRC | 18:06 | |
*** rrutkowski has quit IRC | 18:06 | |
*** xarses has joined #puppet-openstack | 18:06 | |
*** rrutkowski has joined #puppet-openstack | 18:06 | |
*** rrutkowski has quit IRC | 18:09 | |
*** rrutkowski has joined #puppet-openstack | 18:09 | |
*** rrutkowski has quit IRC | 18:10 | |
*** rrutkowski has joined #puppet-openstack | 18:10 | |
*** rrutkowski has quit IRC | 18:11 | |
*** rrutkowski has joined #puppet-openstack | 18:11 | |
*** dtantsur|bbl is now known as dtantsur | 18:22 | |
openstackgerrit | Harald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal ml2 plug-in https://review.openstack.org/537826 | 18:28 |
openstackgerrit | Harald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal - ironic-neutron-agent https://review.openstack.org/539405 | 18:28 |
*** social has joined #puppet-openstack | 18:39 | |
openstackgerrit | Lokesh Jain proposed openstack/puppet-heat stable/pike: Make heat parameter plugin_dirs configurable https://review.openstack.org/541399 | 18:56 |
openstackgerrit | Lokesh Jain proposed openstack/puppet-heat stable/ocata: Make heat parameter plugin_dirs configurable https://review.openstack.org/541400 | 18:59 |
openstackgerrit | Lokesh Jain proposed openstack/puppet-heat stable/newton: Make heat parameter plugin_dirs configurable https://review.openstack.org/541402 | 19:03 |
*** morazi has joined #puppet-openstack | 19:07 | |
*** rrutkowski has quit IRC | 19:08 | |
*** salmankhan has quit IRC | 19:53 | |
*** rrutkowski has joined #puppet-openstack | 20:00 | |
*** jtomasek has quit IRC | 20:00 | |
*** owalsh has quit IRC | 20:02 | |
*** owalsh has joined #puppet-openstack | 20:02 | |
*** amoralej is now known as amoralej|off | 20:02 | |
*** dtantsur is now known as dtantsur|afk | 20:03 | |
*** rmart04 has joined #puppet-openstack | 20:09 | |
*** rmart04 has quit IRC | 20:10 | |
openstackgerrit | Alex Schultz proposed openstack/puppet-nova master: Load puppet-openstack_spec_helper locally during tests https://review.openstack.org/507967 | 20:32 |
openstackgerrit | Merged openstack/puppet-nova master: Add support for VNC TLS https://review.openstack.org/535698 | 20:38 |
openstackgerrit | Harald Jensås proposed openstack/puppet-neutron master: Add networking-baremetal - ironic-neutron-agent https://review.openstack.org/539405 | 20:50 |
*** gfidente|afk has quit IRC | 20:51 | |
openstackgerrit | Merged openstack/puppet-glare master: Add glare client install support https://review.openstack.org/536903 | 20:53 |
*** rrutkowski has quit IRC | 21:06 | |
openstackgerrit | Merged openstack/puppet-swift master: Add facts needed by memcached to avoid failing. https://review.openstack.org/541022 | 21:20 |
openstackgerrit | Mohammed Naser proposed openstack/puppet-swift master: Adding barbican configuration to swift https://review.openstack.org/525321 | 21:40 |
openstackgerrit | Mohammed Naser proposed openstack/puppet-swift master: Zuul: Remove project name https://review.openstack.org/540921 | 21:40 |
*** Lokesh_Jain__ has quit IRC | 21:47 | |
*** etingof has quit IRC | 21:52 | |
*** etingof has joined #puppet-openstack | 22:06 | |
*** openstackgerrit has quit IRC | 22:16 | |
*** rcernin has joined #puppet-openstack | 22:28 | |
*** openstackgerrit has joined #puppet-openstack | 22:29 | |
openstackgerrit | Merged openstack/puppet-panko master: Add Panko expirer class https://review.openstack.org/539591 | 22:29 |
openstackgerrit | Merged openstack/puppet-ovn master: Add OVS Hardware Offloading to OVN https://review.openstack.org/536702 | 22:39 |
openstackgerrit | Merged openstack/puppet-murano master: Add new image_filter_project_id param for dashboard https://review.openstack.org/538740 | 22:48 |
hjensas | mwhahaha: 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 |
mwhahaha | hjensas: yea you'll want to specify puppet 4 | 22:48 |
mwhahaha | hjensas: sec | 22:48 |
openstackgerrit | Merged openstack/puppet-aodh master: Zuul: Remove project name https://review.openstack.org/541231 | 22:49 |
mwhahaha | should still work but try.. | 22:49 |
mwhahaha | hjensas: rm -rf Gemfile.lock ; export PUPPET_GEM_VERSION='~> 4.8.0'; bundle install; bundle exec rake spec | 22:50 |
mwhahaha | hjensas: see if that fixes it for you | 22:50 |
hjensas | mwhahaha: thanks, trying. | 22:50 |
hjensas | mwhahaha: not immideatly, let me try to make a fresh clone of master and try again with PUPPET_GEM_VERSION set. | 22:53 |
hjensas | mwhahaha: nope, this now : http://paste.openstack.org/show/664119/ | 22:55 |
* mwhahaha tries | 22:55 | |
mwhahaha | that's an older version of bundler | 22:56 |
mwhahaha | wonder if it's bad | 22:56 |
mwhahaha | i've got 1.14.6 in my env and it works | 22:57 |
hjensas | hm, updated with 'gem update' - got bundler-1.16.1 . still fails. | 23:05 |
openstackgerrit | Merged openstack/puppet-gnocchi stable/newton: Zuul: Remove project name https://review.openstack.org/541314 | 23:11 |
mwhahaha | hjensas: 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 versions | 23:17 |
hjensas | mwhahaha: 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 |
mwhahaha | so we have a script for centos7 to get you a workable env | 23:20 |
hjensas | mwhahaha: oh, I'll try it! :) | 23:20 |
mwhahaha | the ci stuff should work because we're using stock c7 | 23:21 |
mwhahaha | let me see | 23:21 |
mwhahaha | with zuulv3 it's actually ansible | 23:22 |
mwhahaha | so we do https://github.com/openstack/puppet-openstack-integration/blob/master/playbooks/run-unit-tests.yaml | 23:22 |
mwhahaha | https://github.com/openstack/puppet-openstack-integration/blob/master/playbooks/prepare-node-common.yaml is the dep setup | 23:23 |
hjensas | mwhahaha: thanks, I'll try to see if I can get something working from that. | 23:29 |
mwhahaha | i just use rvm with ruby 2.1.x on my laptop if you don't want to do the vm route | 23:29 |
openstackgerrit | Merged openstack/puppet-keystone stable/newton: Zuul: Remove project name https://review.openstack.org/538579 | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!