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

Check that Podman is installed / Podman Machine is running #701

Closed
cdrage opened this issue Aug 1, 2024 · 0 comments · Fixed by #1499
Closed

Check that Podman is installed / Podman Machine is running #701

cdrage opened this issue Aug 1, 2024 · 0 comments · Fixed by #1499
Assignees
Labels
kind/enhancement ✨ Issue for requesting an improvement
Milestone

Comments

@cdrage
Copy link
Contributor

cdrage commented Aug 1, 2024

Is your enhancement related to a problem? Please describe

We should check to see if podman / podman machine is running before proceeding to the build page or even on the dashboard of bootc.

Right now, we output instead errors to console / functionality does not work.

Describe the solution you'd like

Warnings / errors in the UI that podman is not installed.

Describe alternatives you've considered

No response

Additional context

No response

@cdrage cdrage added the kind/enhancement ✨ Issue for requesting an improvement label Aug 1, 2024
@cdrage cdrage moved this to 📋 Backlog in Podman Desktop Planning Mar 20, 2025
@cdrage cdrage self-assigned this Mar 20, 2025
@slemeur slemeur added this to the 1.8 milestone Mar 26, 2025
@cdrage cdrage moved this from 📋 Backlog to 🚧 In Progress in Podman Desktop Planning Apr 1, 2025
cdrage added a commit to cdrage/podman-desktop-extension-bootc that referenced this issue Apr 4, 2025
### What does this PR do?

On the build page, we should do a prereqs check to see if the podman
machine is even running.

This adds a prereq check at the beginning to make sure that it errors
out before the user fills out the form.

I have also made sure that validation is now captures inside of a try /
catch, which it wasn't before.

### Screenshot / video of UI

<!-- If this PR is changing UI, please include
screenshots or screencasts showing the difference -->

### What issues does this PR fix or reference?

<!-- Include any related issues from Podman Desktop
repository (or from another issue tracker). -->

Closes podman-desktop#701

### How to test this PR?

<!-- Please explain steps to reproduce -->

1. Purposely stop podman machine
2. Go to build
3. See it errors out when trying to press button.

Signed-off-by: Charlie Drage <[email protected]>
cdrage added a commit to cdrage/podman-desktop-extension-bootc that referenced this issue Apr 4, 2025
### What does this PR do?

On the build page, we should do a prereqs check to see if the podman
machine is even running.

This adds a prereq check at the beginning to make sure that it errors
out before the user fills out the form.

I have also made sure that validation is now captures inside of a try /
catch, which it wasn't before.

### Screenshot / video of UI

<!-- If this PR is changing UI, please include
screenshots or screencasts showing the difference -->

### What issues does this PR fix or reference?

<!-- Include any related issues from Podman Desktop
repository (or from another issue tracker). -->

Closes podman-desktop#701

### How to test this PR?

<!-- Please explain steps to reproduce -->

1. Purposely stop podman machine
2. Go to build
3. See it errors out when trying to press button.

Signed-off-by: Charlie Drage <[email protected]>
@cdrage cdrage moved this from 🚧 In Progress to 🚥 In Review in Podman Desktop Planning Apr 4, 2025
@github-project-automation github-project-automation bot moved this from 🚥 In Review to ✔️ Done in Podman Desktop Planning Apr 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement ✨ Issue for requesting an improvement
Projects
Status: ✔️ Done
Development

Successfully merging a pull request may close this issue.

2 participants