Skip to content

Conversation

@squakez
Copy link
Contributor

@squakez squakez commented Nov 25, 2025

Closes apache/camel-quarkus#8003

Description

Target

  • I checked that the commit is targeting the correct branch (Camel 4 uses the main branch)

Tracking

  • If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

  • I checked that each commit in the pull request has a meaningful subject line and body.
  • I have run mvn clean install -DskipTests locally from root folder and I have committed all auto-generated changes.

@github-actions
Copy link
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟

🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run

  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot.

  • You can label PRs using build-all, build-dependents, skip-tests and test-dependents to fine-tune the checks executed by this PR.

  • Build and test logs are available in the Summary page. Only Apache Camel committers have access to the summary.

  • ⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@davsclaus
Copy link
Contributor

@squakez any feedback on this

@squakez
Copy link
Contributor Author

squakez commented Nov 25, 2025

@squakez any feedback on this

It's good to go if approved. We had to revert that previous change based on push/pop because Quarkus runtime does not manage it properly for some reason. We can safely use put/remove and be still consistent with the requirements and compliant with Quarkus runtime as well.

@squakez squakez merged commit e7383b1 into apache:main Nov 25, 2025
5 checks passed
@squakez squakez deleted the fix/ceq_8003 branch November 25, 2025 14:09
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.

[camel-main] Mdc: CAMEL-22717 changes the behavior of the component (in progress), which makes the test in CQ fail

2 participants