| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-10-18 | |||
| 09:44:34 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: compute: enhance compute evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858383 | |
| 10:50:59 | opendevreview | Amit Uniyal proposed openstack/nova master: Adds check for VM snapshot fail while quiesce https://review.opendev.org/c/openstack/nova/+/852171 | |
| 10:56:37 | opendevreview | Rajesh Tailor proposed openstack/nova master: Handle InstanceInvalidState exception https://review.opendev.org/c/openstack/nova/+/861738 | |
| 11:40:02 | opendevreview | Rajesh Tailor proposed openstack/nova master: Correct doc references https://review.opendev.org/c/openstack/nova/+/861741 | |
| 12:27:56 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: compute: enhance compute evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858383 | |
| 12:27:57 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| 12:38:46 | bauzas | reminder: nova operator hours (with a trailing "s" as we have two hours) in 22 mins in the bexar room | |
| 12:40:59 | sean-k-mooney | ack | |
| 12:41:30 | sean-k-mooney | by the way how was my audio in the tc session yesterday | |
| 12:41:37 | sean-k-mooney | was the lag gone? | |
| 12:41:46 | sean-k-mooney | and or did i sound ok | |
| 12:42:22 | sean-k-mooney | because its simpler for me to move to the other room then emma im going to join from there again today | |
| 12:42:39 | sean-k-mooney | and just wonderign if i can continue to use the same setup or if i need my other laptop | |
| 12:42:46 | sean-k-mooney | bauzas: dansmith ^ | |
| 12:43:09 | bauzas | sean-k-mooney: your audio was fine, no worries | |
| 12:43:19 | bauzas | you were no loud | |
| 12:43:40 | sean-k-mooney | as in i was quite? | |
| 12:44:03 | sean-k-mooney | i had my headset mics at 50% so i can increase that if needed | |
| 12:44:27 | sean-k-mooney | for the nova ironic seesion i was lagging because i forgot to use chome and i was joining via the web app | |
| 12:44:43 | sean-k-mooney | for the tc session i used the native zoom app which seamed to work much better | |
| 13:00:38 | bauzas | operator hours NOW at bexar room :) | |
| 13:03:11 | sean-k-mooney | i need to setup downstairs so ill join in a few hopefully there will be a good turn out | |
| 13:03:36 | sean-k-mooney | if not i have a topic i want to add to the ptg adjenda that we might cover breifly. | |
| 13:26:02 | opendevreview | Merged openstack/nova master: Correct doc references https://review.opendev.org/c/openstack/nova/+/861741 | |
| 13:39:39 | bauzas | sean-k-mooney: you're becoming a bit choppy | |
| 13:39:48 | bauzas | your audio is cutting a bit | |
| 13:42:19 | opendevreview | Jorhson Deng proposed openstack/nova master: Allow user defined stop type https://review.opendev.org/c/openstack/nova/+/861333 | |
| 14:39:33 | opendevreview | Arnaud Morin proposed openstack/nova master: Unbind port when offloading a shelved instance https://review.opendev.org/c/openstack/nova/+/853682 | |
| 14:51:18 | opendevreview | Merged openstack/nova stable/zed: Remove mentions of removed scheduler filters https://review.opendev.org/c/openstack/nova/+/858534 | |
| 14:52:48 | bauzas | folks, we start discussing at 1500UTC with the zed retro | |
| 15:12:21 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: api: extend evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858384 | |
| 15:12:21 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova master: compute: enhance compute evacuate instance to support target state https://review.opendev.org/c/openstack/nova/+/858383 | |
| 16:04:12 | gmann | johnthetubaguy: sean-k-mooney : __member_ things is fixed in Horizon https://review.opendev.org/c/openstack/horizon/+/824432 | |
| 16:14:12 | opendevreview | Ghanshyam proposed openstack/placement master: DNM: testign jobs on Ubuntu jammy (22.04) https://review.opendev.org/c/openstack/placement/+/861471 | |
| 17:09:20 | gibi | oops, I pressed end meeting for all | |
| 17:09:29 | sean-k-mooney[m] | hehe | |
| 17:09:41 | gibi | I forget that I got the host :) | |
| 17:09:41 | sean-k-mooney[m] | got a sec to chat about something | |
| 17:09:45 | gibi | sure | |
| 17:10:02 | sean-k-mooney[m] | ill send you an invite | |
| 17:10:04 | gibi | ok | |
| 17:33:28 | johnthetubaguy | gmann: awesome news, thank you for the update! | |
| 17:33:57 | johnthetubaguy | (I didn't know that was a config!) | |
| #openstack-nova - 2022-10-19 | |||
| 01:55:43 | opendevreview | Jorhson Deng proposed openstack/nova master: Allow user defined stop type https://review.opendev.org/c/openstack/nova/+/861333 | |
| 02:17:40 | opendevreview | Jorhson Deng proposed openstack/nova master: Allow user defined stop type https://review.opendev.org/c/openstack/nova/+/861333 | |
| 05:44:08 | opendevreview | Jorhson Deng proposed openstack/nova master: Allow user defined stop type https://review.opendev.org/c/openstack/nova/+/861333 | |
| 05:54:14 | opendevreview | Jorhson Deng proposed openstack/nova master: Allow user defined stop type https://review.opendev.org/c/openstack/nova/+/861333 | |
| 07:53:57 | sahid | o/ | |
| 07:54:53 | sahid | any chance that you have a second look at https://review.opendev.org/c/openstack/nova-specs/+/857838 sean-k-mooney? I hope to have addressed most of your comments I just have kept open one to ensure that this is addressed in the right way | |
| 08:02:49 | opendevreview | Jorhson Deng proposed openstack/nova master: Allow user defined stop type https://review.opendev.org/c/openstack/nova/+/861333 | |
| 09:21:42 | opendevreview | Amit Uniyal proposed openstack/nova stable/zed: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/861856 | |
| 09:21:43 | opendevreview | Amit Uniyal proposed openstack/nova stable/zed: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/861857 | |
| 10:25:59 | mnasiadka | Hello - any reason backport https://review.opendev.org/c/openstack/nova/+/829804 is hanging so long? | |
| 10:28:43 | sean-k-mooney | mnasiadka: it just needs review it fell off my radar so i can take a look now | |
| 10:29:04 | mnasiadka | sean-k-mooney: thanks! | |
| 10:29:07 | sean-k-mooney | in general if a patch is up and not reviewd for a period of tiem we expect the autorp or backporteer to ping us | |
| 10:29:30 | sean-k-mooney | we will eventualy get to it otherwise but the more vocal people are the more likely we are to see it | |
| 10:33:50 | sean-k-mooney | elodilles: if you have time can you take a look at https://review.opendev.org/c/openstack/nova/+/829804 | |
| 10:52:28 | mnasiadka | sean-k-mooney: Actually I need it in Wallaby, will do a cherry-pick today as well | |
| 10:52:53 | sean-k-mooney | ack | |
| 10:53:16 | opendevreview | Michal Nasiadka proposed openstack/nova stable/wallaby: Fix to implement 'pack' or 'spread' VM's NUMA cells https://review.opendev.org/c/openstack/nova/+/861832 | |
| 10:53:34 | mnasiadka | actually it was clean, so simple as a click in Gerrit UI | |
| 10:53:44 | sean-k-mooney | mnasiadka: you might also want the caching change we merged backported | |
| 10:53:58 | sean-k-mooney | mnasiadka: gerrit wont set teh cherry-picked line correctly | |
| 10:54:10 | sean-k-mooney | it only has one | |
| 10:54:16 | sean-k-mooney | it should have one per branch | |
| 10:54:28 | sean-k-mooney | so gerrit only works the first time | |
| 10:55:22 | mnasiadka | ok, I'll update | |
| 10:55:33 | mnasiadka | sean-k-mooney: caching change? which one is that? | |
| 10:55:39 | opendevreview | Amit Uniyal proposed openstack/nova stable/yoga: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/861871 | |
| 10:55:40 | opendevreview | Amit Uniyal proposed openstack/nova stable/yoga: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/861872 | |
| 11:30:30 | sahid | thank you sean-k-mooney[m], I have replied to one of you question, I'm not sure that we need to update the virt driver api but I may have missed something. At leaast the current impl that I have is working as expected for our usage | |
| 11:30:47 | sahid | for the other I will address them when i will get more reviews for sure | |
| 11:31:06 | sean-k-mooney1 | sahid: i belive you will need to chnage rebuild_instance to add the target state | |
| 11:31:31 | sean-k-mooney | so that the driver does not try to start the vm | |
| 11:31:55 | sahid | I have reused the way that we currently handle evacuated instance that are stopped | |
| 11:32:09 | sahid | the driver start the instance and tjere is a process to stop it | |
| 11:32:45 | sean-k-mooney | we should not do that really | |
| 11:32:52 | sean-k-mooney | since the start is going to fail | |
| 11:33:04 | sean-k-mooney | at least in teh encypted volume case | |
| 11:33:20 | sean-k-mooney | ideally you would pass the target state in and only start it if required | |
| 11:34:11 | sean-k-mooney | we can see what others think but if you say targetState=stop i would consider it a bug to attepmt to start it and then stop it | |
| 11:36:13 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/858383/14/nova/compute/manager.py#3826 might be ok | |
| 11:37:28 | sean-k-mooney | no looking at the workflow | |
| 11:37:44 | sean-k-mooney | im pretty sure if you try that with an encypted volume the spawn will fail | |
| 11:38:08 | sean-k-mooney | the admin wont be abel to retirve the key form barbican | |
| 12:04:44 | sahid | sean-k-mooney: oh i see, at least in our use-case it's fine as everything is done using cloud-init and we don't wait for that to happen | |
| 12:06:12 | sahid | I'm wondering whether it's not mandotary to have the vm started to libvirt would have installed the tap and qemu sent the arp, we also know that ovs agent needs to see net devices to configure network | |
| 12:07:35 | sean-k-mooney | that woudl all happen when the user starts the vm | |
| 12:08:08 | sean-k-mooney | we shoudl perhaps discuss this in the ptg or at least get more eye on the review | |
| 12:08:51 | sean-k-mooney | binding the port in neutron makes sense im not sure that plugging the prot into ovs shoudl be done at this point | |
| 12:09:26 | sean-k-mooney | we need to prepare the disk image/instance direcotry | |
| 12:09:41 | sean-k-mooney | but im not sure we should go as far ad defining the domain | |
| 12:10:08 | sean-k-mooney | we could maintain the current beahivor of powering it off after but that does not fell correct to me | |
| 12:11:13 | sahid | i also had this feeling when I have noticed it but then I felt like this was perhaps a necessity | |
| 12:12:08 | sahid | i can make some tests but that will be difficult for me to test all the combinaisons | |
| 12:12:42 | sahid | for the case that is open in your side, does that can be tested easily? | |
| 12:13:26 | sean-k-mooney | and basically ignore it in this case and proceed to powering it off. | |
| 12:13:26 | sean-k-mooney | if we do that we need to catch the expction form looking up the key | |
| 12:15:23 | sahid | i see, i can for sure doing that, but as you mention we probably first have to see whether we want to avoid the vm to be started at driver side, which could brings others implications | |