operators icon indicating copy to clipboard operation
operators copied to clipboard

working setup of flink via kuttl

Open kensipe opened this issue 5 years ago • 3 comments

Signed-off-by: Ken Sipe [email protected]

kensipe avatar Jul 24 '20 15:07 kensipe

Let's update this to use a recent version of flink-demo that uses KUDO's dependency feature. With that, most setup steps aren't necessary anymore, the flink-demo operator will take care of installing it's dependencies. Furthermore, it isn't necessary to change anything regarding local storage in kind (as done in 01-update-storage-class.yaml), because this is part of a default cluster. IMO, if I'm not missing something, the test could be as simple as running kubectl kudo install ../../financial-fraud/demo-operator --instance flink-demo and waiting for the flink-demo Instance to complete. Checking that the dependencies have been installed is also a great test. So it's what currently done in step 3,4,5,6, without steps 1,2. Though all of these assert in 3,4,5,6 can be done in a single step, because they should all be true once the flink-demo instance is complete.

nfnt avatar Oct 16 '20 14:10 nfnt

One thing might be a problem though: We already install O/OVs for a lot of operators as part of the test setup: https://github.com/kudobuilder/operators/blob/master/kuttl-test.yaml. We should run this test in a separate namespace to ensure that new O/OVs are installed for the flink-demo operator and its dependencies.

nfnt avatar Oct 16 '20 14:10 nfnt

@nfnt unless kuttl is configured to run in a specific namespace... a new namespace is create for each test

kensipe avatar Oct 16 '20 20:10 kensipe