Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-08-03
01:34:52 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Remove unused request API sample template https://review.openstack.org/588420
01:44:07 lbragstad melwitt: oh - so don't try and support per user quotas?
01:51:53 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Update installation guide to be more clear about cellsv2 https://review.openstack.org/584244
01:56:47 openstackgerrit zhufl proposed openstack/nova master: Fix none-ascii char in doc https://review.openstack.org/588422
03:03:42 openstackgerrit Vishakha Agarwal proposed openstack/nova master: No change in field 'updated' in server https://review.openstack.org/586446
04:38:48 openstackgerrit melanie witt proposed openstack/nova stable/ocata: [stable only] Handle quota usage during create/delete races https://review.openstack.org/582413
05:57:32 openstackgerrit Vishakha Agarwal proposed openstack/nova master: 'Updated_at' is NULL when show aggregate info https://review.openstack.org/580271
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 openstack Launchpad bug 1785246 in OpenStack Compute (nova) "Compute API reference should describe pause and suspend operations" [Medium,Confirmed]
12:51:06 mriedem sean-k-mooney: i know https://bugs.launchpad.net/nova/+bug/1785246
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

Earlier   Later