Skip to content
This repository has been archived by the owner on Nov 17, 2022. It is now read-only.

Docs out of date for v0.3.0 #67

Open
cep21 opened this issue Feb 17, 2020 · 1 comment
Open

Docs out of date for v0.3.0 #67

cep21 opened this issue Feb 17, 2020 · 1 comment

Comments

@cep21
Copy link

cep21 commented Feb 17, 2020

Docs mention node-role.kubernetes.io/worker=true, but the code for v0.3.0 expects kubernetes.io/role=worker by default (https://github.com/pusher/k8s-spot-rescheduler/blob/v0.3.0/nodes/nodes.go#L31)

Also the example deployment https://github.com/pusher/k8s-spot-rescheduler/blob/master/deploy/deployment.yaml#L31 could change the block

          command:
            - rescheduler
            - -v=2
            - --running-in-cluster=true
            - --namespace=kube-system
            - --housekeeping-interval=10s
            - --node-drain-delay=10m
            - --pod-eviction-timeout=2m
            - --max-graceful-termination=2m
            - --listen-address=0.0.0.0:9235
            - --on-demand-node-label=node-role.kubernetes.io/worker
            - --spot-node-label=node-role.kubernetes.io/spot-worker

into

          args:
            - -v=2
            - --running-in-cluster=true
            - --namespace=kube-system
            - --housekeeping-interval=10s
            - --node-drain-delay=10m
            - --pod-eviction-timeout=2m
            - --max-graceful-termination=2m
            - --listen-address=0.0.0.0:9235
            - --on-demand-node-label=node-role.kubernetes.io/worker
            - --spot-node-label=node-role.kubernetes.io/spot-worker

to both resolve the issue with the binary being renamed (away from rescheduler see #64 (comment)) and to be resistant from future binary renames.

@theobarberbany
Copy link
Contributor

@cep21 Great spot! Thanks for bringing this up. I'll try getting to it this afternoon, unless you fancy opening a PR?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants