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

Changes to start pushing packages to blob Feed and to BAR #361

Merged
merged 2 commits into from Apr 15, 2019

Conversation

joperezr
Copy link
Member

cc @safern @chcosta

Adding required changes in order to publish packages to blob feed and to update BAR with each official build.

@mmitche is there anything additional to these changes needed in order to be included in the official shipping train?

@joperezr joperezr requested review from chcosta and safern April 15, 2019 19:42
@mmitche
Copy link
Member

mmitche commented Apr 15, 2019

@joperezr Are these OOB packages shipping on their own schedule?

@joperezr
Copy link
Member Author

No, they are shipping along with .NET Core 3.0 and its previews. Is that ok?

@mmitche
Copy link
Member

mmitche commented Apr 15, 2019

Okay, are they referenced in other repos?

@joperezr
Copy link
Member Author

No, just referenced by this same repo. No other dotnet repo depends on these two packages.

strategy:
matrix:
release_configuration:
_BuildConfig: Release
Copy link
Member

Choose a reason for hiding this comment

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

nit: you could also just set this as

variables:
- _BuildConfig: Release

Copy link
Member Author

Choose a reason for hiding this comment

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

true in this case since we only build the one configuration but I leave as is if that's ok for consistency with the other jobs which do have more than one configuration.

@@ -239,7 +270,7 @@ jobs:
KeyVaultName: EngKeyVault
SecretsFilter: 'dotnet-myget-org-api-key'
- script: powershell -ExecutionPolicy ByPass -NoProfile -command "& """$(Build.SourcesDirectory)\eng\common\Build.ps1""" -restore" &&
for /f %%x in ('dir /s /b $(Build.SourcesDirectory)\artifacts\packages\pushToMyGet\*.nupkg') do $(Build.SourcesDirectory)\.dotnet\dotnet.exe nuget push %%x -k $(dotnet-myget-org-api-key) -s https://dotnet.myget.org/F/dotnet-core/
for /f %%x in ('dir /s /b $(Build.SourcesDirectory)\artifacts\packages\pushToMyGet\*.nupkg') do dotnet nuget push %%x -k $(dotnet-myget-org-api-key) -s https://dotnet.myget.org/F/dotnet-core/
Copy link
Member

Choose a reason for hiding this comment

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

Does Arcade really not provide support for publishing to myget?

Rather than using the AzureKeyVault task above, I'd suggest you use the "DotNet-MyGet-Publish" variable group.

variables:
  - group: DotNet-MyGet-Publish

Copy link
Member Author

Choose a reason for hiding this comment

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

It might support it, but at least it didn't back when I initially added this since we were moving away from it. I'll switch to use the group as you suggest.

@mmitche
Copy link
Member

mmitche commented Apr 15, 2019

No, just referenced by this same repo. No other dotnet repo depends on these two packages.

Since these packages do not have direct dependencies on the latest core-setup (Microsoft.NETCore.App) and no one else depends on them, they are a bit more like OOB packages and aligning with previews is more about you choosing an appropriate build and doing the package push when needed.

For completeness, you should add a default channel for this repo:

darc add-default-channel --channel ".NET Core 3 Dev" --repo https://github.com/dotnet/iot --branch refs/heads/master

@joperezr joperezr merged commit 909d6dd into dotnet:master Apr 15, 2019
@joperezr joperezr deleted the PublishToBlobFeed branch May 16, 2019 21:40
@github-actions github-actions bot locked and limited conversation to collaborators Dec 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants