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 log_opt and annotaion options to podman_play module #668

Merged
merged 6 commits into from
Nov 26, 2023

Conversation

nishipy
Copy link
Contributor

@nishipy nishipy commented Nov 14, 2023

Fixes #544 and added test.
In addition, this change adds tests for PR #577, which is also related to issue #544.

Signed-off-by: nishipy <goodisonev4@gmail.com>
Signed-off-by: nishipy <goodisonev4@gmail.com>
Signed-off-by: nishipy <goodisonev4@gmail.com>
@nishipy nishipy marked this pull request as ready for review November 15, 2023 01:41
@nishipy
Copy link
Contributor Author

nishipy commented Nov 15, 2023

Hi @sshnaidm, the task Install podman in ci/playbooks/pre.yml failed for some reason (the log says Failed to fetch packages...). Could you have a look? Thank you.

Signed-off-by: nishipy <goodisonev4@gmail.com>
Signed-off-by: nishipy <goodisonev4@gmail.com>
@sshnaidm
Copy link
Member

Hi @sshnaidm, the task Install podman in ci/playbooks/pre.yml failed for some reason (the log says Failed to fetch packages...). Could you have a look? Thank you.

This is a problem with opensuse repositories which we use for installing a new Podman, need to wait until it's fixed. Happens quite a lot, btw.
image

@sshnaidm
Copy link
Member

Checking here: https://status.opensuse.org/

@nishipy
Copy link
Contributor Author

nishipy commented Nov 15, 2023

Thank you for the information.I see. Then, let me retry after it gets fixed.

Signed-off-by: nishipy <goodisonev4@gmail.com>
@nishipy
Copy link
Contributor Author

nishipy commented Nov 24, 2023

@sshnaidm Finally opensuse repositories have come back and all checks have passed. Please review this PR at your convenience. Thank you.

@sshnaidm
Copy link
Member

Awesome, thanks!

@sshnaidm sshnaidm merged commit b7e8711 into containers:master Nov 26, 2023
8 checks passed
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.

Add some genernal parameters in containers.podman.podman_play
2 participants