Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-08-24
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: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: docs: Add docs for vTPM support https://review.opendev.org/739213
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: api: Reject non-spawn operations for vTPM https://review.opendev.org/741500
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: privsep: Add support for recursive chown, move_tree operations https://review.opendev.org/742864
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: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:56 openstackgerrit Stephen Finucane proposed openstack/nova master: Add type hints to 'nova.virt.libvirt.utils' https://review.opendev.org/742865
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
07:46:19 brinzhang_ gibi, bauzas: the cyborg evacuate support patch rebased on this patch, and it merge conflict, we will resolve the merge conflict
07:46:34 brinzhang_ gibi: cool, thanks
08:06:45 openstackgerrit Brin Zhang proposed openstack/nova master: Optimize _create_and_bind_arqs logic in conductor https://review.opendev.org/726564
08:06:46 openstackgerrit Brin Zhang proposed openstack/nova master: Cyborg evacuate support https://review.opendev.org/715326
09:03:49 swp20 gibi: Hi, gibi. https://review.opendev.org/#/c/719163/16/nova/scheduler/client/report.py@L2168. can u give some idea to process these exceptions?
09:04:42 xinranwang__ gibi: sean-k-mooney: Hi all, as we discussed during virtual PTG, I have proposed an overall spec https://review.opendev.org/#/c/742785/ to discuss sriov nic support in nova, neutron, and cyborg. Hope you guys can review it and give comments on it, there still some open questions need to be nailed down which really need you guys' comments and opinion. Let's take this spec as a place to discuss it. Please
09:04:43 xinranwang__ review it when you guys got time. Thanks a lot!
09:07:32 gibi swp20: I will check that comment from efried and try to recover the context
09:07:53 swp20 Thanks gibi.
09:10:03 gibi xinranwang__: you need to re-propose that spec to wallaby as we are past victory spec freeze. I cannot promise I will have time in the coming weeks to dig deep into this as the victoria feature freeze is coming fast. But please add this as a topic for the PTG so we won't forget about it
09:10:18 gibi xinranwang__: https://etherpad.opendev.org/p/nova-wallaby-ptg
09:17:30 xinranwang__ gibi: Understood, sure. I will re-propose it.
09:20:28 xinranwang__ gibi: I'd like to point it out here because it is better that we can do some reviews before next ptg. No rush, maybe after the feature freeze in V.
09:21:05 gibi xinranwang__: sure, I will try to prepare myself for the PTG by reading up on the possible PTG topics
09:23:58 xinranwang__ gibi: great, thanks
09:41:52 Yumeng that we don't need to think and decide all of them at a very short time in PTG.
09:41:52 Yumeng hi gibi and sean-k-mooney: just one more question for the smartnic spec, would you like xinranwang__ and I to split the discussions before next PTG and start discuss(we introduce one or two of the open issues first, and then once you guys have any questions, we can answer and discuss.) the open issues directly one by one at your convenience? so
09:43:24 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Enable real policy fixture by default https://review.opendev.org/745906
09:43:25 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Remove '_FakeImageService' https://review.opendev.org/745908
09:43:25 openstackgerrit Stephen Finucane proposed openstack/nova master: functional: Add and use 'GlanceFixture' https://review.opendev.org/745907
09:45:51 stephenfin gmann: Addressed your comments on that "use real policy" patch ^ Let me know if I missed anything :)
09:58:19 itandops Hi folks, I have some issue to restart nova API after rebooting the controller http://paste.openstack.org/show/797116/
09:58:51 itandops There is a database with empty values nova_cell0
09:59:37 itandops It seems to be the root cause of this issue. Any suggestion to solve this ?
10:03:35 lyarwood itandops: `Lost connection to MySQL server during query` it isn't that the db is empty
10:06:03 itandops lyarwood: I used 2 controllers running nova API.
10:07:16 lyarwood itandops: okay well there's a clear db connectivity issue there for you to debug now :)
10:15:44 sean-k-mooney gibi are you going to try and resolve the sriov attach/detach feature before m3
10:16:50 sean-k-mooney gibi: im still trying to find time to work on numa in placement but im fully expecting that to slip to next cycle. but im going to try and at least have a poc of it before repoposing the spec next cycle

Earlier   Later