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.

Sign up for GitHub

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

Jump to bottom

We should output ANY errors related to Podman Desktop into the image log file as well. #678

Closed
cdrage opened this issue Jul 26, 2024 · 0 comments · Fixed by #779
Closed

We should output ANY errors related to Podman Desktop into the image log file as well. #678

cdrage opened this issue Jul 26, 2024 · 0 comments · Fixed by #779
Assignees
@cdrage
Labels
kind/enhancement ✨ Issue for requesting an improvement

Comments

@cdrage
Copy link
Collaborator

cdrage commented Jul 26, 2024 •
edited
Loading

Is your enhancement related to a problem? Please describe

If there is an issue with the build process BEFORE we start the container, we should link this into the image log file.

The reasoning is that users who experience issues we have to ask them to look at their console logs which may not be very accessible / issues getting them (have to go to troubleshooting then logs tab).

Instead, we should output any errors encountered during the build process into the image log file as well.

I encountered this a lot during implementing #623 where having log output related to issues with the API would of been handy during the build process.

Describe the solution you'd like

For example:

  1. Press build button / build form
  2. Issue starting container related to Podman Desktop
  3. Output log to image log file (it was already created before hand).

Describe alternatives you've considered

No response

Additional context

No response
@cdrage cdrage added the kind/enhancement ✨ Issue for requesting an improvement label Jul 26, 2024
@cdrage cdrage self-assigned this Jul 26, 2024
cdrage added a commit to cdrage/podman-desktop-extension-bootc that referenced this issue Sep 3, 2024
@cdrage
chore: add debugging / error information to log file as well
db1288c 
### What does this PR do?

Adds some simple code that also outputs any errors with the function to
the log file for debugging purposes / being able to help for any
debugging.

### Screenshot / video of UI

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

N/A

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

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

Closes containers#678

### How to test this PR?

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

1. Before pressing build, stop your Podman Machine
2. See that it outputs the connection error into the build log file.

Signed-off-by: Charlie Drage <[email protected]>
@cdrage cdrage mentioned this issue Sep 3, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cdrage cdrage

Labels
kind/enhancement ✨ Issue for requesting an improvement
Projects
Podman Desktop Planning
Status: ✔️ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: add debugging / error information to log file as well cdrage/podman-desktop-extension-bootc
1 participant
@cdrage