Fixed Deployment Preview Workflow#122
Merged
saurabhraghuvanshii merged 3 commits intomeshery-extensions:masterfrom Jan 9, 2026
Merged
Fixed Deployment Preview Workflow#122saurabhraghuvanshii merged 3 commits intomeshery-extensions:masterfrom
saurabhraghuvanshii merged 3 commits intomeshery-extensions:masterfrom
Conversation
github-actions
bot
added
area/ci
SplinterSword
force-pushed
the
deployPreview
branch
from
7fc7fad to
5295753
Compare
SplinterSword
force-pushed
the
deployPreview
branch
from
5295753 to
5c2cde4
Compare
|
@SplinterSword Thank you for your contribution! Let's discuss this during the website call on Monday at 6:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂 |
saurabhraghuvanshii
approved these changes
Jan 9, 2026
Member
|
@SplinterSword I’m merging this to verify if it succeeds. |
saurabhraghuvanshii
merged commit 432dd2b
into
meshery-extensions:master
4 checks passed
Contributor
Author
|
Okay i will look into it as soon as possible, right now i am out of station, will be able to work on it after monday if that is okay. |
Contributor
Author
|
@saurabhraghuvanshii , can you take a look into this one and see if it works |
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
4 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.
Notes for Reviewers
Problem
Preview deployments were intermittently failing in CI with errors such as
artifact not foundor resulting in broken or empty preview sites.Root Cause Analysis
After tracing the CI pipeline end-to-end, the issue was identified as incorrect artifact packaging in the build workflow.
What was happening
The build step generates the static site output in
site/public/.The packaging script (
script.sh) was creating an emptypublic/directory and zipping that directory instead of the actual build output.Solution
This PR fixes the issue by packaging only the actual build output and validating it before uploading the artifact.
Key changes
The packaging script now zips the contents of
site/public/instead of an emptypublic/directory.Verification
The fix was verified both locally and in CI:
make buildlocally and confirmed output insite/public/script.shand inspected the generated zip to confirm it contained a flat site structuredocs/_siteand simulated the Netlify preview locally using:Signed commits