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

Boosters should use standalone dependency if they are running tests in standalone mode #33

Closed
3 tasks
dipak-pawar opened this issue Feb 22, 2018 · 7 comments
Assignees

Comments

@dipak-pawar
Copy link

dipak-pawar commented Feb 22, 2018

I have observed that boosters [1], [2] are using arquillian-junit-container dependency even they want to use standalone mode.

Tasks

[1] - https://github.com/snowdrop/spring-boot-http-booster/blob/master/pom.xml#L56
[2] - https://github.com/openshiftio-vertx-boosters/vertx-http-booster/blob/master/pom.xml#L84

@bartoszmajsak bartoszmajsak mentioned this issue Feb 22, 2018
13 tasks
@hemanik hemanik self-assigned this Feb 26, 2018
@bartoszmajsak
Copy link
Member

bartoszmajsak commented Feb 26, 2018

@dipak-pawar are these all boosters with such dependencies? I think we are missing Wildfly Swarm if not more.

Another thing - as this inherits Cube version from parent pom - shouldn't we first update this one and ask for the release?

Can we get the list of all boosters first @hemanik? Probably something we could script/figure out from https://github.com/fabric8-launcher/launcher-booster-catalog/tree/master

I believe the update procedure to use -starter should be automated.

@bartoszmajsak
Copy link
Member

@dipak-pawar
Copy link
Author

are these all boosters with such dependencies? I think we are missing Wildfly Swarm if not more.

I found above two when I was playing with them. Couldn't find time to check all others. But we should check all others and update it with single dependency.

Another thing - as this inherits Cube version from parent pom - shouldn't we first update this one and ask for the release?

You are right. You can see second task Update boosters parent to use new version of released cube with arquillian/arquillian-cube#993 merged which is for that only.

I believe the update procedure to use -starter should be automated.

👍

@bartoszmajsak
Copy link
Member

Let's prepare automation part of it and do actual PRs/contributions to upstream when we have Cube/Che experience improved.

@hemanik
Copy link
Member

hemanik commented Mar 6, 2018

The following needs to be done for the automation part and to do actual PRs/contributions to upstream:

  • Use GitHub APIs to fork and clone the Booster Repositories.
  1. Spring Boot boosters are at https://github.com/snowdrop/
  2. Vert.x boosters are at https://github.com/openshiftio-vertx-boosters/
  3. Swarm boosters are at https://github.com/wildfly-swarm-openshiftio-boosters/
  • Implementation part of pom bump to update dependencies.
  1. Replace the two arquillian dependencies for standalone with the single starter dependency. (Dropped from the scope of the automation script since it is a one-time thing.)
  2. Update the Arquillian Cube version used if old.
  • Use GitHub APIs to open a PR with the updated pom.xml

@hemanik
Copy link
Member

hemanik commented Mar 20, 2018

Initial Spike for the Automation Script - https://github.com/hemanik/version-bump-automation.

@hemanik
Copy link
Member

hemanik commented Mar 26, 2018

As discussed offline, updating the starter-dependencies can be best done manually since it's a one-time thing.

With regards to this issue then, the following needs to be addressed:

The PRs for the last two tasks on hold for the release of booster-parent with the updated cube version.

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

No branches or pull requests

3 participants