Owen Williams
Owen Williams
due to the risk of instability, we are going to put off merging this until ~Q1 or so.
Design work will be needed for the tsdb changes -- determining how to query data that was written with old versions of prometheus will be tricky, for instance if an...
https://github.com/prometheus/common/pull/537 in progress (name validation, exposition parsers)
https://github.com/ywwg/common/pull/2 in draft (write-side content negotiation)
> @ywwg I guess that means we have to revisit #13271. I didn't remember to check when I reviewed it. Maybe it already works, but we should add tests. noted
> @ywwg I guess that means we have to revisit #13271. I didn't remember to check when I reviewed it. Maybe it already works, but we should add tests. created...
Are GroupingLabels the list inside the parens in "sum by (labels) (foo{})"? Hmm yes that will have to support quoting as well
From other discussions, I understand that this issue has been raised before and value updates are something Prometheus chose not to support because it's out of scope for a monitoring...
Are you in the CNCF slack? https://communityinviter.com/apps/cloud-native/cncf We can discuss the work in the #prometheus-utf8-dev channel there
> FWIW I've implemented the same `storageClassName` fix and it works +1 Does this mean the issue is resolved? Or are you confirming that the fix works? :)