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

Add K8s 1.31 and OCP 4.17 support for powermax driver #349

Closed
wants to merge 13 commits into from

Conversation

abhi16394
Copy link
Contributor

Description

A few sentences describing the overall goals of the pull request's commits.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #

Checklist:

  • Have you run format,vet & lint checks against your submission?
  • Have you made sure that the code compiles?
  • Did you run the unit & integration tests successfully?
  • Have you maintained at least 90% code coverage?
  • Have you commented your code, particularly in hard-to-understand areas
  • Have you done corresponding changes to the documentation
  • Did you run tests in a real Kubernetes cluster?
  • Backward compatibility is not broken

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

tdawe and others added 13 commits October 11, 2024 16:44
Fix PowerMax Integration test failures
PowerMax Integration test failures
Fix Powermax Integration test failures
* get snapshot size from vol size

* Update controller.go

* add log message

* fix quote mark

* %d instead of %s

* debug log

* dumb

* use utkarsh math

* helm fixes

* get rid of hardcoded ns

* Update createFromSnap.yaml

* no more hardcoded ns

* hardcoded ns bad

* hard coded n s

* kubectl exec syntax update

* kubectl exec syntax update

* add note about dependent repos

* get rid of debug printout

* fix unit test target

* fmt fixes

* controller fmt

* gofumpt

* remove debug logs

* undo makefile changes

* Update pvc0.yaml

---------

Co-authored-by: root <[email protected]>
Update go version to 1.23.2
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.

6 participants