Pull to refresh

Comments 10

Идея хорошая.


Каждые 15 секунд (настраивается через опцию --check-period) достаём из очереди порцию наиболее старых образов и проверяем их наличие в registry.

не понял — что значит старых образов? Т.е. у утилиты какой-то внутренний кэш? Или имеется в виду — образы, из которых запущены самые старые подов-деплойментов?

Возможно, стоит перефразировать. Самый старый timestamp последней проверки.
Сейчас поправлю, спасибо!
  template:
    metadata:
      annotations:
        prometheus.io/port: "8080"
        prometheus.io/scrape: "true"


Поправьте deployment, пожалуйста, и примеры. Если прометеус оператор установлен, он подхватит такой деплоймент. Если его нет — хуже не будет.

https://github.com/coreos/kube-prometheus/pull/16#issuecomment-305953469


The annotation is not supported by the operator. ServiceMonitors should be
used instead.
The annotation is very limited by its nature and there's no
feasible way to support anything beyond "scrape on" and a single port. A
per-service scrape interval, multi-port pods, authentication, honorLabels,
and more are all not possible. It also provides no meaningful way to assign
services to be scraped by different Prometheus instances.
ServiceMonitors allow all these without exposing any complex relabelling
and there's really no practical reason for sticking with the annotation.

Представляется, что использование этой аннотации избыточно. Используйте соответствующие CRD. Если есть альтернативное мнение — давайте обсудим.

О как, не знал) Наш форк оператора просто поддерживает много чего, чего нет из коробки.

Протестил вашу штуку — всё отлично. У нас правда свой аналог есть — который чистит лишнее по заданным правилам, ориентируясь по метрикам использования images от k8s api.
gecube не у нас работает, но мы его знаем и уважаем, потому что он всегда читает и критикует наши статьи.

А не проще сразу же манифесты для prometheus-operator положить в deploy/ или даже упаковать всё это в хелм-чарт ?

git pull
на git checkout поменяйте в статье

git clone
Поправил, спасибо.

Only those users with full accounts are able to leave comments. Log in, please.