Skip to content
This repository has been archived by the owner on Oct 14, 2020. It is now read-only.

Wildfly runtime prototype #285

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

awisniew90
Copy link
Member

@awisniew90 awisniew90 commented Mar 20, 2019

Still to do (among many things):

  1. Change context root from default app name
  2. JDBC Booster - flesh out cli commands to add datasource and drivers
  3. General cleanup and more refactoring as needed.

Key Points:

  1. Installs wildfly to target dir
  2. Uses wildfly CLI (jboss-cli.sh) for configuration / app deployment / stopping server
  3. Uses standalone.sh invocation for start and run
  4. New wildfly booster that can be added to use wildfly runtime
  5. One set of general "BoostMojos" that will detect runtime and then call either the Liberty or Wildfly mojos
  6. Two sets of Booster Configurators (for Liberty and Wildfly).

Discussion:

  1. Default port is Port 8080. Should this be 9080 for Boost?
  2. Mojo rerouting - Do we want to do this differently?
  3. Booster Configurator refactoring - Do we want to do this differently too?
  4. How do properties look in all of this?

The refactoring done here is just one of probably a dozen ways we can do this. We should discuss and see what our best option is.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant