-
Notifications
You must be signed in to change notification settings - Fork 130
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
CI does not update private Beta registry if Skip.PublishVcpkg = true
.
#3700
Comments
@danieljurek, it would be great if you can get this fixed before 6/7 - there are a couple packages that we will be releasing this way. |
Thanks for filing @antkmsft, this is a bug. The line here: https://github.com/Azure/azure-sdk-for-cpp/blame/main/eng/pipelines/templates/steps/vcpkg-publish.yml#L49 Should not be indented as deeply. The pipeline run is correctly identifying that a package should not be published to vcpkg in this case: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1483258&view=logs&j=3dbdc0e8-0699-52f6-1880-be74b44e10fc&t=589fe240-e4d3-576e-326a-612f68def396 And yet the step was running anyway. Once this is indented properly I expect skipping behavior to work correctly. |
Thank you @danieljurek! Just to be super clear, what will be the guidance when releasing post-GA Betas, once this is fixed - with cc @Jinming-Hu |
Only set If you leave
In short, we always publish to vcpkg betas and we publish main vcpkg UNLESS the a version already on vcpkg main is a GA version and the version being released is a beta. That is, beta versions of packages do not release over GA packages on the main vcpkg... Customers should not be forced to use a beta package when a GA package is available. |
@danieljurek If I want to ship GA version for storage-common package and beta version for storage-blobs package, do I have to do that in two separate pipelines, or can I do it in one pipeline? |
@Jinming-Hu -- You can do this in the same pipeline execution. Packages are evaluated independently to determine whether to publish to main vcpkg. |
This is not exactly what we want.
When
Skip.PublishVcpkg = true
, we don't want the PR for the vcpkg repo. But we do want to update our private Beta registry.Expected: Beta registry gets updated regardless of whether
Skip.PublishVcpkg
is true or not.The text was updated successfully, but these errors were encountered: