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

Add docker container to run ookla speedtest #390

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

siegy22
Copy link

@siegy22 siegy22 commented Sep 2, 2024

Also create a separate script that makes more sense in a container environment. The current one's error handling is not the best, because it is meant to run as a cronjob it should not output anything. In a container environment that does not really matter.

See the README adjustment for more details.

Also create a separate script that makes more sense in a container environment.
The current one's error handling is not the best, because it is meant to run as
a cronjob it should not output anything. In a container environment that does
not really matter.
@tbblake
Copy link
Contributor

tbblake commented Sep 2, 2024

Interesting and very good point. I admittedly did the docker stuff as an afterthought to just lay on top and didn't give much thought to it beyond that, which I think is obvious and you picked up on it. I'll have to look at these and add them in.

@siegy22
Copy link
Author

siegy22 commented Sep 8, 2024

@tbblake Let me know if you need any inputs, I used the following CronJob on k8s because I deployed zabbix using https://github.com/zabbix-community/helm-zabbix.

apiVersion: batch/v1
kind: CronJob
metadata:
  name: speedtest
spec:
  schedule: "0 * * * *"
  jobTemplate:
    spec:
      template:
        spec:
          containers:
          - name: speedtest
            image: siegy22/speedtest-cli-zabbix:latest
            imagePullPolicy: IfNotPresent
            env:
              - name: ZABBIX_SERVER
                value: "zabbix-zabbix-server.zabbix.svc"
              - name: SPEEDTEST_HOST
                value: "my-speedtest"
          restartPolicy: OnFailure

But as that is pretty k8s specific I didn't include it here, but might be something that people can use as well.

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.

2 participants