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

Recommend podman desktop #39692

Merged
merged 3 commits into from
Mar 27, 2024
Merged

Recommend podman desktop #39692

merged 3 commits into from
Mar 27, 2024

Conversation

maxandersen
Copy link
Member

several issues with out podman page:

  • podman 5.0.0 was released last week but broken for quarkus users on non-linux - 5.0.1 fixes it but not released yet, if users used podman desktop they would have gotten 4.9.x which has better windows/osx testing
  • on osx we tell users to use brew (gets 5.0), install qemu (not compatible with 5), devservices works but podman ps and podman desktop picks 5.0 where testcontainers picks the qemu hosted one
  • alias podman=docker just isn't likely to work - better to use ppodman desktop guided install
  • cross build should be fixed in 4.9 and better in 5.0 (need to be tested - but our current instructions only apply for certain combos)

Here is my attempt on cleaning up so we at least have higher chance of users with quarkus on podman.

Those who want to manually install each part can do so - but with podman desktop we are more likely to have things working for more users.

@quarkus-bot
Copy link

quarkus-bot bot commented Mar 26, 2024

Thanks for your pull request!

The title of your pull request does not follow our editorial rules. Could you have a look?

  • title should preferably start with an uppercase character (if it makes sense!)

This message is automatically generated by a bot.

@maxandersen
Copy link
Member Author

cc @n1hility as discussed - want to get it updated asap.

@geoand geoand changed the title recomend podman desktop Recommend podman desktop Mar 26, 2024
docs/src/main/asciidoc/podman.adoc Outdated Show resolved Hide resolved
docs/src/main/asciidoc/podman.adoc Show resolved Hide resolved
docs/src/main/asciidoc/podman.adoc Outdated Show resolved Hide resolved
@gastaldi
Copy link
Contributor

I had to run the following commands once I installed Podman in my M1 (using brew install podman-desktop) to make DevContainers work:

podman machine init 
podman machine start

After that, I had to close the Podman Desktop app and open it again, so the Podman Machine status is shown as green.

@maxandersen
Copy link
Member Author

Devcontainers or devservices ?

@maxandersen
Copy link
Member Author

And you say you had to run those commands even after having run podman desktop? (Not just install)

@gastaldi
Copy link
Contributor

Devcontainers or devservices ?

Yes, Devservices, sorry

And you say you had to run those commands even after having run podman desktop? (Not just install)

Exactly

maxandersen and others added 2 commits March 27, 2024 01:02
Co-authored-by: Holly Cummins <hcummins@redhat.com>
Signed-off-by: Jason T. Greene <jason.greene@redhat.com>
@n1hility
Copy link
Member

Devcontainers or devservices ?

Yes, Devservices, sorry

And you say you had to run those commands even after having run podman desktop? (Not just install)

Exactly

If you don't have a machine on your system it will prompt to create one for you like so:

Screenshot 2024-03-27 at 1 14 57 AM Screenshot 2024-03-27 at 1 17 50 AM

If you abort the wizard there is also a secondary path:

Screenshot 2024-03-27 at 1 22 21 AM

@n1hility n1hility marked this pull request as ready for review March 27, 2024 06:24
@n1hility
Copy link
Member

Reworked the PR and added additional info. Specific instructions are still needed for Linux setups. Added a very explicit warning not to use brew.

@quarkus-bot
Copy link

quarkus-bot bot commented Mar 27, 2024

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 5c8f0f4.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

Copy link
Member Author

@maxandersen maxandersen left a comment

Choose a reason for hiding this comment

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

+1 LGTM and we can iterate!

@maxandersen maxandersen merged commit 72c7f66 into quarkusio:main Mar 27, 2024
5 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.10 - main milestone Mar 27, 2024
Copy link

🙈 The PR is closed and the preview is expired.

@maxandersen
Copy link
Member Author

merge in but also marked for backport as instructions broken for 3.2 and 3.8 guides too.

@holly-cummins
Copy link
Contributor

I've tested the new instructions, and I can confirm that we'll need to iterate a bit:
image

The gap is that we tell people to install via Podman desktop, but we don't tell them to run Podman desktop before using Quarkus. If they're not launching Podman desktop before trying to use Quarkus, they'll need

podman machine start

as @gastaldi points out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

6 participants