| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-06-28 | |||
| 13:14:07 | sean-k-mooney | the issue with server groups is adding an instance that violates the policy | |
| 13:14:10 | bauzas | just saying this is a can of worms | |
| 13:14:30 | sean-k-mooney | it is | |
| 13:14:38 | bauzas | sean-k-mooney: the problem with server groups is that if we touch it, it creates more races than the ones it fixes | |
| 13:14:42 | sean-k-mooney | but that is a sperate topic form the orginal thing that jkulik raised | |
| 13:15:00 | bauzas | sean-k-mooney: well, this is all about colocality | |
| 13:15:12 | bauzas | we express this in Nova with server groups | |
| 13:15:23 | sean-k-mooney | well affinity and anti affity | |
| 13:15:27 | sean-k-mooney | with differnt granularity | |
| 13:15:29 | sean-k-mooney | but yes | |
| 13:15:52 | bauzas | but the question remains about the best UX we may have | |
| 13:18:24 | bauzas | anyway, me goes back at bug scrub | |
| 14:29:45 | Uggla | question, from the api, I try to check that a scheduling is impossible. To do that I look at 'No valid host found for unshelve instance' in the log. My test seems to work, but I have to wait before checking the logs. Is there a proper way to do that ? | |
| 14:30:54 | Uggla | bauzas, gibi ^ | |
| 14:33:03 | sean-k-mooney | using notifications | |
| 14:33:15 | sean-k-mooney | but in general we dont tend to use logs in tests | |
| 14:33:25 | sean-k-mooney | we do sometimes but there are often better way to do that | |
| 14:33:45 | Uggla | is _wait_for_action_fail_completion a possible option ? | |
| 14:43:15 | gibi | Uggla: you wait for the server to go to ERROR state then you can check the fault in the server to see if it is a no valid host | |
| 14:43:22 | gibi | give me a sec and I will find an example | |
| 14:45:23 | gibi | Uggla: for example https://github.com/openstack/nova/blob/c53ec4e48884235566962bc934cbf292ad5b67b8/nova/tests/functional/test_servers.py#L4100-L4108 | |
| 14:45:28 | Uggla | gibi, ok probably better that what I have just done using wait_for_assert(). | |
| 15:07:24 | bauzas | Uggla: not sure I understand your question | |
| 15:07:36 | bauzas | you're asking how nova-api could know there is a scheduling error ? | |
| 15:07:53 | Uggla | bauzas, no worries Gibi is helping me right now. | |
| 15:08:06 | bauzas | oh, for testing | |
| 15:08:41 | sean-k-mooney | yep testing | |
| 15:08:46 | sean-k-mooney | presumable functional testing | |
| 15:08:51 | sean-k-mooney | rather then unit | |
| 15:11:24 | gibi | yepp func testing | |
| 15:11:58 | gibi | Uggla just found the limitation of the our nova.tests.functional.integrated_helpers.InstanceHelperMixin._wait_for_instance_action_event assert | |
| 15:12:18 | gibi | it always checks the first action of a given type from the list of instance actions | |
| 15:12:27 | gibi | and he had two unshelve actions in the test case | |
| 15:12:35 | gibi | and the assert only checked the first | |
| 15:12:43 | gibi | even though the second was in error state | |
| 15:29:27 | bauzas | reminder: nova meeting in 30 mins | |
| 15:31:07 | sean-k-mooney | oh fun | |
| 15:31:16 | sean-k-mooney | i was not aware we had that limitation | |
| 15:40:06 | gibi | me neither, but now Uggla can improve on that :) | |
| 15:42:15 | Uggla | gibi, sean-k-mooney , currently not completely sure but no the code looks ok. It is more the event I'm looking for which is not the good one. | |
| 15:43:38 | gibi | it can be that the listing of the instance actions are not stable so sometimes the code founds the proper unshelve action sometimes not | |
| 15:48:34 | gibi | I mean the sorting is not stable | |
| 16:00:11 | opendevmeet | Meeting started Tue Jun 28 16:00:11 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:11 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:11 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:00:11 | bauzas | #startmeeting nova | |
| 16:00:15 | bauzas | hello everyone | |
| 16:01:09 | bauzas | mmmm | |
| 16:01:21 | bauzas | heyhoooooooo | |
| 16:02:00 | gibi | o/ | |
| 16:02:19 | bauzas | hah, I hear some sound | |
| 16:02:30 | gibi | crickets ? | |
| 16:02:42 | bauzas | I'm maybe in a cave | |
| 16:03:22 | bauzas | gibi: not sure we have quorum for today's meeting :( | |
| 16:03:23 | elodilles | o/ | |
| 16:03:37 | gibi | sean-k-mooney already gone for today | |
| 16:03:58 | gibi | I guess melwitt is still on PTO | |
| 16:03:59 | bauzas | hah | |
| 16:04:05 | bauzas | yes indeed + artom | |
| 16:04:30 | bauzas | what would you want ? | |
| 16:05:03 | gibi | do we have somebody here for the todays nova meeting to talk about specific things? (if not then we can close this) | |
| 16:05:39 | bauzas | we still have one critical bug | |
| 16:05:41 | elodilles | nothing special news from stable point of view, so - | |
| 16:06:05 | bauzas | I can be the bug baton owner for the next week | |
| 16:06:28 | bauzas | that said, next week we will have a spec review day | |
| 16:06:38 | bauzas | I'll email it | |
| 16:06:49 | bauzas | that's it for me | |
| 16:07:21 | gibi | nothing from me I spent most of the week downstream | |
| 16:07:28 | bauzas | #info Next bug baton is still for bauzas | |
| 16:07:37 | bauzas | #info One Critical bug | |
| 16:07:46 | bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 8 new untriaged bugs (-4 since the last meeting) | |
| 16:07:51 | bauzas | #link https://storyboard.openstack.org/#!/project/openstack/placement 27 open stories (+1 since the last meeting) in Storyboard for Placement | |
| 16:07:54 | bauzas | #link https://storyboard.openstack.org/#!/story/2010108 new Placement bug | |
| 16:08:01 | bauzas | but I'll tell it again next week | |
| 16:08:17 | bauzas | #info Spec review day on July 5th | |
| 16:08:32 | bauzas | that's basically it | |
| 16:08:40 | gibi | I guess in the critical bug we keep the job non-votig while waiting for the fix to be released in centos stream 9 | |
| 16:08:55 | bauzas | gibi: yeah, we'll discuss about it next week | |
| 16:08:59 | gibi | ok | |
| 16:09:07 | bauzas | I have a concern about the centos 9 stream job | |
| 16:09:07 | gibi | then I think we can close early today | |
| 16:09:13 | bauzas | ok, then | |
| 16:09:42 | bauzas | #agreed given we don't have quorum for this meeting, let's punt it for this week until next week | |
| 16:09:59 | bauzas | #info remember we'll have a spec review day next week | |
| 16:10:08 | bauzas | that's it, thanks | |
| 16:10:10 | bauzas | #endmeetingh | |
| 16:10:12 | bauzas | meh | |
| 16:10:14 | bauzas | #endmeeting | |
| 16:10:14 | opendevmeet | Meeting ended Tue Jun 28 16:10:14 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:10:14 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-06-28-16.00.html | |
| 16:10:14 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-06-28-16.00.txt | |
| 16:10:14 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-06-28-16.00.log.html | |
| 16:10:17 | bauzas | even | |
| 16:11:32 | gibi | have a nice evening folks | |
| 16:11:39 | elodilles | you too o/ | |
| 16:15:30 | Uggla | bauzas, already done ! | |
| 16:15:51 | bauzas | Uggla: yeah we didn't had a quorum | |
| 16:16:16 | Uggla | oh ok. | |
| 16:27:35 | opendevreview | Merged openstack/osc-placement master: Replace deprecated assertRaisesRegexp https://review.opendev.org/c/openstack/osc-placement/+/817365 | |
| 16:55:51 | bauzas | gibi: sean-k-mooney: Uggla: others: I forgot to tell I'll be on PTO tomorrow | |