Tell us a bit about yourself before your first pull request. A couple of sentences is plenty.
Who are you?
Sanjeev Verma (@milansanjeev), working at Expedia Group on eg-observe — a paved-road Helm chart that wraps upstream HyperDX for EG's internal observability platform (ClickHouse-backed traces/logs, plus an Okta SSO bridge).
I was already vouched by @jordan-simonovski in #3048 (vouch list PR #3049, merged). @milansanjeev is on .github/VOUCHED.td on main.
What would you like to work on?
#3047 — PromQL proxy preserves Connection host path prefixes (VictoriaMetrics cluster /select/<accountID>/prometheus). Conflicts are resolved and review comments are addressed.
That PR still has the needs-vouch label even though I am on the vouch list. The Vouch workflow only addsneeds-vouch when a PR is opened; it does not strip the label after a later /vouch. I cannot remove labels on hyperdxio/hyperdx from the fork (403).
Could a maintainer drop needs-vouch from #3047 so it can enter the review queue? A /vouch @milansanjeev on this issue is also welcome if anything still needs a refresh.
Related: #3046
Anything we can help with?
Please remove the stale needs-vouch label on #3047. Happy to rebase or answer anything else on that PR.
Tell us a bit about yourself before your first pull request. A couple of sentences is plenty.
Who are you?
Sanjeev Verma (@milansanjeev), working at Expedia Group on eg-observe — a paved-road Helm chart that wraps upstream HyperDX for EG's internal observability platform (ClickHouse-backed traces/logs, plus an Okta SSO bridge).
I was already vouched by @jordan-simonovski in #3048 (vouch list PR #3049, merged).
@milansanjeevis on.github/VOUCHED.tdonmain.What would you like to work on?
#3047 — PromQL proxy preserves Connection host path prefixes (VictoriaMetrics cluster
/select/<accountID>/prometheus). Conflicts are resolved and review comments are addressed.That PR still has the
needs-vouchlabel even though I am on the vouch list. The Vouch workflow only addsneeds-vouchwhen a PR is opened; it does not strip the label after a later/vouch. I cannot remove labels onhyperdxio/hyperdxfrom the fork (403).Could a maintainer drop
needs-vouchfrom #3047 so it can enter the review queue? A/vouch @milansanjeevon this issue is also welcome if anything still needs a refresh.Related: #3046
Anything we can help with?
Please remove the stale
needs-vouchlabel on #3047. Happy to rebase or answer anything else on that PR.