Skip to content
This repository has been archived by the owner on Oct 5, 2021. It is now read-only.

add support for taints #13

Open
m-yosefpor opened this issue Aug 31, 2021 · 4 comments
Open

add support for taints #13

m-yosefpor opened this issue Aug 31, 2021 · 4 comments

Comments

@m-yosefpor
Copy link

m-yosefpor commented Aug 31, 2021

Being able to define taints on nodes which match a selector. for example:

apiVersion: node-labels.openshift.io/v1beta1
kind: Labels
metadata:
  name: taint-sample1
spec:
  nodeNamePatterns:
    - infra-.*
  labels:
    test.openshift.io/foo1: bar1
  taints:
    - effect: NoExecute
      key: node-role.kubernetes.io/infra

There is another node label operator (https://github.com/barpilot/node-labeler-operator) which seems to be deprecated (no activity since 2018), however it supports defining taints on nodes, based on a selector. Also that operator does not support regex name matching. I think it's really useful to have that feature in this operator as well.

@slintes
Copy link
Member

slintes commented Sep 1, 2021

Hi @m-yosefpor, I'm sorry, but we are not maintaining this operator. Our usecase changed before we actually started using it. I will add a note to the readme.

@m-yosefpor
Copy link
Author

Thanks @slintes for informing.

@m-yosefpor
Copy link
Author

I've written another operator with operator-sdk which supports labels, annotations, and taints, namepattern matching, and label matching, based on this operator and also https://github.com/barpilot/node-labeler-operator (the latter is also deprecated). We will open-source it soon, and share the link here if any one else is interested in such operator.

@slintes
Copy link
Member

slintes commented Sep 2, 2021

Sounds great 👍🏼

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