You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "oscerd (via GitHub)" <gi...@apache.org> on 2026/08/06 13:30:36 UTC
[PR] CAMEL-24363: camel-aws - iam doStop, eks catalog defaultValue, parameter-store value header [camel]
oscerd opened a new pull request, #25389:
URL: https://github.com/apache/camel/pull/25389
Three small, independent fixes found during the camel-aws review.
**1. `IAM2Endpoint.doStop()` did not call `super.doStop()`** — it closed the IAM client but skipped the base `ScheduledPollEndpoint`/`DefaultEndpoint` stop logic. Every other aws2 endpoint (including the paired `KMS2Endpoint`) calls it. Fixed.
**2. eks `profileCredentialsName` had a bogus catalog default** — the `String` option was annotated `@UriParam(defaultValue = "false")` (copy-pasted from the adjacent boolean provider flags), publishing a misleading `"defaultValue": "false"` for a profile-name string. Now matches ECS (`@UriParam(label = "security")`); catalog regenerated.
**3. parameter-store `CamelAwsParameterStoreValue` header was advertised but never read** — `putParameter` always took the value from the message body, so the documented producer header was a no-op. It now honors the header when present (falling back to the body). Adds `ParameterStoreProducerTest.putParameterHonorsTheValueHeader`.
All three are low severity. Full 683-module reactor build is green; the new parameter-store test passes. Targeting main (4.22.0).
_Claude Code on behalf of oscerd_
--
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-24363: camel-aws - iam doStop, eks catalog defaultValue, parameter-store value header [camel]
Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus merged PR #25389:
URL: https://github.com/apache/camel/pull/25389
--
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-24363: camel-aws - iam doStop, eks catalog defaultValue, parameter-store value header [camel]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #25389:
URL: https://github.com/apache/camel/pull/25389#issuecomment-5205479849
: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-24363: camel-aws - iam doStop, eks catalog defaultValue, parameter-store value header [camel]
Posted by "davsclaus (via GitHub)" <gi...@apache.org>.
davsclaus commented on PR #25389:
URL: https://github.com/apache/camel/pull/25389#issuecomment-5205633616
There are uncommitted changes
HEAD detached at pull/25389/merge
Changes not staged for commit:
(use "git add <file>..." to update what will be committed)
(use "git restore <file>..." to discard changes in working directory)
modified: catalog/camel-catalog/src/generated/resources/org/apache/camel/catalog/components/aws2-eks.json
modified: dsl/camel-componentdsl/src/generated/java/org/apache/camel/builder/component/dsl/Aws2EksComponentBuilderFactory.java
modified: dsl/camel-endpointdsl/src/generated/java/org/apache/camel/builder/endpoint/dsl/EKS2EndpointBuilderFactory.java
--
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