Skip to content

feat: Send unsynced resources with status updates#597

Merged
maciaszczykm merged 26 commits intomainfrom
marcin/prod-4198-helm-hooks-realization-should-still-send-the-rest-of-the
Dec 3, 2025
Merged

feat: Send unsynced resources with status updates#597
maciaszczykm merged 26 commits intomainfrom
marcin/prod-4198-helm-hooks-realization-should-still-send-the-rest-of-the

Conversation

@maciaszczykm
Copy link
Member

@maciaszczykm maciaszczykm commented Nov 26, 2025

Test Plan

Test environment: https://console.plrl-dev-aws.onplural.sh/

Checklist

  • I have added a meaningful title and summary to convey the impact of this PR to a user.
  • I have deployed the agent to a test environment and verified that it works as expected.
    • Agent starts successfully.
    • Service creation works without any issues when using raw manifests and Helm templates.
    • Service creation works when resources contain both CRD and CRD instances.
    • Service templating works correctly.
    • Service errors are reported properly and visible in the UI.
    • Service updates are reflected properly in the cluster.
    • Service resync triggers immediately and works as expected.
    • Sync waves annotations are respected.
    • Sync phases annotations are respected. Phases are executed in the correct order.
    • Sync hook delete policies are respected. Resources are not recreated once they reach the desired state.
    • Service deletion works and cleanups resources properly.
    • Services can be recreated after deletion.
    • Service detachment works and keeps resources unaffected.
    • Services can be recreated after detachment.
    • Service component trees are working as expected.
    • Cluster health statuses are being updated.
    • Agent logs do not contain any errors.
  • I have added tests to cover my changes.
  • If required, I have updated the Plural documentation accordingly.

@linear
Copy link

linear bot commented Nov 26, 2025

@maciaszczykm maciaszczykm force-pushed the marcin/prod-4198-helm-hooks-realization-should-still-send-the-rest-of-the branch 6 times, most recently from 0b5ee9d to bc9653c Compare November 26, 2025 13:14
@maciaszczykm maciaszczykm force-pushed the marcin/prod-4198-helm-hooks-realization-should-still-send-the-rest-of-the branch from bc9653c to 4c84fc6 Compare November 26, 2025 15:53
…ization-should-still-send-the-rest-of-the' into marcin/prod-4198-helm-hooks-realization-should-still-send-the-rest-of-the
@github-actions github-actions bot added size/L and removed size/M labels Nov 27, 2025
@github-actions github-actions bot added size/XL and removed size/L labels Nov 28, 2025
@github-actions github-actions bot added size/XXL and removed size/XL labels Nov 28, 2025
@maciaszczykm maciaszczykm marked this pull request as ready for review December 1, 2025 11:16
@maciaszczykm maciaszczykm requested a review from a team as a code owner December 1, 2025 11:16
@michaeljguarino
Copy link
Member

I believe the uid required check is only for service component children not service components themselves? Is it happening on service components?

@maciaszczykm maciaszczykm marked this pull request as draft December 1, 2025 16:30
@maciaszczykm
Copy link
Member Author

Converting to draft to verify if parents (root objects) are sent as part of component children set.

@maciaszczykm maciaszczykm marked this pull request as ready for review December 2, 2025 12:23
@maciaszczykm maciaszczykm merged commit ee53521 into main Dec 3, 2025
95 checks passed
@maciaszczykm maciaszczykm deleted the marcin/prod-4198-helm-hooks-realization-should-still-send-the-rest-of-the branch December 3, 2025 09:31
michaeljguarino added a commit that referenced this pull request Dec 5, 2025
michaeljguarino added a commit that referenced this pull request Dec 5, 2025
@maciaszczykm maciaszczykm restored the marcin/prod-4198-helm-hooks-realization-should-still-send-the-rest-of-the branch December 5, 2025 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants