Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: reconcile on project and template changes #124

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

yftacherzog
Copy link
Member

By having the controller watch project and template changes, dev streams will be reconciled when those resources are created or modified.

With that in place, resources can now be created and modified in any order, and resources will be generated per the latest changes and owner ref will be updated between dev stream and project.

@ifireball
Copy link
Member

please rebase

By having the controller watch project and template changes, dev streams
will be reconciled when those resources are created or modified.

With that in place, resources can now be created and modified in any
order, and resources will be generated per the latest changes and
owner ref will be updated between dev stream and project.

Signed-off-by: Yftach Herzog <yherzog@redhat.com>
@ifireball ifireball enabled auto-merge November 19, 2024 08:06
@ifireball ifireball merged commit 25132f7 into konflux-ci:main Nov 19, 2024
7 checks passed
yftacherzog added a commit to yftacherzog/infra-deployments that referenced this pull request Nov 19, 2024
This introduces into production a change in which the controller watches
also over the project and template resources.
konflux-ci/project-controller#124

Signed-off-by: Yftach Herzog <yherzog@redhat.com>
openshift-merge-bot bot pushed a commit to redhat-appstudio/infra-deployments that referenced this pull request Nov 19, 2024
This introduces into production a change in which the controller watches
also over the project and template resources.
konflux-ci/project-controller#124

Signed-off-by: Yftach Herzog <yherzog@redhat.com>
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.

2 participants