Skip to content

✨ bump to CAPI 1.7.2#171

Merged
kranurag7 merged 1 commit into
mainfrom
kr/bump-capi-to-1.7.2
Jun 7, 2024
Merged

✨ bump to CAPI 1.7.2#171
kranurag7 merged 1 commit into
mainfrom
kr/bump-capi-to-1.7.2

Conversation

@kranurag7

@kranurag7 kranurag7 commented May 31, 2024

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:
this commit bumps cluster API to 1.7.2 version.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #170

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

TODOs:

  • squash commits
  • include documentation
  • add unit tests

@kranurag7
kranurag7 requested a review from janiskemper May 31, 2024 13:28

@janiskemper janiskemper left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is image/builder relevant as well?

@kranurag7
kranurag7 force-pushed the kr/bump-capi-to-1.7.2 branch from 859dfda to 5b427b3 Compare June 1, 2024 11:28
@janiskemper

Copy link
Copy Markdown
Member

is image/builder relevant as well?

@kranurag7 what's your plan with the image/builder file? There seem to be some outdated dependencies?!

@kranurag7

Copy link
Copy Markdown
Contributor Author

@kranurag7 what's your plan with the image/builder file? There seem to be some outdated dependencies?!

I'll create another PR for it and that should be something that's automatically updated by renovate bot. See #172

@kranurag7
kranurag7 force-pushed the kr/bump-capi-to-1.7.2 branch from 5b427b3 to 81a090e Compare June 6, 2024 05:08

@janiskemper janiskemper left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. plz squash

Signed-off-by: kranurag7 <anurag.kumar@syself.com>

remove deprecated logger

Signed-off-by: kranurag7 <anurag.kumar@syself.com>
@kranurag7
kranurag7 force-pushed the kr/bump-capi-to-1.7.2 branch from 81a090e to 5a30801 Compare June 6, 2024 08:37
@kranurag7
kranurag7 requested a review from janiskemper June 6, 2024 08:47
@kranurag7
kranurag7 merged commit 0eaeb0a into main Jun 7, 2024
@kranurag7
kranurag7 deleted the kr/bump-capi-to-1.7.2 branch June 7, 2024 07:11
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.

bump to cluster API 1.7.2

2 participants