Skip to content

Usr/sreekb/mount creds port#418

Merged
bharathsreekanth merged 18 commits into
usr/spark/mount-credentialsfrom
usr/sreekb/mount-creds-port
Jan 27, 2025
Merged

Usr/sreekb/mount creds port#418
bharathsreekanth merged 18 commits into
usr/spark/mount-credentialsfrom
usr/sreekb/mount-creds-port

Conversation

@bharathsreekanth

@bharathsreekanth bharathsreekanth commented Jan 22, 2025

Copy link
Copy Markdown
Contributor

Description

Changes to powermax driver and reverseproxy to read from mounted secret.

DISCLAIMER:

  • Basic functionality of the feature has been tested with these changes for both mounted secret and config map approach.
  • This PR is to merge to feature branch and NOT to main. Extensive testing and additional tests will come soon, but the goal of this PR is to get functionality in feature branch so parallel work can proceed.

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?

Manual testing only thus far, with mounted secret as well as regression testing with existing config map approach.
Tested with multiple arrays and unisphere endpoints.
Tested with updating the secrets and configmap on the fly to validate the watcher.

Comment thread csireverseproxy/pkg/config/config.go
Comment thread csireverseproxy/main.go Outdated
Comment thread csireverseproxy/main.go
Comment thread service/service.go
Comment thread csireverseproxy/pkg/config/config.go
@tdawe

tdawe commented Jan 27, 2025

Copy link
Copy Markdown
Contributor

Approving. Checks are failing but this is going into a feature branch. Unit test coverage will need to be updated prior to merging to main.

@bharathsreekanth

Copy link
Copy Markdown
Contributor Author

Approving. Checks are failing but this is going into a feature branch. Unit test coverage will need to be updated prior to merging to main.

That is correct. UTs are being worked on will be addressed before merge to main. These changes are to facilitate having a working solution in a feature branch for parallel development.

@bharathsreekanth bharathsreekanth merged commit d2ee27b into usr/spark/mount-credentials Jan 27, 2025
@bharathsreekanth bharathsreekanth deleted the usr/sreekb/mount-creds-port branch January 27, 2025 14:48
lukeatdell pushed a commit that referenced this pull request Feb 28, 2025
Powermax driver and reverseproxy changes for mounted secret
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.

4 participants