nv-morpheus / Morpheus

Morpheus SDK
Apache License 2.0
310 stars 119 forks source link

[FEA]: Standardize usage of environment variables for constructor args of services #1701

Open mdemoret-nv opened 1 month ago

mdemoret-nv commented 1 month ago

Is this a new feature, an improvement, or a change to existing functionality?

Improvement

How would you describe the priority of this feature request

Medium

Please provide a clear description of problem this feature solves

Many of our service wrappers implement different strategies for how args vs environment variables are handled for things like API keys. This causes extra testing and non-uniform functionality.

Describe your ideal solution

Standardize using a common class and pattern. Only the ABC should be tested and all implementations can ignore this functionality.

Additional context

No response

Code of Conduct