Fix kubernetesVersion references

This commit is contained in:
Vincent Fiduccia 2021-02-02 16:27:18 -07:00
parent e5797ded25
commit aa051feadd
No known key found for this signature in database
GPG Key ID: 2B29AD6BB2BB2582
2 changed files with 40 additions and 25 deletions

View File

@ -119,14 +119,18 @@ export default {
if (name.includes(':')) { if (name.includes(':')) {
const pairs = name.split('\n'); const pairs = name.split('\n');
const clusterVersion = this.currentCluster.kubernetesVersion; const clusterVersion = this.currentCluster.kubernetesVersionDisplay;
pairs.forEach((pair) => { pairs.forEach((pair) => {
const version = (pair.match(/[<>=]+[-._a-zA-Z0-9]+/) || [])[0]; const version = (pair.match(/[<>=]+[-._a-zA-Z0-9]+/) || [])[0];
try {
if (semver.satisfies(clusterVersion, version)) { if (semver.satisfies(clusterVersion, version)) {
name = pair.replace(/[<>=]+[-._a-zA-Z0-9]+: /, ''); name = pair.replace(/[<>=]+[-._a-zA-Z0-9]+: /, '');
} }
} catch (e) {
// Ignore entries with invalid semver
}
}); });
} }
if (name) { if (name) {
@ -189,13 +193,15 @@ export default {
methods: { methods: {
validateBenchmark(benchmark, currentCluster) { validateBenchmark(benchmark, currentCluster) {
const clusterVersion = currentCluster.kubernetesVersion;
if (!!benchmark?.spec?.clusterProvider) { if (!!benchmark?.spec?.clusterProvider) {
if ( benchmark?.spec?.clusterProvider !== currentCluster.status.provider) { if ( benchmark?.spec?.clusterProvider !== currentCluster.status.provider) {
return false; return false;
} }
} }
try {
const clusterVersion = currentCluster.kubernetesVersionDisplay;
if (benchmark?.spec?.minKubernetesVersion) { if (benchmark?.spec?.minKubernetesVersion) {
if (semver.gt(benchmark?.spec?.minKubernetesVersion, clusterVersion)) { if (semver.gt(benchmark?.spec?.minKubernetesVersion, clusterVersion)) {
return false; return false;
@ -206,6 +212,9 @@ export default {
return false; return false;
} }
} }
} catch (e) {
// Ignore error if something is invalid semver
}
return true; return true;
}, },

View File

@ -74,11 +74,13 @@ export default {
// filter benchmarks by spec.clusterProvider and kubernetes min/max version // filter benchmarks by spec.clusterProvider and kubernetes min/max version
// include benchmarks with no clusterProvider defined // include benchmarks with no clusterProvider defined
validateBenchmark(benchmark) { validateBenchmark(benchmark) {
const clusterVersion = this.currentCluster.kubernetesVersion;
if (!!benchmark?.spec?.clusterProvider) { if (!!benchmark?.spec?.clusterProvider) {
return benchmark?.spec?.clusterProvider === this.provider; return benchmark?.spec?.clusterProvider === this.provider;
} }
try {
const clusterVersion = this.currentCluster.kubernetesVersionDisplay;
if (benchmark?.spec?.minKubernetesVersion) { if (benchmark?.spec?.minKubernetesVersion) {
if (semver.gt(benchmark?.spec?.minKubernetesVersion, clusterVersion)) { if (semver.gt(benchmark?.spec?.minKubernetesVersion, clusterVersion)) {
return false; return false;
@ -91,6 +93,10 @@ export default {
} }
return true; return true;
} catch (e) {
// If a version doesn't parse, show it
return true;
}
} }
} }
}; };