Make Kingress reconciled with DomainMapping when adding a new label

[xxinran]

Fixes #13135

Release Note

Fixed domain mapping reconciler to update the Kingress label when adding a new label to domain mapping. 

[xxinran]

/retest

[codecov]

Codecov Report

Merging #13178 (0ff85c7) into main (782df98) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main   #13178   +/-   ##
=======================================
  Coverage   86.70%   86.70%           
=======================================
  Files         196      196           
  Lines       14461    14463    +2     
=======================================
+ Hits        12538    12540    +2     
  Misses       1626     1626           
  Partials      297      297           

Impacted Files	Coverage Δ
pkg/reconciler/domainmapping/reconciler.go	93.68% <100.00%> (+0.06%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[psschwei]

/retest

[nak3]

Thank you so much! LGTM overall.
I guess we can add an unit test like this https://github.com/knative/serving/blob/main/pkg/reconciler/domainmapping/table_test.go#L415 ?

[nak3]

/lgtm

Thank you! Finally, could you please write the release note on the first comment?

[xxinran]

/lgtm

Thank you! Finally, could you please write the release note on the first comment?

Sure, please check it.

[nak3]

/approve

[knative-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: nak3, xxinran

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• pkg/OWNERS [nak3]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment