| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-04-23 | |||
| 12:56:59 | stephenfin | lyarwood: left a comment - could you address that one (happy with the rest being done in a follow-up, as with gibi) | |
| 12:59:47 | lyarwood | ack looking | |
| 12:59:47 | lyarwood | ack looking | |
| 13:03:17 | openstackgerrit | Lee Yarwood proposed openstack/nova master: guestfs: With libguestfs >= v1.41.1 decode returned bytes to string https://review.opendev.org/c/openstack/nova/+/787712 | |
| 13:03:17 | openstackgerrit | Lee Yarwood proposed openstack/nova master: guestfs: With libguestfs >= v1.41.1 decode returned bytes to string https://review.opendev.org/c/openstack/nova/+/787712 | |
| 13:03:51 | stephenfin | thanks | |
| 13:03:59 | lyarwood | np | |
| 13:04:00 | lyarwood | np | |
| 13:04:03 | lyarwood | thanks for review | |
| 13:04:03 | lyarwood | thanks for review | |
| 13:28:50 | bauzas | wow, so Zoom is eating 5GB of my RAM | |
| 13:28:50 | bauzas | wow, so Zoom is eating 5GB of my RAM | |
| 13:28:55 | bauzas | ... | |
| 13:28:55 | bauzas | ... | |
| 13:29:09 | sean-k-mooney | what browser are you using | |
| 13:29:09 | sean-k-mooney | what browser are you using | |
| 13:29:14 | sean-k-mooney | it prefers chrome | |
| 13:29:14 | sean-k-mooney | it prefers chrome | |
| 13:29:30 | sean-k-mooney | or are you using the zoom app | |
| 13:29:30 | sean-k-mooney | or are you using the zoom app | |
| 13:29:42 | sean-k-mooney | browser seams to work better for me at least | |
| 13:29:42 | sean-k-mooney | browser seams to work better for me at least | |
| 13:31:05 | bauzas | I directly use the zoom app | |
| 13:31:05 | bauzas | I directly use the zoom app | |
| 13:31:24 | bauzas | * bauzas tests a few things before the last PTG day | |
| 13:31:27 | sean-k-mooney | ya i had audio issue with that i could hear but people could not hear me | |
| 13:31:27 | sean-k-mooney | ya i had audio issue with that i could hear but people could not hear me | |
| 13:31:47 | bauzas | hopefully this will be our last virtual PTG... | |
| 13:31:47 | bauzas | hopefully this will be our last virtual PTG... | |
| 13:31:49 | sean-k-mooney | so now im using it in google chrome not chromium | |
| 13:31:49 | sean-k-mooney | so now im using it in google chrome not chromium | |
| 13:53:34 | artom | sean-k-mooney, you mean an entirely new nova-manage command? As in `nova-manage sriov-ports add-requester-id`? | |
| 13:53:34 | artom | sean-k-mooney, you mean an entirely new nova-manage command? As in `nova-manage sriov-ports add-requester-id`? | |
| 13:55:32 | sean-k-mooney | almost | |
| 13:55:32 | sean-k-mooney | almost | |
| 13:55:44 | sean-k-mooney | new yes but i was suggesting that it would be automatic | |
| 13:55:44 | sean-k-mooney | new yes but i was suggesting that it would be automatic | |
| 13:56:08 | sean-k-mooney | i.e. you would not have to specify the requester id manually | |
| 13:56:08 | sean-k-mooney | i.e. you would not have to specify the requester id manually | |
| 13:56:23 | sean-k-mooney | it would try and work it out by looking at teh pci claims and the port profile | |
| 13:56:23 | sean-k-mooney | it would try and work it out by looking at teh pci claims and the port profile | |
| 13:56:32 | sean-k-mooney | and if they agreed then setting the value | |
| 13:56:32 | artom | I guess... | |
| 13:56:32 | sean-k-mooney | and if they agreed then setting the value | |
| 13:56:32 | artom | I guess... | |
| 13:56:44 | artom | That would not apply to any instances that are not ACTIVE though | |
| 13:56:45 | artom | That would not apply to any instances that are not ACTIVE though | |
| 13:56:57 | sean-k-mooney | if not it would out put a list of port that were potnetally broken and say hay you might need to fix these | |
| 13:56:57 | sean-k-mooney | if not it would out put a list of port that were potnetally broken and say hay you might need to fix these | |
| 13:57:00 | artom | Which is also my beef with the data migration (currently writing a para of text in the review) | |
| 13:57:00 | artom | Which is also my beef with the data migration (currently writing a para of text in the review) | |
| 13:57:22 | sean-k-mooney | how do you mean | |
| 13:57:22 | sean-k-mooney | how do you mean | |
| 13:57:38 | lyarwood | * lyarwood is going to miss the first 20mins of the nova track talking to the manila folks | |
| 13:57:39 | sean-k-mooney | it woul apply to any vm that is not currently in shleve offloaed or error | |
| 13:57:39 | sean-k-mooney | it woul apply to any vm that is not currently in shleve offloaed or error | |
| 13:57:40 | artom | Well, for instances that are SHELVED(_OFFLOADED) for example... | |
| 13:57:40 | artom | Well, for instances that are SHELVED(_OFFLOADED) for example... | |
| 13:58:06 | sean-k-mooney | ya one that are shleve offloaded currently cant really be fixed without manual intervention | |
| 13:58:07 | sean-k-mooney | ya one that are shleve offloaded currently cant really be fixed without manual intervention | |
| 13:58:28 | artom | And don't we get into weird race conditions for instances in MIGRATING? | |
| 13:58:28 | artom | And don't we get into weird race conditions for instances in MIGRATING? | |
| 13:58:32 | gibi | lyarwood: do you have any hard opinion about droping eventlet? (we will start with that topic in nova) | |
| 13:58:32 | gibi | lyarwood: do you have any hard opinion about droping eventlet? (we will start with that topic in nova) | |
| 13:58:49 | sean-k-mooney | maybe we could determin what subset are valid in the reivew | |
| 13:58:49 | sean-k-mooney | maybe we could determin what subset are valid in the reivew | |
| 13:58:51 | artom | As in, depending on when you run the data migration, we might have already updated the port binding, or maybe we haven't | |
| 13:58:51 | artom | As in, depending on when you run the data migration, we might have already updated the port binding, or maybe we haven't | |
| 13:59:03 | sean-k-mooney | but basically any migration or command shoudl be a sperate patch after the fix | |
| 13:59:03 | sean-k-mooney | but basically any migration or command shoudl be a sperate patch after the fix | |
| 13:59:25 | sean-k-mooney | we have prescende downstream for backporting only the fix without the data migration | |
| 13:59:25 | sean-k-mooney | we have prescende downstream for backporting only the fix without the data migration | |
| 13:59:38 | sean-k-mooney | that is what we did for the network info cache force refresh | |
| 13:59:38 | sean-k-mooney | that is what we did for the network info cache force refresh | |
| 13:59:45 | artom | Oh, I remember that | |
| 13:59:45 | artom | Oh, I remember that | |
| 13:59:55 | artom | I'd argue that it can also be useful upstream | |
| 13:59:55 | artom | I'd argue that it can also be useful upstream | |
| 14:00:11 | artom | Its still fixes an issue, even if any of your pre-existing instances don't get it | |
| 14:00:11 | artom | Its still fixes an issue, even if any of your pre-existing instances don't get it | |
| 14:00:18 | sean-k-mooney | yes | |
| 14:00:20 | sean-k-mooney | i would too | |
| 14:00:20 | sean-k-mooney | i would too | |
| 14:00:34 | sean-k-mooney | that is why i want the data mighation or command in a sperate patch that is after the fix | |
| 14:00:34 | sean-k-mooney | that is why i want the data mighation or command in a sperate patch that is after the fix | |
| 14:00:45 | sean-k-mooney | so we can backport the fix only if we chosse too | |
| 14:00:45 | sean-k-mooney | so we can backport the fix only if we chosse too | |
| 14:00:50 | artom | stephenfin, you cool with that? ^^ I can push the fixes that I have for your comments, and we can figure out the online data migration thing in a separate patch? | |
| 14:00:50 | artom | stephenfin, you cool with that? ^^ I can push the fixes that I have for your comments, and we can figure out the online data migration thing in a separate patch? | |
| 14:00:51 | sean-k-mooney | to prevent new vms form breaking | |
| 14:00:51 | sean-k-mooney | to prevent new vms form breaking | |
| 14:01:27 | sean-k-mooney | then we can see how the stable team feals about backporting the nova manage command or the data migration after its fixed on master | |
| 14:01:27 | sean-k-mooney | then we can see how the stable team feals about backporting the nova manage command or the data migration after its fixed on master | |
| 14:02:08 | lyarwood | gibi: I don't :) | |
| 14:02:08 | lyarwood | gibi: I don't :) | |
| 14:02:16 | gibi | lyarwood: ack | |
| 14:07:53 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Update SRIOV port pci_slot when unshelving https://review.opendev.org/c/openstack/nova/+/784168 | |
| 14:07:53 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Test SRIOV port move operations with PCI conflicts https://review.opendev.org/c/openstack/nova/+/783084 | |
| 14:07:53 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Update SRIOV port pci_slot when unshelving https://review.opendev.org/c/openstack/nova/+/784168 | |
| 14:07:53 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: Test SRIOV port move operations with PCI conflicts https://review.opendev.org/c/openstack/nova/+/783084 | |