Skip to content

Conversation

sanjuyadav24
Copy link
Contributor

@sanjuyadav24 sanjuyadav24 commented Jul 21, 2025

Context

This shows the changes that we must make on the pipeline tasks side to allow canary rollouts

A new release field is added which will be used on server side for canary rollout
"release": {
"sprint": 259,
"ordinal": 1
}
The Sprint corresponds to the current sprint number (259 in this case), and the Ordinal is a sequence number.

When [useSemverBuildConfig = false]
Tasks generated with traditional versioning (different patch numbers)
No breaking changes to existing functionality
Example: PowerShellV2: 2.259.0, PowerShellV2_Node20: 2.259.1

When [useSemverBuildConfig = true]
All task configs share the same major.minor.patch version
Non-default configs get build suffixes from their constMappingKey
Default config has no build suffix
Example: PowerShellV2: 2.259.0, PowerShellV2_Node20: 2.259.0+Node20-225


Task Name

NA


Description

This shows the changes that we must make on the pipeline tasks side to allow canary rollouts. Adds a new field release in the task to associate task with sprint versions


Risk Assessment (Low / Medium / High)

Low (Flag based check, for older task no changes are required)


Unit Tests Added or Updated (Yes / No)

No ( changes are in task build process)


Additional Testing Performed

Manual testing to generate tasks in new format


Documentation Changes Required (Yes / No)

Yes


Checklist

  • Related issue linked (if applicable)
  • Task version was bumped — see versioning guide
  • Verified the task behaves as expected

@sanjuyadav24 sanjuyadav24 requested review from tarunramsinghani and a team as code owners July 21, 2025 04:54
@sanjuyadav24
Copy link
Contributor Author

/azp run

@sanjuyadav24 sanjuyadav24 changed the title Users/sanjuyadav/canary testing 1 Canary Task Rollouts Jul 23, 2025
@sanjuyadav24
Copy link
Contributor Author

/azp run

Copy link
Contributor

@merlynomsft merlynomsft left a comment

Choose a reason for hiding this comment

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

Feedback is semantic, not functional "approving w/comments".

@sanjuyadav24
Copy link
Contributor Author

/azp run

2 similar comments
@sanjuyadav24
Copy link
Contributor Author

/azp run

@sanjuyadav24
Copy link
Contributor Author

/azp run

@sanjuyadav24 sanjuyadav24 force-pushed the users/sanjuyadav/canary_testing_1 branch from d40bb7c to 74f44b0 Compare August 4, 2025 16:33
@sanjuyadav24
Copy link
Contributor Author

/azp run

1 similar comment
@sanjuyadav24
Copy link
Contributor Author

/azp run

@sanjuyadav24 sanjuyadav24 force-pushed the users/sanjuyadav/canary_testing_1 branch from 00fb344 to cb6ae04 Compare August 12, 2025 06:45
@sanjuyadav24
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines failed to run 3 pipeline(s).

@sanjuyadav24
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines failed to run 3 pipeline(s).

@sanjuyadav24
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines failed to run 3 pipeline(s).

@sanjuyadav24
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines failed to run 3 pipeline(s).

@sanjuyadav24
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines failed to run 3 pipeline(s).

@sanjuyadav24
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@sanjuyadav24
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@sanjuyadav24
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@sanjuyadav24 sanjuyadav24 merged commit 9307f34 into master Aug 12, 2025
11 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.

5 participants