-
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
fix(executor): GODEBUG=x509ignoreCN=0 #5562
Conversation
Signed-off-by: Alex Collins <alex_collins@intuit.com>
Codecov Report
@@ Coverage Diff @@
## master #5562 +/- ##
==========================================
- Coverage 47.04% 47.01% -0.03%
==========================================
Files 240 240
Lines 15013 15016 +3
==========================================
- Hits 7063 7060 -3
- Misses 7052 7058 +6
Partials 898 898
Continue to review full report at Codecov.
|
workflow/controller/workflowpod.go
Outdated
@@ -535,6 +535,13 @@ func (woc *wfOperationCtx) createEnvVars() []apiv1.EnvVar { | |||
Name: common.EnvVarContainerRuntimeExecutor, | |||
Value: woc.getContainerRuntimeExecutor(), | |||
}, | |||
// This flag was introduced in Go 16 and will be removed in Go 16. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
xxx 16 and xxxx 16?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will fix this
Next steps.
|
@simster7 @sarabala1979 this appears to have fixed the issue on GCP. |
Signed-off-by: Alex Collins <alex_collins@intuit.com>
Signed-off-by: Alex Collins <alex_collins@intuit.com>
To test this,lets merge it and make sure https://workflows.apps.argoproj.io/ can run coin flip