Skip to content

close shared informers + minor cleanup in tests#2122

Merged
ryanjbaxter merged 1 commit intospring-cloud:mainfrom
wind57:stop-informers-and-some-cleaup
Jan 21, 2026
Merged

close shared informers + minor cleanup in tests#2122
ryanjbaxter merged 1 commit intospring-cloud:mainfrom
wind57:stop-informers-and-some-cleaup

Conversation

@wind57
Copy link
Contributor

@wind57 wind57 commented Jan 21, 2026

No description provided.

Signed-off-by: wind57 <eugen.rabii@gmail.com>
@wind57 wind57 marked this pull request as ready for review January 21, 2026 20:55
@wind57
Copy link
Contributor Author

wind57 commented Jan 21, 2026

@ryanjbaxter ready to be looked at. thank you

@@ -40,37 +40,33 @@ class KubernetesClientCatalogWatchAutoConfigurationApplicationContextTests {

Copy link
Contributor Author

Choose a reason for hiding this comment

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

from here and below, all that is done is to remove : spring.cloud.config.enabled=false

@github-project-automation github-project-automation bot moved this to Todo in 2025.1.1 Jan 21, 2026
@ryanjbaxter ryanjbaxter added this to the 5.0.1 milestone Jan 21, 2026
@ryanjbaxter
Copy link
Contributor

Thanks!

As you have probably noticed I have been making a lot of changes for the github action. We have a goal of moving solely to github actions over the coming months so I have been experimenting with k8s. I THINK I have things in a good state, but let me know if you notice anything weird.

@ryanjbaxter ryanjbaxter merged commit 6cc2646 into spring-cloud:main Jan 21, 2026
26 checks passed
@github-project-automation github-project-automation bot moved this from Todo to Done in 2025.1.1 Jan 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants