Skip to content

Commit cdfd52c

Browse files
committed
fix(om): decouple Azure storage provider
Keep only Azure.Core as a runtime dependency of the Azure Storage state-manager companion. Move Table and Blob SDK references to tests and leave Orleans provider selection to consumers, avoiding unwanted provider and version coupling.
1 parent 74b6de7 commit cdfd52c

4 files changed

Lines changed: 7 additions & 7 deletions

File tree

Egil.Orleans.Messaging/Directory.Packages.props

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
</ItemGroup>
2020
<ItemGroup Label="Orleans">
2121
<PackageVersion Include="Microsoft.Orleans.Reminders" Version="10.1.0" />
22-
<PackageVersion Include="Microsoft.Orleans.Persistence.AzureStorage" Version="10.1.0" />
2322
<PackageVersion Include="Microsoft.Orleans.Runtime" Version="10.1.0" />
2423
<PackageVersion Include="Microsoft.Orleans.Sdk" Version="10.1.0" />
2524
<PackageVersion Include="Microsoft.Orleans.Server" Version="10.1.0" />

Egil.Orleans.Messaging/README.md

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,9 +49,11 @@ Register the default state manager factory on the silo:
4949
siloBuilder.AddDefaultStateManager("state");
5050
```
5151

52-
For Orleans Azure Table or Blob grain storage, install
53-
`Egil.Orleans.Messaging.State.AzureStorage` and register the Azure-aware
54-
factory instead:
52+
For Orleans Azure Table or Blob grain storage, install and configure the
53+
Orleans storage provider separately. The Messaging companion works through
54+
`IPersistentState<T>` and Azure SDK exceptions; it does not select or install
55+
the underlying provider. Install `Egil.Orleans.Messaging.State.AzureStorage`
56+
and register the Azure-aware factory instead:
5557

5658
```csharp
5759
siloBuilder.AddAzureStorageStateManager("state");

Egil.Orleans.Messaging/src/Egil.Orleans.Messaging.State.AzureStorage/Egil.Orleans.Messaging.State.AzureStorage.csproj

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,6 @@
3434
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
3535
</PackageReference>
3636
<PackageReference Include="Azure.Core" />
37-
<PackageReference Include="Azure.Data.Tables" />
38-
<PackageReference Include="Azure.Storage.Blobs" />
39-
<PackageReference Include="Microsoft.Orleans.Persistence.AzureStorage" />
4037
</ItemGroup>
4138

4239
<ItemGroup>

Egil.Orleans.Messaging/test/Egil.Orleans.Messaging.State.AzureStorage.Tests/Egil.Orleans.Messaging.State.AzureStorage.Tests.csproj

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,8 @@
2323
</ItemGroup>
2424

2525
<ItemGroup>
26+
<PackageReference Include="Azure.Data.Tables" />
27+
<PackageReference Include="Azure.Storage.Blobs" />
2628
<PackageReference Include="coverlet.msbuild">
2729
<PrivateAssets>all</PrivateAssets>
2830
<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>

0 commit comments

Comments
 (0)