Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-11-12
20:59:17 efried relevant in that I would need to make sure I don't hit the key manager in such operations.
20:59:30 dansmith efried: but still, I imagine you can't define the xml for the guest referencing a secret that you've since deleted (which you said you were going to do) right?
20:59:46 efried I can, as long as I don't try to boot the thing.
21:00:03 dansmith okay I would have expected that to fail at define time
21:00:05 efried the guest XML only contains the UUID, which (if the instance was ever booted) I've stored in its sysmeta.
21:00:20 sean-k-mooney ya i dont think you will hit that in novas usage
21:00:29 efried I may be misunderstanding, but I think _get_guest_xml just returns an etree, doesn't actually try to libvirtify it.
21:00:40 dansmith efried: sure
21:00:59 dansmith efried: I guess I thought you meant generate the xml as in... update the definition in libvirt
21:01:05 sean-k-mooney the only case i am aware off was the requet to "create a server but not boot it" but we dont supprot that currently
21:01:20 efried I possibly need to rethink the flow here...
21:01:42 efried because I *actually* only want to define the secret to libvirt on boot-ish operations anyway. And then delete it right after.
21:02:00 dansmith right so in driver.spawn()
21:02:20 mriedem dansmith: if moving this get_newer_obj method to NovaObjectBase makes it less gross for you i can do that
21:02:21 efried also cold boot?
21:02:32 dansmith efried: cold boot? :)
21:02:36 dansmith efried: start/
21:02:38 mriedem i thought about it but didn't since this was origially much more special case to this api method
21:02:38 efried power_on
21:02:52 sean-k-mooney cold boot or start call power_on which for libvirt calls hard_reboot which calls spawn
21:03:12 dansmith mriedem: honestly I just hate the way that logic is laid out, probably irrationally, but before even looking for test gaps I knew it would be annoying to validate
21:03:31 dansmith mriedem: the fact that there was one just makes me scared, but I'm clearly the only one, so just do whatever you want, I say
21:04:31 mriedem i expected something like this to already exist but didn't find anything
21:05:22 mriedem if i'm left to do whatever i want, i'm going to address mdbooth's comments and keep the series going. if there are things around this to be done later, i say let's do them later.
21:05:49 dansmith aight
21:06:56 sean-k-mooney efried: for libivrt you might want to create and remove the securte here https://github.com/openstack/nova/blob/7aa88029bbf6311033457c32801963da01e88ecb/nova/virt/libvirt/driver.py#L6188
21:07:36 sean-k-mooney in libvirt that is
21:07:52 sean-k-mooney you would have to add the secret to the xml before
21:09:07 sean-k-mooney anyway night o/
21:17:02 openstackgerrit Matt Riedemann proposed openstack/nova master: Start functional testing for cross-cell resize https://review.opendev.org/636253
21:17:02 openstackgerrit Matt Riedemann proposed openstack/nova master: Filter duplicates from compute API get_migrations_sorted() https://review.opendev.org/636224
21:17:03 openstackgerrit Matt Riedemann proposed openstack/nova master: Validate image/create during cross-cell resize functional testing https://review.opendev.org/642592
21:17:03 openstackgerrit Matt Riedemann proposed openstack/nova master: Handle target host cross-cell cold migration in conductor https://review.opendev.org/642591
21:17:04 openstackgerrit Matt Riedemann proposed openstack/nova master: Add negative test for cross-cell finish_resize failing https://review.opendev.org/643451
21:17:04 openstackgerrit Matt Riedemann proposed openstack/nova master: Add zones wrinkle to TestMultiCellMigrate https://review.opendev.org/643450
21:17:05 openstackgerrit Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize_at_source compute method https://review.opendev.org/637058
21:17:05 openstackgerrit Matt Riedemann proposed openstack/nova master: Add negative test for prep_snapshot_based_resize_at_source failing https://review.opendev.org/669013
21:17:06 openstackgerrit Matt Riedemann proposed openstack/nova master: Add confirm_snapshot_based_resize conductor RPC method https://review.opendev.org/637075
21:17:06 openstackgerrit Matt Riedemann proposed openstack/nova master: Add ConfirmResizeTask https://review.opendev.org/637070
21:17:07 openstackgerrit Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize_at_dest compute method https://review.opendev.org/637630
21:17:07 openstackgerrit Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize from the API https://review.opendev.org/637316
21:17:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Add finish_revert_snapshot_based_resize_at_source compute method https://review.opendev.org/637647
21:17:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Deal with cross-cell resize in _remove_deleted_instances_allocations https://review.opendev.org/639453
21:17:09 openstackgerrit Matt Riedemann proposed openstack/nova master: Add revert_snapshot_based_resize conductor RPC method https://review.opendev.org/638047
21:17:09 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add RevertResizeTask https://review.opendev.org/638046
21:17:10 openstackgerrit Matt Riedemann proposed openstack/nova master: Confirm cross-cell resize while deleting a server https://review.opendev.org/638268
21:17:10 openstackgerrit Matt Riedemann proposed openstack/nova master: Revert cross-cell resize from the API https://review.opendev.org/638048
#openstack-nova - 2019-11-13
00:12:37 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Add emulated TPM support to Nova https://review.opendev.org/631363
00:12:38 openstackgerrit Eric Fried proposed openstack/nova master: WIP: vTPM request_filter https://review.opendev.org/678325
00:12:38 openstackgerrit Eric Fried proposed openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.opendev.org/639934
01:21:49 melwitt gmann: johnthetubaguy has updated the spec for your comments, if you could take another look when you get a chance https://review.opendev.org/602201
05:06:07 openstackgerrit ya.wang proposed openstack/nova-specs master: Add "live migration without performance impact" spec. https://review.opendev.org/693655
08:53:30 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Wire up a force disconnect_volume flag https://review.opendev.org/584849
09:06:05 openstackgerrit Balazs Gibizer proposed openstack/nova stable/train: Use admin neutron client to query ports for binding https://review.opendev.org/694013
09:07:08 openstackgerrit Balazs Gibizer proposed openstack/nova stable/train: Use admin neutron client to gather port resource requests https://review.opendev.org/694015
09:10:14 openstackgerrit Balazs Gibizer proposed openstack/nova stable/train: Use admin neutron client to gather port resource requests https://review.opendev.org/694015
09:21:00 openstackgerrit Balazs Gibizer proposed openstack/nova stable/train: Use admin neutron client to see if instance has qos ports https://review.opendev.org/694018
09:27:23 stephenfin sean-k-mooney: Approved https://review.opendev.org/#/c/683174 FYI
09:37:10 openstackgerrit Merged openstack/nova-specs master: Add spec for VM-scoped SR-IOV NUMA affinity https://review.opendev.org/683174
10:25:14 openstackgerrit Lee Yarwood proposed openstack/os-traits master: WIP Add COMPUTE_RESCUE_STABLE_DEVICES and COMPUTE_RESCUE_BFV traits https://review.opendev.org/694033
12:01:27 sean-k-mooney stephenfin: when you wrote the orginial alias based numa affintiy feature did you add docs? i dont see it referenced here https://docs.openstack.org/nova/train/admin/pci-passthrough.html
12:02:09 sean-k-mooney there is https://docs.openstack.org/nova/train/configuration/config.html#pci.alias
12:02:29 sean-k-mooney whcich mention the numa_policy field but it does not explain them
12:45:53 openstackgerrit Balazs Gibizer proposed openstack/nova stable/train: Use admin neutron client to see if instance has qos ports https://review.opendev.org/694018
12:47:16 gibi_off elod: ^^ additional diff was needed to make the backport work properly due to feature merged in ussuri
12:50:00 openstackgerrit Balazs Gibizer proposed openstack/nova master: Mask the token used to allow access to consoles https://review.opendev.org/220622
12:55:54 openstackgerrit Lee Yarwood proposed openstack/nova-specs master: Virtual instance rescue with stable disk devices https://review.opendev.org/693849
12:55:55 openstackgerrit Lee Yarwood proposed openstack/nova-specs master: Boot from volume instance rescue https://review.opendev.org/694063
13:01:02 elod gibi_off: thanks, i'm not there yet, but will look into it :)
13:02:03 stephenfin sean-k-mooney: Think I just documented it in the config option?
13:02:53 stephenfin though we should really have it documented in doc/source/admin/pci-passthrough.rst
13:02:56 stephenfin I can do that now
13:03:23 stephenfin good way to get used to having working internet again
13:12:06 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove functional test specific nova code https://review.opendev.org/683609
13:31:49 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove functional test specific nova code https://review.opendev.org/683609
13:40:55 efried stephenfin: alex_xu already has to deal with Intel's firewall, the gfwoc is nbd.
13:41:20 efried I work around it by using a personal computer for real work.
13:41:21 bauzas I wonder whether we should propose some Chinese IRC server
13:41:30 efried bauzas: isn't that called WeChat?
13:41:42 bauzas efried: sure, but WeChat doesn't support IRC
13:41:47 efried oh, you mean a server for actual IRC, got it.
13:41:50 bauzas having the same client would be nice
13:42:06 efried I'm not sure the technology is the problem; it's the connectivity.
13:42:07 bauzas like, I could use this Chinese server plus the Freenode one
13:42:27 efried isn't the problem that the bits can't get in & out?
13:42:31 bauzas of course, but I'm pretty sure a IRC server would be supported by the chinese government :)
13:42:59 bauzas I mean, when running in a chinese cloud ;)
13:43:45 bauzas alex_xu: FWIW, I'll still continue to use WeChat
13:54:04 openstack Launchpad bug 1852437 in OpenStack Compute (nova) "Allow ability to disable individual CPU features via `cpu_model_extra_flags`" [Undecided,New]
13:54:04 kashyap efried: Hi, when you get a moment, is a Blueprint required for this: https://bugs.launchpad.net/nova/+bug/1852437 (Allow ability to disable individual CPU features via `cpu_model_extra_flags`)
13:55:28 efried kashyap: at a glance, a blueprint seems entirely necessary. A spec, not as sure. Perhaps put it on tomorrow's meeting agenda for discussion?
13:56:00 kashyap efried: Spec / BP seems like an overkill, IMHO. But sure, can discuss tomm
13:56:42 efried we're talking about enhancing syntax of a conf option in what seems like a nontrivial way.
13:56:46 kashyap It is essentially aims to implement what I stated at the end of this commit in paranthesis: https://opendev.org/openstack/nova/commit/cc27a2007f314
13:56:57 kashyap efried: Hmm, phrased that way...
13:57:18 kashyap efried: But what do you think of it? Is there a better way you can think of than the +/- notion?
13:59:08 efried kashyap: no, that seems reasonable, just seems like a thing that ought to have a "design" somewhere written down that we can agree on. Just having it in a RFE bug might be sufficient, but really that's what blueprints are for. And it's the kind of thing I would not expect to backport.
14:01:36 kashyap efried: Sure, can file a simple BluePrint
14:01:59 efried thanks kashyap.
14:02:40 kashyap I first wanted to file it, but went the bug route, thinking it is a "simple idea". :D

Earlier   Later