Skip to content

Add feature flag guard for ACTIONS_ORCHESTRATION_ID environment variable#4181

Merged
TingluoHuang merged 2 commits intousers/tihuang/setorchidenvfrom
copilot/sub-pr-4178
Jan 6, 2026
Merged

Add feature flag guard for ACTIONS_ORCHESTRATION_ID environment variable#4181
TingluoHuang merged 2 commits intousers/tihuang/setorchidenvfrom
copilot/sub-pr-4178

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 6, 2026

  • Add feature flag constant to Constants.Runner.Features
  • Wrap ACTIONS_ORCHESTRATION_ID setting with feature flag check in NodeScriptActionHandler.cs
  • Format code
  • Build code to validate changes
  • Request code review

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Update to address feedback on actions orchestration ID Add feature flag guard for ACTIONS_ORCHESTRATION_ID environment variable Jan 6, 2026
Copilot AI requested a review from TingluoHuang January 6, 2026 17:21
@TingluoHuang
Copy link
Copy Markdown
Member

@copilot where is the changes?

- Add SetOrchestrationIdEnvForNodeActions feature flag constant
- Wrap ACTIONS_ORCHESTRATION_ID env var setting with feature flag check
- Follows same pattern as ACTIONS_CACHE_SERVICE_V2

Co-authored-by: TingluoHuang <[email protected]>
@TingluoHuang TingluoHuang marked this pull request as ready for review January 6, 2026 18:13
@TingluoHuang TingluoHuang requested a review from a team as a code owner January 6, 2026 18:13
@TingluoHuang TingluoHuang merged commit 4822c46 into users/tihuang/setorchidenv Jan 6, 2026
1 check passed
@TingluoHuang TingluoHuang deleted the copilot/sub-pr-4178 branch January 6, 2026 18:13
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Jan 6, 2026

@copilot where is the changes?

The changes have been implemented in commit d63009f. The implementation adds:

  1. A new feature flag constant SetOrchestrationIdEnvForNodeActions in Constants.Runner.Features
  2. A feature flag check wrapping the ACTIONS_ORCHESTRATION_ID environment variable assignment in NodeScriptActionHandler.cs

This follows the project's coding guidelines to safeguard changes with feature flags.

Copilot stopped work on behalf of TingluoHuang due to an error January 6, 2026 18:13
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.

2 participants