profile
viewpoint
Heath Stewart heaths @Microsoft Kirkland, WA https://heaths.dev Senior Software Engineer at @microsoft who enjoys hiking, backpacking, climbing, and snowshoeing the great Pacific Northwest when AFK.

Azure/azure-sdk-for-rust 218

This repository is for active development of the *unofficial* Azure SDK for Rust. This repository is *not* supported by the Azure SDK team.

AlexGhiondea/IssueCreator 6

Tool to streamline the creation of GitHub issues

heaths/Caffeine 4

Keeps your display or machine awake, and prevents or suspends non-critical shutdowns.

dtzar/oms-go 3

Golang program to submit to OMS's log analytics data collector API

heaths/azure-samples-kek 2

Azure SDK sample for client-side encryption and decryption of blobs

heaths/azure-sdk-for-net 2

Microsoft Azure SDK for .NET

heaths/autorest.csharp 0

Extension for AutoRest (https://github.com/Azure/autorest) that generates C# code

heaths/aziz 0

"Aziz - light!" This project is a simple application to turn switches on and off using SmartThings.

heaths/azpipelines-samples 0

Sample Azure Pipelines

heaths/azsdk-sample-getcert 0

Simple sample to get a private key for a Key Vault certificate

delete branch heaths/azure-sdk-for-net

delete branch : clu-tests

delete time in 3 days

push eventAzure/azure-sdk-for-net

Heath Stewart

commit sha ff8e1b2eaf005fd3554569ff65946788a8b21eb4

Enable Conversations live tests (#26346) * Enable Conversations live tests * Fix environment variables * Temporarily disable some tests Disable a few tests until #26379 is resolved so we can get nightly live tests working again. * Fix AnswersFollowupKnowledgeBaseQuestion results Had a second result come back that was 0.30 confidence, which I've seen in the past as well. Tends to vary whenever the model is retrained. Ask for a higher threshold now.

view details

push time in 3 days

PR merged Azure/azure-sdk-for-net

Reviewers
Enable Conversations live tests

The pipeline variable group has already been updated.

+152 -152

11 comments

7 changed files

heaths

pr closed time in 3 days

pull request commentAzure/azure-sdk-for-net

Enable Conversations live tests

/azp run net - cognitivelanguage - tests

heaths

comment created time in 3 days

push eventheaths/azure-sdk-for-net

Heath Stewart

commit sha 255b4e9bf831cb00233d9cadd7845eae6fd15472

Fix AnswersFollowupKnowledgeBaseQuestion results Had a second result come back that was 0.30 confidence, which I've seen in the past as well. Tends to vary whenever the model is retrained. Ask for a higher threshold now.

view details

push time in 3 days

pull request commentAzure/azure-sdk-for-net

Enable Conversations live tests

/azp run net - cognitivelanguage - tests

heaths

comment created time in 3 days

push eventheaths/azure-sdk-for-net

Heath Stewart

commit sha 46422c604b356e3aea918d4cd0bd35f22a76a470

Temporarily disable some tests Disable a few tests until #26379 is resolved so we can get nightly live tests working again.

view details

push time in 3 days

issue openedAzure/azure-sdk-for-net

Re-enable AnalyzeConversationWithLanguage tests when language vs. locale issue is resolves

The functional and sample tests named AnalyzeConversationWithLanguage and AnalyzeConversationWithLanguageAsync stopped working live with a rather vague 400 BadRequest. After some experimentation I found that BCP-47 language identifiers like "es" no longer work and a full locale identifier like "es-ES" is required. This is an unexpected changes and I want to clarify with the service team this is expected before making changes to the tests. We should also update the swagger documentation if that is the case.

For now I've disabled the tests as opposed to "fixing" them with what could be an accidental regression.

/cc @AhmedLeithy @ZulaMostafa @annatisch @mshaban-msft

created time in 3 days

issue commentAzure/azure-sdk-for-net

Add Immutable property

But, but...that's the only defense we have left!

heaths

comment created time in 3 days

issue openedAzure/azure-sdk-for-go

Regenerate, make sure immutable property is added

Now that https://github.com/Azure/azure-rest-api-specs/pull/17270 has been merged for 7.3-preview, please regenerate and make sure that the immutable property has been correctly added.

Testing might include setting a release policy and marking immutable true, then trying to change or remove the release policy which should err.

created time in 3 days

issue openedAzure/azure-sdk-for-go

Key

created time in 3 days

issue openedAzure/azure-sdk-for-python

Regenerate, make sure immutable property is added

Now that https://github.com/Azure/azure-rest-api-specs/pull/17270 has been merged for 7.3-preview, please regenerate and make sure that the immutable property has been correctly added.

Testing might include setting a release policy and marking immutable true, then trying to change or remove the release policy which should err.

created time in 3 days

issue openedAzure/azure-sdk-for-js

Regenerate, make sure immutable property is added

Now that https://github.com/Azure/azure-rest-api-specs/pull/17270 has been merged for 7.3-preview, please regenerate and make sure that the immutable property has been correctly added.

Testing might include setting a release policy and marking immutable true, then trying to change or remove the release policy which should err.

created time in 3 days

issue openedAzure/azure-sdk-for-java

Regenerate, make sure immutable property is added

Now that https://github.com/Azure/azure-rest-api-specs/pull/17270 has been merged for 7.3-preview, please regenerate and make sure that the immutable property has been correctly added.

Testing might include setting a release policy and marking immutable true, then trying to change or remove the release policy which should err.

created time in 3 days

issue openedAzure/azure-sdk-for-net

Add Immutable property

created time in 3 days

issue commentAzure/azure-sdk-for-net

[LLC/Data Plane Client] Tests slow down in playback mode beyond timeout limit

Agreed. Perhaps there is something deeper down the stack we can do that would work for both this new LLC waitForCompletion pattern as well as existing Operation/Operation<T> classes.

AhmedLeithy

comment created time in 3 days

pull request commentAzure/azure-rest-api-specs

Add CLU and CT Language authoring 2022-03-01-preview APIs

From internal discussion, since this apparently isn't meant to be complete but only enough for a review, you should be targeting a feature branch. Azure REST guidelines are that any version in main - preview or stable - is immutable (at least for breaking changes, which even adding a property is). I recommend creating one from the current Cognitive Language release feature branch, copying over common.json, and starting from there.

You don't have to start completely over, though. Assuming you're in your current topic branch for this PR:

git fetch upstream release/cognitiveservices/clu-2021-11-01-preview
git rebase FETCH_HEAD

common.json should show up in 2021-11-01-preview which you can copy over into your release folder as well.

moabba

comment created time in 3 days

push eventheaths/vssetup-rs

Heath Stewart

commit sha c54f210460ea636f210101eeab34e9796f765dc6

Support creating apartment when constructing (#7) * Support creating apartment when constructing Also does a bunch of refactoring since, otherwise, the lib.rs will eventually get quite large. * Always return Result<T>

view details

push time in 4 days

delete branch heaths/vssetup-rs

delete branch : refactoring

delete time in 4 days

PR merged heaths/vssetup-rs

Support creating apartment when constructing

Also does a bunch of refactoring since, otherwise, the lib.rs will eventually get quite large.

+232 -132

0 comment

4 changed files

heaths

pr closed time in 4 days

pull request commentAzure/azure-sdk-for-net

[Question Answering] Adding Samples, Tests and Readme

@AhmedLeithy I removed the ProjectReference to Azure.Core since that shipped - or is the new 1.22.0 version not the one you were expecting? - but looks like tests might need to be re-recorded, not to mention the test duration issue we discussed Thursday on Teams is happening.

AhmedLeithy

comment created time in 4 days

pull request commentAzure/azure-sdk-for-net

Enable Conversations live tests

Good news: this almost works! @AhmedLeithy the test AnalyzeConversationWithLanguage is failing with 400 BadRequest but the message isn't very specific as to why. Could you look into this, please? If we can get that fixed, these changes should re-enable the live tests.

/cc @annatisch @ZulaMostafa @mshaban-msft

heaths

comment created time in 4 days

push eventAhmedLeithy/azure-sdk-for-net

Mingzhe Huang

commit sha 9a25af5ac60f5edbafde75957f12a9a487606367

refactor(resourcemanager): invoke`ValidateResourceType` only in debug mode (#26205) * refactor(resourcemanager): invoke`ValidateResourceType` only in debug mode It's considered a smell when invoking virtual methods in constructor (see: https://www.codeproject.com/tips/641610/be-careful-with-virtual-method) In `TrackedResource` constructor, virtual method `ValidateResourceType` is invoked. Although after analysis we think it's OK, we still decided to invoke it in debug mode only, for dev/test purpose. This commit will apply conditional compilation, so that `ValidateResourceType` will be excluded in release build. - remove general warning suppress in `csproj` - add conditional compilation and warning suppress for `ValidateResourceType` inside constructor resolve https://github.com/azure/autorest.csharp/issues/1627 This commit also trims test session recordings of `BlobContainerSoftDelete` of `storage` SDK, since it will timeout in pipeline.

view details

Mingzhe Huang

commit sha 46558ecb588535746b4883d4a44bb8fb05307d1f

feat(resourcemanager): remove check name availability definition (#26199) * feat(resourcemanager): remove check name availability definition Per discussion, we should delegate the check name availability to each RP. So there is little value to keep common types for it. Besides, it will cause over replacement (e.g. `ExtendedLocation` in `connectedVmware`). part of https://github.com/Azure/autorest.csharp/issues/1701

view details

Wei Wei

commit sha 330761af808c10ab1775ffbc545384794852b3c9

[Storage] mgmt - support new API version 2021-08-01 (#26085) * [Storage] Support 2021-08-01 * Update since swagger change * [Storage] mgmt - update from last swagger

view details

Max Hamulyák

commit sha a45799438c64e0b45f6a9d0eee785f354abd76f7

Make ServiceBusAdministrationClient available in IAzureClientBuilder (#26225) * add extension method to register ServiceBusAdministrationClient in IAzureClientBuilder * update based on warnings * Revert "update based on warnings" This reverts commit 54adbca93cea7b6df617e7833ae0cc7c8ee59d93. * move methods as per PR suggestion * remove excessive whitespace * update using eng\scripts\CodeChecks.ps1 -ServiceDirectory servicebus

view details

Mohit Chakraborty

commit sha 4cd840462fdaa450b458f9a127d8937887c99def

[ACR] Prepare for 1.0.0 GA release (#26212) * Prepare for 1.0.0 GA release Rename a couple of enum types. Change package version to 1.0.0. Update Changelog. * Update public API and samples

view details

Jesse Squire

commit sha 19e125bacc4aa7ae6f6d750e16017d8b3e728976

[Event Hubs] Fix Processor Performance Tests (#26238) The focus of these changes is to add the processor performance tests project to the main solution, move to a filtered local solution view, and remove legacy code causing build issues.

view details

Anne Thompson

commit sha eadfc2d0350f45afb76dbe2debaa726c0eebb279

fix build break (#26216)

view details

JoshLove-msft

commit sha 289db656964be73f8cae00adc47b4270cce5fe46

Update Resource events (#26222) * Update Resource events * Export API * regen * Use Core RequestMethod * null assert

view details

m-nash

commit sha 2aec516597ca7d922a59c99f00bc12095abbad96

add nullable to Parent property (#26237) remove ctor warning suppress

view details

m-nash

commit sha 6f1f6aaffd0c827ed17b3fc285739ab4ad7a4a84

remove Root from ResourceType (#26243) * remove Root from ResourceType * update api

view details

Will Smith

commit sha 7ac5d48cd4e5ac2428a086c195143364a683734e

[KubernetesConfiguration] Releasing dotnet SDK for KubernetesConfiguration.Extensions api version 2021-09-01 (#26039) * Adding 2 packages to kubernetesconfiguration sdk * Fixing generated files * Removing extra files * Updating tests to use persistent cluster * Updating subscription ID of tests * Updated session records * Changing SourceControlConfigurations package back to KubernetesConfigurations * Moving all files to one package * Moving sourcecontrolconfigurations back to kubernetesconfiguration package * Updating AssemblyInfo.cs Co-authored-by: willsmith <willsmith@microsoft.com>

view details

Mingzhe Huang

commit sha f6b01aed98e5fc7bcd5f86ae017998da1d6115ae

docs(resourcemanager, core): incorrect return value in `compareTo` The docs of several `CompareTo` APIs says the return value is 0, 1, and -1. But actually for some use cases, the return value could be other numbers. We've found under some environment, our test cases will fail because the return value is 10, -10, 8, etc. The test cases have been fixed. This commit will fix the docs of those API.

view details

YanjunGao

commit sha bf6f25b78c984253bed9d7e346076bc1e08d6f8c

[Synapse] management sdk release Jan.2022 (#26249)

view details

Heath Stewart

commit sha 692e922263af70ef83528066cd4cd650e252cc00

Prepare beta release for Key Vault SDK (#26246)

view details

Jesse Squire

commit sha 5b74f611c7cc6d3a2d19dfb9d6109ac129b5633a

[Service Bus] Migration Guide FAQ (#26235) The focus of these changes is to start a FAQ within the migration guide to capture questions that we see frequently in GitHub issues, starting with documenting why we do not offer an API for batch message settlement.

view details

Heath Stewart

commit sha 861d85a02c2c94e4cd61a6b285e455586a65ba89

Resolved previous PR comments for CHANGELOGs (#26267)

view details

JoshLove-msft

commit sha 856edce4bc3e3fa5891b0e5be57c10059cd8eeec

Add back compat decoding for Avro encoder (#26244) * Add back compat decoding for Avro encoder * record

view details

Caio Saldanha

commit sha 3175ff0f2225385411463580045ea48138358671

[FormRecognizer] Updated testing cloud to Public (#26239)

view details

Jesse Squire

commit sha a28404ab3f4410b12f55f368397072b2d28ee56b

[Messaging] Bump AMQP library version (#26215) * [Messaging] Bump AMQP library version The focus of these changes is to move to v2.5.9 of the `Microsoft.Azure.Amqp` package. Because of visibility changes in some of the overloads, we're now passing the cancellation token in addition to a timeout when using the fault tolerant AMQP object manager. * Updating to v2.5.10 of the AMQP package * Fix lambda to pass the cancellation token rather than refereincing by closure.

view details

JoshLove-msft

commit sha ce5e8dd5138e7c39d884f23a517acc7ff6826f8c

Prepare for release (#26274)

view details

push time in 4 days

PullRequestReviewEvent

pull request commentAzure/azure-rest-api-specs

Add CLU and CT Language authoring 2022-03-01-preview APIs

It would still be good to pull in common.json and use its definitions when defined to make sure your service actually accomodates any differences. The Language pillar is trying to consolidate around common patterns as well, and this will help detect any differences.

moabba

comment created time in 4 days

pull request commentAzure/azure-sdk-for-net

Enable Conversations live tests

/azp run net - cognitivelanguage - tests

heaths

comment created time in 4 days

pull request commentAzure/azure-sdk-for-net

Enable Conversations live tests

/azp run net - cognitiveservices - tests

heaths

comment created time in 4 days

push eventheaths/azure-sdk-for-net

Heath Stewart

commit sha 278027986207daa3e2d4d45383d272ad09f9f77a

Fix environment variables

view details

push time in 4 days

more