ArnoRienstra
commented
1 year ago Would be nice for us if this PR is merged, can a maintainer please have a look?
Open woutersmit opened 2 years ago
ArnoRienstra
commented
1 year ago Would be nice for us if this PR is merged, can a maintainer please have a look?
Change description
Allow overriding the namespace of the orchestrator if you need it to be deployed to a namespace other than the namespace the operator is deployed in.
Project tracking
775
Do any added TODOs have an issue in the backlog?
No
Did you add unit tests? Why not?
No. minor change in helm chart, deploys correctly to private cluster
Does this code need functional testing?
No.
Is a code review walkthrough needed? why or why not?
No.
Should additional test coverage be executed in addition to pre-merge?
No.
Does this code need a note in the changelog?
Yes. e.g.: Allow overriding the namespace of the orchestrator
Does this code require documentation changes?
No
Additional Information