Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-10-21
15:19:08 kashyap clarkb: Yeah, I'm actually quickly pushing a test-only patch to DevStack to see where it fails. If you haven't already done so
15:19:45 clarkb kashyap: remote: https://review.opendev.org/c/openstack/devstack/+/815020 Use Nehalem CPU model by default I just pushed that to have the CI system check things. But thats because I'm in meetings and don't have a current devstack install anywhere to check with
15:20:09 kashyap Ah-ha, thank you
15:20:18 kashyap clarkb: I don't have it either right now. But:
15:20:42 kashyap clarkb: I have tested today outside of DevStack: Nehalem model works with virt_type=qemu *and* with Intel and AMD
15:21:24 clarkb kashyap: ok, ya I suspect the question becomes whether or not the CPUs we get in the host VMs are new enough to support Nehalem
15:22:01 clarkb since they too get virtual capabilities to enable live mgiration in clouds and if they were too conservative we'll have problems. The good news if there is any is that they won't be able to boot centos/rhel 9 either so clouds seem likely to update them
15:22:19 clarkb we can recheck 815020 if it generally works to get it to run across a bunch of clouds and double check that
15:24:38 kashyap clarkb: Nehalem is nearly 13 years old ... I hope there are no such hosts :
15:26:00 clarkb kashyap: the issue is the clouds provide virtual CPUs with custom models too
15:26:13 clarkb because they want to do live mgiration. And if they were too conservative you have this problem. I don't think the actual CPUs are that old
15:26:23 clarkb but the CPUs we get in our instances may be
15:27:34 kashyap Right; I see what you mean. Even if they are conservative w/ the virtual CPUs, I'd be really surprised (wouldn't be the first time) if they're more conservative than Nehalem
15:29:50 kashyap clarkb: Bad me. I was _wrong_ earlier on my stupid "inference" about Fedora: Fedora *did not* switch the baseline ABI to x86-64-v2
15:29:54 kashyap So we can rule that out
15:30:56 clarkb ok good, our problems for booting are differen then :)
15:32:25 kashyap Yeah; grr, now to find out the actual cause
15:32:31 kashyap Do you have the boot console log?
15:33:36 clarkb kashyap: not right now. It is something I can probably dig into tomorrow morning if you want to dive into it (have ptg stuff now then real world school meeting stuff after and by the time I'm done you should be enjoying your evening)
15:34:06 clarkb kashyap: also ianw is interested in the fedora 34 issue and his timezone overlap might be better? I guess it depends on how much of a morning person you are :)
15:34:24 kashyap No problem at all
15:34:52 kashyap Go handle what you need to. This can wait. And yes, will check with ianw. I'm in CEST; he's in Australia...so there should be some overlap :)
15:35:03 clarkb sounds good, thanks again!
15:56:00 gibi stephenfin, artom: another way to test is to move heat from novaclient to sdk ;)
15:56:15 artom gibi, you're an evil, evil man
15:56:30 stephenfin gibi: or tempest. That uses novaclient under the hood, I assume rather than subprocessing to the shell?
15:56:49 artom stephenfin, no, tempest reimplements all the client stuff from scratch in Python in-tree
15:56:57 stephenfin oh, so it does
15:56:57 artom By design
15:58:11 artom All I was saying is - we need to make https://opendev.org/openstack/openstacksdk/src/branch/master/.zuul.yaml#L116 work with 2 nodes do we can test how sdk does the client-side stuff for migrations and such
15:58:13 sean-k-mooney by the way when i was mentioning tempst i ment recreeate teh senario test using sdk in the sdk func test
15:58:17 tosky but in general for non testing stuff gibi's proposed solution is correct (assuming the sdk implementation of orchestration is complete enough)
15:58:17 gibi artom: neutron just realized that they have to do the same switch in heat
15:59:18 sean-k-mooney gibi: well heat need to do it rather then neutron but im sure it will be the same people in either case
15:59:50 gibi sean-k-mooney: except if neutron people want heat support for new neutron features where the only client support is in sdk :)
16:00:10 gibi anyhow starting the transition does not seem scarry https://review.opendev.org/c/openstack/heat/+/813425
16:00:22 sean-k-mooney gibi: but do they want that support :)
16:00:28 stephenfin gibi: artom: I hate to be that guy but we also need to move our internal use of cinderclient, glanceclient and neutronclient to SDK at some point :)
16:00:36 stephenfin I think ironicclient was done a few cycles ago
16:00:41 sean-k-mooney gibi: you are assuming neutron care about heat
16:01:04 sean-k-mooney they proably do but maybe not
16:01:10 stephenfin wait, no, there's still ironicclient. It's just not a mandatory import
16:01:13 stephenfin *requirements
16:01:28 artom stephenfin, yeah...
16:01:55 artom Like, I'm taking part in this debat, but who am I kidding, I won't be the one doing the work - I tried to start, then just ran out of steam
16:02:29 gibi stephenfin: ouch, you are right :)
16:02:47 artom OTOH, what's kind of annoying is that the problem that I'm fixing in https://review.opendev.org/c/openstack/openstacksdk/+/741688 is still a problem.
16:02:57 artom image is *still* optional, when it isn't in the PAI
16:02:59 artom *API
16:03:50 artom And it's *still* missing the kwargs things for all the other params that are in the API
16:04:02 stephenfin artom: I should have looked at that. My bad :( If you got time to rebase it onto the feature/r1 branch (which will be merged into master soon enough) I'll review it in the AM
16:04:19 artom stephenfin, I need to fix the unit test below it and rebase
16:04:31 artom stephenfin, it's just such a slog, and I'm lazy and easily distracted
16:04:42 stephenfin you should try cocaine
16:04:50 artom Speaking from experience?
16:04:58 stephenfin all the bankers I know swear by it
16:05:29 artom Explains the state of the financial system
16:07:53 gibi happy hours already?
16:31:24 sean-k-mooney am are we goint to call it a day or do we want to do the pain points discussion
16:31:29 sean-k-mooney since we got time back
16:31:54 sean-k-mooney bauzas: ^
16:31:56 bauzas sean-k-mooney: I prefer to leave early
16:32:18 sean-k-mooney ok
16:32:38 bauzas sean-k-mooney: I feel we can make all the agenda by tomorrow
16:33:04 bauzas we have 7 topics left
16:33:11 sean-k-mooney well we might but there is also tc sesssions tomorow
16:33:25 sean-k-mooney so we might not have quorm for the full day
16:33:39 sean-k-mooney but yes we likely can finish tomorow
16:34:11 bauzas sean-k-mooney: you're right, we're constrainted by the big TC RBAC thing
16:34:43 bauzas sean-k-mooney: but I feel we can postpone a few topics if we really need
16:35:10 sean-k-mooney as a last resort yes but in general we shoudl try to avoid that
16:35:40 bauzas agreed
16:35:47 bauzas I'll do a timekeeping thing
16:36:36 bauzas and try to not exceed 30 mins per topic
16:37:17 sean-k-mooney ok im going to step away for a few minutes. i was still using my wired headset today since i did not find my wireless one this morning so i have slight headach anyway.
16:38:01 sean-k-mooney it went away after 20 mins of not wareing it yesterday so hopefully the same will hapen today.
16:38:21 bauzas sean-k-mooney: yeah that's also why I wanted to stop earlier
16:38:26 bauzas we were not in the room
16:38:33 bauzas asking people to rejoin was an effort
16:38:50 bauzas so it would have meant 1 topic to discuss
16:39:16 sean-k-mooney yep getting momentum back is hard
16:41:01 bauzas sean-k-mooney: I see the TC discussion around RBAC occuring at 1:30pm until 3pm
16:41:48 bauzas sean-k-mooney: I accordingly flipped topics in the agenda
16:42:12 sean-k-mooney ack
17:09:42 opendevreview Balazs Gibizer proposed openstack/nova master: Add a WA flag waiting for vif-plugged event during reboot https://review.opendev.org/c/openstack/nova/+/813419
17:15:14 gibi artom: I've fixed up your comments ^^
17:17:20 artom gibi, *looks* you have some more asserts in the test that are... unrelated? Like I don't know how anal we want to be about this, but it's really only the last one we care about
17:19:08 gibi artom: I can drop the other asserts
17:19:26 gibi I'm also not sure about our strategy in these tests
17:19:45 gibi I admit I copied a previous test and modified that hence the bigs cope
17:19:48 artom Our unit tests are overly tied to the implementation and confusing? Say it ain't so ;)
17:20:35 gibi I don't like our unit tests either :)
17:20:36 artom gibi, yeah, I figured that was the case :) I think in this situation, with the code being what it is, what you have is OK
17:20:53 artom Well, minus the extraneous asserts
17:29:47 opendevreview Balazs Gibizer proposed openstack/nova master: Add a WA flag waiting for vif-plugged event during reboot https://review.opendev.org/c/openstack/nova/+/813419
17:30:40 gibi artom: ^^
17:32:04 artom gibi, cool, thanks for your patience :)
17:32:28 gibi artom: no worries. I do want to have nice unit tests so at least lets have the new ones nicer
17:35:33 sean-k-mooney i think the unit tests we write as small local tests are nice
17:36:10 sean-k-mooney but some of them are close to funcitonal test then unit

Earlier   Later