Skip to content
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

fix: update pipeline to handle an extra character 'v' in version tag #3089

Merged
merged 2 commits into from
Oct 29, 2024

Conversation

santhoshmprabhu
Copy link
Contributor

Reason for Change:
ACN pipelines used Cilium version tags that didn't include the letter 'v'. This PR updates the pipelines to handle private fork version tags, which include the letter 'v'. This will allow us to set ADO pipeline variables to point to ACN private fork images for Cilium.

Issue Fixed:
Lack of support for ACN Cilium private fork images

Requirements:

Notes:

@santhoshmprabhu santhoshmprabhu added the ci Infra or tooling. label Oct 28, 2024
@santhoshmprabhu santhoshmprabhu self-assigned this Oct 28, 2024
@santhoshmprabhu santhoshmprabhu requested a review from a team as a code owner October 28, 2024 18:21
@santhoshmprabhu santhoshmprabhu changed the title fix: update pipeline to handle v in version tag fix: update pipeline to handle an extra character 'v' in version tag Oct 28, 2024
@santhoshmprabhu
Copy link
Contributor Author

santhoshmprabhu commented Oct 28, 2024

Full Pipeline run with private fork images:
https://msazure.visualstudio.com/One/_build/results?buildId=106853314&view=results

@santhoshmprabhu
Copy link
Contributor Author

/azp run Azure Container Networking PR

@santhoshmprabhu
Copy link
Contributor Author

/azp run Azure Container Networking PR

Copy link
Contributor

@jpayne3506 jpayne3506 left a comment

Choose a reason for hiding this comment

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

Lgtm, @santhoshmprabhu will be adding the node-subnet template changes as part of #3083.

🚀

@santhoshmprabhu
Copy link
Contributor Author

/azp run Azure Container Networking PR

@jpayne3506
Copy link
Contributor

/azp list

@santhoshmprabhu
Copy link
Contributor Author

/run Azure Container Networking PR

@jpayne3506
Copy link
Contributor

/azp list

@jpayne3506
Copy link
Contributor

/azp

@santhoshmprabhu santhoshmprabhu added this pull request to the merge queue Oct 29, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 29, 2024
@jpayne3506
Copy link
Contributor

/azp list

@jpayne3506 jpayne3506 added this pull request to the merge queue Oct 29, 2024
Merged via the queue into master with commit eb059cb Oct 29, 2024
94 checks passed
@jpayne3506 jpayne3506 deleted the sanprabhu/cilium-private-fork-version-update branch October 29, 2024 12:52
@jpayne3506 jpayne3506 added release/1.4 Change affects v1.4 release train release/latest Change affects latest release train needs-backport Change needs to be backported to previous release trains release/1.5 Change affects v1.5 release train labels Oct 30, 2024
jpayne3506 pushed a commit that referenced this pull request Oct 31, 2024
…3089)

* fix: update pipeline to handle v in version tag

* chore: empty commit for pipeline run
github-merge-queue bot pushed a commit that referenced this pull request Nov 1, 2024
* fix: update pipeline to handle an extra character 'v' in version tag (#3089)

* fix: update pipeline to handle v in version tag

* chore: empty commit for pipeline run

* ci: update CILIUM_VERSION_TAG to CILIUM_VERSION_TAG_V1_5

---------

Co-authored-by: Santhosh  Prabhu <6684582+santhoshmprabhu@users.noreply.github.com>
github-merge-queue bot pushed a commit that referenced this pull request Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Infra or tooling. needs-backport Change needs to be backported to previous release trains release/latest Change affects latest release train release/1.4 Change affects v1.4 release train release/1.5 Change affects v1.5 release train
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants