Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-08-24
14:46:15 gibi stephenfin: hi! there is an empty slot in the runways and you have the vtpm in the queue. Are you OK with me moving the vtpm to the runway slot?
14:46:31 stephenfin gibi: hey, welcome back. Sure.
14:46:40 gibi thanks. then it is moved :)
14:48:55 lyarwood jsuchome: oh and these failures were due to the changes I made btw
14:49:06 lyarwood jsuchome: so apologies for that, should have them fixed shortly
14:50:53 jsuchome ah, I see, I did not look closely at your changes
14:55:43 openstackgerrit Stephen Finucane proposed openstack/nova master: Don't unset Instance.old_flavor, new_flavor until necessary https://review.opendev.org/744958
14:55:43 openstackgerrit Stephen Finucane proposed openstack/nova master: manager: Move context manager up one level https://review.opendev.org/745674
14:55:44 openstackgerrit Stephen Finucane proposed openstack/nova master: compute: Add type hints for resize functions https://review.opendev.org/745341
14:55:44 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP: compute: Add more type hints for resize functions https://review.opendev.org/745675
14:55:47 openstackgerrit Stephen Finucane proposed openstack/nova master: Add generic reproducer for bug #1879878 https://review.opendev.org/747744
14:55:47 openstack bug 1879878 in OpenStack Compute (nova) "VM become Error after confirming resize with Error info CPUUnpinningInvalid on source node " [Medium,In progress] https://launchpad.net/bugs/1879878 - Assigned to Stephen Finucane (stephenfinucane)
14:55:47 openstackgerrit Stephen Finucane proposed openstack/nova master: Move confirm resize under semaphore https://review.opendev.org/747745
14:55:47 openstackgerrit Stephen Finucane proposed openstack/nova master: Move revert resize under semaphore https://review.opendev.org/747746
14:55:47 openstackgerrit Stephen Finucane proposed openstack/nova master: Cleanup 'drop_move_claim' and '_drop_move_claim' https://review.opendev.org/747747
14:55:47 openstackgerrit Stephen Finucane proposed openstack/nova master: manager: Address TODO https://review.opendev.org/747748
14:56:00 stephenfin alex_xu: Think I've gotten to the bottom of that ^
15:02:44 openstackgerrit Lee Yarwood proposed openstack/nova master: Add ability to download Glance images into the libvirt image cache via RBD https://review.opendev.org/574301
15:02:45 openstackgerrit Lee Yarwood proposed openstack/nova master: Bring back allowed_direct_url_schemes in support of RBD image download https://review.opendev.org/728095
15:02:45 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM zuul: Enable [glance]/allowed_direct_url_schemes within nova-ceph-multistore https://review.opendev.org/743220
15:02:49 lyarwood jsuchome: ^ there you go
15:05:49 openstackgerrit Merged openstack/nova-specs master: Create specs directory for Wallaby https://review.opendev.org/746408
15:12:32 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Provide default 'host_info' for '_get_connection' https://review.opendev.org/746942
15:12:32 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add 'start_compute' helper https://review.opendev.org/746943
15:12:33 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Use tempdir for CONF.instances_path https://review.opendev.org/746944
15:12:33 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Clean up PCI tests https://review.opendev.org/746945
15:12:34 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add resize tests for PCI servers https://review.opendev.org/746946
15:12:34 openstackgerrit Stephen Finucane proposed openstack/nova master: conductor: Don't use setattr https://review.opendev.org/746947
15:12:35 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add test for SR-IOV neutron ports https://review.opendev.org/746948
15:12:35 openstackgerrit Stephen Finucane proposed openstack/nova master: fixtures: Handle binding of first port https://review.opendev.org/746949
15:12:36 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add live migration tests for PCI, SR-IOV servers https://review.opendev.org/746950
15:42:18 stephenfin lyarwood: can you hit https://review.opendev.org/#/c/741285/ again? Should unblock 4 patches or so
15:56:43 openstackgerrit Merged openstack/nova master: Few todo fixes for API new policies https://review.opendev.org/746610
15:57:17 gibi gmann: is this ^^ the last patch for the policy work for Victoria? Can I close the bp?
15:57:50 openstackgerrit Balazs Gibizer proposed openstack/nova master: Undeprecate the vmwareapi driver https://review.opendev.org/742407
15:58:06 gmann gibi: i hope so, let me check if any TODO left and confirm you by today
15:58:19 gibi gmann: cool, thanks
16:03:33 lyarwood stephenfin: ack looking
16:06:24 artom lyarwood, stephenfin, so https://review.opendev.org/#/c/747451/ is the Neutron _post_live_migration() patch series we talked about
16:06:42 artom Oh, and I need to update the functional test to assert the vif host
16:07:01 artom Lunch first though, kiddos are getting low-blood-sugar-restless
16:33:48 stephenfin lyarwood: Thanks for that. Have the rebase for later patches lined up locally but I'll wait for those to (hopefully) go through the gate first
16:35:51 stephenfin artom: I see https://review.opendev.org/#/c/747443/2//COMMIT_MSG and raise you https://review.opendev.org/#/c/747745/1//COMMIT_MSG
16:36:42 stephenfin incidentally, something you're probably interested in given your live migration work (I suspect that suffers from a similar issue but I haven't investigated that yet)
16:38:26 openstackgerrit Merged openstack/nova master: resolve ResourceProviderSyncFailed issue https://review.opendev.org/742398
16:40:36 stephenfin dansmith, melwitt: Could I bug you for a review on https://review.opendev.org/#/c/744958/ again. It's now a partial fix, owing to further issues alex_xu spotted. Later patches fix it completely though
16:41:44 artom stephenfin, yeah, I looked at the bottom func test in that series
16:41:56 artom Haven't been able to force myself yet to sit down and grok the rest
17:36:23 openstackgerrit Merged openstack/os-vif master: windows: Add missing return https://review.opendev.org/746870
17:42:46 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add reno for deprecated APIs policy changes https://review.opendev.org/747784
17:43:05 openstackgerrit Merged openstack/nova master: tests: Add helpers for suspend, resume and reboot of server https://review.opendev.org/741285
17:43:07 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add reno for deprecated APIs policy changes https://review.opendev.org/747784
17:43:20 openstackgerrit Merged openstack/nova master: libvirt: Pass context, instance to '_create_guest' https://review.opendev.org/741286
18:02:59 artom lyarwood, sean-k-mooney, so actually, there's nothing like a host field in the vifs we get from instance.get_network_info() / network_api.get_instance_nw_info()
18:03:17 artom IOW, we have nothing to assert that we're still using the source vifs
18:14:46 lyarwood artom: fun, something for a fup?
18:15:10 lyarwood artom: I was asking if we could assert that at runtime during post_live_migration btw
18:15:16 lyarwood artom: not in a func test
18:15:18 artom lyarwood, how tho? Like, we just don't have that into in the data structure that we get
18:15:28 artom lyarwood, ah - well, same problem though
18:15:34 lyarwood artom: yeah, same problem
18:16:30 lyarwood artom: so maybe I'm just coming at this wrong, it just seems like something you would want to know assuming vif info *can* be host specific
18:17:03 lyarwood artom: connection_info and volume attachments in general are and that's why we track the host in various places
18:17:17 artom lyarwood, well the port binding is
18:17:36 lyarwood artom: kk maybe I'm just confusing concepts then, my bad
18:17:40 artom lyarwood, the vif info isn't host specific per se, it depends on the ml2 driver
18:18:17 artom But if different hosts are using different Neutron driver then in that sense it's host specific
18:18:30 artom But only the port binding profile has the host info
18:18:51 artom From what I understand, the port is purely a Neutron/Nova API concept, it's an abstraction
18:19:05 artom The vif is the actual thing that gets plugged into an instance
18:19:33 artom So while a port generally corresponds to a single vif, they're different levels of abstractions, and thus hold different into
18:19:45 artom sean-k-mooney can keep me honest here, I barely have a grasp on this myself
19:18:53 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Reject non-spawn operations for vTPM https://review.opendev.org/741500
19:18:53 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Add emulated TPM support to Nova https://review.opendev.org/631363
19:18:53 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Add docs for vTPM support https://review.opendev.org/739213
19:18:54 openstackgerrit Stephen Finucane proposed openstack/nova master: Add type hints to 'nova.compute.manager' https://review.opendev.org/742863
19:18:55 openstackgerrit Stephen Finucane proposed openstack/nova master: Don't unset Instance.old_flavor, new_flavor until necessary https://review.opendev.org/741995
19:18:55 openstackgerrit Stephen Finucane proposed openstack/nova master: privsep: Add support for recursive chown, move_tree operations https://review.opendev.org/742864
19:18:56 openstackgerrit Stephen Finucane proposed openstack/nova master: Add type hints to 'nova.virt.libvirt.utils' https://review.opendev.org/742865
19:18:56 openstackgerrit Stephen Finucane proposed openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.opendev.org/639934
19:18:57 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Add helpers for rebuild, cold migrate, and shelve/unshelve https://review.opendev.org/747792
19:44:29 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add reno for deprecated APIs policy changes https://review.opendev.org/747784
21:51:38 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add reno for deprecated APIs policy changes https://review.opendev.org/747784
23:11:40 openstackgerrit Merged openstack/nova master: api: Reject non-spawn operations for vTPM https://review.opendev.org/741500
23:15:04 openstackgerrit sean mooney proposed openstack/nova master: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/742180
#openstack-nova - 2020-08-25
06:38:22 bauzas good morning Nova
06:52:47 gibi bauzas: welcome back!
06:53:14 bauzas gibi: I was around yesterday but I had email scrubbing + a number of paperwork to do
07:02:14 gibi I mostly did paperwork yestarday too.
07:13:43 bauzas gibi: fwiw, I'm very well aware of victoria-3 milestone in two weeks
07:13:58 gibi same here
07:14:01 gibi it is painful
07:14:09 bauzas gibi: I'll rush this week on the routed-networks stuff
07:14:23 bauzas gibi: but you can surely ping me on any review ask
07:14:47 gibi thanks. I will go through the patches in the runway slots today
07:15:07 bauzas well, I'm used to face the milestone right after my long vacations
07:15:08 bauzas :)
07:15:37 bauzas gibi: good idea, I'll jump onto reviewing those patches
07:45:27 brinzhang_ gibi, bauzas: optimiza create and bind arqs logical patch, need to review https://review.opendev.org/#/c/726564/6
07:46:16 gibi brinzhang_: hi! on my list for today

Earlier   Later