add tamilmani1989 to azure group

[tamilmani1989]

I have been working on integration of cilium in AKS/Azure and would like to volunteer for any work related to this area

[aanm]

@tamilmani1989 Thank you for your offer to volunteer as a reviewer for our group. We appreciate your enthusiasm and willingness to contribute.

While we are actively looking for new reviewers, we would look more specifically individuals who have already contributed and have some experience in the area. For example, contributions that could help demonstrate your motivation and build experience might include updating the current golang Azure dependencies which are deprecated. This task would require re-gaining familiarity with the Azure integration in Cilium and would make a meaningful impact on the project. The upstream GH issue is cilium/cilium#36869 if you would like to give it a go.

Thank you

[joestringer]

I see that both @tamilmani1989 and @wedaly have requested to join the reviewer groups for this area, and I think you're probably both in a similar situation. Needless to say, having software engineers working for Azure being involved in the azure codowner group sounds like a great fit :) .

Similar to my feedback on some of the other recent proposals, the primary question to me is just whether the existing codeowners for the area feel confident that we can continue to provide consistent reviews in the area with the extended membership. This expectation is outlined in the contributor ladder for reviewers regarding track record. I don't doubt your ability to make useful contributions in this area, so demonstrating this with PRs or reviews in the area should hopefully be a fairly straightforward operation. If you can share links to those contributions then I think that can also speak to the question that @aanm is raising.

[wedaly]

pointing to a few of the contributions @tamilmani1989 has made in this area:

• multiple PRs to consolidate/cleanup the Azure IPAM docs:
  • doc: Add Azure CNI Powered by cilium as external installer cilium#28286
  • doc: Update recommended way for installing cilium on AKS cilium#28910
  • doc: Rework the AKS tabs so that only instructions for BYOCNI remain.  cilium#28933
• helping to investigate nil ipconfig from NRP (it's not necessarily obvious from the PR comments, but he helped behind the scenes tracking down the reason for this behavior) Set available addresses when Azure returns nil IP configs cilium#36379 (comment)

Azure SIG doesn't have a huge amount of activity, but I think it would be very beneficial to the project to have TM as a reviewer. It's not an understatement to say that he's the world expert in Azure CNI.