-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Workflow-Controller panic: can't create an event with namespace 'default' in namespace 'argo'' #7613
Comments
@shuker85 how do you install argo workflow? |
Helm chart handled by app-of-apps pattern |
Leader election is confused somehow, thinking it is is the wrong namespace. Are you using managed namespace feature where controller is in different namespace to workflows? |
Hi! I'm also running into this error. In my case, I've installed Argo Workflows as part of Kubeflow, and yes, our workflows are in different namespaces from the controller. Can you suggest any other steps I can try to troubleshoot? Thank you! |
please add stack trace |
Thanks @alexec!
|
Are you using |
The panic is a red herring. It is caused by the shutdown, but will not have impact. Any fix for that would go here, but I think it is just annoying, so not needed: argo-workflows/workflow/metrics/server.go Line 54 in 06d4bf7
The event issue is coming from here, but I can't see the problem:
There is probably a bug in I think all we need to do is this: ReleaseOnCancel: false, @shuker85 would you like to submit a PR? |
Should fix argoproj#7613 Signed-off-by: Shyukri Shyukriev <shyukri.shyukriev@mariadb.com>
Summary
I've noticed a unexpected restart of the pod
What happened/what you expected to happen?
This is instance managed by ArgoCD instance and currently not being actively used
What version is it broken in?
v3.2.6
What version was it working in?
Diagnostics
What Kubernetes provider are you using?
GKE on 1.21.5-gke.1802
What executor are you running? Docker/K8SAPI/Kubelet/PNS/Emissary
Emissary
Message from the maintainers:
Impacted by this regression? Give it a 👍. We prioritise the issues with the most 👍.
The text was updated successfully, but these errors were encountered: