bookingcom / shipper

Kubernetes native multi-cluster canary or blue-green rollouts using Helm
Apache License 2.0
734 stars 38 forks source link

Fixed a flapping test TestReCacheClusterOnSecretUpdate #359

Closed hihilla closed 3 years ago

hihilla commented 3 years ago

This is a cherry-pick for #296

This commit alters clusterclientstore.computeSecretChecksum/1 function so it puts all secret Data key-value pairs in a sorted order. The unordered hashmap key nature caused inconsistencies in the hash value and might have caused some frequent cluster cache invalidations.

Signed-off-by: Oleg Sidorov oleg.sidorov@booking.com