Skip to content

Some improvements for libpod#28812

Open
bit-hope wants to merge 1 commit into
podman-container-tools:mainfrom
bit-hope:improvements-for-libpod
Open

Some improvements for libpod#28812
bit-hope wants to merge 1 commit into
podman-container-tools:mainfrom
bit-hope:improvements-for-libpod

Conversation

@bit-hope
Copy link
Copy Markdown
Contributor

@bit-hope bit-hope commented May 29, 2026

Checklist

Ensure you have completed the following checklist for your pull request to be reviewed:

  • Certify you wrote the patch or otherwise have the right to pass it on as an open-source patch by signing all
    commits. (git commit -s). (If needed, use git commit -s --amend). The author email must match
    the sign-off email address. See CONTRIBUTING.md
    for more information.
  • Referenced issues using Fixes: #00000 in commit message (if applicable)
  • Tests have been added/updated (or no tests are needed)
  • Documentation has been updated (or no documentation changes are needed)
  • All commits pass make validatepr (format/lint checks)
  • Release note entered in the section below (or None if no user-facing changes)

Does this PR introduce a user-facing change?

None

@bit-hope bit-hope changed the title Som improvements for libpod Some improvements for libpod May 29, 2026
Comment thread libpod/pod_status.go Outdated
Copy link
Copy Markdown
Contributor

@Honny1 Honny1 left a comment

Choose a reason for hiding this comment

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

LGTM, once CI is green.

I agree with @baude's suggestion of unit test.

Comment thread libpod/runtime_ctr.go
@TomSweeneyRedHat
Copy link
Copy Markdown
Contributor

THe CI is happy due to the lack of a new test with this PR. Once added, this should be GTG. Changes otherwise LGTM

@bit-hope bit-hope force-pushed the improvements-for-libpod branch 2 times, most recently from c837870 to d87d00d Compare June 1, 2026 07:47
Comment thread libpod/pod_status_test.go Outdated
@bit-hope bit-hope force-pushed the improvements-for-libpod branch from b8de8b0 to a80d6b0 Compare June 1, 2026 10:41
Comment thread libpod/pod_status_test.go Outdated
Copy link
Copy Markdown
Contributor

@Honny1 Honny1 left a comment

Choose a reason for hiding this comment

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

Also, please squish commits. Thanks

@bit-hope bit-hope force-pushed the improvements-for-libpod branch from a80d6b0 to 1fc5b6e Compare June 1, 2026 11:34
Copy link
Copy Markdown
Contributor

@Honny1 Honny1 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Please hold off on rebasing on main until the new CI is working upstream (Cirrus CI is dead). We have a PR ready to fix this and hope it merges soon. I will ping you when you're good to rebase.

@bit-hope
Copy link
Copy Markdown
Contributor Author

bit-hope commented Jun 1, 2026

LGTM, thanks!

Please hold off on rebasing on main until the new CI is working upstream (Cirrus CI is dead). We have a PR ready to fix this and hope it merges soon. I will ping you when you're good to rebase.

Ok.

@Honny1
Copy link
Copy Markdown
Contributor

Honny1 commented Jun 1, 2026

You can do a rebase. @bit-hope

Signed-off-by: zhongchunhe.zch <zhongchunhe.zch@qq.com>
@bit-hope bit-hope force-pushed the improvements-for-libpod branch from 1fc5b6e to 6b9d6da Compare June 2, 2026 01:35
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.

4 participants