fix: argo sempahores ConfigMap metadata name is wrong

linz · Oct 12, 2023 · 40af936 · 40af936
1 parent 1088c44
commit 40af936
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git 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<void> {
-  //   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
@@ -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