-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Added new spk for activebackup-prometheus-exporter #4292
base: master
Are you sure you want to change the base?
Conversation
Not sure what the problem is, but all of the failed Actions run sucessfully when I build the spk in the official Docker container... 😕 |
Sadly not, I did some work on the For the status output, if your code uses the regular |
As statet in my first comment, all archs build sucessfully when I build them on my machine in the spksrc Docker container (Just copied the armv7 binary to one of my RaspberryPis to confirm that the binary is intact). I just found out how to see the full logs of the GH Actions and found this line to be the error:
I have sadly no idea what the GH actions do different than my local container when I run |
I did found where the build error is:
Now its a matter of defining the why.... Here is how to reproduce:
If you start with the x86_64 build, the Something seems to be wrong in the environment when called from a cross-compiled build. To investigate. |
Ok, got a tentative fix at #4383 which forces an amd64 host setting along with enforcing docker image This requires you to clean-up first your |
Thanks @th0ma7 for taking a look. I will try your fix and report if it worked :) |
I just cherry-picked your commit into my branch and can confirm that it's working as intended. I cleaned my own package and the native/go package, and directly build e.g. armv7 without any problems 👍 |
@fleaz I just merged my fix. Can you rebase + force push to confirm it now builds all-right on all platforms? |
802df89
to
3c3211f
Compare
Done :) |
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.
Thanks for contribution. I have a minor improvement request
I still have the problem with the status of the Service. It's running fine in the background but the webinterface tells me "Status: Stopped". |
@fleaz About the logo, we care about credits to author (in SVG metadata we store in Git in addition to png). Please also care about license for usage and distribution, we recommend to select one of creative commons. |
@ymartin59 I will test and check your process status issue with PID file... framework should do the job seamlessly... except if exporter has a specific behavior like internal forking... |
c58c284
to
f7c2999
Compare
Hey, I recently had to update this package for our own NAS because I made an update to the packages software itself, and meanwhile I also rebased this PR and I can confirm that the problem described further up with the wrong detection of the running state is gone. One of the nearly 700 commits on master in between have probably fixed it :D I just build it for our NAS and the state is working fine and I can also start/stop via the GUI without any problems. Last thing is the missing icon. But if this is not a requirement for you, this PR is finished from my side :) |
Motivation:
I work at a company where we are responsible for the infrastructure in some office buildings and we use Disk- and Rackstations together with the backup tools from the ActiveBackup suite to backup virtual machines, physical hosts and Google Drives. Because this was a blank spot in our monitoring I started to build activebackup-prometheus-exporter which is an Prometheus exporter which presents the data scraped from the internal sqlite databases generated by the ActiveBackup tools to get informations about the backup jobs. So if you have a running Prometheus server, you can just add your Synology NAS as a target and scrape the data about the backup jobs after installing this tool and create alerts when e.g. the last backup is older than 24 hours or something like that.
I'm new to this whole Synocommunity thing and this is my first spk package I build. So if I did something wrong, please let my know and I will fix it :)
Due to the similarities I used the spk and cross paccakge from the normal Prometheus node-exporter as a starting point and changed it to my needs. The packages was tested on my DS218+ (apollolake) and on a RS1619xs+ (broadwellnk) both running DSM 6.2.
The only thing that is currently not working is that it's not recognized as "running" after an installation, even though it's running (checked with
ps
and I can curl it). Maybe someone more experienced can give me a hint on how to fix this?Best regards,
Felix
Checklist
all-supported
completed successfully