| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-08-02 | |||
| 23:00:38 | melwitt | lbragstad: re: the comments on our PTG etherpad, there is already support for user_id right? as shown on this doc? https://docs.openstack.org/keystone/queens/admin/identity-unified-limits.html | |
| 23:02:38 | melwitt | we've had discussions with operators in the past and there was consensus that user_id isn't useful if hierarchy is possible, but I don't know if/how we could change the semantics of our existing quotas. so I'm thinking of what's possible if we *don't* change semantics | |
| #openstack-nova - 2018-08-03 | |||
| 00:58:02 | openstackgerrit | melanie witt proposed openstack/nova stable/ocata: [stable only] Handle quota usage during create/delete races https://review.openstack.org/582413 | |
| 00:58:04 | openstackgerrit | melanie witt proposed openstack/nova stable/ocata: Add functional regression test for bug 1783613 https://review.openstack.org/588416 | |
| 00:58:04 | openstack | bug 1783613 in OpenStack Compute (nova) ocata "[ocata only] quota usage not decremented during boot/delete race" [Undecided,In progress] https://launchpad.net/bugs/1783613 - Assigned to melanie witt (melwitt) | |
| 00:58:08 | openstackgerrit | Merged openstack/nova master: In Python3.7 async is a keyword [1] https://review.openstack.org/584365 | |
| 00:59:25 | openstackgerrit | melanie witt proposed openstack/nova stable/ocata: [stable only] Add functional regression test for bug 1783613 https://review.openstack.org/588416 | |
| 00:59:25 | openstack | bug 1783613 in OpenStack Compute (nova) ocata "[ocata only] quota usage not decremented during boot/delete race" [Undecided,In progress] https://launchpad.net/bugs/1783613 - Assigned to melanie witt (melwitt) | |
| 00:59:26 | openstackgerrit | melanie witt proposed openstack/nova stable/ocata: [stable only] Handle quota usage during create/delete races https://review.openstack.org/582413 | |
| 01:15:46 | melwitt | mriedem_afk: I added a functional regression test that might help demonstrate the bug ^ | |
| 01:18:19 | melwitt | customer hit an issue around this so I proposed it upstream too in case it can help | |
| 01:30:38 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove unused request API sample template https://review.openstack.org/588420 | |
| 01:31:45 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove unused request API sample template https://review.openstack.org/588420 | |
| 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 | 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 | |