From e0c64356a96dc9c2441139acd2ca281133d80844 Mon Sep 17 00:00:00 2001 From: Maros Marsalek Date: Tue, 20 Sep 2016 15:53:57 +0200 Subject: HONEYCOMB-222 Fix Interface L2 writer order The writer was placed between Interface and Subitnerface, so it was invoked before subinterfaces were processed. Meaning that if the L2/xconnect targetted subitnerface, it would be submitted prematurely. Moving the Interface L2 writer after Subinterface writer. Change-Id: I192ebde5eaf4b05a104d76bce39de74569cdb608 Signed-off-by: Maros Marsalek --- .../java/io/fd/honeycomb/translate/v3po/InterfacesWriterFactory.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'v3po/v3po2vpp/src/main/java/io/fd/honeycomb/translate/v3po/InterfacesWriterFactory.java') diff --git a/v3po/v3po2vpp/src/main/java/io/fd/honeycomb/translate/v3po/InterfacesWriterFactory.java b/v3po/v3po2vpp/src/main/java/io/fd/honeycomb/translate/v3po/InterfacesWriterFactory.java index 0f7d517c4..c8b4ae458 100644 --- a/v3po/v3po2vpp/src/main/java/io/fd/honeycomb/translate/v3po/InterfacesWriterFactory.java +++ b/v3po/v3po2vpp/src/main/java/io/fd/honeycomb/translate/v3po/InterfacesWriterFactory.java @@ -171,9 +171,9 @@ public final class InterfacesWriterFactory implements WriterFactory { registry.addAfter( new GenericWriter<>(VPP_IFC_AUG_ID.child(Routing.class), new RoutingCustomizer(jvpp, ifcNamingContext)), specificIfcTypes); - // Routing(Execute only after specific interface customizers) = + // L2(Execute only after subinterface (and all other ifc types) = registry.addAfter(new GenericWriter<>(L2_ID, new L2Customizer(jvpp, ifcNamingContext, bdNamingContext)), - specificIfcTypes); + SubinterfaceAugmentationWriterFactory.SUB_IFC_ID); // Proxy Arp (execute after specific interface customizers) registry.addAfter( new GenericWriter<>(VPP_IFC_AUG_ID.child(ProxyArp.class), new ProxyArpCustomizer(jvpp, ifcNamingContext)), -- cgit 1.2.3-korg