Add source index build step#46823
Add source index build step#46823alexperovich merged 8 commits intodotnet:masterfrom alexperovich:sourceIndexBuild
Conversation
|
Tagging subscribers to this area: @directhex Issue Details
|
|
/azp run |
|
You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list. |
|
@safern this is ready. Can you review please? |
eng/pipelines/runtime.yml
Outdated
| # | ||
| # Source Index Build | ||
| # | ||
| - template: ./source-index.yml |
There was a problem hiding this comment.
Should we use the path filters for this as well? I presume that we only need to run this if C# code is changed and only in some code paths?
There was a problem hiding this comment.
If its not difficult that would be fine. What do I put where to do that?
There was a problem hiding this comment.
You can add a subset here: https://github.com/dotnet/runtime/blob/master/eng/pipelines/common/evaluate-default-paths.yml#L66 and then you can condition the job based on that subset, something like this:
runtime/eng/pipelines/runtime.yml
Lines 82 to 86 in c9d1fd6
Note that this only applies for CI, official builds doesn't filter per path.
There was a problem hiding this comment.
I can't do this without more changes in arcade. I can't change this job to depend on something else.
There was a problem hiding this comment.
Can we add the right parameters to arcade, like dependsOn and condition? I'd like to avoid running this job in cases where it is not needed as we already use many resources.
We can merge as is, and follow up, but please open an issue.
There was a problem hiding this comment.
Yea, I can remove the part running this in PRs, then add the parameters to arcade, and put the PR stuff back.
No description provided.