*** edmondsw has joined #openstack-powervm | 01:42 | |
*** YuYangWang has quit IRC | 01:43 | |
*** edmondsw has quit IRC | 01:47 | |
openstackgerrit | Merged openstack/nova-powervm master: Remove unused volume_adapter conf https://review.openstack.org/572995 | 02:14 |
---|---|---|
*** tjakobs has joined #openstack-powervm | 03:15 | |
*** tjakobs has quit IRC | 03:20 | |
*** edmondsw has joined #openstack-powervm | 03:30 | |
*** edmondsw has quit IRC | 03:35 | |
*** chhagarw has joined #openstack-powervm | 05:06 | |
*** edmondsw has joined #openstack-powervm | 05:19 | |
*** edmondsw has quit IRC | 05:23 | |
*** AlexeyAbashkin has joined #openstack-powervm | 07:04 | |
*** edmondsw has joined #openstack-powervm | 07:07 | |
*** edmondsw has quit IRC | 07:11 | |
*** Alexey_Abashkin has joined #openstack-powervm | 07:34 | |
*** AlexeyAbashkin has quit IRC | 07:35 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 07:35 | |
*** AlexeyAbashkin has quit IRC | 07:46 | |
*** AlexeyAbashkin has joined #openstack-powervm | 07:56 | |
*** AlexeyAbashkin has quit IRC | 08:23 | |
*** AlexeyAbashkin has joined #openstack-powervm | 08:26 | |
*** AlexeyAbashkin has quit IRC | 08:33 | |
*** AlexeyAbashkin has joined #openstack-powervm | 08:38 | |
*** AlexeyAbashkin has quit IRC | 08:45 | |
*** edmondsw has joined #openstack-powervm | 08:55 | |
*** edmondsw has quit IRC | 09:00 | |
openstackgerrit | Chhavi Agarwal proposed openstack/nova-powervm master: iSCSI Live Migration Support https://review.openstack.org/567575 | 09:03 |
*** AlexeyAbashkin has joined #openstack-powervm | 09:31 | |
*** AlexeyAbashkin has quit IRC | 09:50 | |
*** AlexeyAbashkin has joined #openstack-powervm | 10:02 | |
*** AlexeyAbashkin has quit IRC | 10:13 | |
*** Alexey_Abashkin has joined #openstack-powervm | 10:13 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 10:16 | |
*** AlexeyAbashkin has quit IRC | 10:23 | |
*** AlexeyAbashkin has joined #openstack-powervm | 10:25 | |
*** Alexey_Abashkin has joined #openstack-powervm | 10:38 | |
*** AlexeyAbashkin has quit IRC | 10:39 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 10:39 | |
*** edmondsw has joined #openstack-powervm | 10:43 | |
*** edmondsw has quit IRC | 10:47 | |
*** AlexeyAbashkin has quit IRC | 11:01 | |
*** edmondsw has joined #openstack-powervm | 11:22 | |
openstackgerrit | Elod Illes proposed openstack/networking-powervm stable/pike: Fix sphinx-docs job for stable branch https://review.openstack.org/573197 | 11:39 |
*** AlexeyAbashkin has joined #openstack-powervm | 11:51 | |
*** esberglu has joined #openstack-powervm | 12:57 | |
*** AlexeyAbashkin has quit IRC | 13:08 | |
*** Alexey_Abashkin has joined #openstack-powervm | 13:08 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 13:10 | |
*** openstackgerrit has quit IRC | 13:34 | |
*** AlexeyAbashkin has quit IRC | 13:43 | |
*** AlexeyAbashkin has joined #openstack-powervm | 13:50 | |
*** Alexey_Abashkin has joined #openstack-powervm | 14:02 | |
*** AlexeyAbashkin has quit IRC | 14:02 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 14:02 | |
*** tjakobs has joined #openstack-powervm | 14:32 | |
*** Alexey_Abashkin has joined #openstack-powervm | 14:38 | |
*** AlexeyAbashkin has quit IRC | 14:41 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 14:41 | |
edmondsw | efried I proposed Rocky-2 tags: https://review.openstack.org/#/c/573263/ | 14:43 |
efried | edmondsw: Ack | 14:43 |
efried | edmondsw: Keep an eye out for ------^ which I fast-approved shortly ago. May be worth including. | 14:47 |
edmondsw | efried ack | 14:48 |
*** chhavi__ has joined #openstack-powervm | 14:48 | |
edmondsw | efried that's not on master, so wouldn't affect Rocky tagging | 14:48 |
efried | edmondsw: oh, mahbad, didn't notice that. | 14:49 |
efried | was absently wondering why the doc job breakage wasn't a more urgent thing. | 14:49 |
*** chhagarw has quit IRC | 14:49 | |
edmondsw | :) | 14:49 |
edmondsw | efried you fast approved that one but not https://review.openstack.org/#/c/572841 ? | 14:50 |
edmondsw | doesn't look like either were fast approved... I can workflow them | 14:51 |
efried | okay. Hm, I thought I had. Anyway, yeah, thanks. | 14:51 |
efried | Maybe I hit it before tests had run. | 14:51 |
efried | looks like it. | 14:52 |
*** openstackgerrit has joined #openstack-powervm | 15:02 | |
openstackgerrit | Merged openstack/ceilometer-powervm stable/pike: Fix sphinx-docs job for stable branch https://review.openstack.org/572841 | 15:02 |
*** gman-tx has joined #openstack-powervm | 15:06 | |
openstackgerrit | Chhavi Agarwal proposed openstack/nova-powervm master: iSCSI Live Migration Support https://review.openstack.org/567575 | 15:10 |
*** Alexey_Abashkin has joined #openstack-powervm | 15:20 | |
*** AlexeyAbashkin has quit IRC | 15:22 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 15:22 | |
*** AlexeyAbashkin has quit IRC | 15:22 | |
*** AlexeyAbashkin has joined #openstack-powervm | 15: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-powervm | 15:37 | |
*** AlexeyAbashkin has quit IRC | 15:37 | |
*** Alexey_Abashkin is now known as AlexeyAbashkin | 15:37 | |
edmondsw | efried please take a look at https://review.openstack.org/#/c/567575/11/nova_powervm/virt/powervm/volume/iscsi.py@428 | 15:43 |
edmondsw | you're our feedtask best-practices expert | 15:43 |
edmondsw | :) | 15:43 |
edmondsw | beyond just my comment, I'm wondering if it's right other than that | 15:43 |
efried | edmondsw: k, got calls for the next hour, but will look. | 15:43 |
edmondsw | tx | 15:44 |
edmondsw | chhavi__ ^ | 15:44 |
edmondsw | I added my comments, but would like eric to look as well | 15:44 |
edmondsw | and did you confirm with gman-tx that it's correct to remove_iscsi before remove_hdisk? | 15:44 |
*** gman-tx has quit IRC | 15:52 | |
*** AlexeyAbashkin has quit IRC | 15: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 hdisk | 15:53 |
*** gman-tx has joined #openstack-powervm | 15:53 | |
*** esberglu has quit IRC | 15:57 | |
efried | edmondsw: couple minutes late to the mtg | 15:59 |
edmondsw | mdrabe you joining? | 16:07 |
mdrabe | edmondsw: What meeting? | 16:09 |
edmondsw | mdrabe check your calendar :) or I slacked it to you | 16:10 |
*** gman-tx has quit IRC | 16: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-infra | 16:10 | |
openstackgerrit | Merged openstack/networking-powervm stable/pike: Fix sphinx-docs job for stable branch https://review.openstack.org/573197 | 16:14 |
*** gman-tx has joined #openstack-powervm | 16: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 |
edmondsw | chhavi__ good, I was wondering why it didn't... | 17:17 |
edmondsw | chhavi__ are you going to post the review? | 17:18 |
chhavi__ | so the only change which we have is for udid | 17:18 |
chhavi__ | yeah, i need to deliver the change today, as it has to be in 1.4.1 | 17:18 |
edmondsw | chhavi__ well no... see slack | 17:20 |
openstackgerrit | Chhavi Agarwal proposed openstack/nova-powervm master: iSCSI Live Migration Support https://review.openstack.org/567575 | 17:31 |
*** gman-tx has quit IRC | 17:32 | |
*** openstackgerrit has quit IRC | 17:34 | |
*** chhavi__ has quit IRC | 17:45 | |
*** gman-tx has joined #openstack-powervm | 18:01 | |
*** gman-tx has quit IRC | 19:12 | |
edmondsw | efried what's the plan for https://review.openstack.org/#/c/557976/ ? Is that still needed? | 19:18 |
efried | edmondsw: I think it's obsolete, since we fixed it in pypowervm, nah? | 19:19 |
edmondsw | I think it's probably obsolete, but I don't think this had anything to do with pypowervm did it? | 19:20 |
edmondsw | efried yeah, looks like this was fixed by https://review.openstack.org/#/c/557975/ | 19:24 |
edmondsw | still, fixed, so https://review.openstack.org/#/c/557974/ and https://review.openstack.org/#/c/557976/ are obsolete I think | 19:25 |
efried | yup | 19:25 |
efried | edmondsw: you marking 'em? | 19:25 |
edmondsw | I'll ping claudiub | 19:25 |
edmondsw | well, don't see him on, so I'll just comment | 19:26 |
efried | edmondsw: You could probably just abandon it for him, I don't think he would mind. | 19:30 |
edmondsw | yeah, maybe I should have. At this point I already left a comment, so I'll leave it for a few week or so | 19:31 |
*** tjakobs has quit IRC | 19:33 | |
*** tjakobs has joined #openstack-powervm | 19:34 | |
efried | ight | 19:42 |
*** gman-tx has joined #openstack-powervm | 20:15 | |
*** gman-tx_ has joined #openstack-powervm | 20:23 | |
*** gman-tx has quit IRC | 20:25 | |
*** gman-tx_ is now known as gman-tx | 20:25 | |
*** gman-tx has quit IRC | 20:28 | |
efried | edmondsw: Reviewed iscsi. | 20:34 |
edmondsw | tx | 20:34 |
edmondsw | efried I replied | 20:40 |
edmondsw | it's confusing | 20:40 |
efried | edmondsw: re-replied, thanks for the clarification. Agree more words are needed. | 20:42 |
*** edmondsw has quit IRC | 21:38 | |
*** tjakobs has quit IRC | 22:13 | |
*** edmondsw has joined #openstack-powervm | 22:36 | |
*** edmondsw has quit IRC | 22:41 | |
*** adreznec has quit IRC | 23:04 | |
*** adreznec has joined #openstack-powervm | 23:06 | |
*** tjakobs has joined #openstack-powervm | 23:32 | |
*** tjakobs has quit IRC | 23:43 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!