Skip to content
This repository has been archived by the owner on Feb 20, 2021. It is now read-only.

Upgrade Microsoft.WindowsAzure.Storage in SemanticLogging.WindowsAzure project #81

Closed
robichaud opened this issue May 12, 2016 · 7 comments
Milestone

Comments

@robichaud
Copy link

SemanticLogging.WindowsAzure references a very old version of Microsoft.WindowsAzure.Storage. Breaking changes in Microsoft.WindowsAzure.Storage v7.0 prevent redirecting.

@ohadschn
Copy link
Contributor

ohadschn commented Jul 24, 2016

Seconded. This is what you get when you try using the new storage SDK:

Exception thrown: 'System.MissingMethodException' in Microsoft.Practices.EnterpriseLibrary.SemanticLogging.WindowsAzure.dll

Additional information: Method not found: 'Void Microsoft.WindowsAzure.Storage.Table.CloudTableClient.set_RetryPolicy(Microsoft.WindowsAzure.Storage.RetryPolicies.IRetryPolicy)'.

As a workaround, you could implement a simple sink that pipes events to the System Diagnostics Trace and then have Azure upload them to Table Storage. See the following related SO discussion for a sample implementation: http://stackoverflow.com/questions/38141018/how-can-i-track-the-source-of-exception-on-trying-to-set-azure-sink-for-a-listen.

Another (better, since you'd keep the schema) workaround would be to upload to Application Insights and use Continuous Export (not available in the free tier). Fortunately there's already a sink for that: https://github.com/fidmor89/SLAB_AppInsights.

@bennage
Copy link
Member

bennage commented Aug 25, 2016

We're in the process of scheduling this work. My apologies for the lengthy delay.

@bennage bennage added this to the 2.1 milestone Aug 25, 2016
@robertmacgrogan
Copy link

I ran into this today when trying to upgrade to all of the latest Azure tools and had to do my own patch to fix the problem. The fix is trivial.

Just replace line 69 in WindowsAzureTableSink with the following:

        client.DefaultRequestOptions = new TableRequestOptions
        {
            RetryPolicy = new ExponentialRetry(TimeSpan.FromSeconds(5), 7),
            MaximumExecutionTime = TimeSpan.FromSeconds(10)
        };

This will compile. However, I have not yet been able to test because upgrading azure tools also blows up lots of other things. I'll post later when I see this working successfully.

@bennage
Copy link
Member

bennage commented Aug 25, 2016

@robertmacgrogan thanks!

@robertmacgrogan
Copy link

I wanted to report that I finally was able to test and my fix as reported above works just fine. Not relevant here, but interesting to note that latest version of Microsoft.AspNet.SignalR.ServiceBus had the exact same issue.

Thanks.

paulirwin added a commit to paulirwin/semantic-logging that referenced this issue Oct 3, 2016
mekoda pushed a commit that referenced this issue Oct 28, 2016
Fixes #81: update Azure Storage dependency to 7.x
@robichaud
Copy link
Author

robichaud commented Jan 25, 2017

@paulirwin @mekoda Any idea when you will release an updated NuGet package?

@NBelham
Copy link

NBelham commented May 7, 2018

For me I downgraded and that solved my issue
<package id="WindowsAzure.Storage" version="3.0.3.0" targetFramework="net45" />

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants