-
Notifications
You must be signed in to change notification settings - Fork 37
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
0.3.1 throws error #224
Comments
This is most likely due to the What does |
Indeed, ansible_lsb is empty on my hosts. |
I think the change came in here #203 and we didn't anticipate what that would change |
I sure think so. I don't know the reasoning of introducing the check for more environment information, let's wait for additional information from @mkayontour. The changes were introduced in #203. |
Same issue here on AlmaLinux 9.3, x86_64. Collection version 0.3.0 works without any issues. |
As I don't have access to a Raspberry Pi right now and won't have until next week, maybe @gianmarco-mameli could provide some context as to why they chose to implement the OS detection for Raspbian via |
Hi, the arm64 Debian Bookworm ❯ ansible -m setup harbor -a 'filter=ansible_lsb,ansible_distribution'
harbor | SUCCESS => {
"ansible_facts": {
"ansible_distribution": "Debian",
"ansible_lsb": {
"codename": "bookworm",
"description": "Debian GNU/Linux 12 (bookworm)",
"id": "Debian",
"major_release": "12",
"release": "12"
},
"discovered_interpreter_python": "/usr/bin/python3"
},
"changed": false
} arm64 Raspbian OS ❯ ansible -m setup rpitest2 -a 'filter=ansible_lsb,ansible_distribution'
rpitest2 | SUCCESS => {
"ansible_facts": {
"ansible_distribution": "Debian",
"ansible_lsb": {
"codename": "bookworm",
"description": "Debian GNU/Linux 12 (bookworm)",
"id": "Debian",
"major_release": "12",
"release": "12"
}
},
"changed": false
} armhf Rasbpian OS ❯ ansible -m setup rpi-node1 -a 'filter=ansible_lsb,ansible_distribution'
rpi-node1 | SUCCESS => {
"ansible_facts": {
"ansible_distribution": "Debian",
"ansible_lsb": {
"codename": "bullseye",
"description": "Raspbian GNU/Linux 11 (bullseye)",
"id": "Raspbian",
"major_release": "11",
"release": "11"
}
},
"changed": false
} So in this case the fastest solution is to modify the previous line that raises the error, because I think that ansible_lsb.id, when defined, is always the same as ansible_distribution, maybe if you have other distribution to test is appreciated adapt this line remove this line that raise the error Gianmarco |
@gianmarco-mameli I would adapt your solution. After all I would not expect everyone to install lsb. |
HI @mkayontour, seems perfect, thank you |
This will be fixed in the upcoming release v0.3.2 |
Hi all,
after upgrade from 0.3.0 to 0.3.1 the collection throws an error:
Rollbacked to 0.3.0 and everything is working fine again.
Maybe one have a look?
Cheers,
Marcus
The text was updated successfully, but these errors were encountered: