[MANOPD-76880] Fix cache package version #181
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
cache_version
options does not work for packages from 'associations' section. For instance:add_node
procedure does not fail if there are several versions of thecontainerd
package in the cluster.Fixes # [MANOPD-76880]
Solution
How to apply
Not applicable
Test Cases
TestCase 1
cache_versions: true
works properly duringadd_node
procedureTest Configuration:
Steps:
containerd
package on one node up to version that has different minor (eg.1.4
becomes1.5
)add_node
procedure.Results:
TestCase 2
cache_versions: false
works properly duringadd_node
procedure.Test Configuration:
Steps:
containerd
package on one node up to version that has different minor (eg.1.4
becomes1.5
)add_node
procedure.Results:
associations
sectionTestCase 3
package.associations
incluster.yaml
works properly duringadd_node
procedure.Test Configuration:
Steps:
cluster.yaml
should havepackage.associations
andcache_versions: false
containerd
package on one node up to version that has different minor (eg.1.4
becomes1.5
)add_node
procedure.cache_versions: true
incluster.yaml
.add_node
procedure.Results:
associations
sectionassociations
sectionChecklist