fix: Skip caching for configs that are set to skip

[tomazpu]

…skip parameters set to true

What this PR does / Why we need it:

Skip caching for configs of type classic and setting that have the skip parameter set to true. As the configs won't be deployed there is no need to chache them. With this being skipped, we also prevent performance degradation compared to the previous version.

Special notes for your reviewer:

Does this PR introduce a user-facing change?

[github-actions[bot]]

Unit Test Results

1 901 tests  ±0   1 900 ✅ ±0   1m 56s ⏱️ ±0s   134 suites ±0       1 💤 ±0      1 files   ±0       0 ❌ ±0 

Results for commit 4ce0c939. ± Comparison against base commit e88aa271.

[github-actions[bot]]

E2E Test Results

    4 files  ±0    270 suites  ±0   41m 29s ⏱️ + 18m 15s 2 016 tests ±0  2 011 ✅ ±0  2 💤 ±0  3 ❌ ±0  2 130 runs  ±0  2 125 ✅ ±0  2 💤 ±0  3 ❌ ±0 

For more details on these failures, see this check.

Results for commit 4ce0c939. ± Comparison against base commit e88aa271.

[jskelin]

Is it possible to add a test (e.g. in Test_gatherPreloadConfigTypeEntries_OneEntryPerConfigType)?

[sonarcloud[bot]]

Quality Gate failed

Failed conditions
B Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint