| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-10-24 | |||
| 13:02:51 | sean-k-mooney | baiscally because of a data integrety, power usage and encypted voluems we felt it would be better to alwasy evacuate as off | |
| 13:03:37 | sean-k-mooney | since evacuate is an admin only opeartion its not reasonable to assume the admin knows if its safe to restart the workload or not escially since there may be a posiablity of data curruption depending on howt eh host fails | |
| 13:04:04 | sean-k-mooney | so if we alwasy evacuate to off then the enduser who should know can decied if they want to start, rescue or delete the instace | |
| 13:05:02 | sean-k-mooney | sahid: for the encypted volume case it would be niceif you could handel and ignore the excption we get during spawn but that can be a follow up patch | |
| 13:05:48 | sean-k-mooney | sahid: evenutlaly we woudl like to split up rebuild/evacuate int 3 steps internally. 1 stop the instace, 2 rebuild the disk, 3 start it if requried. | |
| 13:09:52 | sahid_ | sean-k-mooney: if I understand, making it stop by default (the current spec), then in future have the internal clean that you are mentionning, and finally give users ability to choice active/stop, right? | |
| 13:11:40 | sean-k-mooney | no we dont want to give user a choice explictly. if they want the old behaivor they can use the old microverion but going forward we want evacuate to always mean evacuate to powerered off with the new micorversion | |
| 13:12:08 | sean-k-mooney | well its exiplcit in that its contorlled by the microversion but we dont want a new api parmater for it | |
| 13:12:39 | sean-k-mooney | so short term (A cycle) just allow evacuate to powered off | |
| 13:13:31 | sean-k-mooney | medeium term bug fix to not try and start the vm when the intended state is off as a non backporatble change to all drivers | |
| 13:14:04 | sean-k-mooney | that can be done in the A or later cycles | |
| 13:14:56 | sean-k-mooney | sahid_: does that sound ok to you. | |
| 13:26:21 | sahid_ | sean-k-mooney: yes understood, sounds good | |
| 13:26:38 | sahid_ | thank you for your help on it, I will update the spec to reflect that | |
| 13:27:02 | sahid_ | also open an issue | |
| 13:27:08 | sean-k-mooney | cool once updated ill be happy to re review | |
| 13:27:14 | sahid_ | ++ | |
| 13:28:53 | sahid_ | are we agree with that I will still have to change the RPC and API to pass a targetState that will be based on the microversion, right? | |
| 13:29:59 | sean-k-mooney | yep | |
| 13:30:10 | sean-k-mooney | so you can keep all that code that you already wrote | |
| 13:30:33 | sean-k-mooney | its jsut that instead of having a parmater in the payload to set targetState | |
| 13:30:41 | sean-k-mooney | you will set it based on the micoversion | |
| 13:30:59 | sean-k-mooney | and with the new micorverison it will hardoced to off | |
| 13:31:21 | sean-k-mooney | with the old micorversion you dont need to pass it at all | |
| 13:31:24 | sahid_ | yeah i understand the whole process, perfect | |
| 14:22:32 | bauzas | (just typing here again, I was in the wrong chan) | |
| 14:22:39 | bauzas | man, my fingers bleed | |
| 14:22:48 | bauzas | just a bit of warning, I tried to summarize as much as I can for the PTG recap, but this will be a very long thread | |
| 14:22:58 | bauzas | I'm at 80% of the writing but I need to caffeinate | |
| 14:26:40 | sean-k-mooney | :) | |
| 14:26:52 | sean-k-mooney | well hopefully that means it was productive | |
| 17:26:35 | dansmith | bauzas: we already have --purge for archive that does the purge immediately, right? | |
| 17:27:51 | sean-k-mooney | dansmith: yep | |
| 17:28:04 | sean-k-mooney | dansmith: but we would like to have --deleted for pruge | |
| 17:28:22 | sean-k-mooney | so we can eventrully remove the use of arcive and make our purge work like all the opter project | |
| 17:28:46 | sean-k-mooney | the other service dont have shadow tables so there perge remove the deleted rows | |
| 17:28:49 | dansmith | so instead of archive --purge, purge --deleted ? | |
| 17:29:07 | sean-k-mooney | yep basically that | |
| 17:29:45 | sean-k-mooney | thats a lower priority item but purge everywhere else is the command to remove the deleted rows | |
| 17:29:53 | dansmith | meh, but okay | |
| #openstack-nova - 2022-10-25 | |||
| 02:16:42 | opendevreview | shiyawei proposed openstack/nova master: fix bug: 1994018, the volume multiattach and in-use after retype another backend, then can not detach it Closes-Bug#1994018 https://review.opendev.org/c/openstack/nova/+/862355 | |
| 06:01:39 | opendevreview | shiyawei proposed openstack/nova master: fix bug: 1994018, the volume multiattach and in-use after retype another backend, then can not detach it Closes-Bug#1994018 https://review.opendev.org/c/openstack/nova/+/862355 | |
| 06:34:47 | opendevreview | shiyawei proposed openstack/nova master: fix bug: 1994018, the volume multiattach and in-use after retype another backend, then can not detach it Closes-Bug#1994018 https://review.opendev.org/c/openstack/nova/+/862355 | |
| 06:36:55 | opendevreview | shiyawei proposed openstack/nova master: fix bug: 1994018, the volume multiattach and in-use after retype another backend, then can not detach it Closes-Bug#1994018 https://review.opendev.org/c/openstack/nova/+/862355 | |
| 11:22:44 | sean-k-mooney | bauzas: you have not filed the bug for porting the clients to sdk yet have you | |
| 11:22:54 | sean-k-mooney | if not im going to file one for neutorn client now | |
| 11:23:38 | bauzas | no I had no time to actionate on the items yet | |
| 11:24:19 | sean-k-mooney | cool ill create one for neutron then to let ralonsoh have a bug to track | |
| 11:24:24 | sean-k-mooney | but i wont do the rest | |
| 11:33:18 | sean-k-mooney | bauzas: im not sure we need anytihng else in https://bugs.launchpad.net/nova/+bug/1994139 | |
| 12:03:48 | opendevreview | Amit Uniyal proposed openstack/nova stable/ussuri: add regression test case for bug 1978983 https://review.opendev.org/c/openstack/nova/+/862603 | |
| 12:03:49 | opendevreview | Amit Uniyal proposed openstack/nova stable/ussuri: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/862604 | |
| 12:28:12 | opendevreview | Amit Uniyal proposed openstack/nova stable/ussuri: add regression test case for bug 1978983 https://review.opendev.org/c/openstack/nova/+/862603 | |
| 12:28:13 | opendevreview | Amit Uniyal proposed openstack/nova stable/ussuri: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/862604 | |
| 12:39:39 | bauzas | sean-k-mooney: bug report works for me | |
| 13:53:51 | bauzas | gibi: I forgot I have to visit a python meetup at 7pm so I'll need to either have a quick meeting today for the nova one, or just asking you to lead it after 15 mins | |
| 13:53:54 | bauzas | works for you ? | |
| 13:54:20 | gibi | works for me | |
| 13:54:33 | gibi | I can take the baton | |
| 13:59:48 | bauzas | gibi: cool | |
| 14:00:06 | gibi | but I also guess we will have a short one today | |
| 14:01:44 | bauzas | gibi: yup, hence me saying it should be quick | |
| 14:02:05 | bauzas | we have some questions about specless blueprints if we want but let's punt them for next week | |
| 14:51:49 | zigo | Nova unit tests are getting stuck, but I have no idea how to find out which one... | |
| 14:51:49 | zigo | Anyone knows about https://bugs.debian.org/1022733 ? | |
| 14:53:19 | zigo | I'm seeing 12 processes "[sh] <defunct>" started by stestr ... :/ | |
| 14:53:30 | zigo | (that's the num of threads in my laptop, FYI) | |
| 14:56:22 | zigo | Oh, looks like nova.tests.unit.privsep.test_utils.SupportDirectIOTestCase.test_supports_direct_io_with_exception_in_open | |
| 15:15:29 | zigo | and nova.tests.unit.privsep.test_utils.SupportDirectIOTestCase.test_supports_direct_io_with_exception_in_write | |
| 15:31:52 | sean-k-mooney | sounds like we are missing a mock | |
| 15:32:11 | clarkb | zigo: stestr writes a log file and it should update that log file when every test starts and when the test ends. I would look for test entries in that log file that have a start entry but no end entry. But sounds like you may have sorted it out | |
| 15:46:49 | bauzas | reminder : nova meeting in 15 mins-ish | |
| 16:00:32 | bauzas | #startmeeting nova | |
| 16:00:33 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:33 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:33 | opendevmeet | Meeting started Tue Oct 25 16:00:32 2022 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:42 | bauzas | hey folks and welcome back from the PTG | |
| 16:00:50 | bauzas | hope the jetlag didn't hurt you | |
| 16:00:59 | bauzas | (jk) | |
| 16:01:24 | elodilles | o/ | |
| 16:01:28 | gibi | o/ | |
| 16:01:34 | bauzas | let's try to make a short meeting, I have some meetup to attend in a short | |
| 16:01:53 | bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:02:05 | bauzas | #topic Bugs (stuck/critical) | |
| 16:02:09 | bauzas | #info No Critical bug | |
| 16:02:13 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 8 new untriaged bugs (+4 since the last meeting) | |
| 16:02:18 | bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | |
| 16:02:30 | bauzas | gibi: fancy taking the baton one more week ? | |
| 16:02:36 | gibi | bauzas: yes, I can | |
| 16:02:43 | gibi | I did one triage last week | |
| 16:02:50 | bauzas | oh | |
| 16:02:51 | gibi | and I see there is some fresh incoming | |
| 16:02:52 | bauzas | just saw | |
| 16:03:02 | bauzas | nothing to tell about the one you triaged ? | |
| 16:03:38 | bauzas | looks not | |
| 16:03:45 | bauzas | #info bug baton is being passed to gibi | |
| 16:03:46 | gibi | not muhc | |
| 16:04:05 | bauzas | #topic Gate status | |
| 16:04:10 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | |
| 16:04:14 | bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | |