Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-02-20
14:22:04 NewBruce sean-k-mooney - to avoid cluttering the main chat here have dropped you full details in a DM
14:23:15 kaisers stephenfin: thnx for seeing this through to the current state
14:23:24 sean-k-mooney NewBruce: just finishing a code review but ill pop over in a miniute
14:23:43 NewBruce no worries mate - working late tonight, so ill be here a while
14:23:53 NewBruce have been talking to mnaser about this as well
14:24:51 mnaser i mean it is kindof a nova bug :)
14:30:32 mriedem artom: replied to your questions about the tests in https://review.openstack.org/#/c/631948/
14:33:50 dansmith mriedem: got it up now
14:35:40 gibi efried, jaypipes: updated https://review.openstack.org/#/c/616239 based on what we agreed on Friday. If you could re-review that patch that would be awesome
14:36:03 openstackgerrit Surya Seetharaman proposed openstack/nova master: API microversion 2.69: Handles Down Cells Documentation https://review.openstack.org/635147
14:36:03 openstackgerrit Surya Seetharaman proposed openstack/nova master: [Doc Fix]Best practices for effectively tolerating down cells https://review.openstack.org/638173
14:36:13 bauzas gibi: I need to jump on your b/w series
14:36:21 bauzas gibi: later today
14:37:38 gibi bauzas: thanks in advance
14:58:52 jackding Need another core to approve https://review.openstack.org/#/c/603844/. It's been too long.
15:00:10 kashyap jackding: Hi, I have something unrelated to chat: CPU models
15:00:38 efried gibi: +2
15:01:48 kashyap jackding: So I recall you filed this one: https://review.openstack.org/#/c/620959/ (Select cpu model from a list of cpu models)
15:02:10 kashyap jackding: I thought you might be interested in this one: https://blueprints.launchpad.net/nova/+spec/cpu-selection-with-hypervisor-consideration
15:04:26 gibi efried_pto: thanks
15:04:39 kashyap I'll file a spec too. In short, the core proposal here is to replace the existing two libvirt APIs with the "hypervisor-literate" APIs, which are far more useful.
15:05:40 kashyap jackding: ^ It maps onto the comment I noted here: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L3900,#L3909
15:07:19 ysandeep #openstack-nova A quick query, If my nova backend of compute nodes is ceph. And I evacuate instance.. Will my root disk of instance (rbd in ceph) will remain intact or the root disk will reimage(Old disk will be deleted and a new disk will be created from image)?
15:08:28 mriedem ysandeep: if you're evacuating to another compute host sharing the same ceph cluster the root disk should remain
15:10:08 ysandeep mriedem, thank you so much .. Do you happen to have idea about - from which branch we have this feature?
15:11:00 mriedem ysandeep: that's been around a long time, that's why the evacuate api has an onSharedStorage parameter https://developer.openstack.org/api-ref/compute/?expanded=evacuate-server-evacuate-action-detail#evacuate-server-evacuate-action
15:11:09 mriedem having said that, i'm sure there have been bug fixes over the years
15:11:15 mriedem mdbooth probably knows best about that
15:11:39 ysandeep mriedem, thank you soo much.. really appreciate your help..
15:11:56 mriedem yw
15:11:57 mdbooth ysandeep: It remains intact.
15:12:13 ysandeep mdbooth, thanks
15:12:20 mdbooth ysandeep: Same for NFS.
15:13:22 openstackgerrit Adrian Chiris proposed openstack/nova master: Add free for claimed, allocated devices https://review.openstack.org/616120
15:13:22 ysandeep mdbooth, thanks, i will test for nfs also
15:13:23 openstackgerrit Adrian Chiris proposed openstack/nova master: Allow per-port modification of vnic_type and profile https://review.openstack.org/607365
15:13:23 openstackgerrit Adrian Chiris proposed openstack/nova master: Add get_instance_pci_request_from_vif https://review.openstack.org/619929
15:13:24 openstackgerrit Adrian Chiris proposed openstack/nova master: SR-IOV Live migration indirect port support https://review.openstack.org/620115
15:13:24 openstackgerrit Adrian Chiris proposed openstack/nova master: libvirt: auto detach/attach sriov ports on migration https://review.openstack.org/629589
15:13:48 jackding kashyap: noted, thanks
15:13:59 mdbooth ysandeep: Same for boot from volume too, fwiw.
15:14:08 kashyap jackding: Just wanted to give a heads-up, so we don't duplicate efforts :-)
15:15:01 jackding kashyap: sure thing.
15:15:02 artom mriedem, yep, saw that. I'll un-1 soon
15:15:16 artom I also reviewed that series ass-backwards by looking at the top one first
15:15:38 ysandeep mdbooth, yes i was aware for boot from volume(cinder volume) but I was not sure of nova disk(on cluster storage) and ephermal disk behavior
15:40:29 mriedem tssurya: i'll poke on your novaclient change https://review.openstack.org/#/c/579563/ later this morning to try and sort out the test issue
15:40:39 tssurya dansmith: feel free to author the documentation patch if its only those changes in your comments that need to change I can respin, but considering mriedem and you removed the votes I guess there are some obvious incorrect wordings
15:41:09 dansmith tssurya: it's fine, just let me finish the call I'm on and then I'll push up a rev
15:41:20 tssurya mriedem: yea thanks I am not sure why that address gets printed in the wrong order
15:41:48 mriedem tssurya: my guess is something about dict hashseeds in py27
15:42:44 tssurya mriedem: oh okay
15:42:48 tssurya dansmith: thanks
15:53:07 jaypipes jackding: k, will review after gibi's patch.
15:53:38 jackding jaypipes: thanks
16:06:37 tssurya mriedem: also here https://review.openstack.org/#/c/637217/ I am not entirely sure why we need to consider the non-zero standard classes in the extra_specs
16:07:42 mriedem i don't think we're going to
16:07:56 tssurya ok good
16:08:09 mriedem we want to check if the standard resource class key exists in extra specs and if it's already 0, we consider that instance migrated
16:08:29 mriedem if it's non-zero, then i guess we log a warning and still consider it migrated
16:08:35 tssurya yea that's what I throught too, I was just reading jroll's and efried_pto's comments on it
16:08:46 tssurya which implied otherwise
16:09:11 tssurya ah a warning! ok then
16:09:44 mriedem note that on stable branches (at least stable/rocky) that calculation probably needs to account for report_ironic_standard_resource_class_inventory
16:09:54 mriedem which makes it even more complicated
16:10:01 tssurya yea saw that part :(
16:12:20 gibi jaypipes: it is not me it is the problem that is the source of the hurt :)
16:13:05 openstackgerrit melanie witt proposed openstack/nova master: WIP Count instances from mappings and cores/ram from placement https://review.openstack.org/638073
16:22:04 openstackgerrit Dan Smith proposed openstack/nova master: API microversion 2.69: Handles Down Cells Documentation https://review.openstack.org/635147
16:22:06 dansmith mriedem: tssurya see what you think of that ^
16:22:57 dansmith I was tempted to rename it as well to just say "Handling partial infrastructure failures" or something
16:23:31 tssurya dansmith: I don't mind at all :) feel free to change anything in that patch
16:25:35 jaypipes gibi: :)
16:26:32 openstackgerrit Merged openstack/nova master: Address nits from I9e30a24a4c0640f282f507d0a96640d3cdefe43c https://review.openstack.org/636967
16:34:08 mriedem tssurya: i have the novaclient test sorting thing fixed
16:34:19 mriedem i'll post that as a patch before yours in the series
16:34:38 openstackgerrit Balazs Gibizer proposed openstack/nova master: nova-manage: heal port allocations https://review.openstack.org/637955
16:34:38 openstackgerrit Balazs Gibizer proposed openstack/nova master: cache neutron ports in heal allocation https://review.openstack.org/638207
16:43:15 melwitt mriedem: we have an ironic driver fix proposed that adds a kwarg to the get_info() virt method. we wanted your opinion about whether it's backportable and if not, advice on another way to approach it https://review.openstack.org/636699
16:57:46 mriedem well that's a backward incompatible change to any out of tree drivers on stable
16:57:51 mriedem so it depends on if you care about that
16:57:57 mriedem efried_pto probably cares about it
16:58:50 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: API microversion 2.69: Handles Down Cells https://review.openstack.org/579563
16:58:51 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Make Server.networks use a predictable sort order https://review.openstack.org/638214
16:59:12 melwitt a new kwarg with a default value? I didn't think that's backward incompat
17:01:03 tssurya mriedem: oh thanks a lot!
17:01:22 tssurya and as far as the host name being "host1" always its because the fake has that hardcoded
17:01:34 tssurya I can just change that if you want them to be unique
17:02:31 mriedem melwitt: if compute is passing use_cache=False to an out of tree driver that implements get_info(instance) but doesn't have that kwarg in the method signature, it's going to blow up
17:02:54 melwitt oh, right. urgh
17:03:34 mriedem http://paste.openstack.org/show/745484/
17:04:05 mriedem so the worst that happens is out of tree drivers don't have the sync power states task working until they fix that
17:04:53 melwitt jroll ^
17:05:49 jroll melwitt: yep, that's correct, I just didn't know what the policy was given out of tree drivers are unsupported
17:06:29 jroll I'm not sure there's a different approach we can take, am open to it though
17:06:54 jroll other than reverting the thing that made the sync use the cache, and spamming ironic for that task
17:07:06 mriedem i'm commenting
17:07:11 mriedem cool your jets and hold your horses please
17:07:14 mriedem oath can wait
17:07:31 jroll it's not even for oath, larsks reported this bug and I hacked it up ¯\_(ツ)_/¯

Earlier   Later