v3.1.4 cherry-pick

[sarabala1979]

Few PRs are already released in v3.1.3. Please select your PR that needs to release in v3.1.4

• Saravanan Balasubramanian: fix: Exit template shouldn't fail with max parallelism reached (fix(controller): Exit template shouldn't fail with max parallelism reached #6456) 5fde8fa
• Marcin Gucki: fix(controller): fix retry on different hosts (fix(controller): fix retry on different hosts #6429) c5d2461
• Peixuan Ding: fix(server): Fix nil pointer error when getting artifacts from a step without artifacts (fix(server): Fix nil pointer error when getting artifacts from a step without artifacts #6465) 0f6f362
• Daisuke Taniwaki: docs: Add ubie to users (docs: Add Ubie to users #6462) a250bab
• Mohammad Ali: feat(server): Support OIDC custom claims for AuthN. Closes Allow specifying the payload key for groups claim #5953 (feat(server): Support OIDC custom claims for AuthN. Closes #5953 #6444) 9034152
• Antoine Dao: fix(pods): set resources from script templates (fix(controller): set resources from script templates #6450) 3e9d837
• Alex Collins: docs: add 30m to issue template (docs: add 30m to issue template #6446) a39725d
• Simon Behar: fix: Do not display clipboard if there is no text (fix(ui): Do not display clipboard if there is no text #6452) 3abeb01
• Saravanan Balasubramanian: feat(controller): HTTP Template and Agent support feature (feat(controller): HTTP Template and Agent support  #5750) b16a0a0
• github-actions[bot]: docs: updated CHANGELOG.md (docs: updated CHANGELOG.md #6438) 20efb52
• dpeer6: feat(server): support changing MaxGRPCMessageSize using env variable (feat(server): support changing MaxGRPCMessageSize using env variable #6420) dc043ce
• Alex Collins: fix(controller): Reinstate support for outputs.results for containers. Fixes chore: upgrade argo events to v1.4.0 #6428 (fix(controller): Reinstate support for outputs.results for containers. Fixes #6428 #6434) 51c1576
• jibuji: fix: support archive.none for OSS directory artifacts (fix: support archive.none for OSS directory artifacts #6312) 40b0824
• Saravanan Balasubramanian: fix(controller): Same workflow nodes are not executing parallel even semaphore locks available (fix(controller): Same workflow nodes are not executing parallel even semaphore locks available #6418) 7ec5b3e
• Saravanan Balasubramanian: fix(controller): Randomly expr expression fail to resolve (fix(controller): Randomly expr expression fail to resolve #6410) c29b275
• Yuan (Bob) Gong: docs(executor): emissary executor also runs on GKE autopilot (docs(executor): emissary executor also runs on GKE autopilot #6430) f1792f6
• Huan-Cheng Chang: fix(controller/cli): Resolve global artifacts created in nested workflows (fix(controller/cli): Resolve global artifacts created in nested workflows #6422) dd3c112
• Antoine Dao: fix(emissary): throw argo error on file not exist (fix(emissary): throw argo error on file not exist #6392) b17d1bf
• Alex Collins: fix(executor): Remove 15s guard for Docker executor. Fixes wait container won't exit #6415 (fix(executor): Remove 15s guard for Docker executor. Fixes #6415 #6427) 946e4a4
• Luces Huayhuaca: chore: upgrade to k8s v0.20.4 (chore: upgrade to k8s v0.20.4 #6143) 96b165e
• Alex Collins: fix(executor): remove unused import preventing compilation 29ebc2a
• Joe McGovern: feat(controller): opt-in to sending pod node events as pod (feat(controller): opt-in to sending pod node events as pod #6377) cc701a1
• Tianchu Zhao: feat(artifact): enable gcs ListObjects (feat(executor): enable gcs artifact ListObjects, supports data-transformations #6409) 959ce6b
• Alex Collins: chore: correct CODEOWNERS syntax 8966006
• Alex Collins: chore: add @alexec as proto codeowner f47ada3
• Yuan (Bob) Gong: fix(executor/emissary): fix nonroot sidecars + input/output params & artifacts (fix(executor/emissary): fix nonroot sidecars + input/output params & artifacts #6403) 30e2518
• Saravanan Balasubramanian: fix(controller): Global parameter is not getting updated (fix(controller): Global parameter is not getting updated #6401) 4da8fd9
• Alex Collins: docs: updated sidecar-injection.md c4bdc21
• Alex Collins: fix(controller): Force main container name to be "main" as per v3.0. Fixes invalid container names (i.e. not "main") are allowed and causes bugs #6405 (fix(controller): Force main container name to be "main" as per v3.0. Fixes #6405 #6408) f2d24b1
• Tianchu Zhao: fix(executor): fix GCS artifact retry (fix(executor): fix GCS artifact retry #6302) 2df5f66
• Dominik Deren: docs: updating documentation on how to setup IRSA on AWS (docs: updating documentation on how to setup IRSA on AWS #6406) c80f4bc
• Alex Collins: docs: Updating upgrading.md. Closes WorkflowStep when condition behavior changes after v3.1 #6314 10c0fa5
• Alex Collins: fix(controller): Mark workflows wait for semaphore as pending. Fixes Liveness probe fails with 500 and workflow never reconciled when using semaphores #6351 (fix(controller): Mark workflows wait for semaphore as pending. Fixes #6351 #6356) 092b427
• Tianchu Zhao: fix(controller): allow initial duration to be 0 instead of current_time-0 (fix(controller): allow initial duration to be 0 instead of current_time-0 #6389) 4535396
• Yuan Tang: docs: Add note on additional required permission for createBucketIfNotPresent for OSS driver (docs: Add note on additional required permission for createBucketIfNotPresent for OSS driver #6378) 2bf711b
• Thiago Bittencourt Gil: docs: Add 4intelligence ( docs: Add 4intelligence #6400) b15a79c
• Alexander Matyushentsev: fix: Server crash when opening timeline tab for big workflows (fix: Server crash when opening timeline tab for big workflows #6369) f4b89dc
• Alex Collins: Revert "fix: examples/ci.yaml indent (fix: examples/ci.yaml indent #6328)" 99359a0
• Antoine Dao: fix(gcs): throw argo not found error if key not exist (fix(gcs): throw argo not found error if key not exist #6393) 66c4410
• Zadkiel: chore: import grafana dashboard (chore: import grafana dashboard #6365) 2c25259
• KUNG HO BACK: fix: examples/ci.yaml indent (fix: examples/ci.yaml indent #6328) 3f72fe5
• Tianchu Zhao: fix(controller): fix retry on transient errors when validating workflow spec (fix(controller): fix retry on transient errors when validating workflow spec #6370) 9233a8d
• Tianchu Zhao: fix(controller): allow workflow.duration to pass validator (fix(controller): allow workflow.duration to pass validator #6376) 488aec3
• Yuan Tang: feat(controller): Allow configurable host name label key when retrying different hosts (feat(controller): Allow configurable host name label key when retrying different hosts #6341) d6ec032
• Antoine Dao: fix(fields): handle nexted fields when excluding (fix(fields): handle nexted fields when excluding #6359) bd5a8a9
• BOOK: feat(controller): sortDAGTasks supports sort by field Depends (feat(controller): sortDAGTasks supports sort by field Depends #6307) cfab7db
• Yuan Tang: fix(cli): Overridding name/generateName when creating CronWorkflows if specified (fix(cli): Overridding name/generateName when creating CronWorkflows if specified #6308) 6e58b35
• Tianchu Zhao: docs(users): Add WooliesX (docs(users): Add WooliesX #6358) 5f0d6ab
• github-actions[bot]: docs: updated CHANGELOG.md (docs: updated CHANGELOG.md #6348) 16308fa

[alexec]

@Bobgy can you mark the ones you needed again please?

[Bobgy]

Thanks!
I was talking about #6247.
It's not in the list.

nit: What permission do I need to check boxes here?

[Bobgy]

I want this too

Yuan (Bob) Gong: fix(executor/emissary): fix nonroot sidecars + input/output params & artifacts (fix(executor/emissary): fix nonroot sidecars + input/output params & artifacts #6403) 30e2518

But I think it's already cherry picked

[sarabala1979]

@Bobgy I selected it. Do you have any more issues that need to be included in v3.1.4? I am planning to start release tomorrow

[Bobgy]

Thanks, I didn't hit any other issues so far.

I am just curious, I think some fixes like the following are generally good to have, but they are not picked at the moment. How do you decide?

Antoine Dao: fix(emissary): throw argo error on file not exist (fix(emissary): throw argo error on file not exist #6392) b17d1bf

Alex Collins: fix(executor): Remove 15s guard for Docker executor. Fixes wait container won't exit #6415 (fix(executor): Remove 15s guard for Docker executor. Fixes #6415 #6427)

[sarabala1979]

@Bobgy We will go with the below order

1. Regression issues must be included
2. Other fixes based on usecase/no of user facing/ user request/ risk factor.

[sarabala1979]

• Simon Behar: fix: Consider onExit children of Retry nodes (fix: Consider onExit children of Retry nodes #6451) 7ef0f47

[Bobgy]

Hi @sarabala1979, I'm still not seeing my fix in #6247 cherrypicked into 3.1.4.
https://github.com/argoproj/argo-workflows/blob/v3.1.4/workflow/executor/emissary/binary.go#L19

[sarabala1979]

I did cherry pick and resolve conflict on windows file. Let me check.

…

Sent from my iPhone

On Aug 3, 2021, at 7:08 PM, Yuan (Bob) Gong ***@***.***> wrote:  Hi @sarabala1979, I'm still not seeing my fix in #6247 cherrypicked into 3.1.4. https://github.com/argoproj/argo-workflows/blob/247776d66fa6bf988f861ba82f181e386a972626/workflow/executor/emissary/binary.go#L19 — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

[sarabala1979]

@Bobgy I miss understood. I was including Yuan (Bob) Gong: fix(executor/emissary): fix nonroot sidecars + input/output params & artifacts (fix(executor/emissary): fix nonroot sidecars + input/output params & artifacts #6403) 30e2518.
I missed your comment saying the issue is not on the list.
finally, find it 05c901f
somewhat it was missed in 3.1.3 also. I am releasing v3.1.5

[Bobgy]

Yes, it's a different PR. Sorry I split my fix into multiple PRs in the first place. I see it makes it more confusing than necessary.
I'll test more cases in the initial PR next time. : )