Skip to content

[WIP] OLS-2913 Agentic crds#1641

Open
raptorsun wants to merge 2 commits into
openshift:mainfrom
raptorsun:agentic-crds
Open

[WIP] OLS-2913 Agentic crds#1641
raptorsun wants to merge 2 commits into
openshift:mainfrom
raptorsun:agentic-crds

Conversation

@raptorsun
Copy link
Copy Markdown
Contributor

@raptorsun raptorsun commented May 18, 2026

In progress, do not merge

Description

Type of change

  • Refactor
  • New feature
  • Bug fix
  • CVE fix
  • Optimization
  • Documentation Update
  • Configuration Update
  • Bump-up dependent library

Related Tickets & Documents

  • Related Issue #
  • Closes #

Checklist before requesting a review

  • I have performed a self-review of my code.
  • PR has passed all pre-merge test jobs.
  • If it is a core feature, I have added thorough tests.

Testing

  • Please provide detailed steps to perform tests related to this code change.
  • How were the fix/results from this change verified? Please provide relevant screenshots or results.

@raptorsun
Copy link
Copy Markdown
Contributor Author

/hold

@openshift-ci openshift-ci Bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels May 18, 2026
@openshift-ci openshift-ci Bot requested review from blublinsky and joshuawilson May 18, 2026 16:38
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented May 18, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign blublinsky 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

Adds the agentic stack deployment infrastructure to lightspeed-operator:

Deploy scripts (hack/agentic/):
- deploy.sh: Full deploy with on-cluster builds (--provider=vertex|bedrock)
- redeploy-{operator,agent,console,skills,all}.sh: Fast iteration scripts
- undeploy.sh: Teardown with timeout + finalizer cleanup for stuck CRDs
- lib.sh: Shared build helpers with _run() error wrapper, unified _build
  sync|async, centralized image vars, --wait on oc start-build

Operator integration:
- cmd/main.go: Wire agentic controller with --agentic-console-image and
  --agentic-sandbox-image flags
- Add LightspeedAgents to FeatureGate enum in OLSConfig CRD
- Agentic CRDs: ApprovalPolicy, Agent, LLMProvider, Proposal, results
- config/rbac-agentic/: RBAC for agentic controller
- Dockerfile.dev: Local module builds for agentic-operator dependency
- Add FeatureGateLightspeedAgents and agentic image default constants

Demo proposals use find-token skill from quay.io/harpatil/agentic-skills
(TODO: replace with Konflux-built image when available).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@openshift-ci openshift-ci Bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 18, 2026
…de agentic operator CRD in the manifest generation

Signed-off-by: Haoyu Sun <hasun@redhat.com>
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented May 19, 2026

@raptorsun: all tests passed!

Full PR test history. Your PR dashboard.

Details

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-sigs/prow repository. I understand the commands that are listed here.

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

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants