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

dev, prodでprometheus-operator, prometheus, node-exporterのバージョンを変えられるようにする #1222

Open
oke-py opened this issue Nov 20, 2021 · 7 comments
Assignees

Comments

@oke-py
Copy link
Member

oke-py commented Nov 20, 2021

devもprodもbaseを参照しているため、バージョン(に限りませんが)を更新しようとするとdevもprodもいっぺんに更新されてしまいます。
prometheusぐらいなら大丈夫だろうという気もしないではないですが、dev更新=>動作確認=>prod更新のステップを踏めるようにしておきたいです。

apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
bases:
- ../../base

apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
bases:
- ../../base

こんな感じでしょうか。overlaysを参照するものとbaseを参照するものの間に依存関係がある場合、どのように扱うといいのかkustomizeをよくわかっていません。

apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
namespace: monitoring
bases:
- ../../base
resources:
- https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/${ここを変える}/bundle.yaml
- ...
@mayocream
Copy link
Member

はい~/
やらせてください

@oke-py
Copy link
Member Author

oke-py commented Nov 21, 2021

お願いしますー!
困ったらどんどんここでもSlackでもいいので相談してください。
(私は回答を持ち合わせていませんが、一緒に考えます。他の方も回答してくれるはずです。)

@riita10069
Copy link
Member

素人考えですが、

  1. dev側のkustomize.yamlでバージョンをオーバーラップする
    1. --prometheus-config-reloader=quay.io/prometheus-operator/prometheus-config-reloader:v0.51.2
    2. image: quay.io/prometheus-operator/prometheus-operator:v0.52.1
    3. あたりを変更すればよさそう
  2. Exec KRM Functions + Shell Scriptで ${ここを変える} を変更する
    1. バージョンは別のファイルなどで管理する必要がある

あたりがパッと思いつきました。

kustomizeの機能にもうちょっといい感じのがあるかもです。

@oke-py
Copy link
Member Author

oke-py commented Nov 22, 2021

案1はkustomizeのpatchesを使う想定ですね。
一見それでよさそうですが、CRDに対するフィールド追加も反映しておいた方が無難な気がします。

のdiffをとってみました。descriptionの改行位置が変更されてノイズが多いですが。
https://gist.github.com/oke-py/529731a196b8b11738f11a703861e6fe

@riita10069
Copy link
Member

リリースのコミット見るとこんな感じですね。

bundle.yaml

@riita10069
Copy link
Member

bundle.yamlかなりでかいし、kptで管理してくれればいいのにとか思ったりしないでもないですね〜。

Issue立ててみようかな

@oke-py
Copy link
Member Author

oke-py commented Nov 22, 2021

ひとまずは #1229 の方法でいいかも

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

No branches or pull requests

3 participants