-
Notifications
You must be signed in to change notification settings - Fork 289
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
feat: add raspberry pi spport #1145
feat: add raspberry pi spport #1145
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a change fragment and fix the tests that are failing. I'll be honest, I am not familiar with the ansible_lsb.id
variable and from the little bit of reading I think it may well cause more problems than its worth just to allow Pi integration which is not something we will officially support since there's no way for us to test the integration on it. @D3DeFi what are your thoughts?
Thank you feedback.
I see, I continue doing it.
I understood, anyway I continue to work to make/provide workable patch. |
694abdd
to
f43b88b
Compare
I fixed code and added a fragment.
|
835575c
to
7f6f7b8
Compare
@pyrodie18 |
It looks like it cannot fine the package https://github.com/ansible-collections/community.zabbix/actions/runs/7307787096/job/19913806734?pr=1145#step:7:228 |
in centos script, mysql(maria db) 10.8 repo link is hardcorded in
And the url https://mirror.rackspace.com/mariadb/yum/10.8/centos7-amd64/repodata/repomd.xml return 404. I cheked server in my browser, It seems 10.8 was dropped and 10.8 reached to EOL. So I guess this CI fails is not related to my change and In order to pass CI changing mysql version in censto script is needed. |
bc97d5d
to
f478937
Compare
SUMMARY
in case raspberry pi, role scripts mis-detect os type as debian and try to add debian repo url.
This PR change the detection to detect rasberry pi os to get right repo url.
ISSUE TYPE
COMPONENT NAME
{agent, javagateway, web, server} role
ADDITIONAL INFORMATION