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

Dynamic image tagging using PR-specific variables in Azure DevOps pip… #3974

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

shubhadapaithankar
Copy link
Collaborator

@shubhadapaithankar shubhadapaithankar commented Nov 22, 2024

Which issue this PR addresses:

This PR addresses the need for improved image management in the pipeline by introducing PR-specific tagging for RP images. Currently, it can be challenging to trace images back to specific pull requests, which complicates troubleshooting and version management. This update standardizes image tags, supports artifact caching, and enables caching for dependent images, improving both efficiency and traceability.

Fixes: 9503

What this PR does / why we need it:

This PR adds several key enhancements to the CI pipeline:

  • PR-Specific Tagging: RP images are pushed to the development ACR instance with PR-specific tags (pr-${PR_NUMBER}-${GIT_COMMIT_SHA}). This improves traceability by linking images directly to the pull requests that generated them.
  • The pipeline improvements introduced in this PR support more efficient builds, help with version control, and allow easier tracking of images and artifacts related to specific PRs.
  • The PR-specific tagging also simplifies troubleshooting, as we can directly identify which PR generated a particular image.

Key benefits:

  • Enhanced Troubleshooting: PR-specific tags make it easier to trace images back to the relevant pull requests, simplifying debugging.
  • Consistent Image Management: Standardizing tags across images and dependencies helps maintain consistency, allowing SREs to manage images effectively across environments.

Test plan for issue:

Verify PR-specific tagging (pr-${PR_NUMBER}-${GIT_COMMIT_SHA}) in dev ACR, confirm artifact caching reuses previous builds

Is there any documentation that needs to be updated for this PR?

N/A

How do you know this will function as expected in production?

N/A

…eline.

Fix: Add missing 'tag' parameter to ACR push template in Azure DevOps pipeline

Modified the tag to trigger the pipeline using the GitHub PR

Modified the tag to trigger the pipeline using the GitHub PR

Modified the tag to trigger the pipeline using the GitHub PR

Fix: Use GitHub PR number and commit SHA for dynamic tagging in Azure DevOps pipeline

Fix: Use GitHub PR number and commit SHA for dynamic tagging in Azure DevOps pipeline

Fix: Use GitHub PR number and commit SHA for dynamic tagging in Azure DevOps pipeline

Fix: Use GitHub PR number and commit SHA for dynamic tagging in Azure DevOps pipeline

Fix pipeline to dynamically generate valid Docker image tags using PR ID and commit hash with fallback mechanisms

Fix pipeline to dynamically generate valid Docker image tags using PR ID and commit hash with fallback mechanisms

Fix pipeline to dynamically generate valid Docker image tags using PR ID and commit hash with fallback mechanisms

Fix pipeline to dynamically generate valid Docker image tags using PR ID and commit hash with fallback mechanisms

Fix pipeline to dynamically generate valid Docker image tags using PR ID and commit hash with fallback mechanisms

Fix pipeline to dynamically generate valid Docker image tags using PR ID and commit hash with fallback mechanisms

Added a new Extract_PR_Number job to parse the PR number from System.PullRequest.SourceBranch

Added a new Extract_PR_Number job to parse the PR number from System.PullRequest.SourceBranch

Added a new Extract_PR_Number job to parse the PR number from System.PullRequest.SourceBranch

Added a new Extract_PR_Number job to parse the PR number from System.PullRequest.SourceBranch

Added a new Extract_PR_Number job to parse the PR number from System.PullRequest.SourceBranch

Added a new Extract_PR_Number job to parse the PR number from System.PullRequest.SourceBranch

Added a new Extract_PR_Number job to parse the PR number from System.PullRequest.SourceBranch

Added a new Extract_PR_Number job to parse the PR number from System.PullRequest.SourceBranch

Added a new Extract_PR_Number job to parse the PR number from System.PullRequest.SourceBranch
@shubhadapaithankar shubhadapaithankar force-pushed the shubhadapaithankar/ARO-9503-Image-Management-Tag branch from 54cb90a to 2e334c9 Compare November 23, 2024 00:38
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.

1 participant