-
Notifications
You must be signed in to change notification settings - Fork 83
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
Relabeling to release for precreatednamespaces and also namespaceFreeFormEntry #1887
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems fine. I discussed with with Vishnu yesterday and it seems like this would be a good solution.
We need tests against prometheus-config-configmap.yaml for this new optional block.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rujhan-arora-astronomer please pip install pre-commit
and then run pre-commit install && pre-commit run --all-files
from within the astronomer/astronomer repository and commit the changes.
Done! |
7b7fa90
to
6c0bfc9
Compare
@danielhoherd Made some changes. Plz have a look |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…FormEntry (#1887) * Relabeling to release for precreatednamespaces * Added same config to namespaceFreeFormEntry too. Also wrote tests * Added same config to namespaceFreeFormEntry too. Also wrote tests for this new template * Run pre-commit * Run pre-commit and took merge * Run pre-commit and took merge
…FormEntry (#1887) * Relabeling to release for precreatednamespaces * Added same config to namespaceFreeFormEntry too. Also wrote tests * Added same config to namespaceFreeFormEntry too. Also wrote tests for this new template * Run pre-commit * Run pre-commit and took merge * Run pre-commit and took merge
…1897) * Bump pre-commit, requirements (#1841) * Bump pre-commit, requirements * add tomli * Add exceptiongroup * Improve 'make clean' * Update a bunch of dev components * Allow configuring kubernetes-apiservers with insecure_skip_verify (#1881) * update vendor packages (#1885) * bump ap-grafana 8.5.22 -> 8.5.24 * bump ap-kube-state 2.7.0-1 -> 2.8.2 * bump ap-prometheus 2.37.6 -> 2.37.8 * Relabeling to release for precreatednamespaces and also namespaceFreeFormEntry (#1887) * Relabeling to release for precreatednamespaces * Added same config to namespaceFreeFormEntry too. Also wrote tests * Added same config to namespaceFreeFormEntry too. Also wrote tests for this new template * Run pre-commit * Run pre-commit and took merge * Run pre-commit and took merge * Mask sensitive data in request_uri and referrer in nginx ingress logs (#1889) * When manualNamespaceNames is enabled, attach labels the same was as done for pre-created-ns (#1892) * update images (#1879) * update nginx packages 1.23.4 -> 1.24.0 * update postgresql version 15.2.0 -> 15.2.0-1 * update scan images * update cli install and default backend service * update cli install service * include worker pods for metrics (#1895) * bump houston version for graphql subscriptions testing * elasticsearch and kibana 7.17.10 (#1896) * Update Chart.yaml Co-authored-by: Daniel Hoherd <daniel.hoherd@gmail.com> * airflow-chart 1.8.5, commander 0.32.4 (#1899) * nginx ingress controller 1.7.1 (#1900) * bump houston from 0.32.9 to 0.32.10 --------- Co-authored-by: Daniel Hoherd <daniel.hoherd@gmail.com> Co-authored-by: pgvishnuram <81585115+pgvishnuram@users.noreply.github.com> Co-authored-by: Rujhan Arora <95576577+rujhan-arora-astronomer@users.noreply.github.com>
Description
For pre-created namespaces pods, Relabeling and storing the release names in
release
tag, so that they can be fetched by the query made from houston, under metrics tabRelated Issues
4650
Testing
Kind cluster locally. Also, spinning up a separate cluster.
Merging
0.32.1