Skip to content

[release-v0.80.x] Security: Fix GovCloud Go module vulnerabilities (go-chi, compress, sigstore-go, cel-go, otel) - #3914

Merged
tekton-robot merged 4 commits into
tektoncd:release-v0.80.xfrom
jkhelil:fix/SRVKP-13173-govcloud-go-modules-v0.80.x
Aug 13, 2026
Merged

[release-v0.80.x] Security: Fix GovCloud Go module vulnerabilities (go-chi, compress, sigstore-go, cel-go, otel)#3914
tekton-robot merged 4 commits into
tektoncd:release-v0.80.xfrom
jkhelil:fix/SRVKP-13173-govcloud-go-modules-v0.80.x

Conversation

@jkhelil

@jkhelil jkhelil commented Aug 12, 2026

Copy link
Copy Markdown
Member

Changes

Fix multiple Go module vulnerabilities identified in GovCloud FedRAMP compliance scans (SRVKP-13173, SRVKP-13178, SRVKP-13179).

CVE/Advisory Package Previous Fixed
GHSA-rjr7-jggh-pgcp / GHSA-9g5q-2w5x-hmxf / GHSA-3fxj-6jh8-hvhx github.com/go-chi/chi/v5 v5.2.5 v5.3.0
CVE-2026-49834 / GHSA-9vcr-p3rj-q5q6 github.com/sigstore/sigstore-go v1.1.4 v1.2.0
GCJH-h69q-9w9g github.com/google/cel-go v0.28.1 v0.29.0
GHSA-259r-337f-4rfw github.com/klauspost/compress v1.18.6 v1.18.7
CVE-2026-41178 / GHSA-5wrp-cwcj-q835 go.opentelemetry.io/otel v1.43.0 v1.44.0

Jira Issues

SRVKP-13173, SRVKP-13178, SRVKP-13179

Vulnerability Scan

  • Tool: govulncheck@v1.6.0, GOTOOLCHAIN=go1.26.5
  • Note: Base image CVEs (curl, libacl, glibc, etc.) require base image rebuild — not fixable from source.

Test Results

Status: ✅ All tests passed
Command: go test -short ./pkg/...

Risk Assessment

Low — All updates are patch/minor bumps to indirect Go module dependencies.

Submitter Checklist

  • Run make test lint before submitting a PR
  • Includes tests (if functionality changed/added)
  • Includes docs (if user facing)
  • Commit messages follow commit message best practices

Release Notes

Security: update Go module dependencies to address GovCloud FedRAMP compliance CVEs (go-chi/chi v5.3.0, sigstore-go v1.2.0, cel-go v0.29.0, compress v1.18.7, otel v1.44.0)

🤖 Generated by CVE Fixer Workflow

Made with Cursor

Upgrade vulnerable Go module dependencies identified in GovCloud
FedRAMP compliance scans (SRVKP-13173, SRVKP-13178, SRVKP-13179):

- github.com/go-chi/chi/v5:        v5.2.5  -> v5.3.0
- github.com/klauspost/compress:   v1.18.6 -> v1.18.7
- github.com/sigstore/sigstore-go: v1.1.4  -> v1.2.0
- github.com/google/cel-go:        v0.28.1 -> v0.29.0
- go.opentelemetry.io/otel:        v1.43.0 -> v1.44.0

Addresses: GHSA-rjr7-jggh-pgcp, GHSA-9g5q-2w5x-hmxf,
GHSA-3fxj-6jh8-hvhx, CVE-2026-49834, GHSA-9vcr-p3rj-q5q6,
GCJH-h69q-9w9g, GHSA-259r-337f-4rfw, CVE-2026-41178

Jira: SRVKP-13173 SRVKP-13178 SRVKP-13179

Signed-off-by: Jawed khelil <jkhelil@redhat.com>
Assisted-by: Claude Sonnet 4.6 (via Cursor)
Co-authored-by: Cursor <cursoragent@cursor.com>
@tekton-robot tekton-robot added the do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. label Aug 12, 2026
@tekton-robot tekton-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Aug 12, 2026
Run go mod tidy && go mod vendor to sync vendor/ with the updated
go.mod entries for go-chi/chi v5.3.0, compress v1.18.7,
sigstore-go v1.2.0, cel-go v0.29.0, otel v1.44.0.

Jira: SRVKP-13173 SRVKP-13178 SRVKP-13179

Signed-off-by: Jawed khelil <jkhelil@redhat.com>
Assisted-by: Claude Sonnet 4.6 (via Cursor)
Co-authored-by: Cursor <cursoragent@cursor.com>
@tekton-robot tekton-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Aug 12, 2026
@jkhelil

jkhelil commented Aug 12, 2026

Copy link
Copy Markdown
Member Author

/release-note-none

@tekton-robot tekton-robot added release-note-none Denotes a PR that doesnt merit a release note. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Aug 12, 2026
@jkhelil

jkhelil commented Aug 12, 2026

Copy link
Copy Markdown
Member Author

/retest

4 similar comments
@jkhelil

jkhelil commented Aug 12, 2026

Copy link
Copy Markdown
Member Author

/retest

@jkhelil

jkhelil commented Aug 12, 2026

Copy link
Copy Markdown
Member Author

/retest

@jkhelil

jkhelil commented Aug 12, 2026

Copy link
Copy Markdown
Member Author

/retest

@pratap0007

Copy link
Copy Markdown
Contributor

/retest

@pratap0007 pratap0007 closed this Aug 13, 2026
@pratap0007 pratap0007 reopened this Aug 13, 2026
jkhelil and others added 2 commits August 13, 2026 11:58
Bump contrib and exporter packages to match otel core v1.44.0,
eliminating the mixed semconv/v1.40.0 + semconv/v1.41.0 schema
URL panic at pod startup:

- contrib/instrumentation/net/http/otelhttp: v0.68.0 -> v0.69.0
- contrib/instrumentation/runtime:            v0.68.0 -> v0.69.0
- exporters/otlp/otlpmetric/otlpmetricgrpc:  v1.43.0 -> v1.44.0
- exporters/otlp/otlpmetric/otlpmetrichttp:  v1.43.0 -> v1.44.0
- exporters/otlp/otlptrace:                  v1.43.0 -> v1.44.0
- exporters/otlp/otlptrace/otlptracegrpc:    v1.43.0 -> v1.44.0
- exporters/otlp/otlptrace/otlptracehttp:    v1.43.0 -> v1.44.0
- exporters/prometheus:                      v0.65.0 -> v0.66.0
- exporters/stdout/stdouttrace:              v1.43.0 -> v1.44.0

The previous commit bumped otel core to v1.44.0 but left these
packages behind. The OTel SDK panics when merging resources that
reference different schema URLs (v1.40.0 vs v1.41.0), crashing
all three operator pods at startup and causing all e2e tests
to fail.

Jira: SRVKP-13173 SRVKP-13178 SRVKP-13179

Signed-off-by: Jawed khelil <jkhelil@redhat.com>
Assisted-by: Claude Sonnet 4.6 (via Cursor)
Co-authored-by: Cursor <cursoragent@cursor.com>
Upgrade knative.dev/pkg from v0.0.0-20260406140200-cb58ae50e894
to v0.0.0-20260622140654-39ebae2ee2dc (same as operator main).

The previous otel v1.43.0->v1.44.0 bump changed resource.Default()
to use semconv/v1.41.0 (schema URL "1.41.0"), while knative.dev/pkg
was still passing semconv/v1.40.0 SchemaURL ("1.40.0") into
resource.Merge(). The OTel SDK returns ErrMergeConflictSchemaURL
on mismatch, which knative panics on — crashing all operator
pods at startup and failing all e2e tests.

The new knative.dev/pkg version imports semconv/v1.41.0, aligning
SchemaURLs across the full stack.

Jira: SRVKP-13173 SRVKP-13178 SRVKP-13179

Signed-off-by: Jawed khelil <jkhelil@redhat.com>
Assisted-by: Claude Sonnet 4.6 (via Cursor)
Co-authored-by: Cursor <cursoragent@cursor.com>
@jkhelil

jkhelil commented Aug 13, 2026

Copy link
Copy Markdown
Member Author

/approve

@tekton-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jkhelil

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jkhelil

jkhelil commented Aug 13, 2026

Copy link
Copy Markdown
Member Author

/lgtm

@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 13, 2026
@tekton-robot

Copy link
Copy Markdown
Contributor

@jkhelil: you cannot LGTM your own PR.

Details

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@pratap0007

Copy link
Copy Markdown
Contributor

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Aug 13, 2026
@tekton-robot
tekton-robot merged commit e4bc821 into tektoncd:release-v0.80.x Aug 13, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesnt merit a release note. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants