Skip to content

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