Skip to content

Conversation

Ladicek
Copy link
Contributor

@Ladicek Ladicek commented May 26, 2025

@Ladicek Ladicek requested review from manovotn and mkouba May 26, 2025 15:41
@Ladicek Ladicek added area/arc Issue related to ARC (dependency injection) triage/backport-3.20 labels May 26, 2025
@gsmet
Copy link
Member

gsmet commented May 26, 2025

Adding the backport label too for inclusion in 3.23.

This comment has been minimized.

@gsmet gsmet force-pushed the arc-fix-interception-proxy branch from 3fcce0d to c2e08c4 Compare May 26, 2025 21:17

This comment has been minimized.

@Ladicek Ladicek force-pushed the arc-fix-interception-proxy branch from c2e08c4 to ec992f0 Compare May 27, 2025 06:33
@gsmet gsmet force-pushed the arc-fix-interception-proxy branch from ec992f0 to f7b9612 Compare May 27, 2025 08:12
@gsmet
Copy link
Member

gsmet commented May 27, 2025

Rebased to get CI fixes.

This comment has been minimized.

@Ladicek
Copy link
Contributor Author

Ladicek commented May 27, 2025

Thanks @gsmet for taking care of the CI. I had no idea why this PR should affect the MySQL container like that :-)

@Ladicek Ladicek force-pushed the arc-fix-interception-proxy branch from f7b9612 to 5379030 Compare May 27, 2025 12:25
@manovotn manovotn added the triage/waiting-for-ci Ready to merge when CI successfully finishes label May 27, 2025
@gsmet gsmet force-pushed the arc-fix-interception-proxy branch from 5379030 to 4e3f3cf Compare May 28, 2025 14:47
@gsmet gsmet force-pushed the arc-fix-interception-proxy branch from 4e3f3cf to f48156a Compare May 29, 2025 08:52
@gsmet
Copy link
Member

gsmet commented May 29, 2025

Rebasing one more time after another weird CI issue and fix.

Copy link

quarkus-bot bot commented May 29, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit f48156a.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.


Flaky tests - Develocity

⚙️ JVM Tests - JDK 17

📦 extensions/smallrye-reactive-messaging/deployment

io.quarkus.smallrye.reactivemessaging.hotreload.ConnectorChangeTest.testUpdatingConnector - History

  • Expecting actual: ["-6","-8","-9","-10","-11","-12","-13","-14"] to start with: ["-6", "-7", "-8", "-9"] - java.lang.AssertionError
java.lang.AssertionError: 

Expecting actual:
  ["-6","-8","-9","-10","-11","-12","-13","-14"]
to start with:
  ["-6", "-7", "-8", "-9"]

	at io.quarkus.smallrye.reactivemessaging.hotreload.ConnectorChangeTest.testUpdatingConnector(ConnectorChangeTest.java:41)

@geoand geoand merged commit dc12d3f into quarkusio:main May 30, 2025
57 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.24 - main milestone May 30, 2025
@quarkus-bot quarkus-bot bot added kind/bugfix and removed triage/waiting-for-ci Ready to merge when CI successfully finishes labels May 30, 2025
@Ladicek Ladicek deleted the arc-fix-interception-proxy branch May 30, 2025 13:26
@gsmet gsmet removed this from the 3.24 - main milestone Jun 3, 2025
@gsmet gsmet added this to the 3.23.1 milestone Jun 3, 2025
@jmartisk jmartisk modified the milestones: 3.23.1, 3.20.2 Jul 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/arc Issue related to ARC (dependency injection) kind/bugfix triage/flaky-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Proxied synthetic bean seems to proxy to two different super classes

6 participants