redhat-developer / dotnet-regular-tests

.NET Core tests for .NET Bunny (RHEL & Fedora RPM tests)
MIT License
8 stars 13 forks source link

omnisharp fails with .NET 8/9 #250

Closed tmds closed 9 months ago

tmds commented 1 year ago

Log file:

Executing /tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp/test.sh with arguments 8.0.0-preview.6.23307.4 in working directory /tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp
+ rm -rf workdir
+ mkdir workdir
+ pushd workdir
/tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp/workdir /tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp
++ ../../runtime-id --portable
+ runtime_id=linux-x64
+ omnisharp_urls=("https://github.com/OmniSharp/omnisharp-roslyn/releases/latest/download/omnisharp-${runtime_id}-net6.0.tar.gz" "https://github.com/OmniSharp/omnisharp-roslyn/releases/download/v1.38.2/omnisharp-${runtime_id}.tar.gz" "https://github.com/OmniSharp/omnisharp-roslyn/releases/download/v1.37.17/omnisharp-${runtime_id}.tar.gz")
+ IFS=.-
+ read -ra version_split
+ [[ 8 == \7 ]]
+ for download_url in "${omnisharp_urls[@]}"
++ basename https://github.com/OmniSharp/omnisharp-roslyn/releases/latest/download/omnisharp-linux-x64-net6.0.tar.gz
+ tarball=omnisharp-linux-x64-net6.0.tar.gz
+ rm -f omnisharp-linux-x64-net6.0.tar.gz
+ rm -rf omnisharp
++ uname -m
+ [[ x86_64 == \a\a\r\c\h\6\4 ]]
+ [[ 8 -lt 6 ]]
+ wget --no-verbose https://github.com/OmniSharp/omnisharp-roslyn/releases/latest/download/omnisharp-linux-x64-net6.0.tar.gz
2023-06-13 09:04:27 URL:https://objects.githubusercontent.com/github-production-release-asset-2e65be/27927984/f80bdcc5-69f2-4dd3-ae45-20702dc5659a?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20230613%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20230613T090410Z&X-Amz-Expires=300&X-Amz-Signature=ba65ae9b70c83547f8a15c0df5483911356b8799bd1d1bac4c69659b61b12edb&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=27927984&response-content-disposition=attachment%3B%20filename%3Domnisharp-linux-x64-net6.0.tar.gz&response-content-type=application%2Foctet-stream [43149563/43149563] -> "omnisharp-linux-x64-net6.0.tar.gz" [1]
+ mkdir omnisharp
+ pushd omnisharp
+ tar xf ../omnisharp-linux-x64-net6.0.tar.gz
/tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp/workdir/omnisharp /tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp/workdir /tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp
+ '[' -f omnisharp/bin/mono ']'
+ '[' -f run ']'
+ '[' -f OmniSharp ']'
+ run_omnisharp_against_projects ./OmniSharp
+ for project in blazorserver blazorwasm classlib console mstest mvc nunit web webapp webapi worker xunit
+ rm -rf hello-blazorserver
+ mkdir hello-blazorserver
+ pushd hello-blazorserver
+ dotnet new blazorserver
/tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp/workdir/omnisharp/hello-blazorserver /tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp/workdir/omnisharp /tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp/workdir /tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp

Welcome to .NET 8.0!
---------------------
SDK Version: 8.0.100-preview.6.23311.1

----------------
Installed an ASP.NET Core HTTPS development certificate.
To trust the certificate run 'dotnet dev-certs https --trust' (Windows and macOS only).
Learn about HTTPS: https://aka.ms/dotnet-https
----------------
Write your first app: https://aka.ms/dotnet-hello-world
Find out what's new: https://aka.ms/dotnet-whats-new
Explore documentation: https://aka.ms/dotnet-docs
Report issues and find source on GitHub: https://github.com/dotnet/core
Use 'dotnet --help' to see available commands or visit: https://aka.ms/dotnet-cli
--------------------------------------------------------------------------------------
The template "Blazor Server App" was created successfully.
This template contains technologies from parties other than Microsoft, see https://aka.ms/aspnetcore/8.0-third-party-notices for details.

Processing post-creation actions...
Restoring /tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp/workdir/omnisharp/hello-blazorserver/hello-blazorserver.csproj:
  Determining projects to restore...
  Restored /tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp/workdir/omnisharp/hello-blazorserver/hello-blazorserver.csproj (in 264 ms).
Restore succeeded.

+ popd
/tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp/workdir/omnisharp /tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp/workdir /tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp
+ sleep 5
++ readlink -f hello-blazorserver
+ ./OmniSharp /tmp/tmp.AuZQtbWXJ9/dotnet-regular-tests/omnisharp/workdir/omnisharp/hello-blazorserver
Could not locate MSBuild instance to register with OmniSharp.
+ pkill -P 135265
Process Exit Code: 1

cc @omajid

tmds commented 1 year ago

The upstream smoke tests include an omnisharp test.

I rather remove this than fix it and continue to maintain it.

@omajid wdyt?

tmds commented 9 months ago

@omajid @nicrowe00 anyone opposed to removing this test?

nicrowe00 commented 9 months ago

The upstream smoke tests include an omnisharp test.

If that's the case then I have no issue with removing it.