Skip to content

Conversation

mhofstetter
Copy link
Member

Currently, the daemon implements the Prefilter API handler. With it comes the last dependency from the daemon to the prefilter itself.

Therefore, this commit moves the prefilter API handler from the daemon to the existing prefilter hive cell and removes the dependency from the daemon to the prefilter.

@mhofstetter mhofstetter added kind/enhancement This would improve or streamline existing functionality. release-note/misc This PR makes changes that have no direct user impact. area/modularization Relates to code modularization and maintenance. labels Jun 12, 2024
@mhofstetter mhofstetter force-pushed the pr/mhofstetter/prefilter-api-handler branch from e91ab53 to d087592 Compare June 12, 2024 19:39
@mhofstetter
Copy link
Member Author

/test

@mhofstetter mhofstetter marked this pull request as ready for review June 13, 2024 10:15
@mhofstetter mhofstetter requested review from a team as code owners June 13, 2024 10:15
Currently, the daemon implements the Prefilter API handler. With it
comes the last dependency from the daemon to the prefilter itself.

Therefore, this commit moves the prefilter API handler from the daemon
to the existing prefilter hive cell and removes the dependency from the
daemon to the prefilter.

Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
@mhofstetter mhofstetter force-pushed the pr/mhofstetter/prefilter-api-handler branch from d087592 to 5536dc8 Compare June 14, 2024 07:27
@mhofstetter
Copy link
Member Author

rebased to main to get some fixes for Gateway API checks.

@mhofstetter
Copy link
Member Author

/test

@joamaki joamaki added this pull request to the merge queue Jun 14, 2024
Merged via the queue into cilium:main with commit 7f6df2d Jun 14, 2024
@mhofstetter mhofstetter deleted the pr/mhofstetter/prefilter-api-handler branch June 14, 2024 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/modularization Relates to code modularization and maintenance. kind/enhancement This would improve or streamline existing functionality. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants