| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-10-19 | |||
| 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 | if we do that we need to catch the expction form looking up the key | |
| 12:13:26 | sean-k-mooney | and basically ignore it in this case and proceed to powering it off. | |
| 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 | |
| 12:27:48 | opendevreview | ribaudr proposed openstack/nova-specs master: Draft spec to support scaphandre https://review.opendev.org/c/openstack/nova-specs/+/861881 | |
| 12:38:45 | opendevreview | ribaudr proposed openstack/nova-specs master: Draft spec to support scaphandre https://review.opendev.org/c/openstack/nova-specs/+/861881 | |
| 12:55:09 | bauzas | reminder : our PTG sessions start in 5 mins, see you there in the bexar room | |
| 13:01:54 | bauzas | started ^ | |
| 13:01:59 | bauzas | folks, we need you ! | |
| 13:05:59 | bauzas | we'll start with discussing about ELO'ing stable branches for stein and older | |
| 13:06:03 | bauzas | now* | |
| 14:23:05 | bauzas | chateaulav: I know you're in the zoom room, but we are just discussing secure RBAC | |
| 14:23:15 | bauzas | to sure you are listening | |
| 14:23:20 | bauzas | not* sure | |
| 14:30:18 | stephenfin | sean-k-mooney: Your background blur feature is going nuts :) | |
| 14:30:31 | chateaulav | bauzas: thanks | |
| 14:39:51 | sean-k-mooney | fixed by trunig it off | |
| 14:47:47 | gibi | vmaccel: we will discuss the PCI in placement stuff in https://www.openinfra.dev/ptg/rooms/bexar in 3 minutes | |
| 15:00:34 | opendevreview | Merged openstack/nova-specs master: Update the PCI in placement spec https://review.opendev.org/c/openstack/nova-specs/+/855218 | |
| 15:12:02 | bauzas | sean-k-mooney: your audio becomes a bit robotic | |
| 15:15:32 | stephenfin | sean-k-mooney: Y̶o̶u̶'̶r̶e̶ ̶a̶ ̶w̶i̶z̶a̶r̶d̶,̶ ̶H̶a̶r̶r̶y̶!̶ You're a robot, Sean! 🤖 | |
| 15:17:13 | sean-k-mooney | still ok i might have to try treaking something | |
| #openstack-nova - 2022-10-20 | |||
| 08:15:39 | opendevreview | Alex Chan proposed openstack/nova stable/xena: fix instance action log when deleting shelved instance https://review.opendev.org/c/openstack/nova/+/861951 | |
| 08:21:58 | opendevreview | Alex Chan proposed openstack/nova stable/xena: fix instance action log when deleting shelved instance https://review.opendev.org/c/openstack/nova/+/861951 | |
| 08:36:52 | bauzas | ralonsoh: for our nova-neutron session, would we use our bexar room ? | |
| 08:38:53 | ralonsoh | bauzas, I'll send a mail now. I'll also open our meeting room to redirect people to bexar | |
| 08:41:48 | ralonsoh | bauzas, done | |
| 08:42:01 | bauzas | ralonsoh: cool thanks | |
| 08:42:15 | bauzas | ralonsoh: do you have other topics but the ones we have in our etherpad ? | |
| 08:42:34 | ralonsoh | bauzas, no but I think the agenda is pretty tight | |
| 08:42:50 | ralonsoh | we have 4 topics | |
| 08:46:07 | bauzas | ralonsoh: sorry, I only see 2 topics in our etherpad | |
| 08:46:16 | ralonsoh | bauzas, https://etherpad.opendev.org/p/neutron-antelope-ptg | |
| 08:46:26 | ralonsoh | we have 4 | |
| 09:10:25 | bauzas | ralonsoh: my bad, then I'll remove the 2 topics from the nova etherpad and rather link to the neutron etherpad | |
| 12:33:03 | opendevreview | ribaudr proposed openstack/nova-specs master: Draft spec to support scaphandre https://review.opendev.org/c/openstack/nova-specs/+/861881 | |
| 12:37:29 | opendevreview | ribaudr proposed openstack/nova-specs master: Draft spec to support scaphandre https://review.opendev.org/c/openstack/nova-specs/+/861881 | |
| 12:53:28 | stephenfin | bauzas: I will be an hour late today (14:00 UTC). Feel free to discuss the soft-delete thing in my absence if you need to | |
| 12:54:56 | bauzas | stephenfin: we will first discuss the neutron topics until 1500UTC | |
| 12:55:11 | bauzas | stephenfin: we'll then discuss the nova topics after 1500 | |
| 13:00:25 | bauzas | neutron-nova session starting now in the bexar room | |
| 13:00:43 | bauzas | don't let me feel alone in the dark :) | |
| 13:00:49 | gibi | bauzas: bexar or mitaka? | |
| 13:00:56 | bauzas | bexar, as I wrote :) | |
| 13:01:09 | gibi | OK, the neutron etherpad has mitaka in it | |
| 13:01:31 | sean-k-mooney | lol | |
| 13:01:51 | sean-k-mooney | https://www.openinfra.dev/ptg/rooms/bexar | |
| 13:59:45 | elodilles | (i'll need to change to relmgmt session for now, bbl) | |
| 14:23:08 | bauzas | dang, can't find where we suffix the instance name with multicreate | |
| 14:31:38 | gibi | bauzas: https://github.com/openstack/nova/blob/bcdf5988f6ae902dba9b41144a7b4a60688b627c/nova/compute/api.py#L2031 | |
| 14:31:52 | bauzas | gibi: thanks I was grabbing the compute.api | |
| 14:33:04 | gibi | bauzas: do we continue with nova at 15:00 UTC? | |
| 14:33:12 | gibi | i.e. we have 27m break? | |
| 14:35:09 | bauzas | gibi: we're just discussing one topic about robustify hosntames | |
| 14:35:14 | bauzas | and then we take a break | |
| 14:35:15 | gibi | ack, joined | |
| 17:35:27 | stephenfin | melwitt: Regarding https://paste.opendev.org/show/bTNEGH3Z97Ee1Em5zcGI/, for the first bit that seems to be an argparse limitation https://stackoverflow.com/questions/23032514/argparse-disable-same-argument-occurrences | |
| 18:47:54 | darkhorse | Hi team | |
| 18:48:28 | darkhorse | I am trying to fix a nova issue that instance delete action log is not added when deleting a shelved_offloaded instance. | |
| 18:48:42 | darkhorse | https://review.opendev.org/c/openstack/nova/+/861951 | |
| 18:49:26 | darkhorse | I committed the patchset but the CI build fails. Can anyone help figuring out why it fails? | |
| 18:59:46 | frickler | fyi I kind of answered the above in #openstack-dev | |
| 19:38:26 | darkhorse | @frickler | |
| 19:38:36 | darkhorse | Thank you for your information. | |
| 19:38:55 | darkhorse | How can I verify if the bug exists in master branch? | |
| 22:22:47 | melwitt | stephenfin: ack, I never noticed that before with argparse in general. looking at that stackoverflow, I believe I've seen the if len(args.things) > 1: raise error saying no repeats method before for handling that scenario | |
| 22:23:27 | melwitt | but it's good to know this is "expected behavior" from argparse | |
| #openstack-nova - 2022-10-21 | |||
| 07:14:33 | opendevreview | Amit Uniyal proposed openstack/nova stable/yoga: Adds a repoducer for post live migration fail https://review.opendev.org/c/openstack/nova/+/861871 | |
| 07:14:34 | opendevreview | Amit Uniyal proposed openstack/nova stable/yoga: [compute] always set instance.host in post_livemigration https://review.opendev.org/c/openstack/nova/+/861872 | |