Skip to content

make reapply interval configurable#289

Merged
cbarbian-sap merged 1 commit into
mainfrom
reapply-interval
May 15, 2025
Merged

make reapply interval configurable#289
cbarbian-sap merged 1 commit into
mainfrom
reapply-interval

Conversation

@cbarbian-sap
Copy link
Copy Markdown
Contributor

@cbarbian-sap cbarbian-sap commented May 15, 2025

This PR makes the force-reapply interval configurable on component and object level.
So far, every object was force-reapplied to the cluster every 60 minutes (even it is seemed to be in sync according to digests). Now, that interval can be tweaked

  • on component level by implementing the new interface

    type ReapplyConfiguration interface {
      // Get force-reapply interval. Should be greater than the effective requeue interval. If a value smaller than the
      // effective requeue interval is specified, the force-reapply might be delayed until the requeue happens.
      // A return value of zero means to use the framework default.
      GetReapplyInterval() time.Duration
    }

    or by including the convenience type ReapplySpec into the component's spec.

  • on object level (which has higher precedence than the value specified on component level) by setting the annotation
    mycomponent-operator.mydomain.io/reapply-interval.

@cbarbian-sap cbarbian-sap merged commit d8131ae into main May 15, 2025
9 checks passed
@cbarbian-sap cbarbian-sap deleted the reapply-interval branch May 15, 2025 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant