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(updates): Rework update mechanism #1827

Merged
merged 1 commit into from Mar 25, 2023
Merged

Conversation

benoitf
Copy link
Collaborator

@benoitf benoitf commented Mar 24, 2023

What does this PR do?

While trying the update, it was working but user got several error messages
Now:

  • ensure that when we check for updates, we do not download artifacts (so if user do not want to update, there is no silent update)
  • ensure that we're event based (notify user when update is downloaded, etc)
  • ask the user if he wants to restart Podman Desktop once the update has been successfully downloaded

Screenshot/screencast of this PR

AzkI3jELjs.1.mp4

What issues does this PR fix or reference?

Fixes #1806

How to test this PR?

I don't know how to test the PR on your own as it's requiring signed binaries

I did some signed binaries for macOS there:
https://github.com/benoitf/desktop/releases/tag/v0.2.15

it's a faked Podman Desktop test v0.2.15 app that will ask you to update to a v0.2.210 version

- ensure that when we check for updates, we do not download artifacts
- ensure that we're event based
- ask the user if he wants to restart Podman Desktop once the update has been successfully downloaded

fixes containers#1806

Change-Id: Ib6f7460db601bdae7f17758b52af506a207789b6
Signed-off-by: Florent Benoit <fbenoit@redhat.com>
@benoitf benoitf requested a review from a team as a code owner March 24, 2023 17:23
@benoitf benoitf requested review from jeffmaury and cdrage and removed request for a team March 24, 2023 17:23
Copy link
Contributor

@cdrage cdrage left a comment

Choose a reason for hiding this comment

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

LGTM, glad that we got this sorted out. Thanks @benoitf !

@benoitf benoitf merged commit 32e5c61 into containers:main Mar 25, 2023
7 checks passed
@podman-desktop-bot podman-desktop-bot added this to the 0.13.0 milestone Mar 25, 2023
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.

check for updates is working but it reports message failures
3 participants