Skip to content

[EngSys] Forward Rust opt-in parameters through Cosmos CI template - #49522

Merged
jenny (JennyPng) merged 1 commit into
Azure:mainfrom
JennyPng:engsys/cosmos-rust-parameter-forwarding
Oct 13, 2026
Merged

jenny (JennyPng) merged 1 commit into
Azure:mainfrom
JennyPng:engsys/cosmos-rust-parameter-forwarding

Conversation

@JennyPng

@JennyPng jenny (JennyPng) commented Oct 13, 2026 •

Copy link
Copy Markdown
Member

Summary

Follow-up to #49397.

  • Expose and forward InstallMsRustToolchain, MsRustWorkingDirectory, MsRustToolchainFeed, and MsRustAdditionalTargets from cosmos-sdk-client.yml to archetype-sdk-client.yml.
  • Match the shared parameter types and defaults. Rust installation remains disabled by default.
  • Allow Cosmos to configure the opt-in in its SDK-owned sdk/cosmos/ci.yml. This PR does not activate Rust or select a toolchain feed.

Expose the four Rust build parameters in the Cosmos wrapper and forward them to archetype-sdk-client. Keep Rust disabled by default and leave feed selection to the caller.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 00cb7ea9-d85f-4bee-bfb3-e6c94f9d3b64
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
10 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@JennyPng
jenny (JennyPng) marked this pull request as ready for review October 13, 2026 19:59
Copilot AI balanced review requested due to automatic review settings October 13, 2026 19:59

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The new Cosmos forwarding layer is missing coverage in the existing native-build pipeline contract test.

1 open finding
What changed in this PR

Enables Cosmos CI to opt into the shared Microsoft Rust toolchain setup without activating it by default.

Changes:

  • Adds four Rust configuration parameters with shared defaults.
  • Forwards them to the SDK client archetype.
File Description
eng/​pipelines/​templates/​stages/​cosmos-sdk-client.yml Exposes and forwards Rust opt-in settings.

🧠 Review effort: Balanced

@JennyPng
jenny (JennyPng) enabled auto-merge (squash) October 13, 2026 20:06
@JennyPng
jenny (JennyPng) merged commit 19d1c40 into Azure:main Oct 13, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants