search

parodos-dev / orchestrator-helm-chart

Helm chart to deploy the Orchestrator solution suite.
https://parodos.dev/orchestrator-helm-chart/
Apache License 2.0
2 stars 22 forks source link

orchestrator-k8s: update orchestrator plugin version #266

Closed gciavarrini closed 2 weeks ago

gciavarrini commented 2 weeks ago

Bump @janus-idp/backstage-plugin-orchestrator-backend-dynamic to 1.21.0 @janus-idp/backstage-plugin-orchestrator to 1.22.0 into orchestrator-k8s values.

Part of FLPATH-1576

masayag commented 2 weeks ago

The PR is missing a bump of the k8s-orchestrator chart version.

masayag commented 2 weeks ago

@gciavarrini since we're using this method to install the chart: https://github.com/parodos-dev/serverless-workflows/blob/main/.github/workflows/move2kube-e2e.yaml#L84 if the chart isn't the latest one, we won't see the updates. Done by https://github.com/parodos-dev/orchestrator-helm-chart/commit/6972ba6b2a9ec8b55b7be70384a1642616ce4ca9

rgolangh commented 2 weeks ago

we don't need to bump for every pr, just when it makes sense. specifically here there maybe more planned changes or upcoming prs to merge (I have one in my pipeline)

rgolangh commented 2 weeks ago

so just to make the case, I merged it with knowing there is no need for a version yet

masayag commented 2 weeks ago

it is needed for https://github.com/parodos-dev/serverless-workflows/pull/398

gciavarrini commented 2 weeks ago

@masayag thanks for the fix in https://github.com/parodos-dev/orchestrator-helm-chart/commit/6972ba6b2a9ec8b55b7be70384a1642616ce4ca9 I forgot to increase the version