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

feat: add some eventhub e2e tests #3717

Merged
merged 5 commits into from
Oct 3, 2022

Conversation

JorTurFer
Copy link
Member

Signed-off-by: Jorge Turrado jorge_turrado@hotmail.es

The EventHub Namespace connectionString is needed to run these e2e tests, it should be placed in AZURE_EVENT_HUB_CONNECTION_STRING environment variable

Checklist

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Tests have been added
  • Changelog has been updated and is aligned with our changelog requirements

Fixes #2792

Signed-off-by: Jorge Turrado <jorge_turrado@hotmail.es>
Signed-off-by: Jorge Turrado <jorge_turrado@hotmail.es>
Signed-off-by: Jorge Turrado <jorge_turrado@hotmail.es>
@JorTurFer JorTurFer marked this pull request as ready for review October 2, 2022 15:10
@JorTurFer JorTurFer requested a review from a team as a code owner October 2, 2022 15:10
@JorTurFer
Copy link
Member Author

@tomkerkhove , could you create an eventhub namespace and place the manage connection string in AZURE_EVENT_HUB_CONNECTION_STRING?

Signed-off-by: Jorge Turrado <jorge.turrado@scrm.lidl>
CHANGELOG.md Outdated Show resolved Hide resolved
Co-authored-by: Tom Kerkhove <kerkhove.tom@gmail.com>
Signed-off-by: Jorge Turrado Ferrero <Jorge_turrado@hotmail.es>
@JorTurFer
Copy link
Member Author

JorTurFer commented Oct 3, 2022

/run-e2e event_hub*
Update: You can check the progress here

@JorTurFer JorTurFer enabled auto-merge (squash) October 3, 2022 11:34
@JorTurFer JorTurFer merged commit 14ece68 into kedacore:main Oct 3, 2022
@JorTurFer JorTurFer deleted the add-event-hub-e2e branch October 3, 2022 11:36
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.

Add e2e test for Azure Event Hub scaler
2 participants