Skip to content

Add Helix image for Mariner 2.0#779

Merged
wfurt merged 3 commits intomainfrom
mar2
Feb 2, 2023
Merged

Add Helix image for Mariner 2.0#779
wfurt merged 3 commits intomainfrom
mar2

Conversation

@wfurt
Copy link
Member

@wfurt wfurt commented Jan 24, 2023

based on Mariner 1.0
Request for Mariner packages is tracked by microsoft/msquic#3376

The build runs with warning

WARNING: Running pip as root will break packages and permissions. You should install packages reliably by using venv: https://pip.pypa.io/warnings/venv
WARNING: You are using pip version 21.1.2; however, version 22.3.1 is available.
You should consider upgrading via the '/usr/bin/python -m pip install --upgrade pip' command.

I'm wondering if we should install the python craft after we add helixbot user @MattGal.
I would assume this is not only one distributions where we bend the recommendation.

cc: @ManickaP

{
"platforms": [
{
"dockerfile": "src/cbl-mariner/2.0/helix/amd64",
Copy link
Member

Choose a reason for hiding this comment

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

we already have plain mariner 2 machines, why do it in docker?

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

do you have msquic on it @MattGal ? It feels like the docker is easer to manage for us.

Choose a reason for hiding this comment

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

Why is the docker easier to manage? We are about to delete all these files and docker is going to use the same architecture our other VM's have.

Copy link
Member Author

Choose a reason for hiding this comment

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

one more note that containers are super easy for developers for testing and investigations. Going through the helix-repro lab no so much.

Copy link
Member

Choose a reason for hiding this comment

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

@wfurt currently the "regular" helix machines running mariner 2 on them don't have msquic, that's true, but the same "install a Centos 7 version of it" workaround could be easily applied there. I thought we had tried this as part of dotnet/arcade#10253 and determined it didn't work at runtime.

I'm not blocking this PR but I still think it's strange to do mariner testing in docker when we made specific efforts to have dedicated IaaS Mariner 2 test machines. Running the tests as docker has the downsides of the per-machine cost of downloading the images on every machine as well as the bottlenecking effect of running in the same queue with all the other docker-ified linux scenarios.

Copy link
Member Author

Choose a reason for hiding this comment

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

noted @MattGal. Let's revisit this when the linked MsQuic issue is fixed e.g. we have packages for Mariner.
Also new version of MsQuic is coming out soon so we may see some more turn than usually.

@wfurt wfurt merged commit 50a5cf3 into main Feb 2, 2023
@wfurt wfurt deleted the mar2 branch February 2, 2023 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants