Include RID in SB artifact file names

[mthalman]

Fixes dotnet/source-build#2928

[MichaelSimons]

@lbussell, @crummel - Thoughts on how should we handle this change in prep.sh and the release pipeline. The prep script UX feels a bit odd now in that the default behavior would only work on centos.8. The UX needs refactoring.

[MichaelSimons]

@lbussell, @crummel - Thoughts on how should we handle this change in prep.sh and the release pipeline. The prep script UX feels a bit odd now in that the default behavior would only work on centos.8. The UX needs refactoring.

To clarify this PR is not what is changing the behavior of the prep script - it was the removal of the runtime portable artifacts. This change just calls out the default behavior only works for a single rid.

[lbussell]

cc @premun for artifact name changes that might affect 8.0 servicing pipelines.

[premun]

Thanks for the notice, I will have to double check the VMR CI too. The step that re-uses the previously built SDK might depend on this.

[MichaelSimons]

Thanks for the notice, I will have to double check the VMR CI too. The step that re-uses the previously built SDK might depend on this.

It doesn't look like it does. https://github.com/dotnet/installer/blob/main/eng/pipelines/templates/jobs/vmr-build.yml#L143. It utilizes wild card patterns.

[MichaelSimons]

@lbussell, @crummel - Thoughts on how should we handle this change in prep.sh and the release pipeline. The prep script UX feels a bit odd now in that the default behavior would only work on centos.8. The UX needs refactoring.

This is tracked with dotnet/source-build#3222.