Add support for controller.selectorLabels

[hanyouqing]

Proposed changes

Fix issue #3976

Checklist

Before creating a PR, run through this checklist and mark each as complete.

• I have read the CONTRIBUTING doc
• I have added tests that prove my fix is effective or that my feature works
• I have checked that all unit tests pass after adding my changes
• I have updated necessary documentation
• I have rebased my branch onto main
• I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

[jasonwilliams14]

@hanyouqing Thank you for the PR. We are currently reviewing this on our side and trying to reproduce the issue. If you could respond to our request for additional information in the issue thread you created, that would be great. Thank you.

[hanyouqing]

@hanyouqing Thank you for the PR. We are currently reviewing this on our side and trying to reproduce the issue. If you could respond to our request for additional information in the issue thread you created, that would be great. Thank you.

Thanks for your reply! I'm glad to share the reproduce steps, but I can't reproduce it with Helm. We manage the helm chart with Terraform, It might be some issues for the helm Terraform provider. Let me figure it out, if so I'll close the pr.

[hanyouqing]

Hi @jasonwilliams14, I've added some comments and how to reproduce it see the issue page #3976 (comment).
Thanks!

[lucacome]

Looks good @hanyouqing

[codecov]

Codecov Report

Merging #3977 (ddd2611) into main (2f39705) will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3977      +/-   ##
==========================================
- Coverage   51.87%   51.84%   -0.03%     
==========================================
  Files          59       59              
  Lines       16710    16710              
==========================================
- Hits         8668     8664       -4     
- Misses       7745     7747       +2     
- Partials      297      299       +2     

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more