corvus-dotnet / corvus-dotnet/Corvus.Tenancy
Clarify failure modes around GetXxxContainerForTenantAsync
- Dominant language
- C#
- Stars
- 6
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Description
In https://github.com/marain-dotnet/Marain.UserNotifications/pull/112 it became apparent that we have a global lack of clarity around expected error handling behaviour in cases where the requested container cannot be created for some reason.
There are at least two different reasons this could happen:
1. transient infrastructure failure
2. incorrect configuration
The documentation for all of our tenanted storage providers says nothing about how to distinguish between these. This had percolated up the stack causing the same essential problem to become visible in the PR in `Marain.UserNotifications`. But that PR can't solve that problem on its own. It first needs to be addressed at the `Corvus.Tenancy` level, and once we've established the pattern here, that will have knock-on consequences for anything that consumes this.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by reviewing the behavior discussed in Marain.UserNotifications PR #112 and the GetXxxContainerForTenantAsync APIs in Corvus.Tenancy. Establish how transient infrastructure failures differ from incorrect configuration, then document the resulting pattern for tenanted storage providers and identify its consequences for consumers such as Marain.UserNotifications.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100