chore: update go-common to v1.0.165 and remove stale ReportStats field#15
chore: update go-common to v1.0.165 and remove stale ReportStats field#15
Conversation
- Update go-common for cumulative metric totals in ContainerMetrics proto - Remove stale ReportStats field from stack.go after monitor refactor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
💤 Files with no reviewable changes (1)
📜 Recent review details🔇 Additional comments (5)
📝 WalkthroughWalkthroughThis pull request updates multiple dependencies including the Go toolchain (1.26.0 to 1.26.1), go-common (v1.0.150 to v1.0.165), OpenTelemetry modules to v1.40.0, golang.org/x/sys, and Google API/protobuf packages. Additionally, it removes an explicit ReportStats configuration from the Gravity client setup. Changes
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
Summary
ReportStatsfield that was replaced by the new monitoring report structureWhat Changed
go-commonto v1.0.165 ingo.modandgo.sumReportStatsfield across 3 filesRelated PRs
Summary by CodeRabbit