Skip to content

Add stork registration quickstart#1531

Open
aureamunoz wants to merge 1 commit intoquarkusio:developmentfrom
aureamunoz:add-stork-registration-quickstart
Open

Add stork registration quickstart#1531
aureamunoz wants to merge 1 commit intoquarkusio:developmentfrom
aureamunoz:add-stork-registration-quickstart

Conversation

@aureamunoz
Copy link
Copy Markdown
Member

@aureamunoz aureamunoz commented May 9, 2025

Check list:

Your pull request:

  • targets the development branch
  • uses the 999-SNAPSHOT version of Quarkus
  • has tests (mvn clean test)
  • works in native (mvn clean package -Pnative)
  • has integration/native tests (mvn clean verify -Pnative)
  • makes sure the associated guide must not be updated
  • links the guide update pull request (if needed)
  • updates or creates the README.md file (with build and run instructions)
  • for new quickstart, is located in the directory component-quickstart
  • for new quickstart, is added to the root pom.xml and README.md

Related to quarkusio/quarkus#47997 and quarkusio/quarkus#53252

@quarkus-bot
Copy link
Copy Markdown

quarkus-bot bot commented May 9, 2025

/cc @scrocquesel (amazon)

@aureamunoz aureamunoz changed the base branch from main to development May 9, 2025 15:34
@aureamunoz aureamunoz force-pushed the add-stork-registration-quickstart branch from fde2891 to a849d45 Compare July 16, 2025 13:28
@aureamunoz aureamunoz force-pushed the add-stork-registration-quickstart branch 3 times, most recently from ab72c2e to eba0c7a Compare August 7, 2025 09:33
@aureamunoz aureamunoz marked this pull request as ready for review August 7, 2025 09:33
@quarkus-bot

This comment has been minimized.

@aureamunoz aureamunoz force-pushed the add-stork-registration-quickstart branch from eba0c7a to d8cf4cb Compare August 7, 2025 10:32
@quarkus-bot

This comment has been minimized.

@quarkus-bot

This comment has been minimized.

@quarkus-bot

This comment has been minimized.

@aureamunoz aureamunoz force-pushed the add-stork-registration-quickstart branch from f91e94c to 6ea2ce7 Compare September 24, 2025 12:09
@quarkus-bot

This comment has been minimized.

@aureamunoz aureamunoz force-pushed the add-stork-registration-quickstart branch from 6ea2ce7 to 0999c36 Compare March 17, 2026 12:41
@quarkus-bot

This comment has been minimized.

@aureamunoz aureamunoz force-pushed the add-stork-registration-quickstart branch from 0999c36 to bca0def Compare March 23, 2026 16:05
@aureamunoz aureamunoz requested a review from cescoffier March 23, 2026 16:05
@quarkus-bot

This comment has been minimized.

@aureamunoz aureamunoz marked this pull request as draft March 23, 2026 17:30
@aureamunoz aureamunoz marked this pull request as ready for review March 24, 2026 17:15
@aureamunoz
Copy link
Copy Markdown
Member Author

This can now be reviewed; I hope I’ve included all the necessary information @cescoffier

@quarkus-bot

This comment has been minimized.

@cescoffier
Copy link
Copy Markdown
Member

Hum the stork QuickStarts failing do not look good.

@quarkus-bot

This comment has been minimized.

@aureamunoz
Copy link
Copy Markdown
Member Author

The stork quickstarts pass now. You can review @cescoffier , I've taken into account your comments. I will squash commits

@quarkus-bot

This comment has been minimized.

@aureamunoz aureamunoz force-pushed the add-stork-registration-quickstart branch from f77f3f1 to f7da814 Compare April 7, 2026 15:37
@quarkus-bot
Copy link
Copy Markdown

quarkus-bot bot commented Apr 7, 2026

Status for workflow Pull Request Build - development

This is the status report for running Pull Request Build - development on commit f7da814.

Failing Jobs

Status Name Step Failures Logs Raw logs
Build - JDK 17 Build with Maven Failures Logs Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ Build - JDK 17 #

- Failing: context-propagation-quickstart jta-quickstart 

📦 context-propagation-quickstart

org.acme.context.prices.PriceTest.test line 55 - More details - Source on GitHub

Details
org.awaitility.core.ConditionTimeoutException: Condition with Lambda expression in org.acme.context.prices.PriceTest was not fulfilled within 1 minutes  40 seconds.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:26)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:1160)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:1129)
	at org.acme.context.prices.PriceTest.test(PriceTest.java:55)
	Suppressed: io.quarkus.runtime.BlockingOperationNotAllowedException: @Transactional cannot start a JTA transaction within a reactive pipeline. If the annotated method is intended to be blocking, ensure it is executed on a worker thread, for example by annotating it with @Blocking. If the annotated method is intended to be reactive, consider using Hibernate Reactive, which supports @Transactio...

📦 jta-quickstart

org.acme.quickstart.TransactionalResourceTest.testAsync2Endpoint line 61 - More details - Source on GitHub

Details
java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <500>.

	at org.codehaus.groovy.vmplugin.v8.IndyInterface.fromCache(IndyInterface.java:344)
	at io.restassured.internal.ResponseSpecificationImpl$HamcrestAssertionClosure.validate(ResponseSpecificationImpl.groovy:516)
	at org.codehaus.groovy.vmplugin.v8.IndyInterface.fromCache(IndyInterface.java:344)
	at io.restassured.internal.ResponseSpecificationImpl.validateResponseIfRequired(ResponseSpecificationImpl.groovy:714)

@aureamunoz aureamunoz requested a review from cescoffier April 8, 2026 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants