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

ci: remove --no-build from dotnet pack commands #4808

Merged

Conversation

alex-chew
Copy link
Contributor

This removes the --no-build flag from dotnet pack commands in the publish-release-reusable.yml workflow file. This was causing a failure with the following message:

Error: /usr/share/dotnet/sdk/8.0.100/Sdks/NuGet.Build.Tasks.Pack/build/NuGet.Build.Tasks.Pack.targets(221,5): error NU5026: The file '/home/runner/work/dafny/dafny/dafny/Binaries/net6.0/DafnyDriver.runtimeconfig.json' to be packed was not found on disk. [/home/runner/work/dafny/dafny/dafny/Source/DafnyDriver/DafnyDriver.csproj]
...
Error: Process completed with exit code 1.

from this run: https://github.com/dafny-lang/dafny/actions/runs/6931560381/job/18866468076

This PR should unblock CI jobs for now, at the cost of a (maybe redundant?) rebuild of Dafny during each release job. I've left some notes below for root causing afterwards.


The last successful run of this workflow is here: https://github.com/dafny-lang/dafny/actions/runs/6921119802/job/18826708143

The first obvious difference to me is that, even though dotnet-version: 6.0.x was in the environment in both runs, the successful run used .NET 7 during the dotnet build command:

  .NET SDK:
   Version:   7.0.403
   Commit:    142776d834
  
  Runtime Environment:
   OS Name:     ubuntu
   OS Version:  20.04
   OS Platform: Linux
   RID:         ubuntu.20.04-x64
   Base Path:   /usr/share/dotnet/sdk/7.0.403/
  
  Host:
    Version:      7.0.13
    Architecture: x64
    Commit:       3f73a2f186
  
  .NET SDKs installed:
    6.0.416 [/usr/share/dotnet/sdk]
    6.0.417 [/usr/share/dotnet/sdk]
    7.0.113 [/usr/share/dotnet/sdk]
    7.0.203 [/usr/share/dotnet/sdk]
    7.0.310 [/usr/share/dotnet/sdk]
    7.0.403 [/usr/share/dotnet/sdk]

whereas the failing build used .NET 8.0:

   .NET SDK:
   Version:           8.0.100
   Commit:            57efcf1350
   Workload version:  8.0.100-manifests.6c33ef20
  
  Runtime Environment:
   OS Name:     ubuntu
   OS Version:  20.04
   OS Platform: Linux
   RID:         linux-x64
   Base Path:   /usr/share/dotnet/sdk/8.0.100/
  
  .NET workloads installed:
   Workload version: 8.0.100-manifests.6c33ef20
  There are no installed workloads to display.
  
  Host:
    Version:      8.0.0
    Architecture: x64
    Commit:       5535e31a71
  
  .NET SDKs installed:
    6.0.417 [/usr/share/dotnet/sdk]
    7.0.114 [/usr/share/dotnet/sdk]
    7.0.203 [/usr/share/dotnet/sdk]
    7.0.311 [/usr/share/dotnet/sdk]
    7.0.404 [/usr/share/dotnet/sdk]
    8.0.100 [/usr/share/dotnet/sdk]

Description

How has this been tested?

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

@keyboardDrummer keyboardDrummer added the run-deep-tests Tells CI to run all tests label Nov 21, 2023
@alex-chew alex-chew marked this pull request as ready for review November 21, 2023 00:21
@keyboardDrummer keyboardDrummer enabled auto-merge (squash) November 21, 2023 09:35
@keyboardDrummer keyboardDrummer merged commit 4f7af54 into dafny-lang:master Nov 21, 2023
29 of 35 checks passed
@alex-chew alex-chew deleted the ci/2023-11-20-dotnet-pack-issue branch November 21, 2023 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
run-deep-tests Tells CI to run all tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants