Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v2.11.2: Unsuccessful HTTP response #4231

Closed
meurig opened this issue Oct 7, 2020 · 4 comments · Fixed by #4202
Closed

v2.11.2: Unsuccessful HTTP response #4231

meurig opened this issue Oct 7, 2020 · 4 comments · Fixed by #4202
Labels
Milestone

Comments

@meurig
Copy link

meurig commented Oct 7, 2020

Summary

Upon upgrading from v2.10 to v2.11 I'm faced with the following error:

image

The link address is:
https://argo-dev/api/v1/workflows/argo?fields=metadata,items.metadata.uid,items.metadata.name,items.metadata.namespace,items.metadata.labels,items.status.phase,items.status.finishedAt,items.status.startedAt,items.spec.suspend

Hitting that address directly gives me:
{"code":2,"message":"v1alpha1.WorkflowList.Items: v1alpha1.Workflows: v1alpha1.Workflow.Spec: v1alpha1.WorkflowSpec.Templates: []v1alpha1.Template: v1alpha1.Template.DAG: v1alpha1.DAGTemplate.Tasks: []v1alpha1.DAGTask: v1alpha1.DAGTask.v1alpha1.DAGTask.Template: Name: Arguments: v1alpha1.Arguments.Parameters: []v1alpha1.Parameter: v1alpha1.Parameter.Value: ReadString: expects " or n, but found 1, error found in #10 byte of ...|,"value":1}]},"name"|..., bigger context ...|uments":{"parameters":[{"name":"seconds","value":1}]},"name":"start","template":"sleep-n-sec"},{"arg|..."}

Downgrading to v2.10.2 solves the issue. I've also tried with v2.11.0 and the issue returns.

I've not tried with a fresh install of argo yet.

Diagnostics

What Kubernetes provider are you using?
v1.19.2 (running on hyper-v)

What version of Argo Workflows are you running?
v2.11.2 (installed with help chart v0.12.2)


Message from the maintainers:

Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

@meurig meurig added the type/bug label Oct 7, 2020
@meurig
Copy link
Author

meurig commented Oct 7, 2020

I'm not sure if this is related to this issue:
argoproj/argo-helm#462

@meurig
Copy link
Author

meurig commented Oct 7, 2020

Looking more closely at the response from that address I can see:
ReadString: expects " or n, but found 1, error found in #10 byte of ...|,"value":1}]},"name"|..., bigger context ...|uments":{"parameters":[{"name":"seconds","value":1}]},"name":"start","template":"sleep-n-sec"},{"arg|..."}

This led me to suspect the issue is in one of my workflows. Running the following seems to have fixed it:
argo -n argo delete --all

Sorry for the false alarm!

@alexec
Copy link
Contributor

alexec commented Oct 7, 2020

You have a malformed workflow in your system. This is typically because there is a number where there must be a string. This workflow will not execute.

In your case, you have a number in your input parameters.

Options:

  • Find and delete the malformed workflow.
  • Find and fix the malformed workflow.
  • Wait for v2.11.2 which will tolerate this problem.

@alexec alexec linked a pull request Oct 7, 2020 that will close this issue
6 tasks
@alexec alexec added this to the v2.11 milestone Oct 7, 2020
@meurig
Copy link
Author

meurig commented Oct 7, 2020

Thanks @alexec !

I'll try and remove the workflows that are causing the problem, but I'll also look forward to your fix in the next version (I think it will be v2.11.3).

Thanks for all the options! 👍

@whynowy whynowy closed this as completed Oct 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants