cluster-api-provider-bringyourownhost
cluster-api-provider-bringyourownhost copied to clipboard
Observability for BYOH - Task 1, 2
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):
Fixes #
Additional information Observability for BYOH
Special notes for your reviewer
@Shubbhang351, you must sign our contributor license agreement before your changes are merged. Click here to sign the agreement. If you are a VMware employee, read this for further instruction.
Thanks @shubham14bajpai for the changes, If you are working on this, then can you please raise the PR ?
Hey @shivi28, @Shubbhang351 was working on this for a hackathon, if he no longer has cycles to finish this I could take this up :slightly_smiling_face:
@shubham14bajpai, my apologies for incorrectly pinging you. But if you have bandwidth and you are interested, then feel free to pick this issue. :) . @Shubbhang351 let us know if you are still interested to work on it.
Hi @shivi28, This PR can be merged in the target branch(i.e. feature-observability-branch). Later, this requires reprioritizing and re-consensus of this feature before pushing it to the main. We would need actual use cases where this feature might add value.
Sure @dharmjit , so should we aim to merge it in target branch now ?