SUSE / caasp-salt

A collection of salt states used to provision a kubernetes cluster
Apache License 2.0
64 stars 29 forks source link

don't sync module during update. (fix - bsc#1125444) #760

Closed MalloZup closed 5 years ago

MalloZup commented 5 years ago

This pr is used for seeing how far we can go with the removal.

This call cause race condition and issues

MalloZup commented 5 years ago

Wow this passed, 😮😮😮😮🎸🎸🎶🎸🎶🎸

MalloZup commented 5 years ago

so 2/2 run was ok, i want maybe to test with sles15.

MalloZup commented 5 years ago

@inercia before merging this, i would like to see if it works on sles15 see ( https://github.com/SUSE/caasp-jenkins-library/pull/287)

MalloZup commented 5 years ago

i'm testing it with sles15, let's wait with that. currenlty i need to use the right image

MalloZup commented 5 years ago

i'm closing this since we will have soonish the patch for fixing the race condition merged