NetApp / trident

Storage orchestrator for containers
Apache License 2.0
762 stars 222 forks source link

permit specifying trident-operator resources #929

Open clementnuss opened 1 month ago

clementnuss commented 1 month ago

closes https://github.com/NetApp/trident/issues/927

Change description

Permit specifying the resources of the trident-operator deployment ## Did you add unit tests? Why not? No, minor helm chart modification ## Does this code need functional testing?

No, this just moves a configuration to the .Values

Is a code review walkthrough needed? why or why not?

No

Does this code need a note in the changelog?

trident operator helm chart: permit specifying trident-operator resources

Additional Information

clementnuss commented 1 month ago

@vivintw why did you add your commit ? it was breaking the PR.

can you consider merging this ? would have been nice to see this happening for the Oct. release, although now I doubt it will be possible

drew0ps commented 1 month ago

We get alerts because of the low hardcoded cpu limits since the version 100.2406.1. Merging this would be highly appreciated.