Skip to content
This repository was archived by the owner on Jul 13, 2026. It is now read-only.

Add migration notice to README#135

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-lightspeed:mainfrom
omkarjoshi0304:add-migration-notice
Jul 10, 2026
Merged

Add migration notice to README#135
openshift-merge-bot[bot] merged 1 commit into
openstack-lightspeed:mainfrom
omkarjoshi0304:add-migration-notice

Conversation

@omkarjoshi0304

@omkarjoshi0304 omkarjoshi0304 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Summary by CodeRabbit

  • Documentation
    • Added an archive notice at the top of the README.
    • Pointed contributors to the new repository for future issues and pull requests.

This repository has been migrated to
openstack-k8s-operators/lightspeed-operator.
@openshift-ci openshift-ci Bot requested review from lpiwowar and umago July 9, 2026 10:03
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Added an archive/migration notice at the top of README.md indicating the repository has moved to openstack-k8s-operators/lightspeed-operator, directing new issues and pull requests to that repository.

Changes

README Migration Notice

Layer / File(s) Summary
Archive notice addition
README.md
Adds an upfront notice stating the repository is archived/migrated to openstack-k8s-operators/lightspeed-operator, with new issues/PRs to be opened there.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: Akrog

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the README migration notice added by this change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@README.md`:
- Around line 1-2: The README notice is asserting that the repository “is
archived” before that state is actually enabled. Update the migration message in
the README text to avoid the hard archived claim for now, or phrase it in
future/conditional terms until the repo is officially archived.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5aefa1f9-3811-46b1-9093-8e10793ddb6a

📥 Commits

Reviewing files that changed from the base of the PR and between 33493e6 and 4235c51.

📒 Files selected for processing (1)
  • README.md

Comment thread README.md
Comment on lines +1 to +2
> **This repository has been migrated to [openstack-k8s-operators/lightspeed-operator](https://github.com/openstack-k8s-operators/lightspeed-operator). Please open all new issues and pull requests there. This repository is archived and no longer maintained.**

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "== root files =="
git ls-files | sed -n '1,120p'

echo
echo "== README excerpt =="
if git ls-files README.md >/dev/null 2>&1; then
  nl -ba README.md | sed -n '1,20p'
fi

echo
echo "== git remotes =="
git remote -v || true

echo
echo "== repo metadata hints =="
rg -n --hidden --glob '!**/.git/**' -e 'archiv|archived|no longer maintained|migrated to' README.md .github . || true

Repository: openstack-lightspeed/operator

Length of output: 5345


🏁 Script executed:

#!/bin/bash
set -euo pipefail

# Try to inspect the upstream repository metadata if a GitHub remote exists.
remote_url="$(git remote get-url origin 2>/dev/null || true)"
printf 'origin=%s\n' "${remote_url:-<none>}"

case "$remote_url" in
  *github.com*)
    # Normalize to owner/repo where possible.
    repo="$(printf '%s' "$remote_url" | sed -E 's#(git@|https://)github.com[:/]+##; s#\.git$##')"
    printf 'repo=%s\n' "$repo"
    if command -v gh >/dev/null 2>&1; then
      gh api "repos/$repo" --jq '{full_name, archived, disabled, pushed_at, updated_at}' || true
    else
      echo "gh CLI not available"
    fi
    ;;
  *)
    echo "No GitHub remote detected"
    ;;
esac

Repository: openstack-lightspeed/operator

Length of output: 413


Avoid saying this repository is archived yet.
Keep the notice aligned with the repo state; drop the hard “is archived” claim or switch to future tense until the archive setting is enabled.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@README.md` around lines 1 - 2, The README notice is asserting that the
repository “is archived” before that state is actually enabled. Update the
migration message in the README text to avoid the hard archived claim for now,
or phrase it in future/conditional terms until the repo is officially archived.

@openshift-ci openshift-ci Bot added the approved label Jul 9, 2026

@lpiwowar lpiwowar 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.

/lgtm

@openshift-ci

openshift-ci Bot commented Jul 10, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lpiwowar, omkarjoshi0304, umago

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

The pull request process is described 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

@openshift-merge-bot openshift-merge-bot Bot merged commit cc9a395 into openstack-lightspeed:main Jul 10, 2026
8 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants