fix: overwritten envFrom

[garrett361]

Issue link

What changes have been made

This PR fixes an issue I introduced in #192 which I missed as it wasn't an issue in my testing environment.

#192 gives the user the option to specify an envFrom field. The problem is that if the user also provides a ncclGdrEnvConfigMap value, then this results in another envFrom field being created which overwrites the first. I wasn't providing ncclGdrEnvConfigMap in the env I tested in, so I missed this fact.

This PR merges any provided envFrom fields.

Verification steps

Checks

• I've made sure the tests are passing.
• Testing Strategy
  • Unit tests
  • Manual tests
  • Testing is not required for this change

Now tested in an environment where I also specify ncclGdrEnvConfigMap

[dgrove-oss]

Thanks!