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

bump k8s to 1.23.1 #145

Merged
merged 2 commits into from
Jan 12, 2022
Merged

Conversation

joelanford
Copy link
Member

Signed-off-by: Joe Lanford joe.lanford@gmail.com

@coveralls
Copy link

coveralls commented Dec 22, 2021

Pull Request Test Coverage Report for Build 1688277382

  • 4 of 4 (100.0%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.1%) to 88.701%

Totals Coverage Status
Change from base Build 1579656302: 0.1%
Covered Lines: 1625
Relevant Lines: 1832

💛 - Coveralls

@varshaprasad96
Copy link
Member

@joelanford I have created a PR which does small changes in sample generation (#146). After that is merged, running make generate on this PR should update the samples and sanity-test should pass. Sorry for blocking this one!

@varshaprasad96
Copy link
Member

@joelanford I have created a PR which does small changes in sample generation (#146). After that is merged, running make generate on this PR should update the samples and sanity-test should pass. Sorry for blocking this one!

#146 PR does not do the needful of using the current version to generate samples. It needs to be done manually. I can do a follow up for this. For now, running mark test-sanity to update the license header should make the CI pass.

Signed-off-by: Joe Lanford <joe.lanford@gmail.com>
Signed-off-by: Joe Lanford <joe.lanford@gmail.com>
@varshaprasad96
Copy link
Member

Force merging this, as the apidiff is because of moving controllerutil to internal

@varshaprasad96 varshaprasad96 merged commit 98d5f4f into operator-framework:main Jan 12, 2022
@joelanford joelanford deleted the k8s-1.23 branch September 16, 2022 12:26
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.

None yet

3 participants