Fix: Node set updating global output parameter updates global. #5699

[markterm]

Checklist:

• My organization is added to USERS.md.

[codecov]

Codecov Report

Merging #5700 (666e572) into master (4b3a30f) will increase coverage by 0.27%.
The diff coverage is 90.38%.

❗ Current head 666e572 differs from pull request most recent head 60c2aa7. Consider uploading reports for the commit 60c2aa7 to get more accurate results

@@            Coverage Diff             @@
##           master    #5700      +/-   ##
==========================================
+ Coverage   46.79%   47.07%   +0.27%     
==========================================
  Files         245      246       +1     
  Lines       15398    15485      +87     
==========================================
+ Hits         7206     7289      +83     
- Misses       7280     7284       +4     
  Partials      912      912              

Impacted Files	Coverage Δ
workflow/util/util.go	43.92% <58.33%> (+0.64%)	⬆️
.../workflow/v1alpha1/container_set_template_types.go	100.00% <100.00%> (ø)
pkg/apis/workflow/v1alpha1/validation_utils.go	100.00% <100.00%> (ø)
workflow/controller/operator.go	71.14% <100.00%> (-0.24%)	⬇️
workflow/validate/validate.go	73.38% <100.00%> (+0.25%)	⬆️
cmd/argo/commands/get.go	56.95% <0.00%> (+0.64%)	⬆️
cmd/argoexec/commands/emissary.go	50.00% <0.00%> (+1.56%)	⬆️
workflow/metrics/server.go	16.66% <0.00%> (+4.16%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4b3a30f...60c2aa7. Read the comment docs.

[markterm]

@sarabala1979 please let me know if you have any feedback on this PR.

[sarabala1979]

@markterm Please fix the failures