fix(controller): Only set global parameters after workflow validation succeeded to avoid panics

[terrytangyuan]

Fixes #5473.

Signed-off-by: terrytangyuan terrytangyuan@gmail.com

Checklist:

• My organization is added to USERS.md.

[codecov]

Codecov Report

Merging #5477 (95b35aa) into master (5bd7ce8) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head 95b35aa differs from pull request most recent head 1abdccc. Consider uploading reports for the commit 1abdccc to get more accurate results

@@            Coverage Diff             @@
##           master    #5477      +/-   ##
==========================================
- Coverage   16.49%   16.46%   -0.03%     
==========================================
  Files         243      243              
  Lines       43774    43775       +1     
==========================================
- Hits         7219     7208      -11     
- Misses      35573    35587      +14     
+ Partials      982      980       -2     

Impacted Files	Coverage Δ
workflow/controller/operator.go	70.53% <100.00%> (+0.01%)	⬆️
workflow/metrics/server.go	12.76% <0.00%> (-4.26%)	⬇️
server/workflow/workflow_server.go	40.45% <0.00%> (-2.28%)	⬇️
cmd/argoexec/commands/emissary.go	48.43% <0.00%> (-1.57%)	⬇️

---

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 5bd7ce8...1abdccc. Read the comment docs.