Jan Fajerski

Results 85 comments of Jan Fajerski
trafficstars

> Sorry for confusion. An almost similar PR was created recently, see #14958, and some discussion happened over there. The discussion over there is relevant, but the changes should happen...

@ansh-devs yeah I think you can contribute to this. Please reach out to @SuperQ, he might be working on this already. Join us on slack for a quicker discussion.

yes this would be great to have in 3.0. Ideally along with the fixes @machine424 has proposed for #12858 and #12436.

@machine424 Is there anything else to do here? Iiuc this issue can be closed, right?

@pawarpranav83 Could uou resolve the conflicts here, then we should be good to go.

Seems like https://github.com/prometheus/prometheus/issues/12631 is a prerequisite for this? Could use case specific documentation and respective example configs be a path forward here?

Thanks for the update. I'll remove this from the Prometheus 3 project then.

We should also consider the behavior of the `@` modifier. Based on a quick test this also accepts future timestamps and returns data as described. Going by the documentation its...

> How would someone figure out what change is required? This PR introduces and additional job that `needs` the build_all job and it fails or succeeds based on the result...