search

umati / UA-CloudPublisher

umati-fork of OPC UA cloud publisher --- reference implementation leveraging OPC UA PubSub over MQTT. It runs in a Docker container on standard Docker hosts or on Kubernetes and comes with an easy-to-use web user interface.
Other
0 stars 0 forks source link

chore(deps): update dotnet monorepo to v8 (major) #25

Open renovate[bot] opened 4 months ago

renovate[bot] commented 4 months ago

Mend Renovate

This PR contains the following updates:

Package Type Update Change
mcr.microsoft.com/dotnet/aspnet stage major 6.0 -> 8.0
mcr.microsoft.com/dotnet/sdk final major 6.0 -> 8.0

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about these updates again.

This PR has been generated by Mend Renovate. View repository job log here.

renovate[bot] commented 2 months ago

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

GoetzGoerisch commented 2 months ago

@barnstee would you mind taking a look. I try to bump to dotnet 8, but the arm64 build fails with:

#20 [linux/arm64 build 4/7] RUN dotnet restore "./UA-CloudPublisher.csproj"
#20 12.06   Determining projects to restore...
#20 15.24 /usr/share/dotnet/sdk/8.0.302/NuGet.targets(1113,5): error MSB4018: The "ConvertToAbsolutePath" task failed unexpectedly. [/src/UA-CloudPublisher.csproj::TargetFramework=net8.0]
#20 15.24 /usr/share/dotnet/sdk/8.0.302/NuGet.targets(1113,5): error MSB4018: System.ArgumentNullException: Parameter "Paths" cannot be null. [/src/UA-CloudPublisher.csproj::TargetFramework=net8.0]
#20 15.24 /usr/share/dotnet/sdk/8.0.302/NuGet.targets(1113,5): error MSB4018:    at Microsoft.Build.Shared.ErrorUtilities.ThrowArgumentNull(String parameterName, String resourceName) [/src/UA-CloudPublisher.csproj::TargetFramework=net8.0]
#20 15.24 /usr/share/dotnet/sdk/8.0.302/NuGet.targets(1113,5): error MSB4018:    at Microsoft.Build.Tasks.ConvertToAbsolutePath.Execute() [/src/UA-CloudPublisher.csproj::TargetFramework=net8.0]
#20 15.24 /usr/share/dotnet/sdk/8.0.302/NuGet.targets(1113,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute() [/src/UA-CloudPublisher.csproj::TargetFramework=net8.0]
#20 15.24 /usr/share/dotnet/sdk/8.0.302/NuGet.targets(1113,5): error MSB4018:    at Microsoft.Build.BackEnd.TaskBuilder.ExecuteInstantiatedTask(ITaskExecutionHost taskExecutionHost, TaskLoggingContext taskLoggingContext, TaskHost taskHost, ItemBucket bucket, TaskExecutionMode howToExecuteTask) [/src/UA-CloudPublisher.csproj::TargetFramework=net8.0]
#20 ERROR: process "/bin/sh -c dotnet restore \"./UA-CloudPublisher.csproj\"" did not complete successfully: exit code: 1
#24 [linux/amd64 build 7/7] RUN dotnet build "UA-CloudPublisher.csproj" -c Release -o /app/build
#24 CANCELED

https://github.com/umati/UA-CloudPublisher/actions/runs/9742088762/job/26882650070?pr=25#step:7:399

barnstee commented 2 months ago

Converting to .Net8 is a larger task as you will need to rev all NuGet packages to their .Net8 equivalent, also. Do you need .Net8 or is this a nice to have?

GoetzGoerisch commented 2 months ago

Thanks for having a look.

We normally strive to have the latest LTS version in use.

The project builds fine, amd64 on amd64, native and in container, build fine arm64 on arm64, native and container. It only does not build in buildx container build building arm64 emulated on amd64.

Our other projects converted fine.