Monday, 2018-04-23

*** dpawlik has joined #puppet-openstack00:03
*** dpawlik has quit IRC00:08
*** ilbot3 has quit IRC01:47
*** ilbot3 has joined #puppet-openstack01:57
*** armaan has quit IRC02:29
*** armaan has joined #puppet-openstack02:29
*** ykarel_ has joined #puppet-openstack03:24
*** rcernin has quit IRC03:31
*** rcernin has joined #puppet-openstack03:35
*** ykarel_ is now known as ykarel03:55
openstackgerritMerged openstack/puppet-nova stable/queens: Handle libvirt/cpu_model_extra_flags option  https://review.openstack.org/56300104:11
openstackgerritMerged openstack/puppet-nova stable/pike: Handle libvirt/cpu_model_extra_flags option  https://review.openstack.org/56300204:11
*** skramaja has joined #puppet-openstack04:16
openstackgerritMerged openstack/puppet-nova master: Allow live_migration_inbound_addr to be used with non-default port/user/extra_params  https://review.openstack.org/56276404:27
*** armaan has quit IRC04:54
*** armaan has joined #puppet-openstack04:54
*** jtomasek has joined #puppet-openstack05:13
*** agurenko has joined #puppet-openstack05:49
*** jaganathan has joined #puppet-openstack06:24
*** jaganathan has quit IRC06:25
*** jaganathan has joined #puppet-openstack06:25
*** tesseract has joined #puppet-openstack07:04
*** jpena|off is now known as jpena07:10
*** rcernin has quit IRC07:19
*** dpawlik has joined #puppet-openstack07:19
*** dpawlik has quit IRC07:22
*** dpawlik has joined #puppet-openstack07:22
*** tosky has joined #puppet-openstack07:28
*** amoralej|off is now known as amoralej07:42
*** gfidente has joined #puppet-openstack07:43
*** gfidente has quit IRC07:43
*** gfidente has joined #puppet-openstack07:43
*** ykarel is now known as ykarel|lunch07:56
*** shardy has joined #puppet-openstack08:05
*** jaosorior has joined #puppet-openstack08:15
openstackgerritHamdy Khader proposed openstack/puppet-cinder master: Ensure nvmetcli rpm installed when using nvmeof cinder backend  https://review.openstack.org/56351208:22
*** armaan has quit IRC08:24
*** morazi has joined #puppet-openstack08:24
*** armaan has joined #puppet-openstack08:25
*** derekh has joined #puppet-openstack08:27
*** ykarel|lunch is now known as ykarel08:31
*** jaosorior has quit IRC08:32
*** chem has joined #puppet-openstack08:36
*** jaosorior has joined #puppet-openstack09:07
*** armaan has quit IRC09:13
*** armaan has joined #puppet-openstack09:14
openstackgerritHamdy Khader proposed openstack/puppet-cinder master: Ensure nvmetcli rpm installed when using nvmeof cinder backend  https://review.openstack.org/56351209:33
*** rmart04 has joined #puppet-openstack09:53
zigomnaser: Hey, I believe your patch for adding debian as non-voting can be approved now. https://review.openstack.org/#/c/561085/10:10
zigoEmilienM: mwhahaha: ^10:10
zigoI also would love if we could move forward with these 4 patches: https://review.openstack.org/#/c/562256/ https://review.openstack.org/#/c/561362/ https://review.openstack.org/#/c/562134/ https://review.openstack.org/#/c/562152/10:11
zigoI'm not sure how far I am with having a working Debian env, but probably not very far.10:12
*** dtantsur|pto is now known as dtantsur10:16
*** armaan has quit IRC10:17
*** armaan has joined #puppet-openstack10:18
*** armaan has quit IRC10:22
*** armaan has joined #puppet-openstack10:35
*** jpena is now known as jpena|lunch11:33
*** amoralej is now known as amoralej|lunch11:36
*** atoth has joined #puppet-openstack11:56
openstackgerritThomas Goirand proposed openstack/puppet-neutron master: Debian is using Python 3  https://review.openstack.org/56359012:01
zigoThis one is the one that comes next, after all of the above is fixed...12:02
zigo:)12:02
*** trown|outtypewww is now known as trown12:12
*** amoralej|lunch is now known as amoralej12:25
*** jpena|lunch is now known as jpena12:27
*** trozet has quit IRC12:32
*** trozet has joined #puppet-openstack12:44
*** dmsimard|off is now known as dmsimard12:51
*** vnogin has joined #puppet-openstack13:13
*** vnogin has quit IRC13:24
*** vnogin has joined #puppet-openstack13:38
*** vnogin has quit IRC13:51
*** rmart04 has quit IRC13:51
*** rmart04 has joined #puppet-openstack13:52
*** vnogin has joined #puppet-openstack13:52
*** trown is now known as trown|brb13:56
*** vnogin has quit IRC13:57
*** rmart04 has quit IRC13:57
*** rmart04 has joined #puppet-openstack13:57
*** trown|brb is now known as trown14:01
*** skramaja has quit IRC14:16
*** hjensas is now known as hjensas|afk14:29
openstackgerritMerged openstack/puppet-ironic master: Missing deps for unit test of config.pp  https://review.openstack.org/56247514:40
mnaserzigo: it is approved, i'd love to wait for it tomerge to see the ci logs of these changes14:41
*** ykarel is now known as ykarel|away14:50
*** gfidente is now known as gfidente|afk15:00
*** skramaja has joined #puppet-openstack15:04
*** rmart04 has quit IRC15:06
*** skramaja has quit IRC15:07
*** rajinir has joined #puppet-openstack15:08
zigomnaser: thanks !15:30
zigomnaser: Now, after applying all of my Debian patches, I get this in my virtualbox:15:35
zigo2018-04-23 12:25:14 +0000 /Stage[main]/Openstack_integration::Cinder/Cinder::Backend::Iscsi[BACKEND_1]/Cinder_type[BACKEND_1]/properties (err): change from [] to volume_backend_name=BACKEND_1 failed: Execution of '/usr/bin/openstack volume type set --property volume_backend_name=BACKEND_1 BACKEND_1' returned 1: get() takes 1 positional argument but 2 were given (tried 29, for a total of 170 seconds)15:35
zigoI tried typing the same command on the shell, and I get the same result.15:35
zigoAny idea what this could be?15:35
mnaserzigo: i wonder if thats a recent issue15:38
mnaserzigo: wanna run that with --debug ?15:40
mnaserit might give us a backtrace15:40
zigomnaser: http://paste.openstack.org/show/719754/15:40
zigoHere's the result with -vvvv15:41
zigo(same as debug, I believe...)15:41
zigoAh no, hang on.15:41
zigomnaser: http://paste.openstack.org/show/719756/15:42
mnaseri dont see how it has two arguments given15:43
*** jpena is now known as jpena|brb15:43
mnaserzigo: id raised that with the openstack client team15:44
zigook15:44
*** ykarel|away has quit IRC15:49
*** gfidente|afk has quit IRC15:56
*** agurenko has quit IRC15:56
*** owalsh is now known as owalsh_afk15:58
*** armaan has quit IRC16:03
*** armaan has joined #puppet-openstack16:04
*** dpawlik has quit IRC16:06
*** armaan has quit IRC16:09
*** armaan has joined #puppet-openstack16:09
*** vnogin has joined #puppet-openstack16:13
*** hjensas|afk is now known as hjensas16:18
*** jpena|brb is now known as jpena16:22
openstackgerritMerged openstack/puppet-cinder master: Ensure nvmetcli rpm installed when using nvmeof cinder backend  https://review.openstack.org/56351216:26
*** vnogin has quit IRC16:29
openstackgerritMerged openstack/puppet-ec2api master: Missing deps for unit test of config.pp  https://review.openstack.org/56247016:39
openstackgerritMerged openstack/puppet-congress master: Missing deps for config.pp unit test  https://review.openstack.org/56243916:40
openstackgerritMerged openstack/puppet-cinder master: Missing deps for config.pp unit test  https://review.openstack.org/56243716:41
openstackgerritMerged openstack/puppet-ceilometer master: Fix unit test for config.pp  https://review.openstack.org/56243516:41
openstackgerritMerged openstack/puppet-freezer master: Missing deps for unit test of config.pp  https://review.openstack.org/56247116:42
openstackgerritMerged openstack/puppet-aodh master: Missing deps for config.pp unit test  https://review.openstack.org/56243316:42
openstackgerritMerged openstack/puppet-cloudkitty master: Missing deps for config.pp unit test  https://review.openstack.org/56243816:42
openstackgerritMerged openstack/puppet-heat master: Missing deps for unit test of config.pp  https://review.openstack.org/56247416:42
openstackgerritMerged openstack/puppet-ceilometer master: Missing deps for config.pp unit test  https://review.openstack.org/56243616:43
openstackgerritMerged openstack/puppet-rally master: Missing deps for config.pp unit test  https://review.openstack.org/56248416:44
openstackgerritMerged openstack/puppet-mistral master: Missing deps for config.pp unit test  https://review.openstack.org/56247816:44
openstackgerritMerged openstack/puppet-murano master: Missing deps for config.pp unit test  https://review.openstack.org/56247916:44
openstackgerritMerged openstack/puppet-sahara master: Missing deps for config.pp unit test  https://review.openstack.org/56248516:44
openstackgerritMerged openstack/puppet-panko master: Missing deps for config.pp unit test  https://review.openstack.org/56248316:44
openstackgerritMerged openstack/puppet-magnum master: Missing deps for unit test of config.pp  https://review.openstack.org/56247716:45
openstackgerritMerged openstack/puppet-gnocchi master: Missing deps for unit test of config.pp  https://review.openstack.org/56247316:45
openstackgerritMerged openstack/puppet-barbican master: Missing deps for config.pp unit test  https://review.openstack.org/56243416:45
openstackgerritMerged openstack/puppet-designate master: Missing deps for config.pp unit test  https://review.openstack.org/56246916:45
openstackgerritMerged openstack/puppet-keystone master: Missing deps for unit test of config.pp  https://review.openstack.org/56247616:45
*** dpawlik has joined #puppet-openstack16:46
openstackgerritMerged openstack/puppet-trove master: Missing deps for config.pp unit test  https://review.openstack.org/56248616:47
openstackgerritMerged openstack/puppet-glance master: Missing deps for unit test of config.pp  https://review.openstack.org/56247216:47
*** jpena is now known as jpena|off16:50
*** dpawlik has quit IRC16:50
openstackgerritMerged openstack/puppet-neutron master: Missing deps for config.pp unit test  https://review.openstack.org/56248016:55
openstackgerritMerged openstack/puppet-nova master: Missing deps for config.pp unit test  https://review.openstack.org/56248116:55
openstackgerritMerged openstack/puppet-vitrage master: Missing deps for config.pp unit test  https://review.openstack.org/56248716:56
*** trown is now known as trown|lunch17:00
*** jaosorior has quit IRC17:05
*** derekh has quit IRC17:08
*** morazi has quit IRC17:11
openstackgerritMerged openstack/puppet-watcher master: Deprecate auth_uri option  https://review.openstack.org/55843717:23
*** tesseract has quit IRC17:23
*** dtantsur is now known as dtantsur|afk17:33
*** jaosorior has joined #puppet-openstack17:49
openstackgerritMerged openstack/puppet-openstacklib master: Add support for keystone options to wsgi::apache  https://review.openstack.org/55901517:51
*** dpawlik has joined #puppet-openstack17:58
*** dpawlik has quit IRC18:02
*** trown|lunch is now known as trown18:05
*** Lokesh_Jain___ has quit IRC18:50
*** jaosorior has quit IRC18:55
*** amoralej is now known as amoralej|off19:06
*** rajinir has quit IRC19:08
zigoEmilienM: Hi there! Got a bit of time for reviewing my patches?19:09
zigomwhahaha: You?19:09
mwhahahawhich ones19:09
zigomwhahaha: https://review.openstack.org/#/c/561362/ https://review.openstack.org/#/c/562134/19:11
zigoAnd for Cinder: https://review.openstack.org/#/c/562152/19:11
zigoHowever, I'm getting a few "104: connection reset by peer" when using uwsgi, so I'm not 100% sure about it. Maybe my package is setting-up wrong uwsgi params, got to fix that, though it's unrelated to puppet-openstack.19:12
mwhahahatests would be nice19:13
mwhahahalet me see how hard the tests would be for the placement api19:14
zigomwhahaha: The first 2 patches are fixing the issue with nova-placement-api binding on the same port as Apache.19:15
zigoSo it's quite important.19:15
mwhahahayea i'm looking at how hard the tests would be right now19:15
zigo:)19:16
*** atoth has quit IRC19:20
openstackgerritAlex Schultz proposed openstack/puppet-nova master: Debian: make placement-api run on uwsgi  https://review.openstack.org/56136219:27
mwhahahazigo: -^ tests19:27
zigomwhahaha: Thanks a lot.19:29
zigoThe nice thing is, it will show logs for Debian now! :P19:29
mwhahahayea19:30
*** dpawlik has joined #puppet-openstack20:04
*** dpawlik has quit IRC20:05
*** dpawlik has joined #puppet-openstack20:05
mnasermwhahaha: https://review.openstack.org/#/c/561110/3 that's a dep for the debian change to merge20:06
mwhahahak20:06
*** owalsh_afk is now known as owalsh20:08
zigomwhahaha: mnaser: Any idea what's wrong in my patch ? https://review.openstack.org/#/c/563590/20:12
*** ChanServ sets mode: +o mnaser20:12
*** mnaser changes topic to "CI Status: GREEN | Puppet OpenStack modules https://docs.openstack.org/puppet-openstack-guide/latest/"20:13
zigoIt's needed, if python3-neutron-lbaas isn't installed, then neutron-server just fails ... :/20:13
mnaseroh thats a lot of red20:13
mwhahahazigo: isn't it backwards?20:13
zigoOh, silly me.20:13
zigoIndeed !20:13
zigoFixing ...20:13
*** armaan has quit IRC20:14
openstackgerritThomas Goirand proposed openstack/puppet-neutron master: Debian is using Python 3  https://review.openstack.org/56359020:14
EmilienMzigo: I'll look asap20:28
EmilienMzigo: was lunching :D20:28
zigoEmilienM: Cheers !20:28
*** dpawlik has quit IRC20:34
*** dpawlik has joined #puppet-openstack20:37
openstackgerritTobias Urdin proposed openstack/puppet-openstacklib master: Add support for multiple WSGI script aliases  https://review.openstack.org/55911020:37
*** jtomasek has quit IRC20:43
*** rajinir has joined #puppet-openstack20:45
*** dpawlik has quit IRC20:49
*** sai_p has joined #puppet-openstack20:49
*** dpawlik has joined #puppet-openstack20:50
openstackgerritMerged openstack/puppet-neutron master: Change package name for OVN metadata agent  https://review.openstack.org/56260820:54
*** trown is now known as trown|outtypewww21:02
openstackgerritTobias Urdin proposed openstack/puppet-openstacklib master: openstacklib::openstackclient wrong and hardcoded package name  https://review.openstack.org/56377521:30
*** vnogin has joined #puppet-openstack21:30
zigomwhahaha: Much better when it's the correct order: https://review.openstack.org/#/c/563590/ :)21:30
mwhahahaindeed21:30
openstackgerritTobias Urdin proposed openstack/puppet-openstacklib master: openstacklib::openstackclient wrong and hardcoded package name  https://review.openstack.org/56377521:32
*** vnogin has quit IRC21:35
openstackgerritMerged openstack/puppet-openstack-integration master: Change libxslt-dev to libxslt1-dev on debian system  https://review.openstack.org/56111021:35
openstackgerritMerged openstack/puppet-openstack-integration master: Add Debian testing  https://review.openstack.org/56108521:35
openstackgerritMerged openstack/puppet-nova stable/ocata: Handle libvirt/cpu_model_extra_flags option  https://review.openstack.org/56300321:41
*** dpawlik has quit IRC21:47
*** rcernin has joined #puppet-openstack22:31
*** tosky has quit IRC23:54

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