keda
keda copied to clipboard
Provide support for SAS token auth in Azure Event Hubs Scaler
Report
Based on current documentation https://keda.sh/docs/2.6/scalers/azure-event-hub/
- when defining Azure EventHub Scaler there is no option to use EventHub SAS token (which was generated using: https://docs.microsoft.com/en-us/rest/api/eventhub/generate-sas-token)
- or pass SAS token for the checkpoint blob storage Both settings expected to get connection string values - which in some cases is not a valid option
Expected Behavior
To have ability to use SAS token
Actual Behavior
Get only connection string
Steps to Reproduce the Problem
- Setup EventHub scaler
- Try to use SAS token for the EventHub or for Checkpoint blob storage
- Only receive connection string
Logs from KEDA operator
No need
KEDA Version
2.5.0
Kubernetes Version
No response
Platform
No response
Scaler Details
No response
Anything else?
No response
Fair request, adding to backlog. Are you willing to contribute this?
For sure, I will try to find some time for this. Thx
@yaronpri Do you any branch/commit for proving this feature? Would love to help with you
@v-shenoy If you are bored, is this something you want to pick up?
@tomkerkhove I can take them (both event hub and service bus) up next week.
That would be awesome, but don't feel pressured!
@tomkerkhove I Can take up this parallelly with https://github.com/kedacore/keda/issues/2920
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.
Bump
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.
Hi @SushmithaVReddy is there a plan to add support for this ? I saw that https://github.com/kedacore/keda/issues/2920 (Azure Service Bus) - was closed but this issue is still valid.
is this something you have in progress @v-shenoy ?
is this something you have in progress @v-shenoy ?
I was planning to do this. But last I checked the SDK that we use does not support SAS tokens. I am not sure if that has changed.
is this something you have in progress @v-shenoy ?
I was planning to do this. But last I checked the SDK that we use does not support SAS tokens. I am not sure if that has changed.
Okay, I am taking a look at the docs, and there is an interface that mentions SAS tokens. I will take a look and send a PR if it's supported.
After going through the docs for azure-event-hubs-go/v3, it relies on the azure-amqp-common-go/sas package for a SAS token provider which requires either providing the key name & value, or a connection string with the same format as the one KEDA uses. Not the SAS token format.
@yaronpri Maybe you know more about using the SAS tokens of the form you provided with the SDK that we use in KEDA?
Hi @v-shenoy from what I see the way to generate SAS token for EventHub is similar to the way it's generated for ServiceBus. So same concept which was done for ServiceBus could work also for EventHub https://learn.microsoft.com/en-us/azure/service-bus-messaging/service-bus-sas https://learn.microsoft.com/en-us/rest/api/eventhub/generate-sas-token