Skip to content

[azure-*-cpp] Update to April Releases#38085

Merged
JavierMatosD merged 5 commits into
microsoft:masterfrom
azure-sdk:azsdk-cpp-2024-04
Apr 15, 2024
Merged

[azure-*-cpp] Update to April Releases#38085
JavierMatosD merged 5 commits into
microsoft:masterfrom
azure-sdk:azsdk-cpp-2024-04

Conversation

@azure-sdk

Copy link
Copy Markdown
Contributor

Update vcpkg ports for Azure SDK release. This release may contain multiple ports.

## 1.11.3 (2024-04-09)

### Bugs Fixed

- [[microsoft#5450]](Azure/azure-sdk-for-cpp#5450) Reverted libcurl connection pool to use more conservative caching strategy.
- [[microsoft#4352]](Azure/azure-sdk-for-cpp#5371) Fixed compilation error on Visual Studio 2017. (A community contribution, courtesy of _[morten-ofstad](https://github.com/morten-ofstad)_)

### Acknowledgments

Thank you to our developer community members who helped to make Azure Core better with their contributions to this release:

- Morten Ofstad _([GitHub](https://github.com/morten-ofstad))_
@azure-sdk

Copy link
Copy Markdown
Contributor Author

Adding azure-core to release

cc: @antkmsft

## 1.0.0-beta.8 (2024-04-09)

### Breaking Changes

- Claims Based Security authentication now longer throws a `std::runtime_error`, and instead follows the pattern of the rest of the AMQP library and returns an error.
- Authentication now throws `Azure::Core::Credentials::AuthenticationException` instead of `std::runtime_error`.
- Added `Cancelled` status to `CbsOperationResult` and `ManagementOperationStatus`.

### Bugs Fixed

- [[microsoft#5284]](Azure/azure-sdk-for-cpp#5284) [azure-identity][azure-messaging-eventhubs] Impossible to catch exception resulting in SIGABRT signal.
- [[microsoft#5297]](Azure/azure-sdk-for-cpp#5297): Enabled multiple simultaneous `ExecuteOperation` calls.
- Fixed crash when Link Detach message is received while link is being destroyed.

### Other Changes

- `std::ostream` inserter for message body no longer prints the body of the message.
- Tidied up the output of the `AmqpMessage` `std::ostream` inserter.
- Added several `std::ostream` inserters.
- Pass numeric values to `std::ostream` inserters by value not by reference.
@azure-sdk

Copy link
Copy Markdown
Contributor Author

Adding azure-core-amqp to release

cc: @antkmsft

## 1.0.0-beta.2 (2024-04-09)

### Features Added

- Updates to models, transactions and other features.
@azure-sdk

Copy link
Copy Markdown
Contributor Author

Adding azure-data-tables to release

cc: @gearama

## 1.0.0-beta.7 (2024-04-09)

### Breaking Changes

- Renamed `EventDataBatch::TryAddMessage` to `EventDataBatch::TryAdd` to better reflect the method's use.

### Bugs Fixed

- Fixed [microsoft#5297](Azure/azure-sdk-for-cpp#5297). The actual fix for this was to use a single management client per connection.
@azure-sdk

Copy link
Copy Markdown
Contributor Author

Adding azure-messaging-eventhubs to release

cc: @RickWinter @ahsonkhan @antkmsft @gearama @LarryOsterman

@WangWeiLin-MV WangWeiLin-MV self-assigned this Apr 10, 2024
@WangWeiLin-MV WangWeiLin-MV added the category:port-update The issue is with a library, which is requesting update new revision label Apr 10, 2024
Comment thread ports/azure-core-amqp-cpp/portfile.cmake Outdated
Comment thread ports/azure-core-cpp/portfile.cmake Outdated
Comment thread ports/azure-data-tables-cpp/portfile.cmake Outdated
Comment thread ports/azure-messaging-eventhubs-cpp/portfile.cmake Outdated
@antkmsft

antkmsft commented Apr 10, 2024

Copy link
Copy Markdown
Member

@WangWeiLin-MV, thank you for the suggestion, but please allow us to not have that - we have automation that handles generation of this file (we've had similar conversation in the PR with one of your team members about a year ago).
In our case, the REF there does not always have to have the "azure-...-${VERSION}" format (in our daily pipelines, it is just a SHA, for example - here).

Here we had that discussion - #29514.

@antkmsft

Copy link
Copy Markdown
Member

@WangWeiLin-MV, I am not seeing info:reviewed tag, so I'm just checking, if you think we are good to go?
If there are more questions, or suggestions, or if more time is needed - that's ok, I'll be happy to respond especially if anything is needed from us, but if not, then ideally it would be great we are ready when there is the next slot for PR merges, and our port updates are picked up along with the other PRs.

@WangWeiLin-MV

WangWeiLin-MV commented Apr 11, 2024

Copy link
Copy Markdown
Contributor

@antkmsft Sorry for the long wait , I have approved and added the tag.

@WangWeiLin-MV WangWeiLin-MV added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Apr 11, 2024
@antkmsft

Copy link
Copy Markdown
Member

@WangWeiLin-MV No problem. Thank you!

@WangWeiLin-MV WangWeiLin-MV removed the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Apr 11, 2024

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@antkmsft

Copy link
Copy Markdown
Member

Wow, thank you so much @WangWeiLin-MV for the deep dive, analysis, and your PRs Azure/azure-sdk-for-cpp#5514 and Azure/azure-sdk-for-cpp#5516! Really over the top, above and beyond!
I will test them both (for Azure/azure-sdk-for-cpp#5516, we have a dummy package that we can run through the release automation and "release" it to see if everything is working).
I will test them both today, and will update this PR later in the day. If Azure/azure-sdk-for-cpp#5516 works as well, I will that the ${VERSION} change as well - it looks like we'll need to re-run x-add-version for this PR anyway, and since our build scripts will also be updated, I will take that another change as well.
With dependencies, great catch.
Thanks again!

@antkmsft

Copy link
Copy Markdown
Member

@WangWeiLin-MV, I have applied your feedback to this PR.

@WangWeiLin-MV WangWeiLin-MV left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Port usage tests pass with following triplets:

  • x64-windows
  • x64-windows-static

@WangWeiLin-MV WangWeiLin-MV added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Apr 15, 2024
@JavierMatosD JavierMatosD merged commit d1b2324 into microsoft:master Apr 15, 2024
JavierMatosD pushed a commit that referenced this pull request Apr 23, 2024
In #38085, we released 4 azure SDK libraries, and updated port manifest
files to use `HEAD_REF` and `${VERSION}`.
This PR updates the rest of the
https://github.com/Azure/azure-sdk-for-cpp/ ports to have the same
changes, and also updates the remaining Azure SDK for C++ ports to have
multi-line `$comment`.

This will keep all Azure SDK for C++ ports in uniform state when it
comes to portfiles, reflects the mainfest changes from the repo, and
will help to see minimalistic diffs during the next releases and in
daily validation runs (#34835).

cc @WangWeiLin-MV

---------

Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:port-update The issue is with a library, which is requesting update new revision info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants