Skip to content

Conversation

giorio94
Copy link
Member

@giorio94 giorio94 commented Nov 3, 2023

Now that kvstoremesh got merged into the clustermesh-apiserver, and the CI changes got applied, let's also drop the kvstoremesh dockerfile.

Now that kvstoremesh got merged into the clustermesh-apiserver, and the
CI changes got applied, let's also drop the kvstoremesh dockerfile.

Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
@giorio94 giorio94 added kind/cleanup This includes no functional changes. area/clustermesh Relates to multi-cluster routing functionality in Cilium. release-note/misc This PR makes changes that have no direct user impact. area/build Anything to do with the build, more general than area/CI labels Nov 3, 2023
@giorio94
Copy link
Member Author

giorio94 commented Nov 3, 2023

/test

1 similar comment
@giorio94
Copy link
Member Author

giorio94 commented Nov 6, 2023

/test

@giorio94 giorio94 marked this pull request as ready for review November 6, 2023 12:51
@giorio94 giorio94 requested a review from a team as a code owner November 6, 2023 12:51
@giorio94 giorio94 requested a review from jibi November 6, 2023 12:51
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Nov 10, 2023
@jibi jibi merged commit 582b8c7 into main Nov 10, 2023
@jibi jibi deleted the pr/giorio94/kvstoremesh-cleanup branch November 10, 2023 11:07
mhofstetter added a commit to mhofstetter/cilium that referenced this pull request Dec 15, 2023
Currently, vulnerability scanning fails because kvstoremesh no longer
exists on v1.15. It has been removed with cilium#28961

There are already kvstoremesh exclusions for v1.12 & v1.13. This
commit inverts the exclusion to an inclusion for v1.14, as this is
the only branch that contains the kvstoremesh.

Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
julianwiedmann pushed a commit that referenced this pull request Dec 18, 2023
Currently, vulnerability scanning fails because kvstoremesh no longer
exists on v1.15. It has been removed with #28961

There are already kvstoremesh exclusions for v1.12 & v1.13. This
commit inverts the exclusion to an inclusion for v1.14, as this is
the only branch that contains the kvstoremesh.

Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
ldelossa pushed a commit to ldelossa/cilium that referenced this pull request Feb 29, 2024
[ upstream commit 7661353 ]

Currently, vulnerability scanning fails because kvstoremesh no longer
exists on v1.15. It has been removed with cilium#28961

There are already kvstoremesh exclusions for v1.12 & v1.13. This
commit inverts the exclusion to an inclusion for v1.14, as this is
the only branch that contains the kvstoremesh.

Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build Anything to do with the build, more general than area/CI area/clustermesh Relates to multi-cluster routing functionality in Cilium. kind/cleanup This includes no functional changes. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. 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.

2 participants