Skip to content

πŸ“– Rename Kagenti β†’ Rossoctl across agent-examples#733

Merged
mrsabath merged 1 commit into
rossoctl:mainfrom
Ibrahim2595:rename/kagenti-to-rossoctl
Jul 20, 2026
Merged

πŸ“– Rename Kagenti β†’ Rossoctl across agent-examples#733
mrsabath merged 1 commit into
rossoctl:mainfrom
Ibrahim2595:rename/kagenti-to-rossoctl

Conversation

@Ibrahim2595

Copy link
Copy Markdown
Contributor

Content sweep renaming the project Kagenti β†’ Rossoctl in this repo (agent-examples β†’ examples). All 253 case-insensitive kagenti references updated; grep is clean.

Changes:

  • Brand / prose / comments: Kagenti β†’ Rossoctl
  • Go module: github.com/kagenti/mcp-mitm β†’ github.com/rossoctl/mcp-mitm
  • Images: ghcr.io/kagenti/agent-examples/* β†’ ghcr.io/rossoctl/examples/*
  • Cross-repo links: kagenti/kagenti β†’ rossoctl/rossoctl, kagenti-extensions β†’ rossoctl/rossocortex, agent-examples β†’ examples, workload-harness β†’ rossoctl/workload-harness
  • CODEOWNERS: @kagenti/* β†’ @rossoctl/*
  • CI reusable-workflow refs: kagenti/.github β†’ rossoctl/.github
  • Env vars: KAGENTI_* β†’ ROSSOCTL_*
  • Sample runtime-contract values (this repo ships samples, not live contracts): labels kagenti.io/* β†’ rossoctl.io/*, namespace kagenti-system β†’ rossoctl-system, SPIFFE kagenti.{local,example.com} β†’ rossoctl.{local,example.com}, Keycloak realm/client kagenti β†’ rossoctl

Verification: ruff check + format clean, go build/vet clean, pytest 80 passed. Docker image builds unaffected (image names derive from ${{ github.repository }}).

Related to #1972

Assisted-By: Claude Opus 4.8 noreply@anthropic.com

Summary

Related issue(s)

(Optional) Testing Instructions

Fixes #

Content sweep renaming the project Kagenti β†’ Rossoctl in this repo
(agent-examples β†’ examples). All 253 case-insensitive `kagenti`
references updated; grep is clean.

Changes:
- Brand / prose / comments: Kagenti β†’ Rossoctl
- Go module: github.com/kagenti/mcp-mitm β†’ github.com/rossoctl/mcp-mitm
- Images: ghcr.io/kagenti/agent-examples/* β†’ ghcr.io/rossoctl/examples/*
- Cross-repo links: kagenti/kagenti β†’ rossoctl/rossoctl,
  kagenti-extensions β†’ rossoctl/rossocortex, agent-examples β†’ examples,
  workload-harness β†’ rossoctl/workload-harness
- CODEOWNERS: @kagenti/* β†’ @rossoctl/*
- CI reusable-workflow refs: kagenti/.github β†’ rossoctl/.github
- Env vars: KAGENTI_* β†’ ROSSOCTL_*
- Sample runtime-contract values (this repo ships samples, not live
  contracts): labels kagenti.io/* β†’ rossoctl.io/*, namespace
  kagenti-system β†’ rossoctl-system, SPIFFE kagenti.{local,example.com}
  β†’ rossoctl.{local,example.com}, Keycloak realm/client kagenti β†’ rossoctl

Verification: ruff check + format clean, go build/vet clean,
pytest 80 passed. Docker image builds unaffected (image names derive
from ${{ github.repository }}).

Related to #1972

Assisted-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: IBRAHIM IBRAHIM <66755652+Ibrahim2595@users.noreply.github.com>
Comment thread mcp/github_tool/go.mod
@@ -1,4 +1,4 @@
module github.com/kagenti/mcp-mitm
module github.com/rossoctl/mcp-mitm
Comment thread mcp/github_tool/go.mod
@@ -1,4 +1,4 @@
module github.com/kagenti/mcp-mitm
module github.com/rossoctl/mcp-mitm
@esnible
esnible marked this pull request as draft July 20, 2026 13:11
@esnible

esnible commented Jul 20, 2026

Copy link
Copy Markdown
Member

Converted to "DRAFT" because it is important that this not merge until we have support in other repos.

For example, a2a/a2a_currency_converter/deployment/k8s.yaml refers to K8s labels that are not yet understood by the Rossoctl-operator.

@cwiklik I could not find an issue for label renaming. Is there one? Is the plan to have a period where both the old and new labels are supported?

@mrsabath mrsabath left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary

Approving β€” this is the cleanest of the four #1972 rename PRs. The content rename is faithful: no residual kagenti references added, ghcr.io/rossoctl/examples/* applied consistently, and images derive from ${{ github.repository }} so builds are unaffected by the repo rename. pytest/ruff/go build all green.

Notably, this PR does not carry the go.sum lockfile defect flagged in kagenti-extensions#678 and kagenti-operator#485: the only Go change here is the self-module rename (module github.com/kagenti/mcp-mitm β†’ github.com/rossoctl/mcp-mitm in mcp/github_tool/go.mod), which has no checksum implications, and there's no consumed kagenti/* dependency to break. No go.sum was touched, correctly.

Non-blocking

  • Trivy code-scanning reports "2 new high alerts" β€” pre-existing CVEs, re-flagged, not introduced here. The two are jwt-go CVE-2020-26160 and jsonparser CVE-2026-32285 on mcp/github_tool/go.mod, but that file's diff is only the module line rename β€” no dependency was added or changed. GitHub's code-scanning diff view treats the touched file's findings as "new". The actual filesystem scan job (trivy-scan) passes. Same mass-rename artifact as CodeQL on rossoctl/rossoctl#2230.
  • Follow-up (out of scope for a rename): Trivy also surfaces a broader set of real, pre-existing highs across the uv.lock files β€” the MCP Python SDK (mcp 1.12/1.26/1.27 β†’ fixed in 1.27.2 / 1.28.1: CVE-2026-52869, -52870, -59950) and the two Go deps above. Worth a dedicated dependency-bump PR, but not something this rename should take on.

Nit

  • Title uses a πŸ“– emoji prefix; the org title Ruleset expects an allowed word prefix (e.g. Docs: or Refactor:). Length (49) is fine. Consider Docs: Rename Kagenti β†’ Rossoctl across agent-examples.

Areas reviewed: Go module, Python/uv.lock deps, images, K8s samples, security scanners (both Trivy jobs traced), commit conventions.
Commits: 1, signed-off βœ… (DCO green).
CI status: Trivy code-scanning failing (pre-existing CVEs re-flagged); trivy-scan, CodeQL, lint, test, test-startup, hadolint, dependency-review all passing.

@mrsabath
mrsabath marked this pull request as ready for review July 20, 2026 17:25
@mrsabath
mrsabath merged commit c8a9a81 into rossoctl:main Jul 20, 2026
10 of 11 checks passed
@github-project-automation github-project-automation Bot moved this from New/ToDo to Done in Kagenti Issue Prioritization Jul 20, 2026
mrsabath pushed a commit that referenced this pull request Jul 20, 2026
The extensions repo's final name is `cortex` (rossoctl/cortex), not `rossocortex`.
Updates the remaining reference left after #733 merged β€” the AuthBridge cross-repo
link now points at rossoctl/cortex.

Related to #1972

Assisted-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: IBRAHIM IBRAHIM <66755652+Ibrahim2595@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants