-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Ooops! Snap installation failed while executing #7265
Comments
Files identified in the description: If these files are incorrect, please update the |
CC @russoz |
hi @onknows, thanks for reporting the issue. Could you please paste the output of the playbook when running with |
Also, could you please inform:
|
For the record, the problem in #7130 was that the wrong version of the collection was being used. As soon as the user made sure the correct version (7.3.0 at the time), the problem was gone. |
See provision.log vagrant@c2d-galaxy1:~$ snap --version
snap 2.60.3
snapd 2.60.3
series 16
ubuntu 22.04
kernel 6.2.0-32-generic Controller node onknows@io3:~$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=22.04
DISTRIB_CODENAME=jammy
DISTRIB_DESCRIPTION="Ubuntu 22.04.3 LTS"
onknows@io3:~$ Target node: vagrant@c2d-galaxy1:~$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=22.04
DISTRIB_CODENAME=jammy
DISTRIB_DESCRIPTION="Ubuntu 22.04.1 LTS"
vagrant@c2d-galaxy1:~$ Full list: le-dev]└2 master(+13/-11)* ± ansible-galaxy collection list community.general
# /home/onknows/git/gitlab/c2/ansible-dev/ansible_collections
Collection Version
----------------- -------
community.general 7.3.0
θ100° [:ansible-dev]└2 master(+29/-8)* ± |
Hi @onknows , thanks for the extra info. However, in the output file you attached, I do not see in the output of the module a variable Could you please double check and ensure that we have |
Hi @russoz, my bad, I did not see four |
Encountered the same problem |
@russoz Hi, is there any news on that? |
Hi @Voldemat , sorry, haven't had time to work on it these last days. Hopefully on the weekend. |
@russoz is there any awy I can help? I'm also hitting this issue |
Got same issue @russoz |
Ok, the errors that we have been seeing are coming up during the installation of the snaps. The way the module works today is:
Parsing the installation output to detect whether it was successful or not. However, that seems to be proving less straightforward than expected, so I am changing the logic to:
The snap command is not fast (it takes a couple of seconds just to start) and that extra call will be noticed, but it is more reliable that way. |
hi all, Please check whether the code in the PR #7340 solves your problem. As I never got to replicate the problem, I am not confident it does. Cheers, |
@russoz I can test it in some day, but... how can I test it? |
In an isolated environment, install it with:
Alternatively, if you don't mind getting your hands dirty, you could clone my fork and manually copy the snap.py file into your current installation, provided you remember to revert that later ;-) |
No problem for hands dirty. I use Ansible in Docker ;) |
Please let us know if/when you manage to replicate the situation using the updated module. |
Probably I can test not before the weekend. |
I'll try shortly. |
ERROR! Could not find git executable to extract the collection from the Git repository |
So I copied |
Yup, |
Finally I setup via ansible-galaxy and it works ;) Thank you @russoz |
Summary
This is related to #7130 . This issue #7130 is still not completely fixed. The following code fails on Ubuntu 22.04. I tested this with 7.3.0, 7.4.0. It fails with those versions. The code below does however work when using 6.6.0.
Issue Type
Bug Report
Component Name
snap
Ansible Version
Community.general Version
Configuration
OS / Environment
Ubuntu 22.04
Steps to Reproduce
Expected Results
Snap install like on community.general 6.6.0
Actual Results
Code of Conduct
The text was updated successfully, but these errors were encountered: