zishuo/onepanel
1
0
Fork 0
mirror of https://github.com/onepanelio/onepanel.git synced 2025-10-05 13:46:51 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Checking for istio injection by checking if the sidecar has "tty"

Browse Source 
set to true.
This commit is contained in:
Aleksandr Melnikov
2020-11-06 16:10:10 -08:00
parent 0f226eb0b7
commit d8101bff8e
1 changed files with 7 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
pkg/workflow_execution.go
View File

@@ -279,12 +279,13 @@ func (c *Client) injectAutomatedFields(namespace string, wf *wfv1.Workflow, opts
template.Metadata.Annotations = make(map[string]string)
}
template.Metadata.Annotations["sidecar.istio.io/inject"] = "false"
//todo not sure if we need istio yet
if template.Metadata.Labels != nil {
if sidecar, ok := template.Metadata.Labels["sidecar"]; ok {
if sidecar == "sys-visualization-sidecar" {
//For workflows with accessible sidecars, we need istio
//Istio does not prevent the main container from stopping
for _, s := range template.Sidecars {
if s.TTY == true {
template.Metadata.Annotations["sidecar.istio.io/inject"] = "true"
}
//Only need one instance to require istio injection
break
}
}