Update documentation links from openark/percona to proxysql#15
Update documentation links from openark/percona to proxysql#15renecannao merged 1 commit intomasterfrom
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (16)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request systematically updates outdated GitHub repository links within the project's documentation. The primary goal is to redirect all references from the former Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Pull request overview
Updates documentation to point to the current proxysql/orchestrator repository instead of legacy openark/orchestrator / percona/orchestrator links, aligning docs with the new upstream location while preserving references to separate related repos.
Changes:
- Replaced
openark/orchestratorGitHub URLs withproxysql/orchestratoracross multiple docs. - Replaced
percona/orchestratorURLs withproxysql/orchestratorin install guidance. - Updated various “source code”, CI workflow, releases, and LICENSE links to the new repository.
Reviewed changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| docs/using-the-web-api.md | Updates API and client source links to proxysql/orchestrator. |
| docs/users.md | Updates PR submission link to new repo. |
| docs/risks.md | Updates LICENSE link to new repo. |
| docs/pseudo-gtid-manual-injection.md | Updates pseudo-GTID sample files link to new repo. |
| docs/orchestrator-client.md | Updates orchestrator-client script link to new repo. |
| docs/license.md | Updates LICENSE and vendor directory links to new repo. |
| docs/install.md | Updates tags/conf links from percona/orchestrator to proxysql/orchestrator. |
| docs/faq.md | Updates “available at” repo URL. |
| docs/download.md | Updates GitHub/releases links and go get path to new repo. |
| docs/docker.md | Updates Docker-related source/release links to new repo. |
| docs/configuration.md | Updates config source link to new repo. |
| docs/configuration-discovery-pseudo-gtid.md | Updates pseudo-GTID sample script link to new repo. |
| docs/ci.md | Updates workflow/actions links to new repo. |
| docs/ci-env.md | Updates orchestrator config link to new repo. |
| docs/build.md | Updates clone URL to new repo. |
| docs/bugs.md | Updates issues link to new repo. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| For developers: `orchestrator` is go-gettable. Issue: | ||
|
|
||
| go get github.com/openark/orchestrator/... | ||
| go get github.com/proxysql/orchestrator/... |
There was a problem hiding this comment.
The go get github.com/proxysql/orchestrator/... instruction is likely to fail right now because go.mod still declares the module path as github.com/openark/orchestrator (Go will error on a module path mismatch). Either keep the go get line pointing at github.com/openark/orchestrator/... until the module path migration lands, or adjust the text to explain that cloning/building from source works but Go module consumers must still use the old import path for now.
| go get github.com/proxysql/orchestrator/... | |
| go get github.com/openark/orchestrator/... |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request successfully updates the documentation to reflect the repository's move from openark/orchestrator and percona/orchestrator to proxysql/orchestrator. The changes are consistent and cover all the documentation files as intended. I've added a few minor suggestions to improve the readability of the documentation by converting raw URLs into markdown links.
| `orchestrator` is released as open source and is available at [GitHub](https://github.com/openark/orchestrator). | ||
| Find official releases in https://github.com/openark/orchestrator/releases | ||
| `orchestrator` is released as open source and is available at [GitHub](https://github.com/proxysql/orchestrator). | ||
| Find official releases in https://github.com/proxysql/orchestrator/releases |
There was a problem hiding this comment.
For better readability, this URL could be formatted as a markdown link.
| Find official releases in https://github.com/proxysql/orchestrator/releases | |
| Find official releases on the [releases page](https://github.com/proxysql/orchestrator/releases) |
| ### Is orchestrator open source? | ||
|
|
||
| Yes. `Orchestrator` is released as open source under the Apache License 2.0 and is available at: https://github.com/openark/orchestrator | ||
| Yes. `Orchestrator` is released as open source under the Apache License 2.0 and is available at: https://github.com/proxysql/orchestrator |
There was a problem hiding this comment.
For better readability and consistency, this URL could be formatted as a markdown link.
| Yes. `Orchestrator` is released as open source under the Apache License 2.0 and is available at: https://github.com/proxysql/orchestrator | |
| Yes. `Orchestrator` is released as open source under the Apache License 2.0 and is available on [GitHub](https://github.com/proxysql/orchestrator) |
| - Extract from tarball | ||
|
|
||
| Extract the archive you've downloaded from https://github.com/percona/orchestrator/tags | ||
| Extract the archive you've downloaded from https://github.com/proxysql/orchestrator/tags |
There was a problem hiding this comment.
For better readability, this URL could be formatted as a markdown link.
| Extract the archive you've downloaded from https://github.com/proxysql/orchestrator/tags | |
| Extract the archive you've downloaded from the [tags page](https://github.com/proxysql/orchestrator/tags) |
| See [LICENSE](https://github.com/proxysql/orchestrator/blob/master/LICENSE) | ||
|
|
||
| This repository includes 3rd party libraries which have their own licenses. These libraries are found under https://github.com/openark/orchestrator/tree/master/vendor | ||
| This repository includes 3rd party libraries which have their own licenses. These libraries are found under https://github.com/proxysql/orchestrator/tree/master/vendor |
There was a problem hiding this comment.
For better readability, this URL could be formatted as a markdown link.
| This repository includes 3rd party libraries which have their own licenses. These libraries are found under https://github.com/proxysql/orchestrator/tree/master/vendor | |
| This repository includes 3rd party libraries which have their own licenses. These libraries are found under the [`vendor` directory](https://github.com/proxysql/orchestrator/tree/master/vendor) |
| who are actively using it so it seemed like a good idea to generate a page | ||
| of current users. If your name or your company's name is missing from this and | ||
| you would like to be added please send a pull request to github.com/openark/orchestrator/pulls | ||
| you would like to be added please send a pull request to github.com/proxysql/orchestrator/pulls |
There was a problem hiding this comment.
For better readability and user experience, it would be good to format this URL as a markdown link and improve the sentence structure slightly.
| you would like to be added please send a pull request to github.com/proxysql/orchestrator/pulls | |
| you would like to be added, please [send a pull request](https://github.com/proxysql/orchestrator/pulls). |
Summary
github.com/openark/orchestratorreferences withgithub.com/proxysql/orchestratoracross 15 docs filesgithub.com/percona/orchestratorreferences withgithub.com/proxysql/orchestratorindocs/install.mdopenark/orchestrator-ci-env,openark/orchestrator-agent)docs/superpowers/files (specs and plans reference history correctly)Test plan
grep -rn "openark/orchestrator" docs/ | grep -v "ci-env" | grep -v "superpowers"returns emptygrep -rn "percona/orchestrator" docs/ | grep -v "superpowers"returns emptyopenark/orchestrator-ci-envandopenark/orchestrator-agentreferences are unchanged