[MISC] Updated workflows to use github's cache for faster builds#1368
Merged
chandrasekharan-zipstack merged 13 commits intoJun 18, 2025
Merged
Conversation
chandrasekharan-zipstack
requested review from
hari-kuriakose,
jaseemjaskp and
ritwik-g
Contributor
Why would it even break cloud builds? It is a different workflow all together right? |
ritwik-g
reviewed
Jun 17, 2025
ritwik-g
reviewed
Jun 17, 2025
…nd context resolution
hari-kuriakose
approved these changes
Jun 17, 2025
hari-kuriakose
left a comment
hari-kuriakose
left a comment
Contributor
There was a problem hiding this comment.
@chandrasekharan-zipstack LGTM overall.
Please check comments though.
ritwik-g
approved these changes
Jun 18, 2025
…emoved related fix added to run-platform.sh, updated Github tool build workflow to also use bake
…ofile, removed related fix added to run-platform.sh, updated Github tool build workflow to also use bake" This reverts commit 9e7fe1c.
Contributor
|
|
chandrasekharan-zipstack
deleted the
misc/docker-gha-cache-used-for-workflow-builds
branch
pk-zipstack
pushed a commit
that referenced
this pull request
Aug 20, 2025
* [misc] Updated workflows to use github's cache for docker builds * minor: Updated workflows to not push images for testing * [MISC] Updated workflow build to use correct Dockerfile for services * minor: Updated workflows to push images again post testing * [MISC] Refactored build push workflows to use, marked push to false for testing * [MISC] Fixed production build workflow issue related to VERSION env and context resolution * [MISC] Refactored build workflow to simplify tag resolution * [MISC] Moved commented line in build workflow * [MISC] Uncomment push option in workflow runs that was added for testing * [MISC] Updated build definition to add tools in a separate profile, removed related fix added to run-platform.sh, updated Github tool build workflow to also use bake * [MISC] Updated Dockerfile paths and context for tools to make workflow build easily * Revert "[MISC] Updated build definition to add tools in a separate profile, removed related fix added to run-platform.sh, updated Github tool build workflow to also use bake" This reverts commit 9e7fe1c.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Why
How
Can this PR break any existing features. If yes, please list possible items. If no, please explain why. (PS: Admins do not merge the PR without this section filled)
Relevant Docs
Notes on Testing
Screenshots
For tools
For services
Build summary
Checklist
I have read and understood the Contribution Guidelines.