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.

Sign up for GitHub

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

Jump to bottom

[release/6.0] Update dependencies from dotnet/arcade #107220

Merged
merged 3 commits into from
Sep 5, 2024
Merged

[release/6.0] Update dependencies from dotnet/arcade #107220

merged 3 commits into from
Sep 5, 2024

Conversation

dotnet-maestro[bot]
Copy link
Contributor

This pull request updates the following dependencies

From https://github.com/dotnet/arcade

  • Subscription: 36974d0f-1ebe-4d41-3a49-08d961c5a689
  • Build: 20240813.1
  • Date Produced: August 13, 2024 7:43:19 PM UTC
  • Commit: c3f5fe044e7115fb66c79db5e074cb341dbe21dd
  • Branch: refs/heads/release/6.0
  • Updates to .NET SDKs:
    • Updates sdk.version to 6.0.133
    • Updates tools.dotnet to 6.0.133

@dotnet-maestro
Update dependencies from https://github.com/dotnet/arcade build 20240…
2156c1e 
…813.1

Microsoft.DotNet.ApiCompat , Microsoft.DotNet.Arcade.Sdk , Microsoft.DotNet.Build.Tasks.Archives , Microsoft.DotNet.Build.Tasks.Feed , Microsoft.DotNet.Build.Tasks.Installers , Microsoft.DotNet.Build.Tasks.Packaging , Microsoft.DotNet.Build.Tasks.TargetFramework.Sdk , Microsoft.DotNet.Build.Tasks.Templating , Microsoft.DotNet.Build.Tasks.Workloads , Microsoft.DotNet.CodeAnalysis , Microsoft.DotNet.GenAPI , Microsoft.DotNet.GenFacades , Microsoft.DotNet.Helix.Sdk , Microsoft.DotNet.PackageTesting , Microsoft.DotNet.RemoteExecutor , Microsoft.DotNet.SharedFramework.Sdk , Microsoft.DotNet.VersionTools.Tasks , Microsoft.DotNet.XUnitConsoleRunner , Microsoft.DotNet.XUnitExtensions
 From Version 6.0.0-beta.24360.7 -> To Version 6.0.0-beta.24413.1
@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Aug 31, 2024
@jeffhandley jeffhandley added Servicing-consider Issue for next servicing release review area-Infrastructure and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Aug 31, 2024
@lewing
Copy link
Member

lewing commented Aug 31, 2024

@jeffhandley if this is required infrastructure flow it can be marked as approved

@hoyosjs hoyosjs added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Aug 31, 2024
This was referenced Aug 31, 2024
Open
Open
Open
@jeffhandley
Copy link
Member

As far as I can tell, all the failing checks are unrelated to updating arcade. These are all known:

I don't know how to diagnose the System.IO.Tests failures though, especially where the log merely contains the following:

Console log: 'System.IO.Tests' from job 2b582a72-5bb9-4cbf-afee-c09f36848afe (ubuntu.2204.amd64.open.svc) using docker image mcr.microsoft.com/dotnet-buildtools/prereqs:debian-11-helix-amd64 on a005PK4
running $HELIX_CORRELATION_PAYLOAD/scripts/8e3a716358cf46faa8eb11673920d6f8/execute.sh in /datadisks/disk1/work/B6FF0A3F/w/B2CB09DA/e max 900 seconds

@build-analysis build-analysis bot mentioned this pull request Sep 4, 2024
Open
@ericstj
Copy link
Member

ericstj commented Sep 4, 2024

@lewing browser tests failing without known issue.

Everything appears to be like this:

Connection refused [::ffff:127.0.0.1]:41031 (localhost:41031)
[01:27:55] warn: Failed to start the browser, attempt #0: OpenQA.Selenium.WebDriverException: unknown error: Chrome failed to start: exited abnormally.
                   (unknown error: DevToolsActivePort file doesn't exist)
                   (The process started from chrome location /datadisks/disk1/work/A45A094F/p/chrome-linux/chrome is no longer running, so ChromeDriver is assuming that Chrome has crashed.)
                    at OpenQA.Selenium.Remote.RemoteWebDriver.UnpackAndThrowOnError(Response errorResponse)
                    at OpenQA.Selenium.Remote.RemoteWebDriver.Execute(String driverCommandToExecute, Dictionary`2 parameters)
                    at OpenQA.Selenium.Remote.RemoteWebDriver.StartSession(ICapabilities desiredCapabilities)
                    at OpenQA.Selenium.Remote.RemoteWebDriver..ctor(ICommandExecutor commandExecutor, ICapabilities desiredCapabilities)
                    at OpenQA.Selenium.Chromium.ChromiumDriver..ctor(ChromiumDriverService service, ChromiumOptions options, TimeSpan commandTimeout)
                    at OpenQA.Selenium.Chrome.ChromeDriver..ctor(ChromeDriverService service, ChromeOptions options, TimeSpan commandTimeout)

Seems similar to #43955

@lewing
Copy link
Member

lewing commented Sep 4, 2024

@ilonatommy please take a look the the selenium failures here in release/6.0 they don't appear to be related to this change

@carlossanlop
Copy link
Member

carlossanlop commented Sep 4, 2024 •
edited
Loading

CI is green!

Nevermind! It never ran.

@carlossanlop carlossanlop reopened this Sep 4, 2024
@ericstj
Copy link
Member

ericstj commented Sep 5, 2024

Looks like this is ready, but has a conflict to resolve. Conflicts with #106725. I propose we solve that and fast merge to get the builds going. @lewing can you help resolve? I'm not sure which feed to take - or do we keep all of them and if so in what order?

@carlossanlop
Copy link
Member

I fixed the emsdk feed conflict the same way we were told to fix them whenever we see them (which is quite often): Keep them all (just remove the duplicates). Eventually they get automatically cleaned up by maestro.

@ericstj
Copy link
Member

ericstj commented Sep 5, 2024

We need to take this to unblock servicing.

@ericstj ericstj merged commit ec3f6ab into release/6.0 Sep 5, 2024
140 of 154 checks passed
@dotnet-maestro dotnet-maestro bot deleted the darc-release/6.0-00b12ed6-bd03-4c8f-8f09-ca7b32801c6a branch September 5, 2024 22:42
@build-analysis build-analysis bot mentioned this pull request Sep 11, 2024
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeffhandley jeffhandley jeffhandley approved these changes

Assignees
No one assigned
Labels
area-Infrastructure Servicing-approved Approved for servicing release
Projects
Runtime Infra
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@lewing @jeffhandley @ericstj @carlossanlop @hoyosjs