fix(executor): artifact chmod should only if err != nil (argoproj#4409)

USERS.md

@@ -46,7 +46,9 @@ Currently, the following organizations are **officially** using Argo Workflows:
 1. [Graviti](https://www.graviti.cn)
 1. [Greenhouse](https://greenhouse.io)
 1. [Habx](https://www.habx.com/)
+1. [Helio](https://helio.exchange)
 1. [HOVER](https://hover.to)
+1. [HSBC](https://hsbc.com)
 1. [IBM](https://ibm.com)
 1. [InsideBoard](https://www.insideboard.com)
 1. [Interline Technologies](https://www.interline.io/blog/scaling-openstreetmap-data-workflows/)
@@ -71,6 +73,7 @@ Currently, the following organizations are **officially** using Argo Workflows:
 1. [OVH](https://www.ovh.com/)
 1. [Peak AI](https://www.peak.ai/)
 1. [PDOK](https://www.pdok.nl/)
+1. [Pollination](https://pollination.cloud)
 1. [Preferred Networks](https://www.preferred-networks.jp/en/)
 1. [Prudential](https://www.prudential.com.sg/)
 1. [Quantibio](http:https://quantibio.com/us/en/)
@@ -96,5 +99,3 @@ Currently, the following organizations are **officially** using Argo Workflows:
 1. [UFirstGroup](https://www.ufirstgroup.com)
 1. [Wavefront](https://www.wavefront.com/)
 1. [Wellcome Trust](https://wellcome.ac.uk/)
-1. [Helio](https://helio.exchange)
-1. [HSBC](https://hsbc.com)

workflow/executor/executor.go

@@ -225,7 +225,9 @@ func (we *WorkflowExecutor) LoadArtifacts() error {
  log.Infof("Successfully download file: %s", artPath)
  if art.Mode != nil {
  err = chmod(artPath, *art.Mode, art.RecurseMode)
- return err
+ if err != nil {
+ return err
+ }
  }
  }
  return nil