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

fix: start a pod instead of trying to start each individual container #673

Merged
merged 1 commit into from
Oct 23, 2022

Conversation

benoitf
Copy link
Collaborator

@benoitf benoitf commented Oct 23, 2022

What does this PR do?

at the end of creating the pod, just start the pod.
starting all individual containers make the logic failed
also, we stop all containers first.

Screenshot/screencast of this PR

What issues does this PR fix or reference?

#516

How to test this PR?

Stevan's demo

Change-Id: I41ece2faff39b736a5c85f4e675a624df96b3336
Signed-off-by: Florent Benoit fbenoit@redhat.com

@benoitf benoitf marked this pull request as ready for review October 23, 2022 09:01
@benoitf benoitf marked this pull request as draft October 23, 2022 09:01
Change-Id: I41ece2faff39b736a5c85f4e675a624df96b3336
Signed-off-by: Florent Benoit <fbenoit@redhat.com>
@benoitf benoitf changed the title fix: try to delay ops to avoid podman lock freeze fix: start a pod instead of trying to start each individual container Oct 23, 2022
@benoitf benoitf marked this pull request as ready for review October 23, 2022 09:54
@benoitf benoitf enabled auto-merge (rebase) October 23, 2022 09:54
@benoitf benoitf merged commit 73d235b into containers:main Oct 23, 2022
@benoitf benoitf deleted the delay-ops branch October 23, 2022 10:16
@podman-desktop-bot podman-desktop-bot added this to the 0.9.0 milestone Oct 23, 2022
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.

None yet

2 participants