Repository navigation
cli: use operational data in show container [name] - #1339
Merged
Merged
Conversation
mattiaswal
requested changes
Jan 1, 2026
troglobit
force-pushed
the
container-operational
branch
from
January 2, 2026 18:36
d9c94ec to
5fd0dac
Compare
Contributor
Author
|
@mattiaswal note, new Canvas class added and firewall commands in Update: of course I forgot to check the unit tests! 🤦♂️ |
https://github.com/finit-project/finit/releases/tag/4.15 Signed-off-by: Joachim Wiberg <[email protected]>
Finit is capable of setting up all basic file systems, even /run but Infix relies on it allowing executing scripts (dagger and container) so leave that for now. Signed-off-by: Joachim Wiberg <[email protected]>
This patch adds support for flexible columns. When a column is marked as flexible it means it can be stretched when printing the table to an optional minimal table width. Multiple columns can be marked and when this occurs the padding is applied equally to all columns. Also, make sure to only add 2 char padding between columns, not always at the end, or we will require larger terminal than necessary. Signed-off-by: Joachim Wiberg <[email protected]>
Signed-off-by: Joachim Wiberg <[email protected]>
Signed-off-by: Joachim Wiberg <[email protected]>
The constaint was supposed to be 50% of one CPU core, or 500 millicores as per Kubernetes nomenclature. Signed-off-by: Joachim Wiberg <[email protected]>
Leverage the operational data to present a more human-friendly view of containers in the CLI. Replacing the podman script wrapper. Signed-off-by: Joachim Wiberg <[email protected]>
The original 'show firewall log' just did the same as the 'show log firewall' command. This new implementation allows showing pretty-printed firewall log which is admittedly easier on the eyes. Also, add 'show firewall matrix' to just show the zone matrix overview. Signed-off-by: Joachim Wiberg <[email protected]>
Signed-off-by: Joachim Wiberg <[email protected]>
Signed-off-by: Joachim Wiberg <[email protected]>
troglobit
force-pushed
the
container-operational
branch
from
January 2, 2026 19:09
5fd0dac to
1030bae
Compare
Contributor
Author
Thanks! Awesome to get this one out of the way, it's been bugging me all Christmas and New Years! 😆 |
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
2 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.
Description
This is a follow-up to the
container-limitsbranch posted earlier, PR #1327. It holds:/etc/fstab, Finit can handle most of standard mountsCanvasclass incli-prettySimpleTableand newCanvasclassesChecklist
Tick relevant boxes, this PR is-a or has-a: