Tianchu Zhao

Results 16 issues of Tianchu Zhao

I wonder if we can expose more `Arguments` to `Feature option`? For example, for go-install script, we would like to configure `GOPATH` due to some code generation requirements. We understand...

Fixes #9627 Fixes #9772 Fixes #9858 [view_artifact_archive_wf.webm](https://user-images.githubusercontent.com/20961507/196018889-44324bb1-6a5e-4920-97d6-8b28a6755f6c.webm) Could break the PR into refactor + feat Please do not open a pull request until you have checked ALL of these: *...

## Summary ``` if os.Getenv("CI") == "true" { t.SkipNow() } privateKey, err := os.ReadFile(homedir.HomeDir() + "/.ssh/id_rsa") assert.NoError(t, err) driver := &ArtifactDriver{SSHPrivateKey: string(privateKey)} assert.NoError(t, load(driver, &wfv1.GitArtifact{Repo: "[email protected]:argoproj-labs/private-test-repo.git"})) assert.FileExists(t, path+"/README.md") ``` https://github.com/argoproj/argo-workflows/blob/master/workflow/artifacts/git/git_test.go#L29-L36...

tech-debt

Tested with UI - loading cluster wf template - no cluster wf template - exist cluster wf template --- Please do not open a pull request until you have checked...

# Summary Say for UI features, new feature added in workflow page would likely be useful in archived workflow as well, but right now we have both class and functional...

type/feature
type/tech-debt
area/ui
solution/suggested

# Summary Currently argo `ArchiveLogs` only supports logs from `main` container. From time to time user is asking if it supports `init` and `wait` container log. I'm aware the archivelog...

type/feature
area/archive-logs

Fixes #11984 ### Motivation Given a certain parallelism limit. If a workflow couldn't execute due to workflow level synchronization lock (semaphore or mutex), the blocked workflow will release the parallelism...

area/controller
area/mutex-semaphore
area/parallelism

### Pre-requisites - [X] I have double-checked my configuration - [X] I can confirm the issue exists when I tested with `:latest` - [X] I have searched existing issues and...

type/bug
area/controller
area/templates/dag
P1

Fixes #12997 approach 1, connect dag nodes to the specified dependency instead of the deepest childnode of the dependency breaking change, potentially affects many other area of the dag code...

Fixes #12997 ### Motivation Root cause: say we have dag ``` task: A(containerset) task: B(containerset), depends: A task: C(containerset), depends: A ``` if task B is processed and started before...

area/controller
area/templates/dag