Thursday, 2018-06-07

*** edmondsw has joined #openstack-powervm01:42
*** YuYangWang has quit IRC01:43
*** edmondsw has quit IRC01:47
openstackgerritMerged openstack/nova-powervm master: Remove unused volume_adapter conf  https://review.openstack.org/57299502:14
*** tjakobs has joined #openstack-powervm03:15
*** tjakobs has quit IRC03:20
*** edmondsw has joined #openstack-powervm03:30
*** edmondsw has quit IRC03:35
*** chhagarw has joined #openstack-powervm05:06
*** edmondsw has joined #openstack-powervm05:19
*** edmondsw has quit IRC05:23
*** AlexeyAbashkin has joined #openstack-powervm07:04
*** edmondsw has joined #openstack-powervm07:07
*** edmondsw has quit IRC07:11
*** Alexey_Abashkin has joined #openstack-powervm07:34
*** AlexeyAbashkin has quit IRC07:35
*** Alexey_Abashkin is now known as AlexeyAbashkin07:35
*** AlexeyAbashkin has quit IRC07:46
*** AlexeyAbashkin has joined #openstack-powervm07:56
*** AlexeyAbashkin has quit IRC08:23
*** AlexeyAbashkin has joined #openstack-powervm08:26
*** AlexeyAbashkin has quit IRC08:33
*** AlexeyAbashkin has joined #openstack-powervm08:38
*** AlexeyAbashkin has quit IRC08:45
*** edmondsw has joined #openstack-powervm08:55
*** edmondsw has quit IRC09:00
openstackgerritChhavi Agarwal proposed openstack/nova-powervm master: iSCSI Live Migration Support  https://review.openstack.org/56757509:03
*** AlexeyAbashkin has joined #openstack-powervm09:31
*** AlexeyAbashkin has quit IRC09:50
*** AlexeyAbashkin has joined #openstack-powervm10:02
*** AlexeyAbashkin has quit IRC10:13
*** Alexey_Abashkin has joined #openstack-powervm10:13
*** Alexey_Abashkin is now known as AlexeyAbashkin10:16
*** AlexeyAbashkin has quit IRC10:23
*** AlexeyAbashkin has joined #openstack-powervm10:25
*** Alexey_Abashkin has joined #openstack-powervm10:38
*** AlexeyAbashkin has quit IRC10:39
*** Alexey_Abashkin is now known as AlexeyAbashkin10:39
*** edmondsw has joined #openstack-powervm10:43
*** edmondsw has quit IRC10:47
*** AlexeyAbashkin has quit IRC11:01
*** edmondsw has joined #openstack-powervm11:22
openstackgerritElod Illes proposed openstack/networking-powervm stable/pike: Fix sphinx-docs job for stable branch  https://review.openstack.org/57319711:39
*** AlexeyAbashkin has joined #openstack-powervm11:51
*** esberglu has joined #openstack-powervm12:57
*** AlexeyAbashkin has quit IRC13:08
*** Alexey_Abashkin has joined #openstack-powervm13:08
*** Alexey_Abashkin is now known as AlexeyAbashkin13:10
*** openstackgerrit has quit IRC13:34
*** AlexeyAbashkin has quit IRC13:43
*** AlexeyAbashkin has joined #openstack-powervm13:50
*** Alexey_Abashkin has joined #openstack-powervm14:02
*** AlexeyAbashkin has quit IRC14:02
*** Alexey_Abashkin is now known as AlexeyAbashkin14:02
*** tjakobs has joined #openstack-powervm14:32
*** Alexey_Abashkin has joined #openstack-powervm14:38
*** AlexeyAbashkin has quit IRC14:41
*** Alexey_Abashkin is now known as AlexeyAbashkin14:41
edmondswefried I proposed Rocky-2 tags: https://review.openstack.org/#/c/573263/14:43
efriededmondsw: Ack14:43
efriededmondsw: Keep an eye out for ------^ which I fast-approved shortly ago.  May be worth including.14:47
edmondswefried ack14:48
*** chhavi__ has joined #openstack-powervm14:48
edmondswefried that's not on master, so wouldn't affect Rocky tagging14:48
efriededmondsw: oh, mahbad, didn't notice that.14:49
efriedwas absently wondering why the doc job breakage wasn't a more urgent thing.14:49
*** chhagarw has quit IRC14:49
edmondsw:)14:49
edmondswefried you fast approved that one but not https://review.openstack.org/#/c/572841 ?14:50
edmondswdoesn't look like either were fast approved... I can workflow them14:51
efriedokay.  Hm, I thought I had.  Anyway, yeah, thanks.14:51
efriedMaybe I hit it before tests had run.14:51
efriedlooks like it.14:52
*** openstackgerrit has joined #openstack-powervm15:02
openstackgerritMerged openstack/ceilometer-powervm stable/pike: Fix sphinx-docs job for stable branch  https://review.openstack.org/57284115:02
*** gman-tx has joined #openstack-powervm15:06
openstackgerritChhavi Agarwal proposed openstack/nova-powervm master: iSCSI Live Migration Support  https://review.openstack.org/56757515:10
*** Alexey_Abashkin has joined #openstack-powervm15:20
*** AlexeyAbashkin has quit IRC15:22
*** Alexey_Abashkin is now known as AlexeyAbashkin15:22
*** AlexeyAbashkin has quit IRC15:22
*** AlexeyAbashkin has joined #openstack-powervm15:23
-openstackstatus- NOTICE: Zuul update for Ansible 2.5 in progress. Scheduler crashed as unexpected side effect of pip upgrade. Will be back and running shortly.15:31
*** Alexey_Abashkin has joined #openstack-powervm15:37
*** AlexeyAbashkin has quit IRC15:37
*** Alexey_Abashkin is now known as AlexeyAbashkin15:37
edmondswefried please take a look at https://review.openstack.org/#/c/567575/11/nova_powervm/virt/powervm/volume/iscsi.py@42815:43
edmondswyou're our feedtask best-practices expert15:43
edmondsw:)15:43
edmondswbeyond just my comment, I'm wondering if it's right other than that15:43
efriededmondsw: k, got calls for the next hour, but will look.15:43
edmondswtx15:44
edmondswchhavi__ ^15:44
edmondswI added my comments, but would like eric to look as well15:44
edmondswand did you confirm with gman-tx that it's correct to remove_iscsi before remove_hdisk?15:44
*** gman-tx has quit IRC15:52
*** AlexeyAbashkin has quit IRC15:53
chhavi__As I mentioned, remove_iscsi is to unmap the lun from the iscsi session, after that we need to perform rmdev to clean the hdisk15:53
*** gman-tx has joined #openstack-powervm15:53
*** esberglu has quit IRC15:57
efriededmondsw: couple minutes late to the mtg15:59
edmondswmdrabe you joining?16:07
mdrabeedmondsw: What meeting?16:09
edmondswmdrabe check your calendar :) or I slacked it to you16:10
*** gman-tx has quit IRC16:10
-openstackstatus- NOTICE: The zuul upgrade to ansible 2.5 is complete and zuul is running again. Changes uploaded or approved between 15:25 and 15:45 will need to be rechecked. Please report any problems in #openstack-infra16:10
openstackgerritMerged openstack/networking-powervm stable/pike: Fix sphinx-docs job for stable branch  https://review.openstack.org/57319716:14
*** gman-tx has joined #openstack-powervm16:41
chhavi__gman-tx,edmondsw: there is an update, we don't need remove_hdisk call anymore, since remove_iscsi cleans up the hdisk as well.17:16
edmondswchhavi__ good, I was wondering why it didn't...17:17
edmondswchhavi__ are you going to post the review?17:18
chhavi__so the only change which we have is for udid17:18
chhavi__yeah, i need to deliver the change today, as it has to be in 1.4.117:18
edmondswchhavi__ well no... see slack17:20
openstackgerritChhavi Agarwal proposed openstack/nova-powervm master: iSCSI Live Migration Support  https://review.openstack.org/56757517:31
*** gman-tx has quit IRC17:32
*** openstackgerrit has quit IRC17:34
*** chhavi__ has quit IRC17:45
*** gman-tx has joined #openstack-powervm18:01
*** gman-tx has quit IRC19:12
edmondswefried what's the plan for https://review.openstack.org/#/c/557976/ ? Is that still needed?19:18
efriededmondsw: I think it's obsolete, since we fixed it in pypowervm, nah?19:19
edmondswI think it's probably obsolete, but I don't think this had anything to do with pypowervm did it?19:20
edmondswefried yeah, looks like this was fixed by https://review.openstack.org/#/c/557975/19:24
edmondswstill, fixed, so https://review.openstack.org/#/c/557974/ and https://review.openstack.org/#/c/557976/ are obsolete I think19:25
efriedyup19:25
efriededmondsw: you marking 'em?19:25
edmondswI'll ping claudiub19:25
edmondswwell, don't see him on, so I'll just comment19:26
efriededmondsw: You could probably just abandon it for him, I don't think he would mind.19:30
edmondswyeah, maybe I should have. At this point I already left a comment, so I'll leave it for a few week or so19:31
*** tjakobs has quit IRC19:33
*** tjakobs has joined #openstack-powervm19:34
efriedight19:42
*** gman-tx has joined #openstack-powervm20:15
*** gman-tx_ has joined #openstack-powervm20:23
*** gman-tx has quit IRC20:25
*** gman-tx_ is now known as gman-tx20:25
*** gman-tx has quit IRC20:28
efriededmondsw: Reviewed iscsi.20:34
edmondswtx20:34
edmondswefried I replied20:40
edmondswit's confusing20:40
efriededmondsw: re-replied, thanks for the clarification.  Agree more words are needed.20:42
*** edmondsw has quit IRC21:38
*** tjakobs has quit IRC22:13
*** edmondsw has joined #openstack-powervm22:36
*** edmondsw has quit IRC22:41
*** adreznec has quit IRC23:04
*** adreznec has joined #openstack-powervm23:06
*** tjakobs has joined #openstack-powervm23:32
*** tjakobs has quit IRC23:43

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