Skip to content

plugins: move transfer-issue to issue management#702

Open
Amulyam24 wants to merge 1 commit into
kubernetes-sigs:mainfrom
Amulyam24:transfer-issue
Open

plugins: move transfer-issue to issue management#702
Amulyam24 wants to merge 1 commit into
kubernetes-sigs:mainfrom
Amulyam24:transfer-issue

Conversation

@Amulyam24

@Amulyam24 Amulyam24 commented May 2, 2026

Copy link
Copy Markdown
Contributor

With the introduction of issue-management, move transfer-plugin to the folder.

Signed-off-by: Amulyam24 amulmek1@in.ibm.com
Assisted-by: AI

@netlify

netlify Bot commented May 2, 2026

Copy link
Copy Markdown

Deploy Preview for k8s-prow ready!

Name Link
🔨 Latest commit 1f6d66b
🔍 Latest deploy log https://app.netlify.com/projects/k8s-prow/deploys/6a16fabf3889740008fff7a8
😎 Deploy Preview https://deploy-preview-702--k8s-prow.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@k8s-ci-robot k8s-ci-robot added the area/hook Issues or PRs related to prow's hook component label May 2, 2026
@k8s-ci-robot k8s-ci-robot added area/plugins Issues or PRs related to prow's plugins for the hook component size/L Denotes a PR that changes 100-499 lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels May 2, 2026
@Amulyam24

Copy link
Copy Markdown
Contributor Author

/retest

@Amulyam24

Copy link
Copy Markdown
Contributor Author

/cc @petr-muller @stmcginnis, PTAL!

@Amulyam24

Copy link
Copy Markdown
Contributor Author

/cc @petr-muller @stmcginnis

@Amulyam24

Copy link
Copy Markdown
Contributor Author

/cc @Prucek

@k8s-ci-robot k8s-ci-robot requested a review from Prucek May 27, 2026 07:23
Comment thread pkg/plugins/issue-management/issue_management.go Outdated
Comment thread pkg/plugins/issue-management/issue_management.go Outdated
Signed-off-by: Amulyam24 <amulmek1@in.ibm.com>
@Amulyam24 Amulyam24 requested a review from Prucek June 1, 2026 06:28
@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Jun 1, 2026
@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Amulyam24, Prucek
Once this PR has been reviewed and has the lgtm label, please assign cjwagner for approval. For more information see the Code Review Process.

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

Details Needs approval from an approver in each of these files:

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

@petr-muller

Copy link
Copy Markdown
Contributor

@Amulyam24 this is good, but do we have an idea about how the transition will look like for instances that enable the transfer-issue plugin today:

plugins:
- transfer-issue

Will these blow up with errors on update, or simply lose functionality? I'd like to make sure we do not break instances even if it takes more effort.

And if we're not breaking them but they "only" lose functionality until they enable issue-management plugin it makes me think we should get into a habit of updating https://github.com/kubernetes-sigs/prow/blob/main/site/content/en/docs/announcements.md again. Apparently we stopped doing that which is not good. It is not the greatest comm channel of all but we dont have anything better 😅

@Amulyam24

Copy link
Copy Markdown
Contributor Author

@Amulyam24 this is good, but do we have an idea about how the transition will look like for instances that enable the transfer-issue plugin today:

plugins:
- transfer-issue

Will these blow up with errors on update, or simply lose functionality? I'd like to make sure we do not break instances even if it takes more effort.

Hi @petr-muller, Thanks! This is a good point which I missed, I will try this out and get back.
Then we can take the next step accordingly.

@Amulyam24

Copy link
Copy Markdown
Contributor Author

Hi @petr-muller, I confirmed that with this change, using /transfer-issue would silently fail.
The issue-management plugin should be added to the plugins.yaml.

I can update the announcements page, should it be done post this PR merge?

Also, since the page has become kind of obsolete and this change could be breaking, we might have to make the concerned parties explicitly aware, IMO. Atleast Kubernetes and SIGs as a first step, wdyt?

@Amulyam24

Copy link
Copy Markdown
Contributor Author

Hi @petr-muller, can you please share your input on the above?
Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/hook Issues or PRs related to prow's hook component area/plugins Issues or PRs related to prow's plugins for the hook component cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants