Skip to content

Update Helix SDK manually - #43011

Merged
ViktorHofer merged 3 commits into
masterfrom
ViktorHofer-helixsdk
Oct 5, 2020
Merged

Update Helix SDK manually#43011
ViktorHofer merged 3 commits into
masterfrom
ViktorHofer-helixsdk

Conversation

@ViktorHofer

Copy link
Copy Markdown
Member

For follow up / if you see 504s

For follow up / if you see 504s
@ViktorHofer
ViktorHofer requested a review from MattGal October 4, 2020 10:48
@ViktorHofer ViktorHofer self-assigned this Oct 4, 2020
@Dotnet-GitSync-Bot Dotnet-GitSync-Bot added the area-Infrastructure-coreclr Only use for closed issues label Oct 4, 2020
@ViktorHofer
ViktorHofer requested review from a team and akoeplinger October 4, 2020 10:49
@ViktorHofer

Copy link
Copy Markdown
Member Author

@MattGal based on the errors CI is down because openssl is missing on the clients. Does that ring a bell?

cc @dotnet/runtime-infrastructure

@Gnbrkm41

Gnbrkm41 commented Oct 4, 2020

Copy link
Copy Markdown
Contributor

RE: CI failure due to missing openssl on OSX builds, Hitting the same issue on a different PR. https://dev.azure.com/dnceng/public/_build/results?buildId=841276&view=logs&j=1f8f000c-1adc-5434-677a-95dd9e006aba&t=09bee337-6423-532b-2fb6-6276210830fd&l=20

@ViktorHofer

Copy link
Copy Markdown
Member Author

cc @dotnet/dnceng

@lukas-lansky

Copy link
Copy Markdown
Contributor

Comment thread eng/Version.Details.xml
<Dependency Name="Microsoft.DotNet.Helix.Sdk" Version="5.0.0-beta.20502.2">
<Dependency Name="Microsoft.DotNet.Helix.Sdk" Version="5.0.0-beta.20502.7">
<Uri>https://github.com/dotnet/arcade</Uri>
<Sha>baebe30c7969b4148a6a3b90cf5f4e3a96d4a11e</Sha>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Technically this value should be b931c53efeaac8270b319defc0905e9c152bac1a (guessing you did this manually) but it may not matter

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I never update the commit SHA when I do these kinds of manual updates and it doesn't seem to be a problem.

@MattGal

MattGal commented Oct 5, 2020

Copy link
Copy Markdown
Member

@MattGal based on the errors CI is down because openssl is missing on the clients. Does that ring a bell?

cc @dotnet/runtime-infrastructure

We've seen lots of stuff like this... generally it's one of two things, either the hosted agents themselves change or the code that bootstraps the dependency changes. If it's 100% and you can point to "this is specifically wrong with Hosted MacOS" we can certainly open an IcM post-haste; I don't see what's specifically broken yet though.

When you see logs like this though, it's not our machines, so all we can do is support tickets.

Starting: Initialize job
Agent name: 'Hosted Agent'
Agent machine name: 'Mac-1601807338640'
Current agent version: '2.175.2'
Current image version: '20200904.1'

@safern

safern commented Oct 5, 2020

Copy link
Copy Markdown
Member

Install native dependencies script hasn't changed since July 28th... so it might be worth following up with AzDo to know if something changed in the build machines.

https://github.com/dotnet/runtime/blob/master/eng/install-native-dependencies.sh

@safern

safern commented Oct 5, 2020

Copy link
Copy Markdown
Member

I do see that OpenSsl was installed:

Using autoconf
Installing automake
Installing cmake
Using icu4c
Using libtool
Using openssl@1.1
Using pkg-config
Using python3

@ViktorHofer

Copy link
Copy Markdown
Member Author

See #43037 in which @akoeplinger is asserting on the openssl@1.1 symlink instead.

@ViktorHofer

Copy link
Copy Markdown
Member Author

/azp run runtime

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@safern

safern commented Oct 5, 2020

Copy link
Copy Markdown
Member

See #43037 in which @akoeplinger is asserting on the openssl@1.1 symlink instead.

Great, thanks 😄

@ViktorHofer

Copy link
Copy Markdown
Member Author

/azp run runtime

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@ViktorHofer

Copy link
Copy Markdown
Member Author

The OSX failure was a temporary NuGet outage.

@ViktorHofer
ViktorHofer merged commit 3887343 into master Oct 5, 2020
@ViktorHofer
ViktorHofer deleted the ViktorHofer-helixsdk branch October 5, 2020 22:24
@ghost ghost locked as resolved and limited conversation to collaborators Dec 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-Infrastructure-coreclr Only use for closed issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants