Skip to content

Switch to 1ES servicing pools on release/dev16.7-vs-deps#57504

Merged
JoeRobich merged 2 commits intodotnet:release/dev16.7-vs-depsfrom
jonfortescue:1esSvcPoolsReleaseDev167vsdeps
Dec 16, 2021
Merged

Switch to 1ES servicing pools on release/dev16.7-vs-deps#57504
JoeRobich merged 2 commits intodotnet:release/dev16.7-vs-depsfrom
jonfortescue:1esSvcPoolsReleaseDev167vsdeps

Conversation

@jonfortescue
Copy link
Copy Markdown
Contributor

@jonfortescue jonfortescue self-assigned this Nov 1, 2021
@jonfortescue jonfortescue requested a review from a team as a code owner November 1, 2021 20:12
@ghost ghost added the Area-Infrastructure label Nov 1, 2021
@jonfortescue
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 2 pipeline(s).

@333fred
Copy link
Copy Markdown
Member

333fred commented Nov 9, 2021

@JoeRobich why do we even still have this branch? Shouldn't the -vs-deps branches all be removed at this point?

@JoeRobich
Copy link
Copy Markdown
Member

@JoeRobich why do we even still have this branch? Shouldn't the -vs-deps branches all be removed at this point?

We opted to keep the -vs-deps branches as our servicing branches. Helps keep things consistent when releases all come from -vs-deps branches.

@JoeRobich
Copy link
Copy Markdown
Member

All that being said. This branch is not buildable because it still references myget feeds that are no longer available. We should add the dotnet-public and myget-legacy feeds to the nuget.config. Even then there may be packages we have to hunt down and get published.

@jonfortescue
Copy link
Copy Markdown
Contributor Author

@JoeRobich would you be okay merging this even though the builds are failing in the meantime so we can migrate this off the BuildPools?

@JoeRobich
Copy link
Copy Markdown
Member

@JoeRobich would you be okay merging this even though the builds are failing in the meantime so we can migrate this off the BuildPools?

Let me try updating the nuget.config to see if this is an easy fix.

@lpatalas
Copy link
Copy Markdown

@JoeRobich Do you have any update on this PR? This is one of the last issues that's blocking us from completing 1ES migration.

@JoeRobich
Copy link
Copy Markdown
Member

Do you have any update on this PR?

Well, since the branch isn't buildable, there is no reason not to take the change. We would have to do a fair bit of work either way to service this release. Let us merge it and may April come soon.

@JoeRobich JoeRobich merged commit f8c8414 into dotnet:release/dev16.7-vs-deps Dec 16, 2021
@lpatalas
Copy link
Copy Markdown

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants