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

chore: add resources to deploy with helm and terraform connection-str… #59

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

xoanmm
Copy link

@xoanmm xoanmm commented Apr 16, 2023

Resources have been created for the Process Azure Service Bus Queue by using connection string authentication scenario, both for the creation of resources in Azure using terraform through the files located in infra-as-code/connection-string and for the deployment of the manifests located in folder deploy/connection-string using a helm chart created through the files located in deploy/helm/keda-servicebus-connection-string for that purpose.

Also a Dockerfile was created for the order generator, and the code located in src/Keda.Samples.Dotnet.OrderGenerator/Program.cs has been modified to be able to check the messages from a variable called ORDER_AMOUNT to be able to launch this job in a disatended way. For this to work it's necessary to create a new image with the code with modifications, and push this image to registry ghcr.io/kedacore/sample-dotnet-order-generator-servicebus using the latest tag.

The purpose of these change are facilitate the task of testing this scenario by reducing the number of manual steps and automating the process.

Checklist

  • Commits are signed with Developer Certificate of Origin (DCO)
  • A PR is opened to update the documentation on our docs repo

@xoanmm xoanmm requested a review from tomkerkhove as a code owner April 16, 2023 17:13
@xoanmm
Copy link
Author

xoanmm commented Apr 16, 2023

@tomkerkhove I didn't add a PR in docs repo because I think it's not necessary, I have marked only to pass the checks and let the pipelines be launched, please feel free to comment any improvement or change to be made.

@zroubalik
Copy link
Member

@tomkerkhove PTAL

@tomkerkhove
Copy link
Member

Jup, still on my list but need some more time - Sorry!

@xoanmm
Copy link
Author

xoanmm commented Nov 23, 2023

Any chance to take a look @tomkerkhove ?

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.

3 participants