Skip to content

Conversation

@safern
Copy link
Member

@safern safern commented Nov 19, 2019

This will allow use to use platform-matrix to fan out different architectures and then in xplat-setup we can start having shared stuff like directory locations, artifacts conventions, extensions, etc.

This is the first step, wanted to keep the PR as small as possible. More cleanup will be done afterwards.

@safern safern force-pushed the MoveCommonStuffToTemplate branch 4 times, most recently from f8d03e4 to 1562f89 Compare November 19, 2019 08:37
@safern safern marked this pull request as ready for review November 19, 2019 08:45
@ViktorHofer
Copy link
Member

This is the first step, wanted to keep the PR as small as possible. More cleanup will be done afterwards.

I assume that you will change the libraries yml files to depend on the platform-matrix as well? It's a bit confusing that you already add support for it here but don't consume it. It's hard to see if the matrix is correct for the libraries. Would it be feasible to also consume it as part of this PR?

@ViktorHofer
Copy link
Member

I assumed you were adding the support for libraries in this PR already because of the "corefx" option. If this is just code-cleanup then this looks good. I won't review the platform matrix then and wait for the subsequent PR.

@safern
Copy link
Member Author

safern commented Nov 19, 2019

I assumed you were adding the support for libraries in this PR already because of the "corefx" option. If this is just code-cleanup then this looks good. I won't review the platform matrix then and wait for the subsequent PR.

Yes that bit is confusing. This is a step towards getting platform-matrix closer to a point where it can be consumed from the libraries pipelines. Also, in order to include that consumption I would need to change all the libraries logic on how we expand our matrix and this PR would be huge and easy to miss any possible break.

I have a draft PR to get the libraries build ready to consume platform-matrix if you're curious: #71

Thanks for the review and sorry for the confusion, should've been more explicit on the description.

@ViktorHofer
Copy link
Member

No problem at all.

@jashook jashook requested review from echesakov and sbomer November 19, 2019 17:34
Copy link
Contributor

@jashook jashook left a comment

Choose a reason for hiding this comment

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

lgtm thank you!

@safern safern force-pushed the MoveCommonStuffToTemplate branch from 1562f89 to 763d47a Compare November 19, 2019 21:19
@safern
Copy link
Member Author

safern commented Nov 19, 2019

763d47a is fixing the TODOs:

enabling RedHat6 helix queues in the public and also it is enabling Nano testing and moving the existing Nano queue to use the 1809 docker image.

@safern
Copy link
Member Author

safern commented Nov 20, 2019

Merging as CI is green except but the crossgen-comparison job which is a known issue being investigated by @jashook

@safern safern merged commit db3c0c1 into master Nov 20, 2019
@safern safern deleted the MoveCommonStuffToTemplate branch November 20, 2019 18:49
@karelz karelz added this to the 5.0.0 milestone Aug 18, 2020
MichalStrehovsky pushed a commit to MichalStrehovsky/runtime that referenced this pull request Oct 12, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants