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

Enlist Cosmos DB scaler on Artifact Hub #10

Merged

Conversation

JatinSanghvi
Copy link
Contributor

@JatinSanghvi JatinSanghvi commented Nov 10, 2021

Added package for 'KEDA external scaler for Azure Cosmos DB' in 'KEDA Official External Scalers' repository on Artifact Hub.

Checklist

  • Commits are signed with Developer Certificate of Origin (DCO)

Fixes kedacore/external-scaler-azure-cosmos-db#7

Copy link
Member

@tomkerkhove tomkerkhove left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We'll need a bit more docs

artifacthub/azure-cosmos-db/0.0.1/README.md Outdated Show resolved Hide resolved
@tomkerkhove
Copy link
Member

Can you also provide the install section as per https://github.com/artifacthub/hub/blob/master/docs/metadata/artifacthub-pkg.yml#L31 please?

This should help them install the external scaler.

Example: https://artifacthub.io/packages/keptn/keptn-integrations/dynatrace-service?modal=install (source)

@JatinSanghvi JatinSanghvi marked this pull request as draft November 10, 2021 15:28
@JatinSanghvi
Copy link
Contributor Author

Marked PR as draft. I will get the release created and Helm chart ready for the external scaler before working on this one.

@tomkerkhove
Copy link
Member

Feel free to let me know if you need help here @JatinSanghvi (but take your time)

@JatinSanghvi JatinSanghvi force-pushed the JatinSanghvi-enlist-cosmosdb-scaler branch from 1ef5b7d to c10ed0e Compare November 22, 2021 05:10
@JatinSanghvi JatinSanghvi marked this pull request as ready for review November 22, 2021 05:11
@JatinSanghvi JatinSanghvi force-pushed the JatinSanghvi-enlist-cosmosdb-scaler branch 2 times, most recently from 46ff9ca to a077598 Compare November 22, 2021 05:29
Copy link
Member

@tomkerkhove tomkerkhove left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

artifacthub/azure-cosmos-db/0.1.0/artifacthub-pkg.yml Outdated Show resolved Hide resolved
artifacthub/azure-cosmos-db/0.1.0/artifacthub-pkg.yml Outdated Show resolved Hide resolved
artifacthub/azure-cosmos-db/0.1.0/artifacthub-pkg.yml Outdated Show resolved Hide resolved
artifacthub/azure-cosmos-db/0.1.0/artifacthub-pkg.yml Outdated Show resolved Hide resolved
Signed-off-by: Jatin Sanghvi <20547963+JatinSanghvi@users.noreply.github.com>
Signed-off-by: Jatin Sanghvi <20547963+JatinSanghvi@users.noreply.github.com>
Signed-off-by: Jatin Sanghvi <20547963+JatinSanghvi@users.noreply.github.com>
@JatinSanghvi JatinSanghvi force-pushed the JatinSanghvi-enlist-cosmosdb-scaler branch from 704946e to 7bfef81 Compare November 24, 2021 11:25
Copy link
Member

@tomkerkhove tomkerkhove left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks a ton!

@tomkerkhove tomkerkhove merged commit 9066847 into kedacore:main Nov 24, 2021
@JatinSanghvi JatinSanghvi deleted the JatinSanghvi-enlist-cosmosdb-scaler branch November 24, 2021 13:11
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.

List Cosmos DB external scaler on Artifact Hub
2 participants