Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[azure-iot-sdk-c] Update public-preview feature to branch with fixed telemetry #9059

Merged
merged 6 commits into from
Nov 22, 2019

Conversation

jspaith
Copy link
Member

@jspaith jspaith commented Nov 21, 2019

Updates public-preview SDK

@PhoebeHui PhoebeHui requested a review from LilyWangL November 21, 2019 03:07
REF d1cdf78b5160af8e08354e102a6b96395eee79e1
SHA512 0efbfc19e5eef4831b55ded0e8d88e83194bc0f26886841ddc83405c15b7f1bae983e22dc569e22846acd78b843b9e7492883b7c502f4eed92ff80ef45a9942d
REF 1d0402c80944ec7ec24ed24314457166fef98814
SHA512 0c3e173fa9e21efedbdb5efad155494595d1246420b590649e81a8c43e45eff4d659197d201c664a6490e5099be41546ac606c4f7d6c88e26ff32873aaff801f
HEAD_REF public-preview
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This commit commits on the branch public-preview-merge, so need to modify HEAD_REF to public-preview-merge.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@LilyWangL - thanks for feedback. You're correct on public-preview-merge branch. I wanted to get a CI pass of the fix as it was even though our public-preview branch was being updated in parallel. public-preview now has the changed merged into it, and I've updated our port file to point at public-preview's SHA.

I'm moving PR as ready-to-review now.

@LilyWangL
Copy link
Contributor

Note: Feature public-preview test pass with following triplets:

  • x86-windows (This port does not support build dynamic library)
  • x86-windows-static (Pass)
  • x64-windows-static (Pass)
  • arm64-windows (Pass)
  • arm-windows (Pass)
  • x64-uwp (Public-preview due to dependency build failure)
  • x64-linux (Pass)

The feature use_prov_client has nothing to do with this change.

@grdowns grdowns changed the title Update for telemetry issues (public-preview only) [azure-iot-sdk-c] Update public-preview feature to branch with fixed telemetry Nov 21, 2019
@jspaith jspaith marked this pull request as ready for review November 21, 2019 22:24
@ras0219-msft ras0219-msft merged commit 4411b9f into microsoft:master Nov 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants