You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2022/05/18 07:25:47 UTC
[GitHub] [camel-k] astefanutti opened a new pull request, #3285: fix: Use status change predicate to filter updates on owned resources
astefanutti opened a new pull request, #3285:
URL: https://github.com/apache/camel-k/pull/3285
This PR fixes the issue where the Integration controller enters an infinite loop, that keeps its work-queue constantly full, as Integrations are being re-enqueued indirectly via update events on owned resources.
This is caused by kubernetes/kubernetes#100024, which has been fixed in recent versions of Kubernetes with kubernetes/kubernetes#100032.
In order to fix the issue in older Kubernetes versions, this PR adds a predicate to the Integration owned resources watch handlers, to filter out update events on these resources, that have been triggered by changes without any semantic changes to their status blocks.
**Release Note**
```release-note
fix: Use status change predicate to filter updates on owned resources
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] astefanutti merged pull request #3285: fix: Use status change predicate to filter updates on owned resources
Posted by GitBox <gi...@apache.org>.
astefanutti merged PR #3285:
URL: https://github.com/apache/camel-k/pull/3285
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] astefanutti commented on pull request #3285: fix: Use status change predicate to filter updates on owned resources
Posted by GitBox <gi...@apache.org>.
astefanutti commented on PR #3285:
URL: https://github.com/apache/camel-k/pull/3285#issuecomment-1130070639
Let's merge it, the test failures are already present in `main` branch.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] lburgazzoli commented on pull request #3285: fix: Use status change predicate to filter updates on owned resources
Posted by GitBox <gi...@apache.org>.
lburgazzoli commented on PR #3285:
URL: https://github.com/apache/camel-k/pull/3285#issuecomment-1129673999
@astefanutti that's an awesome finding and it would be very nice to have a blog post explaining how you get to that point
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel-k] christophd commented on pull request #3285: fix: Use status change predicate to filter updates on owned resources
Posted by GitBox <gi...@apache.org>.
christophd commented on PR #3285:
URL: https://github.com/apache/camel-k/pull/3285#issuecomment-1130057524
@astefanutti Awesome!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org