Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test : Update Spring boot tests for layered jar changes #308

Merged
merged 1 commit into from
Aug 30, 2023

Conversation

rohanKanojia
Copy link
Member

@rohanKanojia rohanKanojia commented Jul 31, 2023

Related to eclipse-jkube/jkube#2309

  • Remove unused variable
  • Update image list assertions to verify layer folders instead of the fat jar
  • Updated all spring test projects to use spring.version constant instead of inheriting version from dependencyManagement
  • Add zero-config-fatjar integration test spring project that will use spring-boot.fat-jar.version (2.3.9.RELEASE). Till this release, spring boot generated fat jars by default.

Signed-off-by: Rohan Kumar rohaan@redhat.com

@manusa
Copy link
Member

manusa commented Jul 31, 2023

Note that when we merge this feature, it'll be better to have an additional test

@rohanKanojia rohanKanojia force-pushed the pr/spring-boot-layered branch 8 times, most recently from 07b0436 to 4055b65 Compare August 1, 2023 15:48
@rohanKanojia rohanKanojia marked this pull request as ready for review August 1, 2023 16:35
@manusa manusa marked this pull request as draft August 4, 2023 12:16
@rohanKanojia rohanKanojia force-pushed the pr/spring-boot-layered branch 2 times, most recently from d9f34c4 to 869b977 Compare August 8, 2023 09:27
import static org.hamcrest.Matchers.hasSize;

@KubernetesTest(createEphemeralNamespace = false)
public class ZeroConfigFatJar implements JKubeCase, MavenCase {
Copy link
Member

Choose a reason for hiding this comment

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

I'm assuming this is because Spring is less than x version. right?

Copy link
Member Author

Choose a reason for hiding this comment

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

You're right. I've updated PR description

@rohanKanojia rohanKanojia force-pushed the pr/spring-boot-layered branch 2 times, most recently from eb23434 to a998b72 Compare August 30, 2023 14:11
+ Remove spring boot dependencies from dependencyManagement in parent
  pom. Spring boot projects would not use spring-boot.version property
  explicitly in poms
+ Add ZeroConfigFatJarK8sITCase to verify kubernetes maven
  plugin workflow in case of spring boot projects that don't generate a
  layered jar

Signed-off-by: Rohan Kumar <rohaan@redhat.com>
@rohanKanojia rohanKanojia marked this pull request as ready for review August 30, 2023 14:26
@manusa manusa merged commit 06557b8 into eclipse-jkube:main Aug 30, 2023
29 checks passed
@rohanKanojia rohanKanojia deleted the pr/spring-boot-layered branch August 30, 2023 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants