neondatabase / autoscaling

Postgres vertical autoscaling in k8s
Apache License 2.0
166 stars 21 forks source link

util: Remove BuildGitInfo #1109

Closed sharnoff closed 1 month ago

sharnoff commented 1 month ago

We already have other ways of determining the version at runtime (namely: the tag associated with the images), and this just adds needless complication to our dockerfiles.

github-actions[bot] commented 1 month ago

Merging this branch will increase overall coverage

Impacted Packages Coverage Δ :robot:
github.com/neondatabase/autoscaling/autoscale-scheduler/cmd 0.00% (ø)
github.com/neondatabase/autoscaling/autoscaler-agent/cmd 0.00% (ø)
github.com/neondatabase/autoscaling/pkg/agent 0.00% (ø)
github.com/neondatabase/autoscaling/pkg/plugin 4.98% (+0.01%) :thumbsup:
github.com/neondatabase/autoscaling/pkg/util 17.23% (+0.56%) :thumbsup:

Coverage by file ### Changed files (no unit tests) | Changed File | Coverage Δ | Total | Covered | Missed | :robot: | |--------------|------------|-------|---------|--------|---------| | github.com/neondatabase/autoscaling/autoscale-scheduler/cmd/main.go | 0.00% (ø) | 29 (-1) | 0 | 29 (-1) | | | github.com/neondatabase/autoscaling/autoscaler-agent/cmd/main.go | 0.00% (ø) | 38 (-1) | 0 | 38 (-1) | | | github.com/neondatabase/autoscaling/pkg/agent/dumpstate.go | 0.00% (ø) | 48 | 0 | 48 | | | github.com/neondatabase/autoscaling/pkg/plugin/dumpstate.go | 0.00% (ø) | 78 | 0 | 78 | | | github.com/neondatabase/autoscaling/pkg/plugin/state.go | 0.00% (ø) | 398 (-3) | 0 | 398 (-3) | | | github.com/neondatabase/autoscaling/pkg/util/buildinfo.go | 0.00% (ø) | 0 (-8) | 0 | 0 (-8) | | _Please note that the "Total", "Covered", and "Missed" counts above refer to ***code statements*** instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code._

HTML Report

Click to open