Skip to content

Fabric8 it part 7#1474

Merged
ryanjbaxter merged 39 commits intospring-cloud:3.0.xfrom
wind57:fabric8-it-part-7
Oct 9, 2023
Merged

Fabric8 it part 7#1474
ryanjbaxter merged 39 commits intospring-cloud:3.0.xfrom
wind57:fabric8-it-part-7

Conversation

@wind57
Copy link
Copy Markdown
Contributor

@wind57 wind57 commented Oct 7, 2023

No description provided.

<modules>
<module>spring-cloud-kubernetes-fabric8-istio-it</module>
<module>spring-cloud-kubernetes-fabric8-client-discovery</module>
<module>spring-cloud-kubernetes-fabric8-client-loadbalancer</module>
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

we do not need this project as it overlaps with discovery related integration test that already exists. Its name contains "loadbalancer", but it reality it's just using the discovery client under the hood, but we already have discovery client tests. As such, remove this one

@wind57 wind57 marked this pull request as ready for review October 9, 2023 13:43
@wind57
Copy link
Copy Markdown
Contributor Author

wind57 commented Oct 9, 2023

@ryanjbaxter one more very simple one, imo. thank you

@ryanjbaxter ryanjbaxter added this to the 3.0.6 milestone Oct 9, 2023
@ryanjbaxter ryanjbaxter merged commit 2597509 into spring-cloud:3.0.x Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants