Skip to content

Conversation

thardeck
Copy link
Collaborator

@thardeck thardeck commented Oct 14, 2025

fleet.cattle.io/created-by-display-name was added by Rancher UI but was redundant because of the additional label fleet.cattle.io/created-by-user-id.

Additional QA

Testing

Automated Testing

Added integration tests to verify that the deprecated label fleet.cattle.io/created-by-display-name is removed if it exists.

QA Testing Considerations

  1. The label was added only in Rancher 2.12.0 and maybe 2.12.1 after adding a Fleet Resource in the Rancher UI.
  2. Verify that the label is there (Rancher UI hides it), it can only be seen in the yaml config.
  3. Upgrade to a Rancher head with support for the migration (or update the Fleet version manually).
  4. Verify that fleet.cattle.io/created-by-display-name is gone but fleet.cattle.io/created-by-user-id is still there.

Regressions Considerations

fleet.cattle.io/created-by-user-id and other user labels should not be affected by the migration.

@thardeck thardeck requested a review from weyfonk October 14, 2025 14:36
@thardeck thardeck self-assigned this Oct 14, 2025
@thardeck thardeck requested a review from a team as a code owner October 14, 2025 14:36
@thardeck thardeck added this to Fleet Oct 14, 2025
@thardeck thardeck moved this to 👀 In review in Fleet Oct 14, 2025
It was added by Rancher UI but was redundant because of the additional
label `created-by-user-id`.
@thardeck thardeck force-pushed the add_label_migration branch from 410ba0c to 21c8f5c Compare October 14, 2025 14:48
@thardeck thardeck removed the request for review from weyfonk October 16, 2025 12:08
@kkaempf kkaempf added this to the v2.13.0 milestone Oct 17, 2025
@thardeck thardeck merged commit 89700a7 into main Oct 17, 2025
34 of 36 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Fleet Oct 17, 2025
@thardeck thardeck deleted the add_label_migration branch October 17, 2025 14:36
@thardeck thardeck moved this from ✅ Done to Needs QA review in Fleet Oct 17, 2025
@sbulage sbulage self-requested a review October 17, 2025 15:50
thardeck added a commit that referenced this pull request Oct 20, 2025
It was added by Rancher UI but was redundant because of the additional
label `created-by-user-id`.
thardeck added a commit that referenced this pull request Oct 20, 2025
It was added by Rancher UI but was redundant because of the additional
label `created-by-user-id`.
thardeck added a commit that referenced this pull request Oct 20, 2025
It was added by Rancher UI but was redundant because of the additional
label `created-by-user-id`.
thardeck added a commit that referenced this pull request Oct 20, 2025
…#4245)

* Add userID to log messages (#4202)

* Add userID to log messages

* Remove confusing comment in userID log tests

* Remove deprecated created-by label (#4225)

It was added by Rancher UI but was redundant because of the additional
label `created-by-user-id`.
thardeck added a commit that referenced this pull request Oct 20, 2025
It was added by Rancher UI but was redundant because of the additional
label `created-by-user-id`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Needs QA review

Development

Successfully merging this pull request may close these issues.

3 participants