summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTomasz Wrobel <tomasz.wrobel@nokia.com>2021-04-12 12:28:16 +0200
committerTomasz Wrobel <tomasz.wrobel@nokia.com>2021-04-12 12:34:26 +0200
commit8b42cf54a9b8fab9a3ac8d37376ec752d00d2883 (patch)
tree829e33979c60c6c66599de25db7f5e10aa148780
parenta39ffbf3cae394c8964e8407b4eee2d39aab4f41 (diff)
Remove support for Helm v21.1.0
- Remove mapping to helm v2 when version desired is not provided - Remove Helm v2 clients from container Issue-ID: SDC-3185 Signed-off-by: Tomasz Wrobel <tomasz.wrobel@nokia.com> Change-Id: I43def179ee35ade76905a88f83bd17b9be787481
-rw-r--r--Changelog.md15
-rw-r--r--Dockerfile2
-rw-r--r--Makefile2
-rw-r--r--README.md5
-rw-r--r--pom.xml2
-rw-r--r--src/main/java/org/onap/sdc/helmvalidator/helm/versions/ChartBasedVersionProvider.java18
-rw-r--r--src/test/java/org/onap/sdc/helmvalidator/helm/versions/ApiVersionsReaderTest.java12
-rw-r--r--src/test/java/org/onap/sdc/helmvalidator/helm/versions/ChartBasedVersionProviderTest.java16
-rw-r--r--version.properties2
9 files changed, 40 insertions, 34 deletions
diff --git a/Changelog.md b/Changelog.md
index 463a1e2..3c167f8 100644
--- a/Changelog.md
+++ b/Changelog.md
@@ -4,10 +4,12 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](http://keepachangelog.com/)
and this project adheres to [Semantic Versioning](http://semver.org/).
-## [0.0.1] - 23/03/2021
-
-- Init Helm client validator project, which is dedicated to be used during Helm packages validation in time of VSP creation
+## [1.1.0] - 12/04/2021
+- Remove mapping to helm v2 when version desired is not provided
+- Remove Helm v2 clients from container
+- Supported Helm Versions:
+ - Helm v3: 3.5.2, 3.4.1, 3.3.4
## [1.0.0] - 24/03/2021
@@ -15,3 +17,10 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
- Supported Helm Versions:
- Helm v3: 3.5.2, 3.4.1, 3.3.4,
- Helm v2: 2.17.0, 2.14.3
+
+## [0.0.1] - 23/03/2021
+
+- Init Helm client validator project, which is dedicated to be used during Helm packages validation in time of VSP creation
+
+
+
diff --git a/Dockerfile b/Dockerfile
index e22a242..a88db20 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -7,7 +7,7 @@ RUN adduser -G onap -D validator
RUN apk add --no-cache bash vim curl wget
-ENV HELM_SUPPORTED_VERSIONS=3.5.2,3.4.1,3.3.4,2.17.0,2.14.3
+ENV HELM_SUPPORTED_VERSIONS=3.5.2,3.4.1,3.3.4
#Installing Helm
COPY scripts/collect_helm_versions_from_web.sh ./opt/helmvalidator/tmp/collect_helm_versions_from_web.sh
diff --git a/Makefile b/Makefile
index a318769..cebd421 100644
--- a/Makefile
+++ b/Makefile
@@ -5,7 +5,7 @@ build-docker:
mvn clean install -Pdocker
build-docker-local:
- export HELM_SUPPORTED_VERSIONS=3.5.2,3.4.1,3.3.4,2.17.0 && \
+ export HELM_SUPPORTED_VERSIONS=3.5.2,3.4.1,3.3.4 && \
cd scripts && \
./collect_helm_versions_from_web.sh local && \
cd ../ && \
diff --git a/README.md b/README.md
index a8c469c..b14820e 100644
--- a/README.md
+++ b/README.md
@@ -49,12 +49,11 @@ This should be a POST with `multipart/form-data` encoding with the following fie
It is possible to provide helm version in three ways:
1. Select exact version from supported versions e.g. "3.4.1"
-2. Select major helm version e.g. "v2" for Helm 2 or "v3" for Helm 3. In this case app will use the latest supported version.
+2. Select major helm version e.g. "v3" for Helm 3. In this case app will use the latest supported version.
3. In case when field version is not present in request then app gets helm version from "apiVersion" field located in the main chart (Chart.yaml).
Mapping rules:
-apiVersion: v1 -> the latest available helm 2
-
+apiVersion: v1 -> the latest available helm 3
apiVersion: v2 -> the latest available helm 3
#### Response
diff --git a/pom.xml b/pom.xml
index 10d2553..b38cca9 100644
--- a/pom.xml
+++ b/pom.xml
@@ -10,7 +10,7 @@
</parent>
<groupId>org.onap.sdc</groupId>
<artifactId>sdc-helm-validator</artifactId>
- <version>1.0.0-SNAPSHOT</version>
+ <version>1.1.0-SNAPSHOT</version>
<name>helmvalidator</name>
<description>Spring-Boot application for validating helm charts</description>
diff --git a/src/main/java/org/onap/sdc/helmvalidator/helm/versions/ChartBasedVersionProvider.java b/src/main/java/org/onap/sdc/helmvalidator/helm/versions/ChartBasedVersionProvider.java
index b892a33..c3e423f 100644
--- a/src/main/java/org/onap/sdc/helmvalidator/helm/versions/ChartBasedVersionProvider.java
+++ b/src/main/java/org/onap/sdc/helmvalidator/helm/versions/ChartBasedVersionProvider.java
@@ -20,6 +20,7 @@
package org.onap.sdc.helmvalidator.helm.versions;
+import java.util.List;
import org.onap.sdc.helmvalidator.helm.versions.exception.NotSupportedApiVersionException;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Service;
@@ -27,6 +28,9 @@ import org.springframework.stereotype.Service;
@Service
public class ChartBasedVersionProvider {
+ private static final List<String> SUPPORTED_API_VERSIONS = List.of("v1", "v2");
+ private static final String HELM_3 = "3";
+
private final SupportedVersionsProvider supportedVersionsProvider;
private final ApiVersionsReader apiVersionsReader;
@@ -40,18 +44,14 @@ public class ChartBasedVersionProvider {
public String getVersion(String chartPath) {
String apiVersion = apiVersionsReader.readVersion(chartPath);
- return mapToChartVersion(apiVersion);
+ return mapToHelmVersion(apiVersion);
}
- private String mapToChartVersion(String apiVersion) {
- switch (apiVersion) {
- case "v1":
- return supportedVersionsProvider.getLatestVersion("2");
- case "v2":
- return supportedVersionsProvider.getLatestVersion("3");
- default:
- throw new NotSupportedApiVersionException("Cannot obtain Helm version from API version: " + apiVersion);
+ private String mapToHelmVersion(String apiVersion) {
+ if (!SUPPORTED_API_VERSIONS.contains(apiVersion)) {
+ throw new NotSupportedApiVersionException("Cannot obtain Helm version from API version: " + apiVersion);
}
+ return supportedVersionsProvider.getLatestVersion(HELM_3);
}
}
diff --git a/src/test/java/org/onap/sdc/helmvalidator/helm/versions/ApiVersionsReaderTest.java b/src/test/java/org/onap/sdc/helmvalidator/helm/versions/ApiVersionsReaderTest.java
index 79ea56a..2dbb6c1 100644
--- a/src/test/java/org/onap/sdc/helmvalidator/helm/versions/ApiVersionsReaderTest.java
+++ b/src/test/java/org/onap/sdc/helmvalidator/helm/versions/ApiVersionsReaderTest.java
@@ -36,13 +36,12 @@ import org.apache.commons.compress.compressors.gzip.GzipCompressorOutputStream;
import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
-import org.junit.jupiter.params.ParameterizedTest;
-import org.junit.jupiter.params.provider.ValueSource;
import org.onap.sdc.helmvalidator.helm.versions.exception.ApiVersionNotFoundException;
import org.onap.sdc.helmvalidator.helm.versions.exception.ReadFileException;
class ApiVersionsReaderTest {
+ private static final String API_VERSION_V2= "v2";
private static final String TEST_RESOURCES_TMP = "src/test/resources/tmp";
private static final Path TEST_CHART_PATH = Path.of(TEST_RESOURCES_TMP).resolve(Path.of("Chart.yaml"));
@@ -55,14 +54,13 @@ class ApiVersionsReaderTest {
apiVersionsReader = new ApiVersionsReader();
}
- @ParameterizedTest
- @ValueSource(strings = {"v1", "v2"})
- void shouldCorrectlyReadApiVersionFromTar(String apiVersion) throws IOException {
- prepareTestTar(apiVersion);
+ @Test
+ void shouldCorrectlyReadApiVersionFromTar() throws IOException {
+ prepareTestTar(API_VERSION_V2);
String helmVersion = apiVersionsReader.readVersion(TEST_TAR_PATH.toString());
- assertThat(helmVersion).isEqualTo(apiVersion);
+ assertThat(helmVersion).isEqualTo(API_VERSION_V2);
}
@Test
diff --git a/src/test/java/org/onap/sdc/helmvalidator/helm/versions/ChartBasedVersionProviderTest.java b/src/test/java/org/onap/sdc/helmvalidator/helm/versions/ChartBasedVersionProviderTest.java
index 64daf2d..7819aa5 100644
--- a/src/test/java/org/onap/sdc/helmvalidator/helm/versions/ChartBasedVersionProviderTest.java
+++ b/src/test/java/org/onap/sdc/helmvalidator/helm/versions/ChartBasedVersionProviderTest.java
@@ -27,8 +27,6 @@ import static org.mockito.Mockito.when;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.extension.ExtendWith;
-import org.junit.jupiter.params.ParameterizedTest;
-import org.junit.jupiter.params.provider.CsvSource;
import org.mockito.Mock;
import org.mockito.Mockito;
import org.mockito.junit.jupiter.MockitoExtension;
@@ -37,6 +35,9 @@ import org.onap.sdc.helmvalidator.helm.versions.exception.NotSupportedApiVersion
@ExtendWith(MockitoExtension.class)
class ChartBasedVersionProviderTest {
+ private static final String EXPECTED_HELM_VERSION = "3.4.3";
+ private static final String API_VERSION = "v2";
+
private final String testChartPath = "test/path";
@Mock
private SupportedVersionsProvider versionsProvider;
@@ -49,15 +50,14 @@ class ChartBasedVersionProviderTest {
chartBasedVersionProvider = new ChartBasedVersionProvider(versionsProvider, apiVersionsReader);
}
- @ParameterizedTest
- @CsvSource({"v1,2.18", "v2,3.11"})
- void shouldGetLatestHelmVersionBasedOnApiVersion(String apiVersion, String expectedHelmVersion) {
- when(apiVersionsReader.readVersion(testChartPath)).thenReturn(apiVersion);
- when(versionsProvider.getLatestVersion(Mockito.anyString())).thenReturn(expectedHelmVersion);
+ @Test
+ void shouldGetLatestHelmVersionBasedOnApiVersion() {
+ when(apiVersionsReader.readVersion(testChartPath)).thenReturn(API_VERSION);
+ when(versionsProvider.getLatestVersion(Mockito.anyString())).thenReturn(EXPECTED_HELM_VERSION);
String helmVersion = chartBasedVersionProvider.getVersion(testChartPath);
- assertThat(helmVersion).isEqualTo(expectedHelmVersion);
+ assertThat(helmVersion).isEqualTo(EXPECTED_HELM_VERSION);
}
@Test
diff --git a/version.properties b/version.properties
index 2ddebb3..7b8b963 100644
--- a/version.properties
+++ b/version.properties
@@ -1,5 +1,5 @@
major=1
-minor=0
+minor=1
patch=0
base_version=${major}.${minor}.${patch}
release_version=${base_version}