Skip to content

Smoke-test RC tarball build #506

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

Closed
dleeapho opened this issue May 3, 2018 · 12 comments
Closed

Smoke-test RC tarball build #506

dleeapho opened this issue May 3, 2018 · 12 comments
Assignees

Comments

@dleeapho
Copy link
Contributor

dleeapho commented May 3, 2018

No description provided.

@crummel
Copy link
Contributor

crummel commented May 3, 2018

I'm using Jenkins build 60, from hash 9ad4704, which is one commit behind dev/release/2.1 right now. I'm still getting:

Running 'dotnet restore' on /home/chris/dotnet-test/testing-smoke/C#_console/C#_console.csproj...
  Restoring packages for /home/chris/dotnet-test/testing-smoke/C#_console/C#_console.csproj...
/home/chris/dotnet-test/testing-smoke/C#_console/C#_console.csproj : error NU1102: Unable to find package Microsoft.NETCore.App with version (>= 2.1.0-rc1-26426-03)
/home/chris/dotnet-test/testing-smoke/C#_console/C#_console.csproj : error NU1102:   - Found 31 version(s) in nuget.org [ Nearest version: 2.1.0-preview2-26406-04 ]
  Generating MSBuild file /home/chris/dotnet-test/testing-smoke/C#_console/obj/C#_console.csproj.nuget.g.props.
  Generating MSBuild file /home/chris/dotnet-test/testing-smoke/C#_console/obj/C#_console.csproj.nuget.g.targets.
  Restore failed in 1,22 sec for /home/chris/dotnet-test/testing-smoke/C#_console/C#_console.csproj.

@weshaggard @dleeapho

@crummel
Copy link
Contributor

crummel commented May 3, 2018

We are producing that package, are we still expecting smoke-test to fail without adding the local package source? cc @dseefeld

@dseefeld
Copy link
Contributor

dseefeld commented May 3, 2018

I have an update to smoke-test to set a local package source for the case when the package is something we build, but isn't available on nuget.org, yet. I'll add a PR now.

@dseefeld
Copy link
Contributor

dseefeld commented May 3, 2018

@crummel see #507

@crummel
Copy link
Contributor

crummel commented May 3, 2018

Thanks Dan, that works for me with the exception of the web projects which I think #504 should fix. I'm going to stack that, #500, and #505 on top of latest and see if that works.

@dagood
Copy link
Member

dagood commented May 3, 2018

I get this error from the web project smoke test, which I don't think #504 helps with:

    running run
Using launch settings from /work/testing-smoke/C#_web/Properties/launchSettings.json...
It was not possible to find any compatible framework version
The specified framework 'Microsoft.AspNetCore.App', version '2.1.0-preview2-final' was not found.
  - Check application dependencies and target a framework version installed at:
      /work/testing-smoke/builtCli/
  - Installing .NET Core prerequisites might help resolve this problem:
      http://go.microsoft.com/fwlink/?LinkID=798306&clcid=0x409
  - The .NET Core framework and SDK can be installed from:
      https://aka.ms/dotnet-download

My understanding is this is being addressed with aspnet/Universe#1130 and https://github.com/dotnet/cli/pull/9178/files#diff-63122f55987fb7fe1993257c06b472b3 (_AspNetCoreAppSharedFxIsEnabled) so I'm not sure what we can do for RC.

@dagood
Copy link
Member

dagood commented May 3, 2018

Ah, yep: #456 (comment). We don't expect the ASP.NET experience to work in RC1.

@crummel
Copy link
Contributor

crummel commented May 4, 2018

With e6a3f04 I'm getting a new error, looks like it might be an issue with the MSBuild upgrade?
Could not load type 'Microsoft.Build.Logging.ProfilerLogger' from assembly 'Microsoft.Build, Version=15.1.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.

Logged #508 for it.

@markwilkie markwilkie added this to the S135 Apr 30 - May 18 (4/30/2018) milestone May 4, 2018
@dseefeld
Copy link
Contributor

dseefeld commented May 4, 2018

I submitted #509 to fix the smoke test issue.

@crummel
Copy link
Contributor

crummel commented May 4, 2018

16a2bab works for me building the tarball, now building from the tarball on a clean machine.

@crummel
Copy link
Contributor

crummel commented May 5, 2018

Smoke test passes for me on the clean machine as well.

@dleeapho
Copy link
Contributor Author

dleeapho commented May 8, 2018

I also validated with RHEL 7.4 Docker

@dleeapho dleeapho closed this as completed May 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants