-
Notifications
You must be signed in to change notification settings - Fork 1
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
default datasource not created since Swarm 2016.12.0 #8
Comments
Note, the exact same code works with swarmtool 2016.11.0. |
So I guess it's a result of this breaking change in Swarm. |
That would be my guess. We changed to allow you to add a JDBC driver as a dependency and we'd auto create the module.xml for it |
@kenfinnigan OK, but the problem is we don't use Maven, and instead call swarmtool using |
Ah. Could you raise a JIRA issue, as I suspect that's a gap we hadn't thought of |
OK. (I did try |
Issue reported as https://issues.jboss.org/browse/SWARM-1439. |
Thanks @gavinking |
Hi, @kenfinnigan is there some easy workaround for this issue? Some config file I can easily add to my project to force the default datasource to be created? Thanks. |
You're attempting this against a pre-built .war that you can't rebuild? If you can somehow get your driver within your WEB-INF/lib, then it should create the default DS based on that driver. |
What if you add a dep to the h2 module in the Ceylon module, @gavinking? |
When testing EAP quickstarts with Swarm, I found the same obstacle. I added the So maybe |
FTR, this was tracked under https://issues.jboss.org/browse/SWARM-1100 |
Didn't help. |
Where did it end up in the |
Where precisely? My |
Oh, OK, it's added in newer versions. Fine. |
@Ladicek alright, so here's my problem. But after 2017.3.2, the server starts, but simply doesn't deploy my JAX-RS app. This is all that is logged by the server startup:
So apparently there is also something else that changed. |
Note that this is with an app that worked fine up until it was broken by swarm 2016.12.0. |
@FroMage @bobmcwhirter On Swarm 2016.12.0 I tried adding But the datasource still didn't get created.
|
There is probably some additional metadata required by Swarm 2017.3+, but I don't know what it might be. |
@kenfinnigan @bobmcwhirter Do I need some sort of yaml file or something in the newer versions of Swarm? |
You don't have to use yaml, but we're moving that way instead of custom main() usage |
Well I have neither a |
Not unless you need to configure fractions in any way, such as DS config |
So then I don't understand why it doesn't discover and start my JAX-RS application in the latest releases. What changed? |
No error messages? Endpoint not running at all? |
Exactly. |
Very weird. Is the project on GH somewhere we can play with? |
Yes it is, it is https://github.com/DiegoCoronel/ceylon-jboss-swarm/ but you need to explicitly specify the version of swarm and the jdbc driver on the command line. Wait till I get home I will tell you the command I'm using. |
OK, @kenfinnigan, the commands you need to compile, build, and run the jar are:
Note that the command line options I've added are:
Let me know how you go. |
Thanks for the details @gavinking, will take a look |
Appreciated. |
@gavinking Was able to reproduce the problem locally. In the logs I see that Is |
It's just a wrapper that directly calls swarmtool |
When using Swarm from version 2016.12.0 onward, I get:
My guess is that there must be something missing from the fractions, but I'm not sure what.
@FroMage, @kenfinnigan, @bobmcwhirter any idea?
The text was updated successfully, but these errors were encountered: