Feature skip dryrun in all cases

[Nheijmink19]

This is an extention to solve the issue that the validation fails on creating resources in a not yet existing namespace.
argoproj/argo-cd#21788
and a followup to:
#708

By setting the skipDryRun you can skip the validation in your sync action allowing the resources that can already be created (the namespace) to be created and the other resources in another sync wave.
In the old situation it only worked for CRD's which made it redundant to skipDryRunOnMissingResource. By moving it out of the err!=nil check it can be used by all type of resources.

If oke I will join the meeting next thursday to explain the requested change.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 53.80%. Comparing base (8849c3f) to head (13f10ff).
Report is 47 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #728      +/-   ##
==========================================
- Coverage   54.26%   53.80%   -0.46%     
==========================================
  Files          64       64              
  Lines        6164     6486     +322     
==========================================
+ Hits         3345     3490     +145     
- Misses       2549     2724     +175     
- Partials      270      272       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Nheijmink19]

if agreed on the change we can squash or I can create a new branch

[agaudreault]

Closing due to #730

Will be replace by creating the namespace before the DryRun if the namespace does not exist.

Eventually, a new option should be added to perform the dry run on the current wave instead of on all the resource before the sync.

[Nheijmink19]

sounds like a good alternative

[Nheijmink19]

@agaudreault I did create something like that in the past:
https://github.com/argoproj/gitops-engine/pull/664/files
I am not sure if my implementation is the way you planned on building it, but if it is you already have a start.