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

spread: band-aid docker installation on fedora-37 #277

Merged
merged 2 commits into from
May 25, 2023

Conversation

tigarmo
Copy link
Collaborator

@tigarmo tigarmo commented May 23, 2023

Latest version of the docker snap fails to install on Fedora with the following message:

 error: cannot install "docker": snap "docker" assumes unsupported features:
       snapd2.59.1 (try to refresh snapd)

Trying to refresh snapd doesn't actually solve the issue. While we investigage, installing from the older stable channel should keep the spread tests running.

Also do some minor cleanup from the fedora 36 -> 37 update.

  • Have you signed the CLA?

Latest version of the docker snap fails to install on Fedora with the
following message:

```
 error: cannot install "docker": snap "docker" assumes unsupported features:
       snapd2.59.1 (try to refresh snapd)
```

Trying to refresh snapd doesn't actually solve the issue. While we
investigage, installing from the older stable channel should keep the
spread tests running.

Also do some minor cleanup from the fedora 36 -> 37 update.
@mr-cal
Copy link
Collaborator

mr-cal commented May 23, 2023

I think this is the best workaround. Fedora looks like they are ~3 months behind for snapd releases (https://packages.fedoraproject.org/pkgs/snapd/snapd/).

Can you add a comment to the spread test to mention that we can remove the workaround once snapd is bumped on fedora?

@tigarmo
Copy link
Collaborator Author

tigarmo commented May 24, 2023

I'm going to add a comment for sure, but I haven't yet because I'm not clear on the issue/fix:

a) I tried "hackingly" updating snapd via snap install snapd. No idea if that's even possible/advisable, but even after snapd 2.59.2 was installed that error persisted;

b) The "assumes" keyword there makes me think that this is related to this recent Snapcraft bugfix, and this docker-snap merge is another indicator. @sergiusens do you happen to have any idea why our bugfix would lead to this breaking?

This latest update to the docker snap has a bunch of fairly major changes: https://github.com/docker-snap/docker-snap/releases

@sergiusens
Copy link
Collaborator

sergiusens commented May 24, 2023 via email

@tigarmo tigarmo requested a review from mr-cal May 24, 2023 15:07
@tigarmo tigarmo marked this pull request as ready for review May 24, 2023 15:07
@tigarmo tigarmo merged commit 9e99b2e into canonical:main May 25, 2023
@tigarmo tigarmo deleted the fedora-docker-install branch May 25, 2023 13:21
@zyga
Copy link

zyga commented Apr 10, 2024

Hey, just to mention this as I found it referenced from https://github.com/canonical/snapcraft-rocks/blob/core22-7/spread.yaml#L68C7-L68C54

Fedora is now well maintained and should get up-to-date snapd quickly. Re-execution is technically not implemented on Fedora and the remaining work is non-trivial.

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.

6 participants