-
Notifications
You must be signed in to change notification settings - Fork 2.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Add ability to migrate owner ID when using the TXT registry #3631
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: mkilchhofer The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Welcome @mkilchhofer! |
980f22e
to
0e3b9fb
Compare
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Hello @mkilchhofer , Do you think you can sign the CLA & rebase this PR ? |
No activity and no response in the last month. |
@mloiseleur: Closed this PR. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@mkilchhofer could you please consider signing CLA, or if you agree I can fork and rebase your commit as mine and open a new PR. But I don't want to steal your work ;) |
What is happening here folks? Number of attempts failed. Not a most favorable solution, community issue, organizational issue? This feature could be impactful 👍 |
+1, I can also drive the work if we're just waiting on a CLA @mkilchhofer, are you okay if one of us rebased your commits and tried to contribute again to this? |
Sure yes 👍 |
Description
This is a follow up of the stale PR over there:
As the contributor @Inorysky is no longer active in the initial PR, I forked his/hers work into our organization, rebased and added a unit test.
Initial PR description:
I tested this feature also manually on:
Fixes #2036
Checklist