kudobuilder / kudo

Kubernetes Universal Declarative Operator (KUDO)
https://kudo.dev
Apache License 2.0
1.17k stars 103 forks source link

Child params in operators with dependencies failing in pkg resolver #1773

Closed alenkacz closed 3 years ago

alenkacz commented 3 years ago

What this PR does / why we need it:

Fixes #1772

alenkacz commented 3 years ago

The operator test failure is OK, it's because I've cleaned up the operators folder

asekretenko commented 3 years ago

Hi @alenkacz @kensipe, are you planning to proceed with this PR?

FWICT, upgrade-test failure is also unrelated - this particular flake is observed elsewhere in KUDO CI (I've filed https://github.com/kudobuilder/kudo/issues/1783, but most likely it is not KUDO and probably not even KUTTL)

alenkacz commented 3 years ago

@asekretenko merged :) thanks for review @kensipe I missed that you approved, appreciate it! It is such a long time I've implemented this though that I don't remember some of the implementation details :D