-
Notifications
You must be signed in to change notification settings - Fork 40
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Use locks to protect the MultiError, Syncing, and Apply methods independently. This is a hack to improve thread-safety without changing the Applier interface. In the future, the interface should probably be rewritten to use channels instead. - Replace `parser.SetSyncStatus` with `updateSyncStatus`, which handles prepending management conflict errors from the remediator. - Move the periodic sync status update from the reconciler into the parser. This removes the need for some synchronization that wasn't previously thread-safe. - Remove several parser methods which are obsolete with the above changes. - Rename applier.Interface to KptApplier to make way for the KptDestroyer interface later. - Route the retry delay and status update delay up through the parser and reconciler. These are now hard-coded with the rest of the constants. They could be made into options later, but I didn't want to expose new options until we need them exposed. - Rename -Frequency and -Period options to -Delay which more accurately reflects how they are used. Period is how long something takes. Frequency is how often something occurs. Delay is the time between occurances. Change-Id: I1c3373f98bb5d842ce113ba61dc0c87e1119c665
- Loading branch information
Showing
14 changed files
with
302 additions
and
364 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.