Skip to content
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 alternative podman command for memory info #961

Merged
merged 2 commits into from
Oct 30, 2024

Conversation

aressem
Copy link
Member

@aressem aressem commented Oct 30, 2024

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@aressem aressem requested a review from thomasht86 October 30, 2024 13:51
thomasht86
thomasht86 previously approved these changes Oct 30, 2024
Copy link
Collaborator

@thomasht86 thomasht86 left a comment

Choose a reason for hiding this comment

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

Great! Think we should probably add this to the example notebooks running docker as well, but this is a good start.

@thomasht86
Copy link
Collaborator

@thomasht86
Copy link
Collaborator

I am working on fixing the notebook tests in another branch, so we can just merge this after re-approval

@thomasht86 thomasht86 merged commit f860691 into master Oct 30, 2024
39 of 45 checks passed
@thomasht86 thomasht86 deleted the aressem/add-podman-cmd branch October 30, 2024 14:40
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.

2 participants