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.

Sign up for GitHub

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

Jump to bottom

4.x: small archetype issues #7446

Closed
4 tasks done
barchetta opened this issue Aug 24, 2023 · 1 comment
Closed
4 tasks done

4.x: small archetype issues #7446

barchetta opened this issue Aug 24, 2023 · 1 comment
Assignees
@tvallin
Labels
4.x Version 4.x archetypes
Projects
Backlog

Comments

@barchetta
Copy link
Member

barchetta commented Aug 24, 2023 •
edited by tvallin

Minor issues found while testing the 4.0 archetypes:

database-mp

  • README says "that uses the dbclient API". This uses JPA.

quickstart-se

  • README: health URL uses PORT instead of 8080. This is different from all other urls in the README
  • pom has two copies of dependencies for helidon-webserver-observe-health and helidon-health-checks
  • Exception when test runs:
2023.08.24 11:05:46.430 Attempt to start in unexpected state state STARTED; ignored
java.lang.IllegalStateException
        at io.helidon.metrics.PeriodicExecutor.lambda$startExecutor$1(PeriodicExecutor.java:139)
        at io.helidon.metrics.PeriodicExecutor.sync(PeriodicExecutor.java:175)
        at io.helidon.metrics.PeriodicExecutor.startExecutor(PeriodicExecutor.java:124)
        at io.helidon.metrics.PeriodicExecutor.start(PeriodicExecutor.java:91)
        at io.helidon.metrics.RegistryFactory.start(RegistryFactory.java:199)
        at io.helidon.webserver.observe.metrics.MetricsFeature.beforeStart(MetricsFeature.java:166)
@github-actions github-actions bot added this to Triage in Backlog Aug 24, 2023
@barchetta barchetta mentioned this issue Aug 24, 2023
Merged
@barchetta barchetta changed the title 4.x: small archetype fixes 4.x: small archetype issues Aug 24, 2023
@barchetta barchetta added 4.x Version 4.x archetypes labels Aug 25, 2023
@tvallin tvallin mentioned this issue Aug 25, 2023
Merged
@barchetta barchetta moved this from Triage to Sprint Scope in Backlog Aug 25, 2023
@tvallin
Copy link
Member

tvallin commented Aug 29, 2023

Fixed by #7459

@tvallin tvallin closed this as completed Aug 29, 2023
Backlog automation moved this from Sprint Scope to Closed Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tvallin tvallin

Labels
4.x Version 4.x archetypes
Projects
Backlog
Archived in project
Backlog
  
Closed
Milestone
No milestone
Development

No branches or pull requests
2 participants
@barchetta @tvallin