Skip to content

Move VPack to OneBranch#3763

Merged
csujedihy merged 7 commits intomainfrom
huanyi/onecore-vpack
Jul 19, 2023
Merged

Move VPack to OneBranch#3763
csujedihy merged 7 commits intomainfrom
huanyi/onecore-vpack

Conversation

@csujedihy
Copy link
Collaborator

Description

Move VPack to OneBranch pipeline.

Testing

Tested in OneBranch pipeline

Yi Huang (from Dev Box) added 2 commits July 19, 2023 10:53
@csujedihy csujedihy requested a review from a team as a code owner July 19, 2023 17:56
ob_createvpack_owneralias: quicdev
ob_createvpack_description: msquic.$(Build.SourceBranchName)
ob_createvpack_versionAs: string
ob_createvpack_version: 2.3.0-$(Build.BuildId)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does our update version ps1 script still work for this, or does it need to be updated?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

that doesn't work for this. I have a plan to read the version number from a json file like you did in a separate PR.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Actually, there is an issue. The BuildId in vpack version won't match MSQUIC_METADATA_ONEBRANCH_BUILD_ID.

Copy link
Collaborator Author

@csujedihy csujedihy Jul 19, 2023

Choose a reason for hiding this comment

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

MSQUIC_METADATA_ONEBRANCH_BUILD_ID won't match the build ID of the pipeline that pushes vpack and creates the PR either because it's from the previous pipeline.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Looks like we have been doing all the time. So, probably not a big deal.

Copy link
Collaborator Author

@csujedihy csujedihy Jul 19, 2023

Choose a reason for hiding this comment

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

Does our update version ps1 script still work for this, or does it need to be updated?

nvm. I updated update-version.ps1.

@codecov
Copy link

codecov bot commented Jul 19, 2023

Codecov Report

Merging #3763 (b9cd771) into main (307a53a) will decrease coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3763      +/-   ##
==========================================
- Coverage   75.64%   75.54%   -0.11%     
==========================================
  Files          56       56              
  Lines       15544    15544              
==========================================
- Hits        11759    11743      -16     
- Misses       3785     3801      +16     

see 12 files with indirect coverage changes

@csujedihy csujedihy merged commit 9a09ade into main Jul 19, 2023
@csujedihy csujedihy deleted the huanyi/onecore-vpack branch July 19, 2023 20:19
nibanks pushed a commit that referenced this pull request Jul 20, 2023
nibanks pushed a commit that referenced this pull request Jul 20, 2023
nibanks added a commit that referenced this pull request Jul 20, 2023
Co-authored-by: Yi Huang <huanyi@microsoft.com>
nibanks added a commit that referenced this pull request Jul 20, 2023
Co-authored-by: Yi Huang <huanyi@microsoft.com>
gaurav2699 pushed a commit that referenced this pull request Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants