Closed markusthoemmes closed 4 years ago
/approve /lgtm
Makes a lot of sense. I know we're not really going to use this operator's pkg as a library, but aligning with the new operator is nice.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: aliok, markusthoemmes
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Proposed Changes
To align with https://github.com/knative-sandbox/operator/pull/21 and to be able to use this controller as a "library" (to compile multiple operators into one binary), this moves the flag definition to the main.go file and passes the built config through.
Release Note
/assign @jcrossley3 @houshengbo