Add support for listening to Azure Maintenance Events by amsoedal · Pull Request #1876 · StackExchange/StackExchange.Redis
and others added 21 commits
September 21, 2021 17:08Doing a set of suggestions here following our call - cc @amsoedal! I've just merged up latest, will do a test run here but wanted to get changes to make it a bit more extensible for other clouds and easier to consume in front of your eyes. Philo added some color around the purposes for the public events (makes sense!), so was trying to make using them a bit cleaner on the client side, e.g.: ```cs if (evt is AzureMaintenanceEvent ame && ame.NotificationType == AzureMaintenanceEvent.NotificationTypes.NodeMaintenanceStarting) ``` The extensibility model would be other cloud providers adding a PR here, which would consist in total of their handler (if different) and adding to the base `ServerMaintenanceEvent` (not meant to be externally extensible at the moment). Can you please provide thoughts? Co-authored-by: Michelle Soedal <ansoedal@microsoft.com>
NickCraver
deleted the
amsoedal/az_maintenance_events
branch
NickCraver pushed a commit that referenced this pull request
Nov 4, 2021Documentation for the feature introduced here: #1876
This was referenced
Jan 8, 2022This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters