cli: use operational data in show container [name]
#1339
Merged
Conversation
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
mattiaswal
requested changes
Jan 1, 2026
troglobit
force-pushed
the
container-operational
branch
from
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 <troglobit@gmail.com>
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 <troglobit@gmail.com>
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 <troglobit@gmail.com>
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
The constaint was supposed to be 50% of one CPU core, or 500 millicores as per Kubernetes nomenclature. Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
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 <troglobit@gmail.com>
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 <troglobit@gmail.com>
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
troglobit
force-pushed
the
container-operational
branch
from
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! 😆 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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: