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: add suggestion when WSL seems to require a reboot #7007

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

lstocchi
Copy link
Contributor

@lstocchi lstocchi commented Apr 29, 2024

What does this PR do?

it adds a suggestion to try installing wsl without any distribution if after having installed virtual platform and wsl and have rebooted the machine, there are still sign the machine needs to be rebooted.

Screenshot / video of UI

image

What issues does this PR fix or reference?

it fixes #6978

How to test this PR?

  1. run tests
  • Tests are covering the bug fix or the new feature

Signed-off-by: lstocchi <lstocchi@redhat.com>
@lstocchi lstocchi requested review from benoitf and a team as code owners April 29, 2024 16:34
@lstocchi lstocchi requested review from cdrage and feloy and removed request for a team April 29, 2024 16:34
@lstocchi
Copy link
Contributor Author

cc @odockal

Copy link
Collaborator

@benoitf benoitf left a comment

Choose a reason for hiding this comment

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

LGTM but have not tested on Windows

@odockal
Copy link
Contributor

odockal commented Apr 30, 2024

Gonna take a look. Thx!

@odockal
Copy link
Contributor

odockal commented Apr 30, 2024

LGTM. Tried on Windows. User is now guided to finish the system preparation.

@lstocchi lstocchi merged commit 341e70d into containers:main Apr 30, 2024
8 checks passed
@lstocchi lstocchi deleted the i6978 branch April 30, 2024 09:17
@podman-desktop-bot podman-desktop-bot added this to the 1.11.0 milestone Apr 30, 2024
benoitf pushed a commit to benoitf/desktop that referenced this pull request Apr 30, 2024
benoitf pushed a commit to benoitf/desktop that referenced this pull request Apr 30, 2024
Signed-off-by: lstocchi <lstocchi@redhat.com>
Signed-off-by: Florent Benoit <fbenoit@redhat.com>

Signed-off-by: Luca Stocchi <49404737+lstocchi@users.noreply.github.com>
benoitf pushed a commit that referenced this pull request Apr 30, 2024
Signed-off-by: lstocchi <lstocchi@redhat.com>
Signed-off-by: Florent Benoit <fbenoit@redhat.com>

Signed-off-by: Luca Stocchi <49404737+lstocchi@users.noreply.github.com>
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.

Podman Onboarding check does not offer further guidance for the users on windows regarding WSL2
4 participants