Skip to content

Update the default pipeline to upload artifacts to internal feed for nightly builds and consolidated pipeline updates #4422

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 15 commits into from
Jun 2, 2025

Conversation

aishwaryabh
Copy link
Contributor

@aishwaryabh aishwaryabh commented May 15, 2025

Issue describing the changes in this PR

Resolves #4420

I'm fixing the consolidated pipeline to automatically trigger when a build is done on either the core-tools.official pipeline or the core-tools-host pipeline. However we want to distinguish between nightly builds and regular builds that were triggered by CI. Therefore, we can separate the nightly build artifacts from the CI triggered artifacts by uploading the nightly build artifacts to an internal feed.

The format of the artifact version will be: YYYY.MMDD.buildId (where MM has no leading zeros).

Here is an example of the feed:
https://azfunc.visualstudio.com/internal/_artifacts/feed/core-tools-nightly-build

This PR also conditionally downloads the artifacts from the nightly build feed in the consolidated pipeline, if the consolidated pipeline run is also only a nightly build. Otherwise, we just take whatever was triggered by CI and use that build.

Note if the consolidated pipeline is manually run, we just use whatever is the latest run (regardless of nightly build or not).

The artifacts have also been updated to being with func when published, such as func-cli

Pull request checklist

  • My changes do not require documentation changes
    • Otherwise: Documentation issue linked to PR
  • My changes do not need to be backported to a previous version
    • Otherwise: Backport tracked by issue/PR #issue_or_pr
  • I have added all required tests (Unit tests, E2E tests)

Sorry, something went wrong.

Verified

This commit was signed with the committer’s verified signature.
parkerbxyz Parker Brown

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was signed with the committer’s verified signature.
parkerbxyz Parker Brown

Verified

This commit was signed with the committer’s verified signature.
parkerbxyz Parker Brown

Verified

This commit was signed with the committer’s verified signature.
parkerbxyz Parker Brown

Verified

This commit was signed with the committer’s verified signature.
parkerbxyz Parker Brown

Verified

This commit was signed with the committer’s verified signature.
parkerbxyz Parker Brown

Verified

This commit was signed with the committer’s verified signature.
parkerbxyz Parker Brown
@aishwaryabh aishwaryabh requested a review from a team as a code owner May 15, 2025 20:53

Verified

This commit was signed with the committer’s verified signature.
parkerbxyz Parker Brown

Verified

This commit was signed with the committer’s verified signature.
parkerbxyz Parker Brown

Verified

This commit was signed with the committer’s verified signature.
parkerbxyz Parker Brown
@aishwaryabh aishwaryabh force-pushed the aibhandari/consolidated-pipeline-default-updates branch from 740664f to 6ed52de Compare May 22, 2025 22:42
@aishwaryabh aishwaryabh force-pushed the aibhandari/consolidated-pipeline-default-updates branch from 424a288 to bc4bd2a Compare May 27, 2025 20:51
@aishwaryabh aishwaryabh merged commit 43a76b3 into main Jun 2, 2025
12 checks passed
@aishwaryabh aishwaryabh deleted the aibhandari/consolidated-pipeline-default-updates branch June 2, 2025 16:43
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.

Upload nightly builds to an internal feed and have consolidated pipeline trigger on CI build completion
3 participants