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

WFLY-18471 ha-singleton-deployment Quickstart Common Enhancements CY2023Q3 #914

Merged
merged 1 commit into from
May 27, 2024

Conversation

rhusar
Copy link
Member

@rhusar rhusar commented May 20, 2024

  • Synchronize properties definitions.
  • Convert the deployment to war from jar and introduce a welcome page via static index.html.
  • Introduce a simple integration test and -Pintegration-testing Maven profile.
  • Cleanup README file.
  • Introduce automatic CI for the quickstart.

n.b. no OpenShift support and no bootable-jar support.

Resolves
https://issues.redhat.com/browse/WFLY-18471 – ha-singleton-deployment Quickstart Common Enhancements CY2023Q3

Related PR for ha-singleton-service QS: #915 (not conflicting)

@rhusar rhusar requested a review from emmartins as a code owner May 20, 2024 14:23
Comment on lines +147 to +152
<!-- Tests -->
<dependency>
<groupId>org.junit.jupiter</groupId>
<artifactId>junit-jupiter-engine</artifactId>
<scope>test</scope>
</dependency>
Copy link
Member Author

Choose a reason for hiding this comment

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

@emmartins Please remind me, are you OK with JUnit 5 here?

Copy link
Contributor

Choose a reason for hiding this comment

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

yup

…023Q3

Synchronize properties definitions.
Convert the deployment to war from jar and introduce a welcome page via static index.html.
Introduce a simple integration test and -Pintegration-tests Maven profile.
Cleanup README file.
Introduce automatic CI for the quickstart.
@rhusar rhusar changed the title WFLY-18471 ha-singleton-deployment Quickstart Common Enhancements CY2… WFLY-18471 ha-singleton-deployment Quickstart Common Enhancements CY2023Q3 May 20, 2024
Comment on lines +147 to +152
<!-- Tests -->
<dependency>
<groupId>org.junit.jupiter</groupId>
<artifactId>junit-jupiter-engine</artifactId>
<scope>test</scope>
</dependency>
Copy link
Contributor

Choose a reason for hiding this comment

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

yup

@emmartins emmartins merged commit e7feb5c into wildfly:main May 27, 2024
21 checks passed
@emmartins
Copy link
Contributor

good work, thanks

@rhusar
Copy link
Member Author

rhusar commented May 28, 2024

Thanks @emmartins and again sorry for taking a while to finish this off.

@rhusar rhusar deleted the WFLY-18471 branch June 1, 2024 11:46
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.

2 participants