Skip to content

Commit

Permalink
Disable repository verification
Browse files Browse the repository at this point in the history
Repository verification is a big constraints when testing different images
This change remove the validation in the webhook
The operator will only update the image if the version of the image is also changed
  • Loading branch information
geobeau committed Feb 1, 2022
1 parent 2edf0d9 commit 6f4a2b5
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions pkg/api/v1/cluster_validation.go
Original file line number Diff line number Diff line change
Expand Up @@ -124,9 +124,10 @@ func checkValues(c *ScyllaCluster) error {

func checkTransitions(old, new *ScyllaCluster) error {
// Check that repository remained the same
if !reflect.DeepEqual(old.Spec.Repository, new.Spec.Repository) {
return errors.Errorf("repository change is currently not supported, old=%v, new=%v", *old.Spec.Repository, *new.Spec.Repository)
}
// We actually want to be able to change the repository, it should retain equivalent tag scheme
// if !reflect.DeepEqual(old.Spec.Repository, new.Spec.Repository) {
// return errors.Errorf("repository change is currently not supported, old=%v, new=%v", *old.Spec.Repository, *new.Spec.Repository)
// }

// Check that the datacenter name didn't change
if old.Spec.Datacenter.Name != new.Spec.Datacenter.Name {
Expand Down

0 comments on commit 6f4a2b5

Please sign in to comment.