Skip to content

Prepares infrastructure for Staging resource reconciliation#10

Merged
DanielBlei merged 1 commit intomainfrom
set-staging-reconcilliation
Nov 15, 2025
Merged

Prepares infrastructure for Staging resource reconciliation#10
DanielBlei merged 1 commit intomainfrom
set-staging-reconcilliation

Conversation

@DanielBlei
Copy link
Owner

@DanielBlei DanielBlei commented Nov 15, 2025

Pull Request

Description

Adds status tracking helpers (InternalStatus, SetInternalStatus, SetStagingStatus), resource validation functions for ingestion resources (CronJob, Job, Trigger), and component creation utilities.

Implements observed generation tracking to prevent unnecessary reconciliations and initial bootstrap mode support for CronJob creation.

Implementation Notes

  • Uses observed generation pattern to skip reconciliations when only status changes
  • Creates deep copy before status updates for safe patching
  • Bootstrap mode currently supports CronJob creation only (Job/Trigger marked as TODO)
  • Component helpers apply default resource requirements when not specified
  • Controller filters events to reconcile only on spec/annotation changes, not status updates

Testing

  • Code builds and runs locally
  • Relevant tests added or updated
  • Manual/automated tests performed

Checklist

  • Code style is consistent with the project
  • Documentation updated if needed
  • Ready for review

Thank you for your contribution!

Add status tracking helpers, resource validation functions, and observed
generation tracking to prepare for staging reconciliation. Includes
InternalStatus helpers and component creation utilities.
@DanielBlei DanielBlei force-pushed the set-staging-reconcilliation branch from b7ecc6d to 1b0002d Compare November 15, 2025 09:55
@DanielBlei DanielBlei merged commit d9b2460 into main Nov 15, 2025
4 checks passed
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