-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
UI (when 2 tabs open) 3.6.0rc4 - submit workflowtemplate greyed out #13892
Comments
for this to happen looks like you need 2 workflow tabs open (in any of chrome or safari), workaround is to hard refresh via hold down command+shift keys and click the refresh icon in address bar @MasonM maybe caused by 68adbcc#diff-0494089cd084aa91c5d8e043b283fe327f53ecf94623bf82be945dea2e2587fd ? |
I can reproduce this on firefox 131 on linux. I don't need a hard refresh, just a normal refresh to make the button appear, and the button appears greyed out without 2 tabs. |
@tooptoop4 The "submit" button was completely broken before that change for the reasons described at #13593, so it definitely didn't cause this. If this was a regression, it dates back at least to #13069 |
Looking at this more, I think this is a consequence of reference equality checking. In my original fix for #13593, I had it use deep equality comparisons, which doesn't have this problem. However, I reverted that in 24c0206 because Anton had concerns about performance (see #13593 (comment)). I did figure out a way to fix this without affecting performance (see #13593 (comment)), but the problem is that's a significant change, and I don't know if there's anyone left that can code review such a change. So I think we're just going to have to live with this until there's someone available to code review UI changes. |
@MasonM I did initially start on argo with some UI contributions, I am a bit rusty, but I can review your code. |
@isubasinghe Thanks! I cleaned up the POC and entered #13915 for this |
) Signed-off-by: instauro <[email protected]> Co-authored-by: instauro <[email protected]> (cherry picked from commit 2fd5488)
… (#13915) Signed-off-by: Mason Malone <[email protected]> (cherry picked from commit 1392ef5)
Pre-requisites
:latest
image tag (i.e.quay.io/argoproj/workflow-controller:latest
) and can confirm the issue still exists on:latest
. If not, I have explained why, in detail, in my description below.What happened? What did you expect to happen?
in the ui it lets me create a new workflowtemplate and update it, but the button to submit it is greyed out. i'm using the quickstart
Version(s)
3.6.0rc4
Paste a minimal workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
default
Logs from the workflow controller
Logs from in your workflow's wait container
The text was updated successfully, but these errors were encountered: