Bump Dependencies

[kensipe]

Summary:

controller-runtime to 0.5.1
kubectl to 0.17.3
controller-tools v0.2.6

Closely tied to: kudobuilder/kudo#1420
In fact the PR had more affect on testing and kuttl than on the rest of kudo.

Gotchas:

For some harness tests, I had to edit our test CRDs like:

-  name: mycrds.mycrd.k8s.io
+  name: mycrds.mycrd.example.com

With new kubectl version, the original name had multiple problems:

• names ending with k8s.io are restricted and now require an api-approved.kubernetes.io annotation pointing to a PR which introduced them
• adding this annotation proved challenging since we then see a new transition condition with the reason: ApprovedAnnotation thus bringing the number of status.condition s to three. These conditions would sometimes arrive out of order, however, test harness does not support the same order of elements when comparing two arrays, thus making the tests flaky. See this thread for more details.

Signed-off-by: Ken Sipe kensipe@gmail.com

[zen-dog]

Looks awfully familiar 🚢