Skip to content

Review podman examples#166

Open
colisee wants to merge 4 commits intoLibreBooking:masterfrom
colisee:podman
Open

Review podman examples#166
colisee wants to merge 4 commits intoLibreBooking:masterfrom
colisee:podman

Conversation

@colisee
Copy link
Collaborator

@colisee colisee commented Mar 6, 2026

Changes

  • Switch podman cli example to a pod utilization from a network utilization
  • Add an example of utilization with a pod-kind Kubernetes yaml file

@colisee colisee requested a review from ikke-t March 6, 2026 09:13
@colisee colisee added the documentation Improvements or additions to documentation label Mar 6, 2026
@colisee colisee changed the title Reviw podman examples Review podman examples Mar 6, 2026
@ikke-t
Copy link
Collaborator

ikke-t commented Mar 6, 2026

I tested the changes and they do work. Now I would perhaps change the headers of the file. They say local access in some, some not. They are all 8080 port so none of them is more to local access than another one. Perhaps the word local could be removed, or it should be in all. I think it relates to the docker example having LB in front.

Copy link
Collaborator

@ikke-t ikke-t left a comment

Choose a reason for hiding this comment

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

the local in headers

@colisee
Copy link
Collaborator Author

colisee commented Mar 7, 2026

I agree with you. I will stop mentioning local access.

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

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants