Review Event Grid Reaction

[ruokun-niu]

Description

Updated the EventGrid Reaction to use the nuget Reaction SDK and enabled authentication via managed identity. Currently the reaction only supports EventGrids that have the input schema of CloudEventSchemaV1_0.

Configuration

The reaction takes the following configuration properties:

Property	Description
eventGridUri	Topic Endpoint for your Azure EventGrid topic
eventGridKey	Access key to the EventGrid topic. This field is optional if using managed identity

Example with Access Key

kind: Reaction
apiVersion: v1
name: eventgrid
spec:
  kind: EventGrid
  queries:
    <query-id>:
  properties: 
    eventGridUri: <endpoint>
    eventGridKey: <access-key>

Example with Managed identity

kind: Reaction
apiVersion: v1
name: eventgrid
spec:
  kind: EventGrid
  queries:
    <query-id>:
  identity:
    kind: MicrosoftEntraWorkloadID
    clientId: <client-id>
  properties: 
    eventGridUri: <endpoint>

Type of change

• This pull request adds or changes features of Drasi and has an approved issue (issue link required).

Fixes: #issue_number

Copilot reviewed 6 out of 16 changed files in this pull request and generated no comments.

Files not reviewed (10)

• reactions/azure/eventgrid-reaction/.dockerignore: Language not supported
• reactions/azure/eventgrid-reaction/.gitignore: Language not supported
• reactions/azure/eventgrid-reaction/Dockerfile: Language not supported
• reactions/azure/eventgrid-reaction/Makefile: Language not supported
• reactions/azure/eventgrid-reaction/Properties/.gitignore: Language not supported
• reactions/azure/eventgrid-reaction/eventgrid-reaction.csproj: Language not supported
• reactions/azure/eventgrid-reaction/eventgrid-reaction.sln: Language not supported
• reactions/azure/storagequeue-reaction/Dockerfile: Language not supported
• reactions/azure/eventgrid-reaction/Models/ChangeNotification.cs: Evaluated as low risk
• reactions/azure/eventgrid-reaction/Program.cs: Evaluated as low risk

Comments suppressed due to low confidence (1)

reactions/azure/eventgrid-reaction/Services/ControlSignalHandler.cs:57

• Verify that 'ControlSignalNotificationOp.X' is the correct value for the 'Op' property in 'ControlSignalNotification'.

Op = ControlSignalNotificationOp.X,

[danielgerlag]

Does EventGrid have a batch send call? So they could all fail/pass together?

[ruokun-niu]

I think it's possible. We can combine them into a list of cloudevents and send this batch using SendEventAsync

https://learn.microsoft.com/en-us/dotnet/api/azure.messaging.eventgrid.eventgridpublisherclient.sendeventsasync?view=azure-dotnet#azure-messaging-eventgrid-eventgridpublisherclient-sendeventsasync(system-collections-generic-ienumerable((azure-messaging-cloudevent))-system-threading-cancellationtoken)

[ruokun-niu]

Just committed this change