[kube-prometheus-stack] Add instance label information to the prometheusrule description

[daviderli614]

What this PR does / why we need it

I found that the description in the prometheusrule of NodeHighNumberConntrackEntriesUsed and node-network lacks the instance label. Adding this label helps to know the specific alarm instance.

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

• fixes #

Special notes for your reviewer

Checklist

• DCO signed
• Chart Version bumped
• Title of the PR starts with chart name (e.g. [prometheus-couchdb-exporter])

[GMartinez-Sisti]

Hi @daviderli614, please check the header of the files you are changing. There is a specific procedure to update these.

[daviderli614]

Hi @daviderli614, please check the header of the files you are changing. There is a specific procedure to update these.

@GMartinez-Sisti
I modified kube-prometheus, what do I need to modify here next and then run sync_prometheus_rules.py ?
I need to modify this repository? https://github.com/kubernetes-monitoring/kubernetes-mixin

[jkroepke]

Yeah that should be sufficient

[daviderli614]

I want to wait until kube-prometheus is merged before modifying here.

[jkroepke]

Yeah, that mandatory. Otherwise, the changes from this PR may get reverted by scripts.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions.

[stale]

This issue is being automatically closed due to inactivity.