Skip to content

Update go.mod/go.sum for crypto#4697

Merged
Ganiredi merged 2 commits into
aws:mainfrom
Ganiredi:main
Jun 17, 2026
Merged

Update go.mod/go.sum for crypto#4697
Ganiredi merged 2 commits into
aws:mainfrom
Ganiredi:main

Conversation

@Ganiredi

@Ganiredi Ganiredi commented Jun 4, 2026

Copy link
Copy Markdown
Member

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@Ganiredi

Ganiredi commented Jun 4, 2026

Copy link
Copy Markdown
Member Author

/approve
/unhold

@eks-distro-bot

Copy link
Copy Markdown
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Ganiredi

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

@mahakagi

Copy link
Copy Markdown
Member

/lgtm

@eks-distro-bot

Copy link
Copy Markdown
Collaborator

New changes are detected. LGTM label has been removed.

The previous commit (827ff1e) ran a repo-wide find/replace that rewrote
x/crypto version strings INSIDE 14 coredns patch files, creating
version<->hash mismatches and broken diff context. This caused
'git am' to fail during the Prow build (patch does not apply at go.sum:423).

Patch files are pre-generated diffs and must not be edited by dependency
bumps. Restored all 14 from the last good state (166e963); they now match
upstream/main. go.mod/go.sum namesake changes are unaffected.
@Ganiredi Ganiredi merged commit 3964b35 into aws:main Jun 17, 2026
23 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants