Skip to content

Bump k8s to 1.35 along with go deps#1106

Closed
CodesbyUnnati wants to merge 1 commit intodevelopfrom
bump-k8s-1.35
Closed

Bump k8s to 1.35 along with go deps#1106
CodesbyUnnati wants to merge 1 commit intodevelopfrom
bump-k8s-1.35

Conversation

@CodesbyUnnati
Copy link
Copy Markdown
Member

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Does this PR introduce a user-facing change?


Additional Notes for your reviewer:

Review Checklist:
  • Follows the developer guidelines
  • Relevant tests are added or updated
  • Relevant docs in this repo added or updated
  • Relevant carvel.dev docs added or updated in a separate PR and there's
    a link to that PR
  • Code is at least as readable and maintainable as it was before this
    change

Additional documentation e.g., Proposal, usage docs, etc.:


Signed-off-by: Unnati Mishra <unnati.mishra@broadcom.com>
@CodesbyUnnati
Copy link
Copy Markdown
Member Author

CodesbyUnnati commented Jan 14, 2026

Depends on #1107

@github-project-automation github-project-automation Bot moved this to Closed in Carvel Jan 17, 2026
@CodesbyUnnati CodesbyUnnati reopened this Jan 29, 2026
@github-project-automation github-project-automation Bot moved this from Closed to In Progress in Carvel Jan 29, 2026
@CodesbyUnnati CodesbyUnnati mentioned this pull request Jan 29, 2026
5 tasks
@Siegfriedk
Copy link
Copy Markdown
Contributor

I wanted to create a new PR switching this dependencies:

github.com/k14s/difflib v0.0.0-20240118055029-596a7a5585c3
github.com/k14s/ytt v0.36.0

To the migrated version under

https://github.com/carvel-dev/difflib
https://github.com/carvel-dev/ytt

Any chance you could do that here instead? Otherwise I will just wait for this to be merged. Waiting for the k8s 1.35 too

@CodesbyUnnati
Copy link
Copy Markdown
Member Author

I wanted to create a new PR switching this dependencies:

github.com/k14s/difflib v0.0.0-20240118055029-596a7a5585c3
github.com/k14s/ytt v0.36.0

To the migrated version under

https://github.com/carvel-dev/difflib https://github.com/carvel-dev/ytt

Any chance you could do that here instead? Otherwise I will just wait for this to be merged. Waiting for the k8s 1.35 too

Which version of difflib you are talking about? I only see only version here and here.

For ytt, last week ytt's new release was publish. So, I will update that in the go.mod file.

@Siegfriedk
Copy link
Copy Markdown
Contributor

@CodesbyUnnati my main concern is the migration to carvel-dev to get rid of a cyclic dependency issue between the arvel-dev repository and the original one.

I'm not sure if you can just reference the carvel-dev one directly and it would pull it. At least the k14s redirects to carvel so the reference change might just work

@CodesbyUnnati
Copy link
Copy Markdown
Member Author

github.com/carvel-dev/difflib

@Siegfriedk Thanks for pointing out! I am using this PR #1108 to update the versions.

@CodesbyUnnati
Copy link
Copy Markdown
Member Author

PR #1108 has been merged!

@github-project-automation github-project-automation Bot moved this from In Progress to Closed in Carvel Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants