Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-03-29
06:09:35 openstackgerrit Michael Still proposed openstack/nova master: Remove unused FP device creation and deletion methods. https://review.openstack.org/635433
06:09:35 openstackgerrit Michael Still proposed openstack/nova master: Privsepify ipv4 forwarding enablement. https://review.openstack.org/635431
06:09:36 openstackgerrit Michael Still proposed openstack/nova master: Move adding vlans to interfaces to privsep. https://review.openstack.org/635436
06:09:36 openstackgerrit Michael Still proposed openstack/nova master: Privsep the ebtables modification code. https://review.openstack.org/635435
06:09:37 openstackgerrit Michael Still proposed openstack/nova master: Move dnsmasq restarts to privsep. https://review.openstack.org/639280
06:09:37 openstackgerrit Michael Still proposed openstack/nova master: Move iptables rule fetching and setting to privsep. https://review.openstack.org/636508
06:09:38 openstackgerrit Michael Still proposed openstack/nova master: Move calls to ovs-vsctl to privsep. https://review.openstack.org/639282
06:09:38 openstackgerrit Michael Still proposed openstack/nova master: Move router advertisement daemon restarts to privsep. https://review.openstack.org/639281
06:09:39 openstackgerrit Michael Still proposed openstack/nova master: Move final bridge commands to privsep. https://review.openstack.org/639580
06:09:39 openstackgerrit Michael Still proposed openstack/nova master: Move setting of device trust to privsep. https://review.openstack.org/639283
06:09:40 openstackgerrit Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438
06:09:40 openstackgerrit Michael Still proposed openstack/nova master: Cleanup the _execute shim in nova/network. https://review.openstack.org/639581
06:09:41 openstackgerrit Michael Still proposed openstack/nova master: Style corrections for privsep usage. https://review.openstack.org/648615
06:09:41 openstackgerrit Michael Still proposed openstack/nova master: Cleanup no longer required filters and add a release note. https://review.openstack.org/639826
06:09:42 openstackgerrit Michael Still proposed openstack/nova master: WIP: Add test coverage for nova.privsep.libvirt. https://review.openstack.org/648616
06:28:51 openstackgerrit ya.wang proposed openstack/nova-specs master: expose auto converge in rest api https://review.openstack.org/648619
07:51:04 gibi alex_xu: looking
08:38:25 gibi alex_xu: answered in https://review.openstack.org/#/c/639608
08:38:57 gibi alex_xu: instead of revert and re-add the functional reproduce can I just push a patch that changes what exception we throw in the functiional test?
09:10:50 openstackgerrit Magnus Bergman proposed openstack/nova master: Adding tests to demonstrate bug #1821824 https://review.openstack.org/647957
09:10:51 openstack bug 1821824 in OpenStack Compute (nova) "Forbidden traits in flavor properties don't work" [Undecided,In progress] https://launchpad.net/bugs/1821824 - Assigned to Magnus Bergman (magnusbe)
09:13:05 trident gibi: There. Got stuck in some other work yesterday, but here it is. First commit with new functional and unit tests demonstrating the forbidden traits issue. Next commit will add the fix as well.
09:14:31 gibi trident: ack. I will look at it shortly
09:23:33 openstackgerrit Balazs Gibizer proposed openstack/nova master: Fix exception type in test_boot_reschedule_fill_provider_mapping_raises https://review.openstack.org/648651
09:23:33 openstackgerrit Balazs Gibizer proposed openstack/nova master: Handle placement error during re-schedule https://review.openstack.org/639608
09:25:06 gibi alex_xu: ^^
09:29:04 openstackgerrit Magnus Bergman proposed openstack/nova master: Fix bug preventing forbidden traits from working https://review.openstack.org/648653
09:31:19 trident gibi: I hope I got it right with the commits in series.
09:32:17 gibi trident: based on a quick look in gerrit you put the patches in a good order. I will review both
09:32:30 trident Great, thanks!
09:48:07 gibi trident: left comments in https://review.openstack.org/#/c/647957
09:50:13 gibi trident: some formal changes needed but the test your created seems good to me
10:03:17 trident gibi: Thanks! Will look into it shortly!
10:09:59 openstackgerrit Surya Seetharaman proposed openstack/nova-specs master: Support adding the reason behind a server lock https://review.openstack.org/638629
10:23:29 openstackgerrit sean mooney proposed openstack/nova master: make lower-constratints env use lower-constratins https://review.openstack.org/645392
10:38:33 openstackgerrit Surya Seetharaman proposed openstack/python-novaclient master: [WIP] Support adding the reason behind a server lock https://review.openstack.org/648659
11:00:08 tssurya stephenfin: any idea on how to fix http://logs.openstack.org/29/638629/7/check/openstack-tox-docs/f14d940/job-output.txt.gz#_2019-03-29_10_28_35_792167 , seems like the docs CI is failing..not sure if there is a bug already
11:07:35 openstackgerrit Surya Seetharaman proposed openstack/nova master: [WIP] Support adding the reason behind a server lock https://review.openstack.org/648662
11:08:08 openstackgerrit Merged openstack/nova master: Error out migration when confirm_resize fails https://review.openstack.org/647546
11:12:41 openstackgerrit sean mooney proposed openstack/nova master: make lower-constratints env use lower-constratins https://review.openstack.org/645392
11:17:19 openstackgerrit sean mooney proposed openstack/nova master: make lower-constratints env use lower-constratins https://review.openstack.org/645392
11:19:20 sean-k-mooney by the way i have confimed that cdent's version works too https://review.openstack.org/#/c/622972/14
11:20:06 sean-k-mooney so im fine with abandoning my version in favor of his if people prefer but we should merge one of the two
11:25:40 stephenfin tssurya: \o/ Looks like Sphinx 2.0 has broken some stuff
11:25:45 stephenfin I think that got released today
11:26:15 stephenfin tssurya: What repo is that against? I didn't think we were still using oslosphinx anywhere
11:29:15 sean-k-mooney launchpad is broken...
11:30:04 sean-k-mooney i should just never file bugs buzilla was broken when i tried yesterday too.
11:32:03 openstackgerrit sean mooney proposed openstack/nova master: make lower-constraints env use lower-constraints https://review.openstack.org/645392
11:35:34 openstackgerrit sean mooney proposed openstack/nova master: make lower-constraints env use lower-constraints https://review.openstack.org/645392
11:39:36 openstackgerrit Helena proposed openstack/nova-specs master: Spec for a new nova virt driver to manage an RSD, composable infrastructure deployment https://review.openstack.org/648665
11:50:59 stephenfin tssurya: https://review.openstack.org/648666
12:11:03 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: docs: Remove cruft from 'conf.py' https://review.openstack.org/648670
12:11:04 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: docs: Fix multiple issues with the 'redirect' extension https://review.openstack.org/648671
12:11:05 openstackgerrit Stephen Finucane proposed openstack/nova-specs master: docs: Switch to openstackdocstheme https://review.openstack.org/648672
12:11:27 stephenfin gibi, bauzas: Can you look at those to unblock tssurya/anyone else proposing specs ^
12:11:49 stephenfin (https://review.openstack.org/648666 might be helpful for additional context)
12:19:45 gibi stephenfin: on a meeting I will look at it latae
12:19:47 gibi later
12:49:57 openstackgerrit Balazs Gibizer proposed openstack/nova master: Only call _fill_provider_mapping if claim succeeds https://review.openstack.org/648676
13:06:04 elbragstad gmann nice - thanks for those
13:09:15 fried_rice gibi: Earlier mlavalle asked about https://bugs.launchpad.net/neutron/+bug/1821058 / https://review.openstack.org/#/c/645173/
13:09:16 fried_rice Any initial thoughts on this work?
13:09:16 fried_rice I can see you're already reviewing the latter.
13:09:16 openstack Launchpad bug 1821058 in neutron "[RFE] Port binding event extended information for Nova" [Wishlist,Confirmed]
13:11:46 gibi fried_rice: my position is that the problems listed in the spec are valid, however it is not clear to me how the proposed solution will impact nova (and os-vif) especially how much dependency will be added between nova and the neutron backends
13:12:11 gibi fried_rice: but I haven't read sean-k-mooney's reply in the spec yet
13:12:37 fried_rice gibi: Okay. I suppose it will be something for the xproj session at the ptg. But obviously the more we can have resolved before then, the better.
13:12:53 gibi fried_rice: totally agree
13:13:01 gibi fried_rice: this will be a cross project effort
13:13:36 sean-k-mooney fried_rice: so i have been putting of writing the nova side of the spec abit untill i get a vague yes or no form the neutron side
13:14:01 fried_rice sean-k-mooney: At some point we'll want it tied to a bp
13:14:02 sean-k-mooney there are likely other ways to adress the issues i that were raised
13:14:15 fried_rice but I guess there's no point in filing one of those if we're still not sure we want to do it at all.
13:14:25 sean-k-mooney fried_rice: i can file that now if you like.
13:14:38 fried_rice sean-k-mooney: Or it doesn't exist :P
13:15:00 sean-k-mooney well i can file a blueprint for the geneal problem of fix neutron event handleing
13:15:20 sean-k-mooney the solution is still open
13:15:38 fried_rice sean-k-mooney: it will require *something* in nova?
13:18:07 tssurya stephenfin: thanks for the fix !
13:20:09 sean-k-mooney fried_rice: yes
13:21:05 fried_rice sean-k-mooney: Then yes please, let's get a bp on the docket
13:21:32 sean-k-mooney fried_rice: put simplely if we want to ensure that networking is configured correclty for all instance lifecycle operations and not just spawn/power on then we need to wait for the netwrok events in several placees in the code
13:21:52 fried_rice several places in the nova code
13:21:55 sean-k-mooney fried_rice: we cant today simpley because we dont know what neutron backend is being used and if it will send them
13:22:04 sean-k-mooney fried_rice: its all in the virt dirvres
13:22:34 fried_rice interesting problem.
13:23:10 sean-k-mooney interesting proablem i have been trinig to fix for 4 years... but yes :)
13:24:15 sean-k-mooney fried_rice: we have some comments like https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L2846-L2855 in the code where we know we would like to wait but cant
13:24:51 sean-k-mooney but https://bugs.launchpad.net/nova/+bug/1813789 is also jsut another sideffect of the same large problem
13:24:53 openstack Launchpad bug 1813789 in OpenStack Compute (nova) "Evacuate test intermittently fails with network-vif-plugged timeout exception" [Medium,In progress] - Assigned to Artom Lifshitz (notartom)
13:35:38 openstackgerrit Theodoros Tsioutsias proposed openstack/nova-specs master: Enable rebuild for instances in cell0 https://review.openstack.org/648686
13:36:15 openstackgerrit Theodoros Tsioutsias proposed openstack/nova-specs master: Add PENDING vm state https://review.openstack.org/648687
13:37:04 alex_xu gibi: yea, I'm ok with another fix
13:38:12 gibi alex_xu: cool, I pushed the test fix below the actual bugfix
13:38:42 alex_xu gibi: thanks, checking now
13:38:54 openstackgerrit Matt Riedemann proposed openstack/nova master: Delete allocations even if _confirm_resize raises https://review.openstack.org/647566
13:38:55 openstackgerrit Matt Riedemann proposed openstack/nova master: api-ref: add more details to confirmResize troubleshooting https://review.openstack.org/647741
13:39:25 gibi stephenfin: did you missed a depends-on in https://review.openstack.org/#/c/648670/ ?
13:39:36 gibi stephenfin: the doc job fails

Earlier   Later