Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

grafana-agent-operator/0.43.0 package update #28140

Merged
merged 2 commits into from
Sep 15, 2024

Conversation

octo-sts[bot]
Copy link
Contributor

@octo-sts octo-sts bot commented Sep 11, 2024

@octo-sts octo-sts bot added request-version-update request for a newer version of a package automated pr P1 This label indicates our scanning found High, Medium or Low CVEs for these packages. labels Sep 11, 2024
Copy link
Contributor Author

octo-sts bot commented Sep 11, 2024

Open AI suggestions to solve the build error:

The error message is: "panic: pattern \"GET /debug/pprof/\" (registered at net/http/pprof/pprof.go:100) conflicts with pattern \"/debug/pprof/delta_heap\" (registered at github.com/grafana/pyroscope-go/[email protected]/http/pprof/pprof.go:24):\n    GET /debug/pprof/ matches fewer methods than /debug/pprof/delta_heap, but has a more general path pattern\n\n goroutine 1 [running]:\n net/http.(*ServeMux).register(...)\n     net/http/server.go:2797\n net/http.HandleFunc({0x40001666d8?, 0x18ee69c?}, 0x16598c0?)\n     net/http/server.go:2791 +0x9c\n net/http/pprof.init.0()\n     net/http/pprof/pprof.go:100 +0x94"

1. Identify conflicting routes in your codebase.
2. Modify one of the conflicting routes to ensure unique paths.
3. Update the registration of the modified route.
4. Rebuild and test the application to ensure the conflict is resolved.

@mamccorm mamccorm force-pushed the wolfictl-13429d1f-f184-431e-ab19-6bb8e801e41e branch from fb77f34 to 3a03618 Compare September 11, 2024 22:15
Copy link
Contributor Author

octo-sts bot commented Sep 11, 2024

Open AI suggestions to solve the build error:

The error message is: "panic: pattern \"GET /debug/pprof/\" (registered at net/http/pprof/pprof.go:100) conflicts with pattern \"/debug/pprof/delta_heap\" (registered at github.com/grafana/pyroscope-go/[email protected]/http/pprof/pprof.go:24): GET /debug/pprof/ matches fewer methods than /debug/pprof/delta_heap, but has a more general path pattern"

1. Identify the conflicting routes in your codebase.
2. Modify the route patterns to ensure they do not overlap.
3. Ensure that more specific routes are registered before more general ones.
4. Test the changes to confirm the conflict is resolved.

@mamccorm mamccorm force-pushed the wolfictl-13429d1f-f184-431e-ab19-6bb8e801e41e branch from 3a03618 to 602fca7 Compare September 11, 2024 23:01
Copy link
Contributor Author

octo-sts bot commented Sep 11, 2024

Open AI suggestions to solve the build error:

The error message is: "panic: pattern \"GET /debug/pprof/\" (registered at net/http/pprof/pprof.go:100) conflicts with pattern \"/debug/pprof/delta_heap\" (registered at github.com/grafana/pyroscope-go/[email protected]/http/pprof/pprof.go:24): GET /debug/pprof/ matches fewer methods than /debug/pprof/delta_heap, but has a more general path pattern"

1. Identify conflicting routes in your code.
2. Modify one of the conflicting routes to ensure unique paths.
3. Rebuild and test the application to confirm the conflict is resolved.

@kranurag7 kranurag7 force-pushed the wolfictl-13429d1f-f184-431e-ab19-6bb8e801e41e branch from 602fca7 to efe6b74 Compare September 14, 2024 06:20
@kranurag7
Copy link
Member

kranurag7 commented Sep 14, 2024

locally this package is building successfully.

[sdk] ❯ wolfictl scan ./packages/x86_64/grafana-agent-operator-0.43.0-r0.apk
🔎 Scanning "./packages/x86_64/grafana-agent-operator-0.43.0-r0.apk"
✅ No vulnerabilities found

I've rebased it to main, the CI should be green hopefully.

EDIT: it's the testing pipeline which is failing.

Copy link
Contributor Author

octo-sts bot commented Sep 14, 2024

Open AI suggestions to solve the build error:

The error message is: "panic: pattern \"GET /debug/pprof/\" (registered at net/http/pprof/pprof.go:100) conflicts with pattern \"/debug/pprof/delta_heap\" (registered at github.com/grafana/pyroscope-go/[email protected]/http/pprof/pprof.go:24): GET /debug/pprof/ matches fewer methods than /debug/pprof/delta_heap, but has a more general path pattern"

1. Identify the conflicting routes in your codebase.
2. Modify the route patterns to ensure they do not overlap.
3. Ensure that more specific routes are registered before more general ones.
4. Test the changes to confirm the conflict is resolved.

@Dentrax
Copy link
Member

Dentrax commented Sep 15, 2024

The panic because of probably: grafana/agent#6972

Copy link
Contributor

Package grafana-agent-operator: Click to expand/collapse

Package grafana-agent-operator:

.PKGINFO metadata:

  (
  	"""
  	# Generated by melange
  	pkgname = grafana-agent-operator
- 	pkgver = 0.42.0-r1
+ 	pkgver = 0.43.0-r0
  	arch = x86_64
- 	size = 52150204
+ 	size = 52314792
  	origin = grafana-agent-operator
  	pkgdesc = Grafana Agent Operator is a Kubernetes operator for the static mode of Grafana Agent. It makes it easier to deploy and configure static mode to collect telemetry data from Kubernetes resources.
  	url = 
- 	commit = faa4fff2a135c1aaeb886bd21902ca05ea5dfbcc
- 	builddate = 1721839521
+ 	commit = 56090656810ece20138c22f3908f419f873f8885
  	license = Apache-2.0
- 	provides = cmd:grafana-agent-operator=0.42.0-r1
- 	datahash = 4894ec4afee2df9747e1c27dba3720d0c084bd3837a7c07e1feb3d226fdbbdf0
+ 	provides = cmd:grafana-agent-operator=0.43.0-r0
+ 	datahash = 9408ca43cb41c7f62a4d7338485a6059ca8dd73df51ca4006dfcf09122484f7f
  	"""
  )

Modified: /usr/bin/grafana-agent-operator

bincapz found differences: Click to expand/collapse

Moved: grafana-agent-operator/var/lib/db/sbom/grafana-agent-operator-0.42.0-r1.spdx.json -> /tmp/wolfictl-apk-3169304820/grafana-agent-operator/var/lib/db/sbom/grafana-agent-operator-0.43.0-r0.spdx.json (similarity: 0.99)

@kranurag7 kranurag7 enabled auto-merge (squash) September 15, 2024 19:37
@Dentrax Dentrax self-assigned this Sep 15, 2024
@kranurag7 kranurag7 merged commit 13c058e into main Sep 15, 2024
10 checks passed
@kranurag7 kranurag7 deleted the wolfictl-13429d1f-f184-431e-ab19-6bb8e801e41e branch September 15, 2024 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automated pr P1 This label indicates our scanning found High, Medium or Low CVEs for these packages. request-version-update request for a newer version of a package service:bincapz/pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants