oscerd opened a new pull request, #23540:
URL: https://github.com/apache/camel/pull/23540
Backports #23438 to `camel-4.18.x`.
Cherry-pick of merge commit `902504dd360` with one adaptation: the original
PR's upgrade-guide entry was added to `camel-4x-upgrade-guide-4_21.adoc`,
which does not exist on the 4.18.x maintenance branch. Per the project's
backport policy the same content has been added to
`camel-4x-upgrade-guide-4_18.adoc` instead.
## Summary
Renames the Exchange header string values across `OpenstackConstants`,
`KeystoneConstants`, `NovaConstants`, `CinderConstants`, `GlanceConstants`,
`NeutronConstants` and `SwiftConstants` that were **not** in the `Camel`
namespace to `CamelOpenstack<Subsystem><Name>`. 55+ constants across the 6
subsystems (cinder/glance/keystone/neutron/nova/swift) plus the common
`OpenstackConstants`.
## Scope decisions preserved from main PR
- **Common headers** in `OpenstackConstants` use a `CamelOpenstack<Name>`
prefix (`CamelOpenstackOperation`, `CamelOpenstackId`,
`CamelOpenstackName`,
`CamelOpenstackDescription`, `CamelOpenstackProperties`).
- **Subsystem-specific headers** use `CamelOpenstack<Subsystem><Name>`.
- **`SwiftConstants.CONTAINER_METADATA_PREFIX`, `VERSIONS_LOCATION`,
`CONTAINER_READ`, `CONTAINER_WRITE` are unchanged** because they are
Swift HTTP protocol contract values (`X-Container-Meta-`,
`X-Versions-Location`, `X-Container-Read`, `X-Container-Write`) used by
openstack4j.
- **Non-header constants** (`SCHEME_*`, `CREATE`, `UPDATE`, `GET_ALL`, `GET`,
`DELETE`, subsystem-name constants) keep their previous values.
The cherry-pick includes the fix to `ContainerProducerTest.getTest` that was
shipped with the main-branch PR — the test had been using
`SwiftConstants.LIMIT`/`DELIMITER` to look up values in the openstack4j
`ContainerListOptions` internal map; with the rename, the test correctly uses
the literal openstack4j parameter names (`"limit"`, `"delimiter"`).
## Test plan
- [x] `mvn test` in `components/camel-openstack` on 4.18.x — 142 tests pass
- [x] Cherry-pick applied cleanly except for the 4_21 → 4_18 guide-file
rename described above (no merge marker shipped)
- [x] Upgrade-guide entry added under `=== camel-openstack - potential
breaking change` in `camel-4x-upgrade-guide-4_18.adoc`
Tracker: CAMEL-23577
_Reported by Claude Code on behalf of Andrea Cosentino_
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
