Skip to content

ci: Improve pre-staging forge container image size#7329

Merged
ppawlowski merged 7 commits into
mainfrom
ci-redice-pre-staging-forge-image-size
May 27, 2026
Merged

ci: Improve pre-staging forge container image size#7329
ppawlowski merged 7 commits into
mainfrom
ci-redice-pre-staging-forge-image-size

Conversation

@ppawlowski
Copy link
Copy Markdown
Contributor

@ppawlowski ppawlowski commented May 25, 2026 •
edited
Loading

Description

This pull request introduces an improved version of the multistage container image build used for pre-staging environments.
The main goal of this change is to:

  • reduce the image size
  • validate changes on pre-staging environments before pushing these adjustments to the Dockerfile used by FFC deployments

Related Issue(s)

Checklist

  • I have read the contribution guidelines
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
    • Upgrade instructions
    • Configuration details
    • Concepts
  • Changes flowforge.yml?
    • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
    • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production
  • Link to Changelog Entry PR, or note why one is not needed.

Labels

  • Includes a DB migration? -> add the area:migration label

@ppawlowski ppawlowski marked this pull request as ready for review May 25, 2026 11:02
@ppawlowski ppawlowski requested a review from hardillb May 25, 2026 11:02
@ppawlowski ppawlowski changed the title ci: Improve pre-staging image size ci: Improve pre-staging forge container image size May 25, 2026
@hardillb
Copy link
Copy Markdown
Contributor

hardillb commented May 26, 2026 •
edited
Loading

validate changes on pre-staging environments before pushing these adjustments to the Dockerfile used by FFC deployments

If this is going to end up on production, then it needs to run the script to install the Device Agent Editor cache Node-RED UI parts

https://github.com/FlowFuse/helm/blob/f5271711c66e0569c0388342c7737c6cd781bc8c/flowforge-container/Dockerfile#L12-L13

hardillb
hardillb requested changes May 26, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@hardillb hardillb left a comment

Choose a reason for hiding this comment

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

Device-agent editor cache ^

@ppawlowski ppawlowski requested a review from hardillb May 26, 2026 20:37
@ppawlowski ppawlowski merged commit 0460f02 into main May 27, 2026
28 checks passed
@ppawlowski ppawlowski deleted the ci-redice-pre-staging-forge-image-size branch May 27, 2026 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hardillb hardillb hardillb approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ppawlowski @hardillb