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

rethink configurations spec #1680

Closed
lburgazzoli opened this issue Aug 31, 2020 · 4 comments
Closed

rethink configurations spec #1680

lburgazzoli opened this issue Aug 31, 2020 · 4 comments
Labels
area/core Core features of the integration platform kind/question Further information is requested status/never-stale
Milestone

Comments

@lburgazzoli
Copy link
Contributor

lburgazzoli commented Aug 31, 2020

As today providing configurations is done by a list of key/value pairs but that may not be very handing for tooling and in general for comprehension:

  configuration:
    - type: "property"
      value: "key=value"
    - type: "secret"
      value: "my-secret"

Wonder if we should provide a better organization, something like:

  configuration:
    - secret:
        name: twitter-directmessage
    - property:
        name: user
        value: my-user
    - property:
        name: password
        valueFrom:
          secretKeyRef:
            name: the-password
            key: password

which could give more freedom and would allow to use patterns commonly used in other kubernetes resources like referencing an entry in a secret or configmap

@lburgazzoli lburgazzoli added the kind/question Further information is requested label Aug 31, 2020
@lburgazzoli lburgazzoli added this to the 2.0.0 milestone Aug 31, 2020
@lburgazzoli
Copy link
Contributor Author

Relates to #1676

@lburgazzoli lburgazzoli added the area/core Core features of the integration platform label Aug 31, 2020
@lburgazzoli lburgazzoli changed the title review configurations spec rethink configurations spec Aug 31, 2020
@github-actions
Copy link
Contributor

This issue has been automatically marked as stale due to 90 days of inactivity.
It will be closed if no further activity occurs within 15 days.
If you think that’s incorrect or the issue should never stale, please simply write any comment.
Thanks for your contributions!

@squakez
Copy link
Contributor

squakez commented Dec 15, 2021

#2771 should be close this. With that work, the way to provide a configuration will be through container trait.

@squakez
Copy link
Contributor

squakez commented Feb 2, 2023

We manage them now via mount trait: https://camel.apache.org/camel-k/next/traits/mount.html

@squakez squakez closed this as completed Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/core Core features of the integration platform kind/question Further information is requested status/never-stale
Projects
None yet
Development

No branches or pull requests

2 participants