Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

Label legacy TKRs as such #3858

Merged
merged 2 commits into from
Nov 9, 2022
Merged

Conversation

imikushin
Copy link
Contributor

@imikushin imikushin commented Nov 8, 2022

What this PR does / why we need it

TKRs without bootstrap packages will now be labeled with label run.tanzu.vmware.com/legacy-tkr: ''

Bootstrap packages were introduced in TKR API v1alpha3, so any TKRs converted from previous API versions won't have them.

Which issue(s) this PR fixes

Fixes #3873.

Describe testing done for PR

Added unit tests.

Release note

TKRs converted from API versions before v1alpha3 will now be labeled with `run.tanzu.vmware.com/legacy-tkr: ''`.

Additional information

Special notes for your reviewer

TKRs without bootstrap packages will now be labeled with label
run.tanzu.vmware.com/legacy-tkr: ''

Bootstrap packages were introduced in TKR API v1alpha3, so any TKRs
converted from previous API versions won't have them.

Signed-off-by: Ivan Mikushin <imikushin@vmware.com>
@imikushin imikushin requested review from a team as code owners November 8, 2022 03:39
@imikushin imikushin added area/api area/tkr ok-to-merge PRs should be labelled with this before merging labels Nov 8, 2022
@codecov
Copy link

codecov bot commented Nov 8, 2022

Codecov Report

Merging #3858 (7cb9d6b) into main (c57f9ac) will decrease coverage by 0.87%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3858      +/-   ##
==========================================
- Coverage   46.56%   45.69%   -0.88%     
==========================================
  Files         400      426      +26     
  Lines       39830    41688    +1858     
==========================================
+ Hits        18548    19049     +501     
- Misses      19584    20908    +1324     
- Partials     1698     1731      +33     
Impacted Files Coverage Δ
apis/run/v1alpha3/tanzukubernetesrelease_types.go 33.33% <ø> (ø)
addons/controllers/machine_controller.go 65.65% <0.00%> (-3.04%) ⬇️
...til/clusterbootstrapclone/clusterbootstrapclone.go 67.36% <0.00%> (-3.01%) ⬇️
addons/controllers/clusterbootstrap_controller.go 63.36% <0.00%> (-0.13%) ⬇️
cmd/cli/plugin/cluster/machinehealthcheck.go 100.00% <0.00%> (ø)
cmd/cli/plugin/cluster/scale.go 17.85% <0.00%> (ø)
cmd/cli/plugin/cluster/available_upgrade.go 16.32% <0.00%> (ø)
cmd/cli/plugin/cluster/set_node_pool.go 14.63% <0.00%> (ø)
.../cli/plugin/cluster/get_machinehealthcheck_node.go 9.30% <0.00%> (ø)
cmd/cli/plugin/cluster/kubeconfig_get.go 8.82% <0.00%> (ø)
... and 24 more

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

Copy link
Contributor

@prkalle prkalle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

Could you please create an Issue and refer it in PR summary

tkr/controller/tkr-source/tkr/tkr_test.go Outdated Show resolved Hide resolved
Signed-off-by: Ivan Mikushin <imikushin@vmware.com>
@imikushin imikushin merged commit d4d226e into vmware-tanzu:main Nov 9, 2022
xiujuanx pushed a commit that referenced this pull request Nov 14, 2022
TKRs without bootstrap packages will now be labeled with label
run.tanzu.vmware.com/legacy-tkr: ''

Bootstrap packages were introduced in TKR API v1alpha3, so any TKRs
converted from previous API versions won't have them.

Signed-off-by: Ivan Mikushin <imikushin@vmware.com>
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
area/api area/tkr cla-not-required ok-to-merge PRs should be labelled with this before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Label legacy TKRs as such in TKGm
3 participants