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

Allow nested metadata properties #590

Open
jeffhollan opened this issue Jan 30, 2020 · 2 comments
Open

Allow nested metadata properties #590

jeffhollan opened this issue Jan 30, 2020 · 2 comments
Labels
feature All issues for new features that have been committed to needs-discussion

Comments

@jeffhollan
Copy link
Member

Creating as this came up as a request during a standup. Some scalers like Azure Monitor may require a number of metadata options to configure, and today we require metadata be a flat list:

(hypothetical example)

metadata:
  connectionName:
  connectionType:
  connectionSubscription:
  metricType:
  metricName:
  metricThreshold:

We could consider allowing nesting or other ways to more easily define like:

metadata:
  connection:
    name:
    type:
    subscription:
  metric:
    type:
    name:
    threshold:

Open for discussion and upvotes 👍

@jeffhollan jeffhollan added needs-discussion feature-request All issues for new features that have not been committed to labels Jan 30, 2020
@tomkerkhove
Copy link
Member

tomkerkhove commented Jan 31, 2020

More than happy to see this being added 🙌

@stale
Copy link

stale bot commented Oct 14, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale All issues that are marked as stale due to inactivity label Oct 14, 2021
@tomkerkhove tomkerkhove added feature All issues for new features that have been committed to and removed feature-request All issues for new features that have not been committed to labels Oct 14, 2021
@stale stale bot removed stale All issues that are marked as stale due to inactivity labels Oct 14, 2021
@tomkerkhove tomkerkhove moved this to Backlog in Roadmap - KEDA Core Feb 10, 2022
@tomkerkhove tomkerkhove moved this from To Do to Proposed in Roadmap - KEDA Core Feb 11, 2022
SpiritZhou pushed a commit to SpiritZhou/keda that referenced this issue Jul 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature All issues for new features that have been committed to needs-discussion
Projects
Status: Proposed
Development

No branches or pull requests

2 participants