From 40af9362671b5a9172c667c779b6b3b16d7b3271 Mon Sep 17 00:00:00 2001 From: Paul Fouquet Date: Fri, 13 Oct 2023 08:54:32 +1300 Subject: [PATCH] fix: argo sempahores ConfigMap metadata name is wrong --- config/app.ts | 3 --- config/charts/argo.semaphores.ts | 4 ++++ 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/config/app.ts b/config/app.ts index 25ad9aff..0e5100da 100644 --- a/config/app.ts +++ b/config/app.ts @@ -5,9 +5,6 @@ import { ArgoSemaphore } from './charts/argo.semaphores'; const app = new App(); async function main(): Promise { - // const ns = new NamespaceChart(app, 'namespace', { namespace: 'argo' }); - - // FIXME: metadata name is not correct, like 'semaphore-semaphores-c8f222a7' new ArgoSemaphore(app, 'semaphore', {}); app.synth(); diff --git a/config/charts/argo.semaphores.ts b/config/charts/argo.semaphores.ts index 3d3798a4..fe2faddf 100644 --- a/config/charts/argo.semaphores.ts +++ b/config/charts/argo.semaphores.ts @@ -9,6 +9,10 @@ export class ArgoSemaphore extends Chart { super(scope, id, applyDefaultLabels(props, 'argo', 'v1', 'semaphores', 'workflows')); new kplus.ConfigMap(this, 'semaphores', { + metadata: { + name: 'semaphores', + namespace: 'argo', + }, data: { standardising: '2', // Limit of how many standardising workflow instances can run at the same time bulk: '4', // Limit of how many bulk workflow instances can run at the same time