Skip to content

fix(azure): update argo workflows to store artifacts correctly#835

Merged
mrsimonemms merged 1 commit intomainfrom
sje/azure-argo-fix
Nov 27, 2024
Merged

fix(azure): update argo workflows to store artifacts correctly#835
mrsimonemms merged 1 commit intomainfrom
sje/azure-argo-fix

Conversation

@mrsimonemms
Copy link
Copy Markdown
Contributor

@mrsimonemms mrsimonemms commented Nov 26, 2024

Description

Updates the Argo workflow configuration is done so that the artifacts are stored correctly. The storage container uses credentials from the ChartMuseum Vault secret.

This now publishes the Metaphor app to the cluster:

I've added Reload annotations to the server and controller for the configmap and secrets used to reload the pods if the configmap and secret changes - if there's an already workflow, this doesn't affect it as they're run as separate jobs (but it would likely fail anyway as the configuration has changed).

Related Issue(s)

Fixes #

How to test

Deploy an Azure app and watch Metaphor run. I have demos if you want to pair

@mrsimonemms mrsimonemms marked this pull request as ready for review November 26, 2024 15:07
@jokestax
Copy link
Copy Markdown
Contributor

Hey @mrsimonemms ,do we have logic to detokenize <KUBEFIRST_STATE_STORE_CONTAINER_NAME> in api?

@mrsimonemms
Copy link
Copy Markdown
Contributor Author

Hey @mrsimonemms ,do we have logic to detokenize <KUBEFIRST_STATE_STORE_CONTAINER_NAME> in api?

@jokestax yup

@jokestax
Copy link
Copy Markdown
Contributor

cool 👍

@mrsimonemms mrsimonemms merged commit fa380f4 into main Nov 27, 2024
@mrsimonemms mrsimonemms deleted the sje/azure-argo-fix branch November 27, 2024 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants