| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-03 | |||
| 12:53:34 | bauzas | mriedem: hola | |
| 12:53:59 | sean-k-mooney | fried_rice: any idea if the ibm drivers detach pcidevices from the guest on suspend | |
| 12:54:13 | bauzas | mriedem: I was thinking on cherry-picking https://review.openstack.org/#/c/584204 (I mean, the series) to Queens | |
| 12:54:38 | bauzas | mriedem: you okay with it ? | |
| 12:55:25 | fried_rice | sean-k-mooney: I would only be able to answer for PowerVM, and... It's possible suspend is an operation we don't support. /me checks support matrix... | |
| 12:55:45 | sean-k-mooney | fried_rice: im looking at the intree driver now. | |
| 12:55:51 | mriedem | bauzas: no | |
| 12:56:02 | mriedem | RequestSpec.is_bfv is an rpc api bump | |
| 12:56:05 | mriedem | so not backportable | |
| 12:56:11 | bauzas | oh rightg | |
| 12:56:18 | mriedem | dansmith and i talked about that while he was writing the patch | |
| 12:56:56 | sean-k-mooney | mriedem: i might start working on that docs bug by the way but ill need stephenfin ect to check it since my written expression is not always well valid english :) | |
| 12:57:06 | fried_rice | sean-k-mooney: https://docs.openstack.org/nova/latest/user/support-matrix.html#operation_suspend_driver_powervm | |
| 12:57:24 | mriedem | sean-k-mooney: sure | |
| 12:57:42 | mriedem | i'm also not saying we should copy the libvirt description of those operations into the api, | |
| 12:57:47 | mriedem | if it's not the same behavior across virt drivers | |
| 12:59:23 | sean-k-mooney | mriedem: i agree but we should likely add a note for the different backends. https://docs.openstack.org/nova/latest/user/support-matrix.html#operation_suspend should proably have a note too | |
| 13:00:31 | bauzas | mriedem: for some reason, I missed https://review.openstack.org/#/c/580720/ in the series | |
| 13:00:35 | bauzas | my bad | |
| 13:08:38 | sean-k-mooney | interesting... the xenapi appears to just suspend. both hyperv and vspher end up delegating to there repective hyperviors suspend as a result this apears to only be a thing for libvirt. | |
| 13:12:24 | fried_rice | Anyone from HyperV around? | |
| 13:17:13 | sean-k-mooney | fried_rice: i guess not | |
| 13:17:26 | sean-k-mooney | brb going for lunch/coffee | |
| 13:17:28 | fried_rice | doesn't matter, I found what I needed. | |
| 13:19:29 | openstackgerrit | Merged openstack/nova master: Scrub hw:cpu_model from API samples https://review.openstack.org/588371 | |
| 13:20:48 | fried_rice | stephenfin: Any justification for something like this https://review.openstack.org/#/c/588422/ ? | |
| 13:27:25 | stephenfin | fried_rice: Not that I'm aware of, anyway | |
| 13:27:35 | stephenfin | Purely a nice to hav | |
| 13:27:57 | fried_rice | stephenfin: ight, thanks for the look. I'm not opposed to approving the thing once they fix it, I guess. | |
| 13:28:05 | stephenfin | likewise | |
| 13:28:13 | fried_rice | Rather than saying it's a fin u cannot do (hint) | |
| 13:28:25 | finucannot | noted | |
| 13:28:39 | finucannot | :) | |
| 13:41:48 | openstackgerrit | Chris Dent proposed openstack/nova stable/queens: [placement] Retry allocation writes server side https://review.openstack.org/588569 | |
| 13:41:57 | openstackgerrit | Eric Fried proposed openstack/nova master: [placement] Debug log per granular request group https://review.openstack.org/588350 | |
| 13:42:18 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: Don't filter out sibling sets with one core https://review.openstack.org/588570 | |
| 13:42:19 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: Ensure emulator threads are always calculated https://review.openstack.org/588571 | |
| 13:42:20 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: Always pass 'NUMACell.siblings' to _pack_instance_onto_cores' https://review.openstack.org/588572 | |
| 13:42:21 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: trivialfix: cleanup _pack_instance_onto_cores() https://review.openstack.org/588573 | |
| 13:42:22 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: Add unit tests for EmulatorThreadsTestCase https://review.openstack.org/588574 | |
| 13:42:23 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: Not use thread alloc policy for emulator thread https://review.openstack.org/588575 | |
| 13:43:15 | finucannot | lyarwood: Fancy sticking those on your review queue? ^ | |
| 13:44:00 | lyarwood | finucannot: sure thing | |
| 13:44:06 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use placement 1.28 in scheduler report client https://review.openstack.org/583667 | |
| 14:02:33 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Move resource_class_cache into placement hierarchy https://review.openstack.org/584085 | |
| 14:02:34 | openstackgerrit | Chris Dent proposed openstack/nova master: [placement] ensure_rc_cache only at start of process https://review.openstack.org/584086 | |
| 14:26:12 | openstack | Launchpad bug 1785235 in OpenStack Compute (nova) "metadata retrieval fails when using a global nova-api-metadata service" [Undecided,In progress] - Assigned to Liam Young (gnuoy) | |
| 14:26:12 | melwitt | dansmith: do you understand this bug? says guests can't retrieve metadata from the metadata API with multiple cells https://bugs.launchpad.net/nova/+bug/1785235 cc gnuoy | |
| 14:27:26 | dansmith | well, I understand the words in the bug | |
| 14:27:30 | melwitt | I had thought guests retrieved metadata over http, not the MQ | |
| 14:27:31 | cfriesen | mriedem_afk: I think the idea was to allow a management layer to create a new compute node in the DB so that we can "disable" it, set up system-generated host aggregates, boot the node, do some health checks, then "enable" it once everything is ready. | |
| 14:27:46 | dansmith | I also understand that I'm going to -2 the code change | |
| 14:27:47 | dansmith | melwitt: of course they do, | |
| 14:27:49 | melwitt | okay. I didn't understand either | |
| 14:28:03 | dansmith | but they're saying that metadata service then doesn't hit the right db as a result | |
| 14:28:12 | gnuoy | yep | |
| 14:28:26 | melwitt | oh, I see | |
| 14:29:39 | melwitt | so, this method, or another one that we missed cell targeting in? https://github.com/openstack/nova/blob/master/nova/api/metadata/base.py#L677 | |
| 14:30:04 | dansmith | melwitt: L692 | |
| 14:30:46 | melwitt | right | |
| 14:31:16 | melwitt | okay | |
| 14:31:17 | dansmith | commented | |
| 14:32:31 | gnuoy | dansmith, I see context.target_cell getting called and adding the cells mq and db endpoints to cctxt | |
| 14:32:33 | melwitt | cool thanks | |
| 14:32:50 | gnuoy | but when the request executes those endpoints are ignored | |
| 14:33:23 | dansmith | gnuoy: did you open this bug? | |
| 14:33:26 | gnuoy | I did | |
| 14:33:45 | dansmith | gnuoy: please show some logs and config | |
| 14:34:15 | gnuoy | sure | |
| 14:36:40 | dansmith | gnuoy: are you running a standalone metadata server with cmd/api_metadata ? | |
| 14:38:24 | gnuoy | I am running a standalone metadata server. I don't follow the second part of the question | |
| 14:39:08 | dansmith | gnuoy: okay I think I see what's going on and why you want to make the change you're making | |
| 14:39:22 | gnuoy | ah, cool! | |
| 14:39:37 | gnuoy | dansmith, that change was a starter for 10, I'm happy to update it | |
| 14:39:46 | dansmith | gnuoy: so there are two ways to run metadata, either part of the regular api with enabled_apis= | |
| 14:40:06 | dansmith | and then with the standalone cmd/api_metadata thing which just starts up a standalone metadata server | |
| 14:40:19 | dansmith | the latter always forces the indirection api into place, | |
| 14:40:33 | dansmith | which was really intended for the case where you're running metadata on each compute node | |
| 14:40:34 | gnuoy | yep, I saw that | |
| 14:40:48 | gnuoy | ah | |
| 14:40:59 | dansmith | for the global case, you really should be running the regular api server, and just not enable the osapi api if you want to only run metadata | |
| 14:41:11 | dansmith | that will not install the indirection handler and do proper switching | |
| 14:41:14 | dansmith | and will perform better | |
| 14:41:49 | gnuoy | dansmith, ok, I will give that a try, thanks. | |
| 14:42:22 | dansmith | gnuoy: please confirm for us and assuming that shakes out, we should write a doc change to fix this bug | |
| 14:42:39 | gnuoy | dansmith, absolutely, thanks | |
| 14:43:48 | mriedem_afk | cfriesen: there is a config option to keep compute services disabled when they are first created | |
| 14:44:21 | gnuoy | dansmith, If the indirection api is not used will the metadata service expect to be able to talk directly to the cells individual dbs? | |
| 14:44:35 | dansmith | gnuoy: yeah | |
| 14:44:52 | gnuoy | oh, hmm, ok | |
| 14:45:11 | dansmith | but it's global, and thus the access pattern will look like all those other global ones | |
| 14:45:28 | gnuoy | right, I see | |
| 14:51:36 | mriedem | cfriesen: was a bug reported upstream for this? https://github.com/starlingx-staging/stx-nova/commit/71acfeae0d1c59fdc77704527d763bd85a276f9a#diff-516904cc81cade24a9122ecf96707bf0R3359 | |
| 14:54:51 | mriedem | seems like something that could be deal with during _init_instance on restart of the compute service | |
| 14:57:53 | mriedem | cfriesen: heh i can get behind this :) https://github.com/starlingx-staging/stx-nova/commit/71acfeae0d1c59fdc77704527d763bd85a276f9a#diff-516904cc81cade24a9122ecf96707bf0R4217 | |
| 14:59:47 | melwitt | can someone sanity check me on this? when configured to use qcow2 images with libvirt, the backing file on each compute host is expected to be raw format (not qcow2) being that each instance created on the compute will be a COW copy of it? | |
| 15:00:07 | openstack | Launchpad bug 1774730 in OpenStack Compute (nova) "Compute node convert qcow2 to raw even if force_raw_images=false" [Undecided,New] | |
| 15:00:07 | melwitt | I'm trying to triage this https://bugs.launchpad.net/nova/+bug/1774730 | |
| 15:01:07 | mriedem | cfriesen: i could have sworn we had a patch up for this upstream too https://github.com/starlingx-staging/stx-nova/commit/71acfeae0d1c59fdc77704527d763bd85a276f9a#diff-516904cc81cade24a9122ecf96707bf0R4226 | |
| 15:03:41 | melwitt | mdbooth: if you're around, question about force_raw_images=False and images_type=qcow2, the backing file is expected to be raw format right? since each instance is a COW copy of it? ^ | |