Tags: getoutreach/localizer
Tags
fix(stencil): upgrade modules (2025-02-14) (#350) * build(deps): upgrade github.com/go-git/go-git/v5 from v5.12.0 to v5.13.2 * build(deps): upgrade google.golang.org/protobuf from v1.35.1 to v1.36.5 * build(deps): upgrade github.com/getoutreach/gobox from v1.102.0 to v1.103.1 --------- Co-authored-by: getoutreach-ci-1[bot] <95656460+getoutreach-ci-1[bot]@users.noreply.github.com> Co-authored-by: Mark Lee <[email protected]>
fix: logging and pf improvements, skip namespaces & non-ClusterIP svcs ( #249) * fix: logging and pf improvements, skip namespaces & non-ClusterIP svcs This PR contains a number of reliability and UX improvements for using localizer in a larger cluster. * When creating a port-forward, we don't pass a stop channel anymore. This was previously passed `ctx.Done()` which would silently stop the port-forward on ^C. This caused invalid "use of closed connection" logging that was confusing to the user but also possibly could end up in a port-forward not being fully stopped. * When shutting down, pass a temporary context that has a timeout of 30 seconds. This ensures that `/etc/host` modifications, ip pool cleanups, and other shutdown functions properly finish instead of possibly being terminated midway through during normal shutdown. * Enabled delibird to remove tracing/metrics information being sent by default in localizer binaries. This is a OSS friendly way of us getting telemetry by logging to disk instead, which a user can opt to send us when reporting bugs (more on that in future work!) * Upgraded all dependencies, including `client-go`. We still need a fork, but thankfully our fork is a single line now so it's much more likely we can upstream the change we made (the other change was upstreamed a year ago by another user!) * Exposes `stderr` from the port-forwarder to the console with a colored prefix. This makes it easier to tell when a port-forward was busted but localizer didn't detect it (or if it did, why it failed!) * Skipped namespaces by default (`kube-system`) and enabled the user to pass `--skip-namespace` to provide more (helps #212). * Skip non-clusterIP services. These aren't addressable in the cluster, so we shouldn't create a tunnel for them. This helps with `NodePort` and `LoadBalancer` service overhead that some larger clusters may have. * Reduced logging to be easier to read, while keeping a lot of helpful logs in the `debug` level. Defaults to `info` logging instead of `debug` logging (pass `--log-level debug` for that!) * make linter happy and shutdown easier to read
PreviousNext