Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-09-09
13:25:18 openstackgerrit Federico Ressi proposed openstack/nova master: Log stack trace when fails launching a defined domain https://review.opendev.org/750652
14:04:34 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP zuul: Introduce nova-multinode-evacuate https://review.opendev.org/744883
14:06:02 sean-k-mooney lyarwood: is that a job ^
14:06:14 sean-k-mooney i assume its not some new api action right
14:06:25 sean-k-mooney it sjust the job where we will test evacuate
14:06:46 tosky it's part of the removal of the remaining live-migration job
14:06:58 sean-k-mooney ya that is what i was assuming
14:07:38 sean-k-mooney i know lee has been workign on that for a while and we discussed that in the past but have not been following this too closely
14:17:26 openstackgerrit Artom Lifshitz proposed openstack/nova stable/train: post live migration: don't call Neutron needlessly https://review.opendev.org/750670
14:21:25 bauzas gibi: fwiw, looking at your change, no question yet
14:21:32 bauzas stephenfin: have you seen my point ?
14:21:39 gibi bauzas: cool, thanks
14:24:10 stephenfin bauzas: oh, sorry, I missed that
14:24:22 stephenfin Lemme refresh that real quick
14:24:28 bauzas np, have you time for respinning it ?
14:24:32 bauzas cool ta
14:24:47 stephenfin sean-k-mooney: See, ta is catching on :P ^
14:25:14 sean-k-mooney hehe it is a very irish thing, slowly we will take over
14:27:00 openstackgerrit Artom Lifshitz proposed openstack/nova stable/rocky: post live migration: don't call Neutron needlessly https://review.opendev.org/750673
14:27:43 artom Oh wait, I f'ed that up, it's r s t
14:27:50 artom So stein before rocky
14:28:52 openstackgerrit Artom Lifshitz proposed openstack/nova stable/stein: post live migration: don't call Neutron needlessly https://review.opendev.org/750674
14:30:51 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Remove resize, cold migration from vTPM limitations https://review.opendev.org/750675
14:30:55 stephenfin bauzas: ^
14:30:58 openstackgerrit Artom Lifshitz proposed openstack/nova stable/stein: post live migration: don't call Neutron needlessly https://review.opendev.org/750674
14:31:44 stephenfin I didn't add anything to the release note since we already decided not to list specific operations in there. See Ie470af3e738327c6f2800f386dbe43319f896222
14:31:58 bauzas stephenfin: err, I'd have loved to have the doc in the same patch, but kk
14:32:47 bauzas ah you're right and I'm old
14:32:51 bauzas nevermind this then
14:35:53 openstackgerrit Artom Lifshitz proposed openstack/nova stable/rocky: post live migration: don't call Neutron needlessly https://review.opendev.org/750673
15:03:29 stephenfin bauzas: What's the chances of looking at https://review.opendev.org/#/q/topic:sriov-functional-tests+status:open too?
15:03:54 bauzas stephenfin: atm looking at the hairy patch on SRIOV attach/detach from gibi
15:03:58 bauzas maybe next one ?
15:04:19 bauzas stephenfin: do the above also checks sriov attach/detach ?
15:04:43 gibi bauzas: nope, but my patch has functional test for it
15:04:50 stephenfin No, not needed for that
15:04:53 bauzas kk
15:04:57 stephenfin and SR-IOV attach/detach is higher priority, for sure
15:04:58 bauzas so it's independent
15:05:07 gibi it is
15:05:10 bauzas k, continuing to review gibi's homework
15:05:35 gibi yeah I working from home so the code I push is basically a homework :D
15:05:38 bauzas and fwiw, the above series is functional tests adds, right? if so, we don't need to merge them before tomorrow's FF
15:05:51 stephenfin bauzas: correct
15:06:10 bauzas kk all good, back to gerrit and github
15:06:54 stephenfin gibi: I assume XenAPI-removal will have to be punted come M3? :-\
15:07:14 stephenfin I'm waiting on the Tempest fix to merge but gate failures are killing me :-(
15:07:49 gibi stephenfin: if nobody else showing up reviewing that series then I'm affraid so
15:08:13 gibi ohh the tempest part is still open that does not help either
15:09:21 lyarwood sorry been stuck downstream for a while but let me know if I can help with anything
15:09:45 bauzas stephenfin: well, if that's not a Facebook poke, you can surely do it
15:09:58 bauzas but I'm already married fwiw
15:10:59 stephenfin bauzas: keep your smut off IRC, plz
15:11:07 stephenfin ;)
15:12:42 stephenfin lyarwood: It's clearly not the highest priority, but I have two series that I'd like to close off: the XenAPI-removal and the SR-IOV tests. Former is feature'ish so that's more important rn and starts here https://review.opendev.org/#/c/749300/
15:13:25 stephenfin You won't be able to look at anything above the 'remove os-agents' change until the Tempest change merges but the first half should be good to go
15:26:10 openstackgerrit Stephen Finucane proposed openstack/nova master: network: Remove unused 'affect_auto_assigned' parameter https://review.opendev.org/744871
15:26:11 openstackgerrit Stephen Finucane proposed openstack/nova master: network: Add type hints for various allocate functions https://review.opendev.org/744870
15:31:16 bauzas gibi: couple of questions in https://review.opendev.org/#/c/740995/17/nova/compute/manager.py
15:31:25 gibi looking
15:31:41 bauzas gibi: mostly about exception handling in order to make sure we have correct error behaviours
15:32:01 bauzas the overall logic looked to me good
16:15:20 gibi bauzas: replied in https://review.opendev.org/#/c/740995
16:21:44 gibi I can do the changes tomorrow morning
16:27:31 bauzas gibi: k, no worries, I will continue the review for the func test part
16:27:46 bauzas gibi: I'm okay with having those as a FUP since it's like a bugfix
16:27:50 bauzas ie. closing the gaps
16:36:19 gibi bauzas: cool o/
16:40:58 bauzas gibi: sent to the gate
16:41:09 bauzas I'll hold reviewing the FUP for now
16:41:15 bauzas ++
16:41:55 gibi thanks
16:42:05 openstackgerrit Merged openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.opendev.org/639934
16:42:20 openstackgerrit Merged openstack/nova master: Migrate default policy file from JSON to YAML https://review.opendev.org/748059
16:42:28 openstackgerrit Merged openstack/nova master: Only unplug vif after the device is detached from libvirt https://review.opendev.org/744934
16:46:44 openstackgerrit Ghanshyam Mann proposed openstack/nova master: [Trivial] Replace ref of policy.json to policy.yaml https://review.opendev.org/749821
19:41:03 openstackgerrit Merged openstack/nova master: [Trivial] Replace ref of policy.json to policy.yaml https://review.opendev.org/749821
#openstack-nova - 2020-09-10
01:21:52 openstackgerrit Brin Zhang proposed openstack/nova master: Cyborg shelve/unshelve support https://review.opendev.org/729563
02:41:01 openstackgerrit Merged openstack/nova master: Set migrate_data.vifs only when using multiple port bindings https://review.opendev.org/742180
04:39:07 openstackgerrit MIKI Nobuhiro proposed openstack/nova master: libvirt: add IP address to libvirt guest metadata https://review.opendev.org/750552
05:31:04 openstackgerrit Luyao Zhong proposed openstack/nova master: Rollback when reset running live migration https://review.opendev.org/747899
05:31:04 openstackgerrit Luyao Zhong proposed openstack/nova master: Track error migrations in resource tracker https://review.opendev.org/745281
05:31:05 openstackgerrit Luyao Zhong proposed openstack/nova master: Improve orphans tracking in resource tracker https://review.opendev.org/745282
05:57:01 openstackgerrit Federico Ressi proposed openstack/nova master: Log stack trace when fails launching a defined domain https://review.opendev.org/750652
06:57:04 gibi good morning Nova
07:26:27 bauzas gibi: good morning
07:29:13 bauzas gibi: I need to go off for around 2 hours this morning for an appointment, ping me your new revision for the FUP and I'll try to look at it around noon
07:29:29 gibi bauzas: OK, I will ping you
08:31:46 openstackgerrit Merged openstack/nova master: Use UUID as vif and network_id in vif tests https://review.opendev.org/748722
09:08:18 openstackgerrit Balazs Gibizer proposed openstack/nova master: Follow up for I67504a37b0fe2ae5da3cba2f3122d9d0e18b9481 https://review.opendev.org/750184
09:08:55 gibi bauzas: the friendly ping I promised ^^
09:15:52 luyao stephenfin: Hi, I reply you at https://review.opendev.org/#/c/745281/, thanks for review
09:17:39 luyao alex_xu: could you have a look at https://review.opendev.org/#/q/topic:bp/vpmem-enhancement+(status:open+OR+status:merged) if you get time, there are some updates and discussion. thanks in advance.
09:36:58 bauzas gibi: +2d
09:37:13 bauzas stephenfin: your +2 is needed https://review.opendev.org/750184
09:37:44 bauzas taxiing
09:38:54 gibi bauzas: thanks
09:45:31 openstackgerrit OpenStack Release Bot proposed openstack/os-vif stable/victoria: Update .gitreview for stable/victoria https://review.opendev.org/750924
09:45:32 openstackgerrit OpenStack Release Bot proposed openstack/os-vif stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria https://review.opendev.org/750925
09:45:34 openstackgerrit OpenStack Release Bot proposed openstack/os-vif master: Update master for stable/victoria https://review.opendev.org/750926

Earlier   Later