This is a follow-up to #50. I mistakenly left out the change in the ACL controller where we store the client, meaning that while the Disconnect method is called, there isn't anything available to actually disconnect. Whoops!
I have:
[x] Read and followed Crossplane's [contribution process].
[x] Run make reviewable test to ensure this PR is ready for review.
How has this code been tested
I deployed it into a test cluster and have verified that the goroutine count remains steady instead of linearly increasing
Description of your changes
This is a follow-up to #50. I mistakenly left out the change in the ACL controller where we store the client, meaning that while the
Disconnectmethod is called, there isn't anything available to actually disconnect. Whoops!I have:
make reviewable testto ensure this PR is ready for review.How has this code been tested
I deployed it into a test cluster and have verified that the goroutine count remains steady instead of linearly increasing