Use private kn-workflow for gen-manifests

[rgolangh]

Signed-off-by: Roy Golan [email protected]

[rgolangh]

/hold
I think I see a problem with the created manifests.

[rgolangh]

mm the name of the manifest of the workflow file is now 03-sonataflow_mtaanalysis.yaml. And not 01...
We would have to 1. remove the existing manifests (just once) 2. change gen-manifest script to look for *-sonataflow_$workflow_id.yaml

[masayag]

mm the name of the manifest of the workflow file is now 03-sonataflow_mtaanalysis.yaml. And not 01... We would have to 1. remove the existing manifests (just once) 2. change gen-manifest script to look for *-sonataflow_$workflow_id.yaml

Would you prefer to include the changes before submitting this PR or as part of it?

[masayag]

Let's merge this on Sunday :-)
But before that, shellcheck complains:

In scripts/gen_manifests.sh line 50:
SONATAFLOW_CR="$(printf '%s' manifests/*-sonataflow_${workflow_id}.yaml)"
                                                    ^------------^ SC208[6](https://github.com/parodos-dev/serverless-workflows/actions/runs/10075226922/job/27853023783?pr=335#step:4:7) (info): Double quote to prevent globbing and word splitting.

[rgolangh]

Let's merge this on Sunday :-) But before that, shellcheck complains:

In scripts/gen_manifests.sh line 50:
SONATAFLOW_CR="$(printf '%s' manifests/*-sonataflow_${workflow_id}.yaml)"
                                                    ^------------^ SC208[6](https://github.com/parodos-dev/serverless-workflows/actions/runs/10075226922/job/27853023783?pr=335#step:4:7) (info): Double quote to prevent globbing and word splitting.

why wait? we need every bit to help :)
I saw the shellcheck error, will fix

[masayag]

@rgolangh do the call to m2k might also need a token which is missed?

++ curl -XGET -s -o /dev/null -w '%{http_code}' http://localhost:9080/api/orchestrator/workflows/m2k
Handling connection for 9080
+ M2K_STATUS=401

[masayag]

@rgolangh going to merge this as-is since the only remaining changes are around adding authorization header to other MTA calls.

[rgolangh]

@rgolangh do the call to m2k might also need a token which is missed?

++ curl -XGET -s -o /dev/null -w '%{http_code}' http://localhost:9080/api/orchestrator/workflows/m2k
Handling connection for 9080
+ M2K_STATUS=401

was fixed with latest push