Skip to content

Add gemini-agent Prow plugin#730

Open
ameukam wants to merge 1 commit into
kubernetes-sigs:mainfrom
ameukam:gemini-agent-plugin
Open

Add gemini-agent Prow plugin#730
ameukam wants to merge 1 commit into
kubernetes-sigs:mainfrom
ameukam:gemini-agent-plugin

Conversation

@ameukam

@ameukam ameukam commented May 23, 2026

Copy link
Copy Markdown
Member

Implements a Prow plugin triggered by /gemini-agent comments on GitHub issues and PRs. The plugin collects GitHub context (issue metadata, PR diffs, recent comments) and sends it to Gemini on Vertex AI for analysis.

Assisted by Opus 4.6

@netlify

netlify Bot commented May 23, 2026

Copy link
Copy Markdown

Deploy Preview for k8s-prow ready!

Name Link
🔨 Latest commit 27bec9f
🔍 Latest deploy log https://app.netlify.com/projects/k8s-prow/deploys/6a1eca4e37f13b0008a78802
😎 Deploy Preview https://deploy-preview-730--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 23, 2026
@k8s-ci-robot k8s-ci-robot added area/plugins Issues or PRs related to prow's plugins for the hook component size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels May 23, 2026
@ameukam ameukam force-pushed the gemini-agent-plugin branch from 2aeb35e to 760dfd1 Compare May 23, 2026 16:42
@k8s-ci-robot k8s-ci-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels May 23, 2026
@ameukam

ameukam commented May 24, 2026

Copy link
Copy Markdown
Member Author

/assign @petr-muller @krzyzacy

@petr-muller

Copy link
Copy Markdown
Contributor

TBH I'm not a fan of these integrations being internal plugins (how many /X-agent implementations would we maintain inside hook?), I think they have a potential to grow in functionality and in how they are coupled with the given AI service to be standalone external plugins.

@ameukam

ameukam commented May 25, 2026

Copy link
Copy Markdown
Member Author

TBH I'm not a fan of these integrations being internal plugins (how many /X-agent implementations would we maintain inside hook?), I think they have a potential to grow in functionality and in how they are coupled with the given AI service to be standalone external plugins.

I don't think we have clear policy on what's eligible for internal plugins. it's not only the AI service but everything that's interact with a third-party service.
I'll defer do your decision but IMHO we should provide more clarity on internal.

@midnightconman

midnightconman commented May 27, 2026

Copy link
Copy Markdown
Contributor

Internal / external plugins should have better docs, but the gist of it is that external plugins (like cherrypicker) aren't compiled into hook and as such are easier to manage.

@petr-muller if this was converted to an external plugin, could it still be staged here in the prow repo as an example?

@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 2, 2026
@ameukam ameukam force-pushed the gemini-agent-plugin branch from 760dfd1 to f9e1ca6 Compare June 2, 2026 12:10
@k8s-ci-robot k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 2, 2026
@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ameukam
Once this PR has been reviewed and has the lgtm label, please ask for approval from krzyzacy. 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

@k8s-ci-robot k8s-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Jun 2, 2026
…b issues and PRs.

The plugin collects GitHub context (issue metadata, PR diffs, recent comments) and sends it to Gemini on Vertex AI for analysis.

Assisted by Opus 4.6

Signed-off-by: Arnaud Meukam <ameukam@gmail.com>
@ameukam ameukam force-pushed the gemini-agent-plugin branch from f9e1ca6 to 27bec9f Compare June 2, 2026 12:19
@ameukam

ameukam commented Jun 2, 2026

Copy link
Copy Markdown
Member Author

@petr-muller @midnightconman rewritten as an external plugin. PTAL!. 🙏🏾

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. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants