One invalid WorkflowTemplate make all WorkflowTemlate invisible

[toyamagu-2021]

Pre-requisites

• I have double-checked my configuration
• I can confirm the issues exists when I tested with :latest
• I'd like to contribute the fix myself (see contributing guide)

What happened/what you expected to happen?

• If we have one invalid WorkflowTemplates the argo-server returns error response InternalServeError, then entire WorkflowTemplates are invisible.

• 

• cli

$ argo template -n argo-workflows list
2023/06/29 21:08:17 rpc error: code = Internal desc = json: cannot unmarshal string into Go struct field Container.items.spec.templates.container.command of type []string

Version

latest(v3.4.8)

Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.

apiVersion: argoproj.io/v1alpha1
kind: WorkflowTemplate
metadata:
  name: good
  namespace: argo-workflows
spec:
  entrypoint: whalesay-template
  templates:
    - name: whalesay-template
      container:
        image: docker/whalesay
        command: [cowsay]
---
apiVersion: argoproj.io/v1alpha1
kind: WorkflowTemplate
metadata:
  name: bad
  namespace: argo-workflows
spec:
  entrypoint: whalesay-template
  templates:
    - name: whalesay-template
      container:
        image: docker/whalesay
        command: this-is-invalid


-

Logs from the workflow controller

Logs from workflow server


argo-workflows-server-69ffff554c-7qskg argo-server time="2023-06-29T12:08:03.355Z" level=info duration=6.17825ms method=GET path=/api/v1/workflows/argo size=52 status=0
argo-workflows-server-69ffff554c-7qskg argo-server time="2023-06-29T12:08:17.239Z" level=error msg="finished unary call with code Internal" error="rpc error: code = Internal desc = json: cannot unmarshal string into Go struct field Container.items.spec.templates.container.command of type []string" grpc.code=Internal grpc.method=ListWorkflowTemplates grpc.service=workflowtemplate.WorkflowTemplateService grpc.start_time="2023-06-29T12:08:17Z" grpc.time_ms=2.287 span.kind=server system=grpc
argo-workflows-server-69ffff554c-7qskg argo-server time="2023-06-29T12:08:17.239Z" level=info duration=2.655417ms method=GET path=/api/v1/workflow-templates/argo-workflows size=140 status=500

Logs from in your workflow's wait container

N/A

[terrytangyuan]

This is an issue on Argo Server side. It's using the generated Argo client's list method. Not sure if there's anything we can customize though.

[toyamagu-2021]

Thanks! Agree.
I dug into the code but only knew it's not simple.
We might change the list logic considerably...

• https://github.com/argoproj/argo-workflows/blob/f180335b370643b731edcd133b7ef35de36a83e6/pkg/client/clientset/versioned/typed/workflow/v1alpha1/workflowtemplate.go#L64C10-L78

[terrytangyuan]

Yes exactly

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

[tooptoop4]

related to #10229 / #10264 / #9316