-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add podman system events alias to podman events #16464
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rhatdan The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
idea is LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add the changelog to the commit message as well.
@edsantiago PTAL, I could not figure out how to get hack/xref-helpmsgs-manpages to recognize that there was a links file for the podman-system-events man page |
diff --git a/docs/source/markdown/podman-system.1.md b/docs/source/markdown/podman-system.1.md
index c41124510..12b77f5b8 100644
--- a/docs/source/markdown/podman-system.1.md
+++ b/docs/source/markdown/podman-system.1.md
@@ -15,7 +15,7 @@ The system command allows you to manage the podman systems
| ------- | ------------------------------------------------------------ | ------------------------------------------------------------------------ |
| connection | [podman-system-connection(1)](podman-system-connection.1.md) | Manage the destination(s) for Podman service(s) |
| df | [podman-system-df(1)](podman-system-df.1.md) | Show podman disk usage. |
-| events | [podman-system-events(1)](podman-system-events.1.md) | Monitor Podman events |
+| events | [podman-system-events(1)](podman-events.1.md) | Monitor Podman events |
| info | [podman-system-info(1)](podman-info.1.md) | Displays Podman related system information. |
| migrate | [podman-system-migrate(1)](podman-system-migrate.1.md) | Migrate existing containers to a new podman version. |
| prune | [podman-system-prune(1)](podman-system-prune.1.md) | Remove all unused pods, containers, images, networks, and volume data. | |
Thanks @edsantiago I love it when it is right in front of your face. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to change eventsCommand
to cmd
in the new eventsFlags function, otherwise tab completion will not work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/hold
New changes are detected. LGTM label has been removed. |
Looks like the format test is failing, I think you need to add this: $ git diff
diff --git a/test/system/610-format.bats b/test/system/610-format.bats
index 8f74634d1..3ee95bdff 100644
--- a/test/system/610-format.bats
+++ b/test/system/610-format.bats
@@ -37,6 +37,7 @@ search | $IMAGE
pod inspect | mypod
events | --stream=false --events-backend=file
+system events | --stream=false --events-backend=file
"
# podman machine is finicky. Assume we can't run it, but see below for more. |
Signed-off-by: Daniel J Walsh <[email protected]>
Signed-off-by: Daniel J Walsh <[email protected]>
/hold cancel |
Signed-off-by: Daniel J Walsh [email protected]
Does this PR introduce a user-facing change?