Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-17
19:01:28 aspiers efried: but it seems that there are many other code paths which could reach update_provider_tree(), so your previous question about locking of the region modifying _domain_caps now will have a different answer
19:02:10 efried aspiers: I don't have a problem with the call chain through init_host as you've currently got it setting *some* instance variable on the libvirt driver.
19:02:14 efried which upt can then just go read
19:02:17 aspiers although ResourceTracker._update_available_resource has a semaphore, so maybe that's OK anyway?
19:02:27 efried it just shouldn't be in the capability dict.
19:02:44 aspiers OK, just wondering where's the best place to put it
19:03:00 efried self._supports_sev
19:03:02 efried whatever
19:03:07 aspiers yeah I guess the way you suggest works
19:03:36 efried you can put a comment saying we do this during init_host for efficiency and because it doesn't change
19:03:55 aspiers right - if I put it in resource_tracker.py then it would probably get called too often (i.e. more than once)
19:04:31 efried right. And the singleton-ness you've got set up on it would partially mitigate that - but there's no reason to even have the extra call stacks in the way.
19:04:44 efried memoizing, not singleton
19:04:58 aspiers yep
19:05:28 efried how's that whole stopping-working-because-it's-eod-friday thing going for you?
19:05:44 aspiers really great, just fantastic
19:06:16 efried You got a taste for it in Denver, now you can't get enough.
19:07:24 aspiers sad but true
19:23:05 openstackgerrit Eric Fried proposed openstack/nova stable/stein: Skip _exclude_nested_providers() if not nested https://review.opendev.org/659206
19:23:05 openstackgerrit Eric Fried proposed openstack/nova stable/stein: DNM: Revert "Use external placement in functional tests" https://review.opendev.org/659682
19:25:23 aspiers efried: last question before I go, you're saying that I don't need a driver capability at all, and u_p_t() would just provide the trait directly?
19:25:37 efried aspiers: Correct
19:25:46 aspiers OK good, thought so
19:25:54 efried aspiers: From the operator's point of view:
19:26:05 efried if they want mem encryption, they request resource MEM_ENCRYPTION_CONTEXT=1
19:26:13 efried if they want SEV specifically, they request required=SEV
19:26:15 aspiers BTW the kernel module parameter file is readable non-root, so I'm assuming privsep not required
19:26:23 aspiers right
19:26:24 efried okay.
19:26:46 aspiers IIRC, what you just wrote is captured in the spec
19:26:53 aspiers I remember us going over that before
19:26:54 efried I think so, yes
19:26:59 aspiers sweet
19:27:01 efried but I guess we never talked about driver capabilities
19:27:06 aspiers right
19:27:20 aspiers OK, *really* going now :)
19:27:23 efried yeah, it just seems silly for PowerVM, which will never ever ever have AMD SEV support, to have to say no every time.
19:27:23 aspiers have a great w/e!
19:27:27 efried later o/
19:27:42 aspiers yes totally, felt wrong to have to add to powervm driver
19:27:46 aspiers cya! o/
19:32:45 openstackgerrit Matt Riedemann proposed openstack/nova master: api-ref: mention default project filtering when listing servers https://review.opendev.org/659873
21:48:40 openstackgerrit Eric Fried proposed openstack/nova stable/stein: Skip _exclude_nested_providers() if not nested https://review.opendev.org/659206
21:49:52 openstackgerrit Merged openstack/nova-specs master: Follow up for I9200c2b18369da8b9194a8ccbbf6f1baf6a9613c https://review.opendev.org/659300
21:50:15 efried mriedem, melwitt: ---^ stein & rocky backports are ready for your stable eyes now.
21:50:31 melwitt ack
21:52:15 mriedem my eyeballs are currently embargoed
21:52:31 mriedem my sweet dance moves, however, are not
22:09:51 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Allow searching for hypervisors and getting back details https://review.opendev.org/659886
#openstack-nova - 2019-05-18
01:56:18 openstackgerrit Merged openstack/nova master: Use update_provider_tree in vmware virt driver https://review.opendev.org/651615
02:04:31 openstackgerrit Merged openstack/nova master: Fix target used in nova.policy.check_is_admin https://review.opendev.org/624185
03:15:12 openstackgerrit Guo Jingyu proposed openstack/nova master: Skip existing VMs when hosts apply force_config_drive https://review.opendev.org/659703
18:47:53 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Allow searching for hypervisors and getting back details https://review.opendev.org/659886
18:50:00 openstackgerrit Matt Riedemann proposed openstack/nova master: Drop pre-cinder 3.44 version compatibility https://review.opendev.org/621061
18:53:14 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix guestfs.set_backend_settings call https://review.opendev.org/524748
18:54:20 openstackgerrit Matt Riedemann proposed openstack/nova master: Create volume attachment during boot from volume in compute https://review.opendev.org/541420
18:54:36 openstackgerrit Matt Riedemann proposed openstack/nova master: Migrate old style volume attachments on nova-compute startup https://review.opendev.org/549130
19:27:24 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove ServersTestBase.setUp() https://review.opendev.org/608780
19:28:07 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: don't log error if guest gone during interface detach https://review.opendev.org/610727
20:55:40 openstackgerrit Eric Fried proposed openstack/nova master: Remove redundant conductor from ServersTestBase.setUp() https://review.opendev.org/608780
21:22:02 openstackgerrit Merged openstack/nova stable/pike: Do not dump all instances in the scheduler https://review.opendev.org/659832
21:50:21 openstackgerrit Merged openstack/nova master: Remove redundant conductor from ServersTestBase.setUp() https://review.opendev.org/608780
22:14:20 openstackgerrit Matt Riedemann proposed openstack/nova stable/ocata: Do not dump all instances in the scheduler https://review.opendev.org/659833
#openstack-nova - 2019-05-19
01:15:36 openstackgerrit Guo Jingyu proposed openstack/nova master: Skip existing VMs when hosts apply force_config_drive https://review.opendev.org/659703
07:20:18 openstackgerrit Adrian Chiris proposed openstack/nova master: [FUP] Follow-up patch for SR-IOV live migration https://review.opendev.org/659101
15:50:02 openstackgerrit Adrian Chiris proposed openstack/nova master: Allow driver to properly unplug VIFs on destination on confirm resize https://review.opendev.org/643024
15:50:02 openstackgerrit Adrian Chiris proposed openstack/nova master: Move get_pci_mapping_for_migration to MigrationContext https://review.opendev.org/643023
#openstack-nova - 2019-05-20
06:39:48 openstackgerrit Boxiang Zhu proposed openstack/nova master: Add host and hypervisor_hostname flag to create server https://review.opendev.org/645520
07:51:33 kashyap efried: aspiers: Some great sleuthing, both of you there: https://review.opendev.org/#/c/655268/
08:46:39 openstackgerrit Rui Zang proposed openstack/nova-specs master: Virtual persistent memory libvirt driver implementation https://review.opendev.org/622893
08:46:39 openstackgerrit Rui Zang proposed openstack/nova-specs master: support virtual persistent memory https://review.opendev.org/601596
09:29:37 openstackgerrit Stephen Finucane proposed openstack/nova master: Stop handling 'update_cells' on 'BandwidthUsage.create' https://review.opendev.org/651303
09:29:37 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove 'instance_update_from_api' https://review.opendev.org/651302
09:29:38 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove cells code https://review.opendev.org/651306
09:29:38 openstackgerrit Stephen Finucane proposed openstack/nova master: Stop handling cells v1 for instance naming https://review.opendev.org/651304
09:29:39 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove conductor_api and _last_host_check from manager.py https://review.opendev.org/651059
09:36:31 openstackgerrit Hamdy Khader proposed openstack/nova master: [WIP] OVS DPDK port representors support https://review.opendev.org/658785
09:38:53 openstackgerrit Hamdy Khader proposed openstack/os-vif master: [WIP] OVS DPDK port representors support https://review.opendev.org/658786
09:40:46 stephenfin lyarwood: If you're about, fancy sending this three line patch to the gate? https://review.opendev.org/#/c/659833/
09:47:35 openstackgerrit Boxiang Zhu proposed openstack/nova master: Add host and hypervisor_hostname flag to create server https://review.opendev.org/645520
09:55:13 lyarwood stephenfin: ack
10:07:53 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: Add 'flavor-extra-spec-image-property-validation-extended' spec https://review.opendev.org/638734
10:15:58 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Do not reraise DiskNotFound exceptions during resize https://review.opendev.org/571410
10:18:12 openstackgerrit Stephen Finucane proposed openstack/nova master: hacking: Resolve E731 (do not assign a lambda expression, use a def) https://review.opendev.org/651554
10:18:13 openstackgerrit Stephen Finucane proposed openstack/nova master: hacking: Resolve E741 (ambiguous variable name) https://review.opendev.org/652103
10:18:13 openstackgerrit Stephen Finucane proposed openstack/nova master: hacking: Resolve W503 (line break occurred before a binary operator) https://review.opendev.org/651555
10:18:14 openstackgerrit Stephen Finucane proposed openstack/nova master: hacking: Resolve W605 (invalid escape sequence) https://review.opendev.org/652104
10:26:15 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: Add 'flavor-extra-spec-image-property-validation-extended' spec https://review.opendev.org/638734
10:57:18 openstackgerrit Matthew Booth proposed openstack/nova stable/rocky: Fix retry of instance_update_and_get_original https://review.opendev.org/659318
10:57:44 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Support for changing deleted_on_termination after boot https://review.opendev.org/580336
10:57:55 openstackgerrit Brin Zhang proposed openstack/nova-specs master: Support delete_on_termination in volume attach api https://review.opendev.org/612949
10:58:04 openstackgerrit Matthew Booth proposed openstack/nova stable/queens: Fix retry of instance_update_and_get_original https://review.opendev.org/659320
11:11:20 openstackgerrit Boxiang Zhu proposed openstack/nova master: Fix failure to boot instances with qcow2 format images https://review.opendev.org/640271
11:34:14 openstackgerrit guangchao.pan proposed openstack/nova master: Add a name for the automatically created volume https://review.opendev.org/660096
11:40:07 openstackgerrit Merged openstack/os-resource-classes master: Dropping the py35 testing https://review.opendev.org/654653
11:51:58 openstackgerrit Merged openstack/os-traits master: Replace git.openstack.org URLs with opendev.org URLs https://review.opendev.org/655066
12:43:10 openstackgerrit Merged openstack/nova master: api-ref: mention default project filtering when listing servers https://review.opendev.org/659873
12:48:58 sean-k-mooney dansmith: this is proably a little early for you but if you are about can i get your input on something related to bug #1813789
12:48:59 openstack bug 1813789 in OpenStack Compute (nova) "Evacuate test intermittently fails with network-vif-plugged timeout exception" [Medium,In progress] https://launchpad.net/bugs/1813789 - Assigned to Artom Lifshitz (notartom)
12:49:54 sean-k-mooney dansmith: artom is away for a few days on PTO so i have been asked to take a look at ^

Earlier   Later