Skip to content

Fix: action#6146

Merged
c121914yu merged 3 commits intolabring:mainfrom
c121914yu:4.14.5-dev
Dec 24, 2025
Merged

Fix: action#6146
c121914yu merged 3 commits intolabring:mainfrom
c121914yu:4.14.5-dev

Conversation

@c121914yu
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings December 24, 2025 09:09
@c121914yu c121914yu changed the title 4.14.5 dev Fix: action Dec 24, 2025
@gru-agent
Copy link
Contributor

gru-agent bot commented Dec 24, 2025

TestGru Assignment

Summary

Link CommitId Status Reason
Detail b86a5bb 🚫 Skipped No files need to be tested {".github/workflows/fastgpt-preview-image.yml":"File path does not match include patterns."}

History Assignment

Tip

You can @gru-agent and leave your feedback. TestGru will make adjustments based on your input

@c121914yu c121914yu merged commit e53242e into labring:main Dec 24, 2025
5 of 6 checks passed
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the Docker build cache writing configuration from the FastGPT preview image workflow. The change removes the --cache-to flag from the docker buildx command, which prevents build cache layers from being written back to storage.

Key Changes:

  • Removed the --cache-to=type=local,dest=/tmp/.buildx-cache flag from the docker buildx build command

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -73,7 +73,6 @@ jobs:
--label "org.opencontainers.image.description=${{ steps.config.outputs.DESCRIPTION }}" \
--push \
--cache-from=type=local,src=/tmp/.buildx-cache \
Copy link

Copilot AI Dec 24, 2025

Choose a reason for hiding this comment

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

Removing the cache-to configuration will prevent the Docker build cache from being updated. The workflow still reads from the cache using --cache-from on line 75, but without --cache-to, new build layers won't be written back to /tmp/.buildx-cache. This means the cache will become stale after the first build, reducing the effectiveness of the caching strategy and potentially increasing build times in subsequent runs. The cache-to line should be retained to maintain proper cache lifecycle management.

Suggested change
--cache-from=type=local,src=/tmp/.buildx-cache \
--cache-from=type=local,src=/tmp/.buildx-cache \
--cache-to=type=local,dest=/tmp/.buildx-cache,mode=max \

Copilot uses AI. Check for mistakes.
@github-actions
Copy link

Preview mcp_server Image:

registry.cn-hangzhou.aliyuncs.com/fastgpt/fastgpt-pr:fatsgpt_mcp_server_b86a5bb87baef63c1cd823c196df9d9896b976d0

@github-actions
Copy link

Preview sandbox Image:

registry.cn-hangzhou.aliyuncs.com/fastgpt/fastgpt-pr:fatsgpt_sandbox_b86a5bb87baef63c1cd823c196df9d9896b976d0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant