Skip to content

chore: update go-common to v1.0.165 and remove stale ReportStats field#15

Open
jhaynie wants to merge 1 commit intomainfrom
task/hadron-monitoring
Open

chore: update go-common to v1.0.165 and remove stale ReportStats field#15
jhaynie wants to merge 1 commit intomainfrom
task/hadron-monitoring

Conversation

@jhaynie
Copy link
Member

@jhaynie jhaynie commented Mar 9, 2026

Summary

  • Update go-common dependency to v1.0.165 for monitoring proto changes
  • Remove deprecated ReportStats field that was replaced by the new monitoring report structure

What Changed

  • Updated go-common to v1.0.165 in go.mod and go.sum
  • Removed references to the deprecated ReportStats field across 3 files

Related PRs

  • ion: Infrastructure monitoring WebSocket hub and auth
  • hadron: Container log streaming and metrics
  • app: Monitoring UI

Summary by CodeRabbit

  • Chores
    • Updated Go toolchain from version 1.26.0 to 1.26.1.
    • Upgraded dependencies including go-common, OpenTelemetry modules, system packages, and protobuf libraries to latest compatible versions.
    • Removed unnecessary explicit configuration default.

- Update go-common for cumulative metric totals in ContainerMetrics proto
- Remove stale ReportStats field from stack.go after monitor refactor
@coderabbitai
Copy link

coderabbitai bot commented Mar 9, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3dc2a6ab-17d0-4c76-bcf7-ed4a3c42ef40

📥 Commits

Reviewing files that changed from the base of the PR and between 292241f and a404b3d.

⛔ Files ignored due to path filters (1)
  • go.sum is excluded by !**/*.sum
📒 Files selected for processing (2)
  • go.mod
  • internal/stack/stack.go
💤 Files with no reviewable changes (1)
  • internal/stack/stack.go
📜 Recent review details
🔇 Additional comments (5)
go.mod (5)

3-3: Go 1.26.1 is a valid release. No action needed.


6-6: go-common v1.0.165 version confirmed.

The updated dependency version has been verified to exist and is accessible. No breaking changes are evident in the codebase related to this version bump.


51-53: No action needed—these are indirect dependencies.

The Google packages are valid versions and exist in the public repositories. However, the codebase does not contain any .proto files or generated protobuf code, and there are no direct imports of google.golang.org/protobuf. These packages are indirect dependencies pulled in by other packages, so manual regeneration is not required.


40-44: OpenTelemetry v1.40.0 versions are correctly configured.

The OpenTelemetry module versions (v1.40.0) have been confirmed to exist. Since these are indirect dependencies brought in transitively through other packages, no code adjustments are required in this repository—any compatibility concerns would be managed by the direct dependency maintainers.


48-48: golang.org/x/sys v0.40.0 is valid and secure.

The version v0.40.0 exists in the official release history. The Go Vulnerability Database shows no active CVEs affecting this version—the only recorded vulnerability (CVE-2022-29526) is fixed in earlier versions that v0.40.0 exceeds.


📝 Walkthrough

Walkthrough

This 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

Cohort / File(s) Summary
Dependency Updates
go.mod
Upgraded Go toolchain to 1.26.1, go-common to v1.0.165, OpenTelemetry modules to v1.40.0, golang.org/x/sys to v0.40.0, and updated Google APIs/protobuf packages to newer versions.
Configuration Adjustments
internal/stack/stack.go
Removed explicit ReportStats: false setting from Gravity client configuration, allowing the field to use its default value.
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant