Skip to content

[release/7.0] Fix servicing failures when building coreclr nuget packages - #75899

Merged
ViktorHofer merged 1 commit into
release/7.0from
backport/pr-75884-to-release/7.0
Sep 21, 2022
Merged

[release/7.0] Fix servicing failures when building coreclr nuget packages#75899
ViktorHofer merged 1 commit into
release/7.0from
backport/pr-75884-to-release/7.0

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Backport of #75884 to release/7.0

/cc @ViktorHofer

Customer Impact

Testing

Risk

IMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.

Fixes the failures in the 7.0.1 test servicing build:
```
/__w/1/s/.packages/microsoft.dotnet.build.tasks.packaging/7.0.0-beta.22416.1/build/Packaging.targets(800,5): error : No VersionSuffix was set. Ensure it is set before targets in packaging are ran. [/__w/1/s/src/coreclr/.nuget/Microsoft.NETCore.ILAsm/Microsoft.NETCore.ILAsm.pkgproj]
##[error].packages/microsoft.dotnet.build.tasks.packaging/7.0.0-beta.22416.1/build/Packaging.targets(800,5): error : (NETCORE_ENGINEERING_TELEMETRY=Build) No VersionSuffix was set. Ensure it is set before targets in packaging are ran.
/__w/1/s/.packages/microsoft.dotnet.build.tasks.packaging/7.0.0-beta.22416.1/build/Packaging.targets(800,5): error : No VersionSuffix was set. Ensure it is set before targets in packaging are ran. [/__w/1/s/src/coreclr/.nuget/Microsoft.NETCore.ILDAsm/Microsoft.NETCore.ILDAsm.pkgproj]
##[error].packages/microsoft.dotnet.build.tasks.packaging/7.0.0-beta.22416.1/build/Packaging.targets(800,5): error : (NETCORE_ENGINEERING_TELEMETRY=Build) No VersionSuffix was set. Ensure it is set before targets in packaging are ran.
```
@ghostghost added the area-Infrastructure-coreclr Only use for closed issues label Sep 20, 2022
@ghost

Copy link
Copy Markdown

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of #75884 to release/7.0

/cc @ViktorHofer

Customer Impact

Testing

Risk

IMPORTANT: Is this backport for a servicing release? If so and this change touches code that ships in a NuGet package, please make certain that you have added any necessary package authoring and gotten it explicitly reviewed.

Author:github-actions[bot]
Assignees:-
Labels:

area-Infrastructure-coreclr

Milestone:-

@ViktorHofer

Copy link
Copy Markdown
Member

@jeffschwMSFT this infrastructure change is necessary to successfully build 7.0.x servicing bits. This was already part of release/6.0 but was never forward ported into main. Please see the linked issue for more details.

I assume this is a tell-mode only change and can be merged without additional approvals?

@jeffschwMSFTjeffschwMSFT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

approved. agree this is tell mode. is the failing ci related?

@ViktorHofer

Copy link
Copy Markdown
Member

I don't think so. Seems to be another AzDO flakiness/outage:

##[warning]Git fetch failed with exit code 128, back off 7.517 seconds before retry.
git --config-env=http.extraheader=env_var_http.extraheader fetch --force --tags --prune --prune-tags --progress --no-recurse-submodules origin --depth=20 +1986e7a0aa0866dfc873111fe13f1907888f5ec4:refs/remotes/origin/1986e7a0aa0866dfc873111fe13f1907888f5ec4
fatal: unable to access 'https://github.com/dotnet/runtime/': The requested URL returned error: 429
##[error]Git fetch failed with exit code: 128

@ViktorHofer
ViktorHofer merged commit 473b312 into release/7.0Sep 21, 2022
@ViktorHofer
ViktorHofer deleted the backport/pr-75884-to-release/7.0 branch September 21, 2022 11:19
@ghostghost locked as resolved and limited conversation to collaborators Oct 21, 2022
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Infrastructure-coreclrOnly use for closed issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@ViktorHofer@jeffschwMSFT