| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-09 | |||
| 13:30:51 | mriedem | brinzhang: can you rebase https://review.opendev.org/#/c/673485/12 on https://review.opendev.org/#/c/670790/8 ? | |
| 13:30:53 | mriedem | there is a merge conflict | |
| 13:32:25 | brinzhang | mriedem: Today, I was rebased it, now has a merge conflict? | |
| 13:33:05 | mriedem | it has to be rebased on the latest patch set of 670790 | |
| 13:33:25 | sean-k-mooney | mriedem: by the way i configured the jobs to run in expermental and perodic since we only have 1 provider but would you be open to adding it as non voting in check at some point. non voiting jobs dont break the pipeline if they cant run because the provider is down. | |
| 13:33:34 | brinzhang | mriedem: Okay, I will reabse it now. | |
| 13:33:44 | ganso | hello folks! I am trying to use "openstack usage list" which according to this commit https://review.opendev.org/#/c/639652/2/openstackclient/compute/v2/usage.py should be able to handle microversions 2.40, but using --debug I see it is always using 2.1. Is there any way to force openstack CLI to use nova microversions 2.40? My OpenStack CLI is version 3.19 | |
| 13:34:23 | mriedem | sean-k-mooney: maybe | |
| 13:34:29 | artom | ganso, openstack --help, you'll see an option named something like --os-compute-api-version | |
| 13:34:46 | artom | ganso, osc defaults to using the lowest possible microversion, aka 2.1 | |
| 13:35:01 | ganso | artom: thanks! | |
| 13:35:01 | sean-k-mooney | mriedem: ok well im beign conservitve and wanted to get some confidence form the periodic job before proposing that. | |
| 13:35:25 | artom | ganso, also, in the future, please direct similar questions to #openstack (see /topic) | |
| 13:35:32 | mriedem | the help on that osc command should be updated to mention using 2.40 or above for paging otherwise the user would have no way of knowing that w/o looking at code | |
| 13:36:16 | ganso | artom: will do! thank you very much! | |
| 13:36:18 | ganso | mriedem: +1 | |
| 13:37:41 | openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Microversion 2.79: Add delete_on_termination to volume-attach API https://review.opendev.org/673485 | |
| 13:38:41 | brinzhang | mriedem: Done of that reabased. | |
| 13:39:27 | luyao | dansmith: Hi Dan, could you have a look again at https://review.opendev.org/#/c/678447/ and https://review.opendev.org/#/c/678448/, about 'resource' db and object. I'll appreciate that . | |
| 13:55:52 | stephenfin | If I cold migrate an instance but don't confirm it, I should expect to see VCPU usage for the instance on both the source and destination, right? | |
| 13:56:48 | bauzas | stephenfin: what's your concern ? | |
| 13:56:53 | bauzas | allocations, you mean ? | |
| 13:57:00 | stephenfin | yeah | |
| 13:57:04 | bauzas | if so, yeah | |
| 13:57:04 | stephenfin | allocations | |
| 13:57:12 | stephenfin | an allocation for the instance, an allocation for the migration record | |
| 13:57:16 | bauzas | correct | |
| 13:57:23 | artom | stephenfin, you mean based on other existing behaviour? | |
| 13:57:35 | artom | I believe so - the instance is consuming resources on the source, the migration on the dest | |
| 13:57:41 | stephenfin | sweet | |
| 13:58:00 | bauzas | the original allocation is transformed to have the consumer to be the migration UUID instead of the instance | |
| 13:58:18 | bauzas | and then we create a new allocation for the instance | |
| 13:58:32 | bauzas | having the consumer be the instance UUID | |
| 13:58:45 | bauzas | artom: the other way | |
| 13:59:02 | artom | bauzas, I was talking about the old-school resources tracker | |
| 13:59:06 | stephenfin | next question: I thought by default a cold migration would never land on another host, but that's not what I'm seeing in functional tests | |
| 13:59:13 | artom | ... which I guess stephenfin no longer cares about? | |
| 13:59:24 | bauzas | artom: stephenfin: https://specs.openstack.org/openstack/nova-specs/specs/queens/implemented/migration-allocations.html | |
| 13:59:58 | artom | stephenfin, other way around, I think | |
| 14:00:11 | artom | By default it always goes to different host | |
| 14:00:19 | bauzas | correct ^ | |
| 14:00:20 | stephenfin | artom: yeah, sorry, never land on the _same_ host | |
| 14:00:27 | artom | And then if you set the allow_resize_to_same_host option, it can land on the same host | |
| 14:00:30 | bauzas | unless you modify an option $ | |
| 14:00:35 | bauzas | yeah what artom said | |
| 14:00:36 | artom | (Or whatever it's called) | |
| 14:00:50 | bauzas | artom: that is the correct option name | |
| 14:02:01 | stephenfin | weeird | |
| 14:02:11 | stephenfin | there must be something else going on here so | |
| 14:02:18 | artom | stephenfin, so, dunno if this is relevant, but while working on the the NUMA LM func tests, I noticed there was an issue in how you started computes with different fakelibvirt connections for each | |
| 14:02:26 | stephenfin | shoot | |
| 14:02:47 | artom | stephenfin, https://review.opendev.org/#/c/672595/45/nova/tests/functional/libvirt/base.py | |
| 14:03:14 | artom | Again, no idea if that's what's causing your thing now, but could be relevant | |
| 14:04:01 | stephenfin | oh, fun | |
| 14:12:55 | stephenfin | artom: The other approach is this one https://review.opendev.org/#/c/678470/24/nova/tests/unit/virt/libvirt/fakelibvirt.py | |
| 14:13:11 | stephenfin | which I think accomplishes much the same thing | |
| 14:13:35 | stephenfin | there are quite a few in-flight changes with those files atm :) | |
| 14:15:09 | openstackgerrit | Brin Zhang proposed openstack/nova master: Follow-up: Add delete_on_termination to volume-attach API https://review.opendev.org/680655 | |
| 14:15:28 | artom | stephenfin, and then you stub that connection into each compute manager I guess? | |
| 14:15:53 | artom | Oh, that's not your patch | |
| 14:15:54 | artom | Hrmm | |
| 14:17:02 | kashyap | aspiers: Any reason you know this method is in virt/hardware.py -- _get_flavor_image_meta()? It can be used by "non-hardware"-related features, too | |
| 14:17:15 | aspiers | kashyap: git blame? | |
| 14:17:17 | kashyap | Maybe a question for stephenfin, as he wrote that :-) | |
| 14:17:37 | kashyap | aspiers: Already did; 48e0c000fefe561ec5cfa1364ef2cac2f6ce9244 | |
| 14:18:37 | kashyap | aspiers: stephenfin: Actually, disregard...it makes sense, because of the 'hw' key :D | |
| 14:35:19 | dansmith | mriedem: assume you'll take another trip through the numa patches today? I'm about to start into the forbidden garden of aggregates | |
| 14:35:53 | bhagyashris | mriedem: Hi | |
| 14:36:30 | mriedem | dansmith: i was going over them over the weekend but stopped short of the big one | |
| 14:36:38 | mriedem | waiting on ci results in the special job | |
| 14:36:42 | dansmith | ah cool | |
| 14:37:00 | mriedem | https://review.opendev.org/#/c/635229/ has new wizardry that is going to require stephenfin and/or sean-k-mooney | |
| 14:42:33 | sean-k-mooney | mriedem: is that the patch you had question relating to the xml generation on | |
| 14:44:36 | sean-k-mooney | mriedem: if there is a specific thing you wanted me to look at i can do it quickly otherwise im going to finish with stephenfin series then start on artoms | |
| 14:45:46 | artom | sean-k-mooney, the new allocation_cpu_tuning, to cover the case when an instance with no NUMA topology is on a host with vcpu_pin_set | |
| 14:45:59 | sean-k-mooney | ah right | |
| 14:46:16 | sean-k-mooney | this https://review.opendev.org/#/c/635229/60/nova/virt/libvirt/migration.py@101 | |
| 14:46:21 | artom | So actually the field is new in https://review.opendev.org/#/c/634827/ | |
| 14:46:47 | artom | sean-k-mooney, well, PS61 is the latest but yeah | |
| 14:47:20 | dansmith | efried: shilpasd: I found something else on the base patch for forbidden aggregates | |
| 14:47:45 | dansmith | I'm pretty sure we don't deepcopy() the default value, which is why I did default=None for the "aggregates" field right above it. | |
| 14:48:11 | dansmith | either way, it should be consistent I think, and I expect =None is more clearly safe than a default of something mutable (like mutable default arguments for methods in python) | |
| 14:48:41 | sean-k-mooney | mriedem: artom so the context for this is that since we added the vcpu_pin_set option, if you have used it we have relied on operators ensuring that all host in the same host aggrate/availablity zone (depending on how you choose to defien the region you can live migrate vm within) has vcpu_pin_set set to the same value | |
| 14:49:13 | shilpasd | dansmith: Thanks for review | |
| 14:49:56 | dansmith | shilpasd: does my comment make sense? | |
| 14:50:10 | dansmith | shilpasd: also, if you're not able to make this change right away I could probably just do it myself | |
| 14:50:18 | sean-k-mooney | mriedem: more specifically if you define vcpu_pin_set we copy the value into <vcpu cpuset=vcpu_pin_set> for all instnace but we never updated that on live migration | |
| 14:50:39 | shilpasd | dansmith: yes, your comment make sense, will have test cases with default config (i.e. no filter remove or append) and with remove and with append | |
| 14:50:53 | sean-k-mooney | so since artom is fixing it for numa instance we asked him to fix non numa instance too | |
| 14:51:03 | stephenfin | TIL 'functional.libvirt.base.ServersTestBase' uses API v2.0 by default. That's two hours I won't get back \o/ | |
| 14:51:03 | mriedem | sean-k-mooney: the config option help for that says nothing about that assumption/requirement https://docs.openstack.org/nova/latest/configuration/config.html#DEFAULT.vcpu_pin_set | |
| 14:51:30 | dansmith | shilpasd: oh you're talking about the later one | |
| 14:51:40 | dansmith | shilpasd: I +2d that already, so don't change it, and/or change it in a followup | |
| 14:51:53 | dansmith | shilpasd: I _just_ left a more complicated comment on the base patch, so go look at that real quick | |
| 14:52:00 | sean-k-mooney | mriedem: that is the entire orginal reason it was intoduced. to define the set of cpu that guest can run on. it was not just for pinned/numa instance | |
| 14:52:43 | mriedem | sean-k-mooney: ok....that doesn't mean much to me since i'm not familiar with when or why that was added, | |
| 14:52:50 | mriedem | i'm just saying, "if you have used it we have relied on operators ensuring that all host in the same host aggrate/availablity zone (depending on how you choose to defien the region you can live migrate vm within) has vcpu_pin_set set to the same value" is not documented | |
| 14:53:07 | mriedem | so undocumented assertions are kind of....hairy | |
| 14:53:31 | artom | mriedem, yep, which why with this new allocation_cpu_tuning stuff, it won't be necessary | |
| 14:53:47 | artom | Previously we hoped both dest and source had it set the same for LM to work | |