| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-03 | |||
| 06:06:56 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove unused request API sample template https://review.openstack.org/588420 | |
| 06:48:14 | alex_xu | stephenfin: sorry, just send at wrong channel, the sample files are deleted by this commit https://review.openstack.org/#/c/149129/, and actually, the api sample test doesn't validate the request body, so there is no complain, those file actually just for document. | |
| 06:52:29 | gmann | alex_xu: stephenfin this will fix - https://review.openstack.org/#/c/588420/4 | |
| 06:53:27 | alex_xu | gmann: thanks | |
| 06:54:07 | alex_xu | gmann: but I'm thinking about why we cleanup those empty file at https://review.openstack.org/#/c/149129/ | |
| 06:55:36 | gmann | alex_xu: not sure why we removed may be because their are just empty | |
| 06:55:52 | gmann | they are just empty | |
| 06:56:25 | alex_xu | gmann: yea, anyway, your fix is better | |
| 07:00:16 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove unused request API sample template https://review.openstack.org/588420 | |
| 07:00:38 | gmann | alex_xu: done ^^ | |
| 07:02:54 | alex_xu | gmann: thanks | |
| 07:23:36 | openstackgerrit | Yongli He proposed openstack/nova master: Load expected attr pci_devices while migrate https://review.openstack.org/588455 | |
| 07:31:32 | openstackgerrit | Merged openstack/nova master: Add another up-call to the cells v2 caveats list https://review.openstack.org/581910 | |
| 08:22:13 | giblet | stephenfin: you found it you can make sure it is fixed ;) https://review.openstack.org/#/c/588420/ | |
| 08:28:24 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Fix nits in resource_provider.py https://review.openstack.org/588470 | |
| 09:12:03 | tobasco | is there any manual process that needs to be performed if you get a lot of this? | |
| 09:12:06 | tobasco | http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/resource_tracker.py#n1308 | |
| 09:12:26 | tobasco | why wouldn't it clear allocations if the instance doesn't exist? | |
| 09:12:37 | openstackgerrit | zhufl proposed openstack/nova master: xx_instance_type_id in list_migrations should be integer https://review.openstack.org/588481 | |
| 09:55:18 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Fix nits in resource_provider.py https://review.openstack.org/588470 | |
| 10:20:08 | cdent | giblet++ | |
| 10:29:07 | panda|rover | Hi, I'm trying to gather console logs for nova instances, but it seems logs resets at boot, is there a way to maintain the console log persistent across reboots ? | |
| 11:01:00 | openstackgerrit | Chen proposed openstack/nova master: Revert task_state to none for LM failure due to invalid dest https://review.openstack.org/588512 | |
| 11:13:31 | sean-k-mooney | tobasco: there are some bugs related to live migration that can cause allocation to leak | |
| 11:15:08 | amarao | Hello. I found that if I remove image instance booted from, migration no longer uses a proper aggregate based on that image meta. Is someone knew something about this? | |
| 11:15:38 | tobasco | so we've been pounding our cloud with rally, so if my logs contain excessive of such statements that would probably be after rally live migrations | |
| 11:15:52 | tobasco | should I be worried, I assume I would want to release those allocations manually somehow | |
| 11:29:40 | openstackgerrit | Merged openstack/nova master: Remove unused request API sample template https://review.openstack.org/588420 | |
| 12:06:03 | openstackgerrit | Liam Young proposed openstack/nova master: Target metadata requests at the correct cell. https://review.openstack.org/588520 | |
| 12:13:12 | openstackgerrit | Merged openstack/nova master: Docs: Add Placement to Nova system architecture https://review.openstack.org/584338 | |
| 12:20:48 | openstackgerrit | Liam Young proposed openstack/nova master: Remove Neutron MetaAPIProxy from cellsv2-layout https://review.openstack.org/588525 | |
| 12:30:53 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: Cleanup comp_node, res_prov, services, aggregate_hosts during cell deletion https://review.openstack.org/546660 | |
| 12:40:26 | mriedem | cfriesen: what was the reason for needing a POST /os-services API to create nova-compute services on a given host? https://github.com/starlingx-staging/stx-nova/commit/71acfeae0d1c59fdc77704527d763bd85a276f9a#diff-f3afe2522f9c92f5705f0ff5cf343865R246 | |
| 12:40:36 | mriedem | which is also, btw, not multi-cell aware since it doesn't rely on the host mapping | |
| 12:44:49 | mriedem | sean-k-mooney: check this out https://github.com/starlingx-staging/stx-nova/commit/71acfeae0d1c59fdc77704527d763bd85a276f9a#diff-99e4b3f7232bf35155ff8b590b0ea589R44 | |
| 12:47:18 | sean-k-mooney | mriedem: clicking but not sure i want too | |
| 12:47:30 | sean-k-mooney | haha | |
| 12:48:23 | sean-k-mooney | mriedem: that in the api. that is not a bad idea to be honest | |
| 12:49:12 | sean-k-mooney | mriedem: we dont document in the api that when using the libvirt dirview we detach all pci/sriov device on suspend which is hostile to a guest application that was uing them | |
| 12:49:26 | sean-k-mooney | pause would not detach the devices. | |
| 12:51:06 | mriedem | sean-k-mooney: i know https://bugs.launchpad.net/nova/+bug/1785246 | |
| 12:51:06 | openstack | Launchpad bug 1785246 in OpenStack Compute (nova) "Compute API reference should describe pause and suspend operations" [Medium,Confirmed] | |
| 12:51:15 | sean-k-mooney | mriedem: we likely can do this where i wanted too in the snapshot case after talking to dansmith due to concern about data curroption by not flushing buffers but this seams ok | |
| 12:51:44 | sean-k-mooney | ya i was think about that yesterday after we were talking about it. | |
| 12:52:01 | mriedem | windriver could have just disabled the suspend/resume apis with policy | |
| 12:52:07 | mriedem | rather than change the behavior | |
| 12:52:24 | sean-k-mooney | its an impentation detail of the libvirt driver im not sure we should be leaking it through the api | |
| 12:52:39 | sean-k-mooney | that said we should document it | |
| 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 | 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: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: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 | |