Skip to content

Allow request resources to be set as well as limits - #410

Merged
hardillb merged 7 commits into
mainfrom
allow-request-resources
Oct 13, 2026
Merged

hardillb merged 7 commits into
mainfrom
allow-request-resources

Conversation

@hardillb

@hardillb hardillb commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

fixes #397

Description

image

If left blank it will use the limits and set them the same (with the existing 128m bump to the memory limit)

I have set the default CPU Request to match the limit, but left the memory empty to match existing behaviour.

Will need a small forge app update to make unset options render nicely

image

Related Issue(s)

#397

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

@hardillb hardillb self-assigned this Oct 6, 2026
Comment thread kubernetes.js Outdated
@hardillb

hardillb commented Oct 6, 2026

Copy link
Copy Markdown
Contributor Author

Merge FlowFuse/flowfuse#8838 after this

@knolleary knolleary left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Small help text typo and clarification suggested. Others 👍🏻

Comment thread kubernetes.js Outdated
Co-authored-by: Nick O'Leary <nick.oleary@gmail.com>
@hardillb
hardillb merged commit 7561aad into main Oct 13, 2026
2 checks passed
@hardillb
hardillb deleted the allow-request-resources branch October 13, 2026 09:58
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.

Allow CPU request and limit to be configured separately

2 participants