You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "Croway (via GitHub)" <gi...@apache.org> on 2026/04/08 14:28:43 UTC
[PR] CAMEL-23300: add connectionString option to camel-couchbase endpoint [camel]
Croway opened a new pull request, #22494:
URL: https://github.com/apache/camel/pull/22494
## Summary
- Add `connectionString` endpoint option to `camel-couchbase` so users can pass a full Couchbase SDK connection string (e.g., `couchbase://host:port`), enabling connections when the KV port is not the default 11210
- Switch test-infra Couchbase container from hardcoded fixed ports to `isFixedPort()`, so tests use random ports by default (fixed ports only when `camel.infra.fixedPort=true` for Camel JBang)
- Update integration tests to pass `connectionString` from the test service
## Test plan
- [x] `camel-couchbase` compiles successfully
- [x] 3/3 producer integration tests pass with random ports (`-Dcouchbase-7`)
- [x] Backwards compatible: when `connectionString` is not set, behavior is unchanged
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] CAMEL-23300: add connectionString option to camel-couchbase endpoint [camel]
Posted by "Croway (via GitHub)" <gi...@apache.org>.
Croway merged PR #22494:
URL: https://github.com/apache/camel/pull/22494
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] CAMEL-23300: add connectionString option to camel-couchbase endpoint [camel]
Posted by "Croway (via GitHub)" <gi...@apache.org>.
Croway commented on PR #22494:
URL: https://github.com/apache/camel/pull/22494#issuecomment-4207156090
I faced an issue in camel-kafka-connectors test, and noticed that couchbase is not starting on a default port during tests. The test infra is configured to always start couch base on the default ports due to camel-couchbase limitation.
In this PR I am removing that limitation (it is now possible to define a connection string), and I am updating the test-infra to start couchbase on a random port (as expected).
This way, the tests in camel-kafka-connectors should be successful... hopefully!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] CAMEL-23300: add connectionString option to camel-couchbase endpoint [camel]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #22494:
URL: https://github.com/apache/camel/pull/22494#issuecomment-4207366427
<!-- ci-tested-modules -->
:test_tube: **CI tested the following changed modules:**
- `catalog/camel-catalog`
- `components/camel-couchbase`
- `dsl/camel-endpointdsl`
- `test-infra/camel-test-infra-couchbase`
:warning: **Some tests are disabled on GitHub Actions** (`@DisabledIfSystemProperty(named = "ci.env.name")`) and require manual verification:
- `components/camel-couchbase`: 6 test(s) disabled on GitHub Actions
<details><summary>All tested modules (7 modules)</summary>
- Camel :: Couchbase
- Camel :: JBang :: MCP
- Camel :: JBang :: Plugin :: Route Parser
- Camel :: JBang :: Plugin :: Validate
- Camel :: Launcher :: Container
- Camel :: YAML DSL :: Validator
- Camel :: YAML DSL :: Validator Maven Plugin
</details>
---
:gear: [View full build and test results](https://github.com/apache/camel/actions/runs/24141921419)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] CAMEL-23300: add connectionString option to camel-couchbase endpoint [camel]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #22494:
URL: https://github.com/apache/camel/pull/22494#issuecomment-4207071950
:star2: Thank you for your contribution to the Apache Camel project! :star2:
:robot: CI automation will test this PR automatically.
:camel: 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 although they are normally detected and executed by CI.
* You can label PRs using `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](https://camel.apache.org/community/team/#committers) have access to the summary.
:warning: Be careful when sharing logs. Review their contents before sharing them publicly.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] CAMEL-23300: add connectionString option to camel-couchbase endpoint [camel]
Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus commented on PR #22494:
URL: https://github.com/apache/camel/pull/22494#issuecomment-4207094470
Is the problem only related to a port number? I wonder if you can use port = 0, to indicate a non default port, if that was the case. Also the description for connection string includes a port number in the example but the text says its needed when you do not have a port number for dynamic assigned port.
So maybe we can allow to use :0 as port number if you want a dynamic port, and then don't need to add a new parameter. And the new parameter is also not tooling friendly.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org