Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-09-24
14:40:15 mriedem it depends on what you care about,
14:40:48 mriedem the immediate issue is failing scheduling b/c of the DiskFilter or claim (by having DiskFilter enabled) and booting a volume-backed server - that is fixed with dan's fix above and disabling the DiskFilter since placement won't filter on DISK_GB
14:41:10 mriedem the resource tracker should also not be counting volume-backed server usage against the compute node (hypervisors API) since rocky
14:41:27 mriedem so i'm not sure what is *not* fixed in rocky
14:42:34 sean-k-mooney right. the only thing i can think of is that we dont default to safe regarding os_compute_api:servers:create:zero_disk_flavor
14:42:46 sean-k-mooney but that is for backwards comaptablity in rocky
14:56:01 openstackgerrit Merged openstack/os-resource-classes master: Update bug link in docs to point to storyboard https://review.opendev.org/681464
15:02:09 openstackgerrit Artom Lifshitz proposed openstack/nova master: Move pre-3.44 Cinder post live migration test to test_compute_mgr https://review.opendev.org/683597
15:02:24 openstackgerrit Merged openstack/os-resource-classes master: Build pdf docs https://review.opendev.org/681463
15:13:38 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Use the "memory" mode 'preferred' for a guest NUMA node https://review.opendev.org/684375
15:14:13 kashyap sean-k-mooney: ^ In your "copious spare time"
15:14:34 kashyap sean-k-mooney: Shouldn't we consider migration here: if an instance from a Compute node with 'strict' is being migrated to a host with 'preferred' - possible explosions?
15:15:01 kashyap sean-k-mooney: Err, that main commit summary is nonsense, let me fix that
15:15:06 sean-k-mooney yes this would need to be taken into account
15:15:38 sean-k-mooney but if we hardcore preferred then it not an issue excapt on upgrde
15:15:49 sean-k-mooney it would be fixed by a hard reboot
15:16:54 sean-k-mooney i would not nessisarly require this to be handeled in live migration given that numa live migration was not really correct until train
15:19:18 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: libvirt: Use the `preferred` memory allocation mode for NUMA https://review.opendev.org/684375
15:19:31 aspiers kashyap: "pfalsh" typo in https://libvirt.org/formatdomaincaps.html#elementsOSBIOS
15:19:47 kashyap aspiers: Ah, you saw the libvirt patch I Cced you on?
15:19:50 aspiers yes
15:20:04 kashyap aspiers: Will fix; it's merged already :D
15:20:11 kashyap Thanks for spotting; feel free to reply there
15:20:25 aspiers kashyap: So I saw. Thanks - don't think I'm subscribed to that list
15:20:46 kashyap sean-k-mooney: Ah-ha; good point on "NUMA + live migration not really being a 'thing' until Train"
15:20:47 aspiers kashyap: While you're at it, you could add some helpful hyperlinks
15:21:13 sean-k-mooney kashyap: well it was a thing but it had a load of edgecases
15:21:15 kashyap aspiers: You don't need to be subscribed; just reply they moderators will (usually, quickly) take care
15:21:27 sean-k-mooney so i dont really considre this edgecase to be important
15:21:40 sean-k-mooney at least not relitive to the other ones that were not handeled
15:21:45 kashyap sean-k-mooney: So, an upgrade note is required, too, along with the 'fixes' note I added -- isn't it?
15:21:56 aspiers kashyap: I'd rather not get involved - too much other stuff to deal with :)
15:21:58 kashyap aspiers: Mind pointing out which ones?
15:22:03 kashyap aspiers: Heh, I hear ya
15:22:16 aspiers kashyap: Here are some suggested edits
15:22:18 aspiers "For the loader element, the following can occur:"
15:22:28 aspiers "the following elements can occur:"
15:22:36 kashyap (Oh, I wanted to fix that, too! Realized it too late)
15:22:43 aspiers but then you list "value" alongside "type" etc. which is incorrect
15:22:55 aspiers type/readonly/secure are values of the name attribute of <enum>
15:22:56 sean-k-mooney we could update the numa migration code in train to also update it to prefered
15:22:59 aspiers but you don't even mention <enum>
15:23:03 sean-k-mooney but i would do that as a seperate patch
15:23:44 aspiers and the confusion over the point of <enum> was the whole reason we started looking at this in the first place
15:24:15 aspiers kashyap: ^^^
15:25:00 kashyap aspiers: Okay, my brain is completely exhausted for the moment; will make a note to get back to this and submit a patch
15:25:03 kashyap Thanks!
15:25:05 aspiers kashyap: "For cases like that, os element exposes ..." is ambiguous about whether it's <os> in domCaps or caps
15:25:12 aspiers which again was the crux of the confusion
15:25:28 aspiers These docs need to be crystal clear about when they are referring to elements of domCaps vs. caps
15:25:50 aspiers so that the parallel between the two is understood
15:26:33 kashyap aspiers: I didn't touch those bits; but you're right, they _do_ need "less careless" documentation. I'll get back to this tomm / this week
15:26:45 aspiers kashyap: thanks! one more pointer
15:26:49 kashyap Non-null?
15:26:53 kashyap :D
15:27:06 aspiers kashyap: I'm suggesting to hyperlink text like "the <loader/> element in the domain XML"
15:27:20 aspiers should link to https://libvirt.org/formatdomain.html#elementLoader
15:27:29 aspiers in both directions, ideally
15:27:39 aspiers kashyap: feel free to send me something to review
15:27:43 aspiers when you have a chance
15:29:29 kashyap aspiers: Will definitely do.
15:29:53 aspiers thanks again!
15:30:48 kashyap You're welcome again
15:33:06 kashyap sean-k-mooney: Yeah, indeed.
15:34:10 kashyap Sean, sorry, disregard that; was replying to a stale message, as had the IRC buffer scrolled up.
15:48:22 mlavalle gmann: hi, I have some questions regarding http://specs.openstack.org/openstack/nova-specs/specs/train/approved/policy-default-refresh.html. when you have some time, please ping me
16:02:58 openstackgerrit Stephen Finucane proposed openstack/nova master: Add missing parameter https://review.opendev.org/682886
16:02:58 openstackgerrit Stephen Finucane proposed openstack/nova master: Use compression by default for 'SshDriver' https://review.opendev.org/684393
16:03:56 stephenfin mriedem: Seeing as we're on a docs roll, could you take a look at https://review.opendev.org/#/c/671298/, please? Maybe efried or gibi too
16:05:37 gibi stephenfin: done
16:05:47 mriedem hmm
16:07:21 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Document global options for nova-manage https://review.opendev.org/677443
16:07:21 openstackgerrit Stephen Finucane proposed openstack/nova master: config: Explicitly register 'remote_debug' CLI opts https://review.opendev.org/677444
16:07:21 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP: docs: Rewrite nova-manage docs to use proper directives https://review.opendev.org/677509
16:12:15 shilpasd mriedem: hi, to verify performance for changes https://review.opendev.org/#/c/612626/ Ignore root_gb for BFV in simple tenant usage API, i have written test program http://paste.openstack.org/show/779196/, please go through once and share any suggestions/corrections if any
16:12:52 mriedem shilpasd: you should paste that into the patch and/or the bug report
16:13:04 shilpasd ok, will do that
16:13:32 shilpasd mriedem: thanks, will paste along with test results
16:15:05 dansmith stephenfin: comment on that db sync one
16:15:32 dansmith stephenfin: If you just did what you said in the commit message, I'd be good, but I feel like you're regressing the verbiage in there
16:15:50 dansmith maybe mriedem has an opinion
16:16:00 dansmith I mean, I bet he does, but maybe he'll tell us what it is
16:17:17 stephenfin makes sense. I can respin
16:17:18 mriedem dansmith: i just said the same thing :)
16:17:29 mriedem and said "maybe dan has an opinion on this"
16:17:37 dansmith heh nice
16:18:48 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Note use of 'nova-manage db sync --config-file' https://review.opendev.org/671298
16:18:49 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Correct 'nova-manage db sync' documentation https://review.opendev.org/677508
16:18:49 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Document global options for nova-manage https://review.opendev.org/677443
16:18:50 openstackgerrit Stephen Finucane proposed openstack/nova master: config: Explicitly register 'remote_debug' CLI opts https://review.opendev.org/677444
16:18:50 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP: docs: Rewrite nova-manage docs to use proper directives https://review.opendev.org/677509
16:22:24 stephenfin dansmith, mriedem: okay, took what dansmith said and made it a thing ^
16:22:47 dansmith looks okay to me, but mriedem was more specific about details, so I'll let him ack it first
16:27:24 mriedem you and gibi can approve
16:50:29 ozzzo after rebuilding controllers in my dev cluster, is is possible to re-register the existing HV, or do I need to rebuild them?
16:51:12 openstackgerrit Merged openstack/nova master: objects: use all_things_equal from objects.base https://review.opendev.org/681397
16:51:18 openstackgerrit Merged openstack/nova master: Func test for migrate reschedule with pinned compute rpc https://review.opendev.org/683385
16:51:25 openstackgerrit Merged openstack/nova master: Add evacuate vs rebuild contributor doc https://review.opendev.org/683245
16:51:59 artom mriedem, so, I think I figured out the NUMA LM thing
16:52:56 artom We filter out 'accepted' migrations in https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L4422

Earlier   Later