Skip to content

chore: remove duplicate credentials x-descriptor#2882

Merged
oscerd merged 1 commit into
apache:mainfrom
oscerd:quick-fix/fix-duplicate-credentials-descriptors
Jun 25, 2026
Merged

chore: remove duplicate credentials x-descriptor#2882
oscerd merged 1 commit into
apache:mainfrom
oscerd:quick-fix/fix-duplicate-credentials-descriptors

Conversation

@oscerd

@oscerd oscerd commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Summary

Follow-up to #2879 (merged). That PR converted the deprecated urn:alm:descriptor:com.tectonic.ui:password x-descriptor to urn:camel:group:credentials on four properties that already carried the canonical urn:camel:group:credentials, leaving a duplicate entry. @apupier spotted it while reviewing #2881.

This removes the redundant descriptor so each property lists urn:camel:group:credentials exactly once.

Affected

  • crypto-encrypt-actionkey
  • crypto-decrypt-actionkey
  • spring-rabbitmq-sinkpassword
  • spring-rabbitmq-sourcepassword

Verification

  • Catalog-wide check: no property has duplicate x-descriptors across the 250 Kamelets.
  • script/validator over all 250 Kamelets: no errors.
  • All four files parse as valid YAML.

Note: the "duplicate x-descriptors" class of bug would be caught automatically by a validator rule; #2881 already hardens script/validator and a dedup rule is noted there as a natural follow-up.


AI-generated by Claude Code on behalf of Andrea Cosentino (@oscerd).

apache#2879 converted the deprecated urn:alm:descriptor:com.tectonic.ui:password
x-descriptor to urn:camel:group:credentials on four properties that already
carried the canonical urn:camel:group:credentials descriptor, leaving a
duplicate entry (spotted by @apupier on apache#2881). Remove the redundant line so
each property lists the credentials descriptor once.

Affected: crypto-encrypt-action, crypto-decrypt-action, spring-rabbitmq-sink,
spring-rabbitmq-source. A catalog-wide check confirms no property has
duplicate x-descriptors.

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>
@oscerd oscerd requested review from apupier and davsclaus June 25, 2026 16:17
@oscerd oscerd merged commit cdceafb into apache:main Jun 25, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant