From 95d18b91f26cae876bec3e946ee1f390007137eb Mon Sep 17 00:00:00 2001 From: Marek Gradzki Date: Tue, 21 Mar 2017 12:39:36 +0100 Subject: HC2VPP-117: workaround for VPP-208 Change-Id: I75d05fc73236f7a768f54d29247054cc20b722a6 Signed-off-by: Marek Gradzki --- .../src/main/java/io/fd/hc2vpp/policer/read/PolicerCustomizer.java | 1 - .../java/io/fd/hc2vpp/vpp/classifier/write/ClassifyTableWriter.java | 5 ++++- .../io/fd/hc2vpp/vpp/classifier/write/ClassifyTableWriterTest.java | 6 +++++- 3 files changed, 9 insertions(+), 3 deletions(-) diff --git a/vpp-classifier/impl/src/main/java/io/fd/hc2vpp/policer/read/PolicerCustomizer.java b/vpp-classifier/impl/src/main/java/io/fd/hc2vpp/policer/read/PolicerCustomizer.java index 9d4c40f50..faf34678b 100644 --- a/vpp-classifier/impl/src/main/java/io/fd/hc2vpp/policer/read/PolicerCustomizer.java +++ b/vpp-classifier/impl/src/main/java/io/fd/hc2vpp/policer/read/PolicerCustomizer.java @@ -122,7 +122,6 @@ final class PolicerCustomizer extends FutureJVppCustomizer builder.setName(toString(details.name)); builder.setCir(UnsignedInts.toLong(details.cir)); builder.setEir(UnsignedInts.toLong(details.eir)); - // TODO(HC2VPP-117): policer init fails if cb is configured (looks like byte ordering issue on VPP side) builder.setCb(toUnsignedBigInteger(details.cb)); builder.setEb(toUnsignedBigInteger(details.eb)); builder.setRateType(PolicerRateType.forValue(details.rateType)); diff --git a/vpp-classifier/impl/src/main/java/io/fd/hc2vpp/vpp/classifier/write/ClassifyTableWriter.java b/vpp-classifier/impl/src/main/java/io/fd/hc2vpp/vpp/classifier/write/ClassifyTableWriter.java index a88620b35..cfa2ae068 100644 --- a/vpp-classifier/impl/src/main/java/io/fd/hc2vpp/vpp/classifier/write/ClassifyTableWriter.java +++ b/vpp-classifier/impl/src/main/java/io/fd/hc2vpp/vpp/classifier/write/ClassifyTableWriter.java @@ -79,7 +79,10 @@ public class ClassifyTableWriter extends VppNodeWriter public void updateCurrentAttributes(@Nonnull final InstanceIdentifier id, @Nonnull final ClassifyTable dataBefore, @Nonnull final ClassifyTable dataAfter, @Nonnull final WriteContext writeContext) throws WriteFailedException { - throw new UnsupportedOperationException("Classify table update is not supported"); + // TODO(HC2VPP-10): memory_size is not part of classify table dump, so initialization triggers update, + // uncomment after VPP-208 is fixed + // throw new UnsupportedOperationException("Classify table update is not supported"); + LOG.error("Classify table update is not supported, ignoring config"); } @Override diff --git a/vpp-classifier/impl/src/test/java/io/fd/hc2vpp/vpp/classifier/write/ClassifyTableWriterTest.java b/vpp-classifier/impl/src/test/java/io/fd/hc2vpp/vpp/classifier/write/ClassifyTableWriterTest.java index 2a8b3db66..0b156a333 100644 --- a/vpp-classifier/impl/src/test/java/io/fd/hc2vpp/vpp/classifier/write/ClassifyTableWriterTest.java +++ b/vpp-classifier/impl/src/test/java/io/fd/hc2vpp/vpp/classifier/write/ClassifyTableWriterTest.java @@ -21,6 +21,7 @@ import static org.junit.Assert.fail; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.times; import static org.mockito.Mockito.verify; +import static org.mockito.Mockito.verifyZeroInteractions; import static org.mockito.Mockito.when; import io.fd.hc2vpp.common.test.write.WriterCustomizerTest; @@ -173,10 +174,13 @@ public class ClassifyTableWriterTest extends WriterCustomizerTest { customizer.deleteCurrentAttributes(id, classifyTable, writeContext); } - @Test(expected = UnsupportedOperationException.class) + @Test + // TODO: uncomment after HC2VPP-10 is fixed + // (expected = UnsupportedOperationException.class) public void testUpdate() throws Exception { final ClassifyTable classifyTableBefore = generateClassifyTable(TABLE_NAME); final InstanceIdentifier id = getClassifyTableId(TABLE_NAME); customizer.updateCurrentAttributes(id, classifyTableBefore, new ClassifyTableBuilder().build(), writeContext); + verifyZeroInteractions(api); } } \ No newline at end of file -- cgit 1.2.3-korg