Skip to content

Sort source requests by dependencies - #1576

Merged
matheuscscp merged 1 commit into
fluxcd:mainfrom
cappyzawa:worktree/issue-1254
Sep 2, 2026
Merged

Sort source requests by dependencies#1576
matheuscscp merged 1 commit into
fluxcd:mainfrom
cappyzawa:worktree/issue-1254

Conversation

@cappyzawa

@cappyzawa cappyzawa commented Aug 31, 2026

Copy link
Copy Markdown
Member

Closes #1254

Assisted-by: Codex/GPT-5

@matheuscscp matheuscscp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@cappyzawa
cappyzawa marked this pull request as ready for review September 1, 2026 12:53
@cappyzawa

Copy link
Copy Markdown
Member Author

@matheuscscp Thanks for the approval 👍

@stefanprodan stefanprodan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@matheuscscp

Copy link
Copy Markdown
Member

Thanks @cappyzawa! Please rebase and force-push for the merge 🙏

Processing dependencies first avoids unnecessary retries when source
changes fan out to multiple HelmReleases.

Signed-off-by: cappyzawa <cappyzawa@gmail.com>
Assisted-by: Codex/GPT-5
@matheuscscp
matheuscscp merged commit f37ac2a into fluxcd:main Sep 2, 2026
7 of 8 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.

Enqueued requests for source event from watcher are not sorted in the dependency order

3 participants