Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #2687: Allow multiple integration platforms in the same namespace #2695

Merged
merged 9 commits into from
Oct 21, 2021

Conversation

nicolaferraro
Copy link
Member

@nicolaferraro nicolaferraro commented Oct 15, 2021

This (together with #2683) enables the upgrade/downgrade of individual integrations, handing them off to two different operators concurrently running in the same namespace. I've created a POC here: https://github.com/nicolaferraro/camel-k-selective-upgrade-demo

I've changed a bit #2683, to use an annotation for the operator.id instead of a label (and added a sibling platform.id annotation), since we're not using label selectors to query for them.

So we have two annotations that can be applied to any Camel K resource:

  • camel.apache.org/operator.id: allows to specify the operator that should reconcile the resource
  • camel.apache.org/platform.id: allows to specify the platform that should be used as global configuration

And another annotation that can be applied on a platform (camel.apache.org/secondary.platform) that can mark a platform as secondary to prevent the controller to mark it as "Duplicate" and allow using it explicitly.

Done:

  • Feature
  • E2E Tests
  • Documentation

Release Note

Resources can be configured to use a secondary integration platform

@nicolaferraro nicolaferraro added kind/feature New feature or request status/wip Work in progress labels Oct 15, 2021
@nicolaferraro nicolaferraro merged commit eb4e32e into apache:main Oct 21, 2021
@nicolaferraro nicolaferraro deleted the 2687-multi-operators branch October 21, 2021 16:12
@nicolaferraro nicolaferraro mentioned this pull request Nov 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants