-
Notifications
You must be signed in to change notification settings - Fork 477
Archive v2.1 documentation #21195
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Archive v2.1 documentation #21195
Conversation
- Delete v2.1 directory and images - Remove sidebar-data-v2.1.json to disable sidebar navigation - Delete _includes/v2.1/ and _includes/releases/v2.1/ directories - Update releases/v2.1.md to archived format with support policy links - Add all 27 v2.1 release anchors to prevent broken links from unsupported-versions page - Fix 78+ cross-version references pointing to v2.1: - Replace v2.1 include references with inline content in v2.0 tutorials - Move DBeaver and Kubernetes images to images/common/ directory - Update all IntelliJ image references to use images/common/intellij/ - Replace v2.1 URLs with stable docs links - Archive follows v19.2 pattern: page exists but shows archived notice - htmltest passes 100% with all 9,369 documents tested
Only archive v2.1 documentation, keep Gemfile unchanged
Files changed:
|
✅ Deploy Preview for cockroachdb-interactivetutorials-docs canceled.
|
✅ Deploy Preview for cockroachdb-api-docs canceled.
|
✅ Netlify Preview
To edit notification comments on pull requests, go to your Netlify project configuration. |
…est config The sidebar_htmltest.rb plugin was generating HTML test files for all versions, including those marked as ignored in .htmltest.yml. This caused htmltest failures for archived versions that have sidebar-data files but include CockroachCloud links that don't exist. Changes: - Read .htmltest.yml to identify ignored version directories - Skip generating sidebar HTML test files for ignored versions - Add logging to show which versions are being skipped This ensures that only versions actually tested by htmltest have corresponding sidebar test files generated.
rmloveland
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great overall! We're very close, please:
- Remove the LLM-generated script that hacks some SSL settings
- Remove the newly introduced cross-version links in some of the later versions
- Add the zip archive of v2.1 to the 'Archived Documentation' page - right now it's missing. Although if you prefer to do that in another PR that also works
| ignored_versions = ignored_dirs.map do |dir| | ||
| match = dir.match(/\^?docs\/?(v\d+\.\d+)/) | ||
| match[1] if match | ||
| end.compact |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
interesting, I don't think end.compact is valid Ruby syntax, but it does pass ruby -c 🤔
| @@ -0,0 +1,27 @@ | |||
| require "openssl" | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please remove this, it doesn't belong in src/current - if there's a scripts directory somewhere you want to stash this away in for the EduEng team to use that's totally fine ofc. This looks like LLM noise to silence an SSL error
| {% for r in rel %} | ||
| {% include releases/{{ page.major_version }}/{{ r.release_name }}.md release=r.release_name release_date=r.release_date %} | ||
| {% endfor %} | ||
| To download the archived documentation for this release, see [Archived Documentation]({% link releases/archived-documentation.md %}). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When I look at the rendered preview of the 'Archived Documentation' page v2.1 is not there. Do you still need to make the zip archive and upload it and then edit that page to add a link? Or were you planning to do that in a separate PR?
| @@ -31,17 +31,17 @@ To work through this tutorial, take the following steps: | |||
|
|
|||
| Start DBeaver, and select **Database > New Connection** from the menu. In the dialog that appears, select **CockroachDB** from the list. | |||
|
|
|||
| <img src="{{ 'images/v2.1/dbeaver-01-select-cockroachdb.png' | relative_url }}" alt="DBeaver - Select CockroachDB" style="border:1px solid #eee;max-width:100%" /> | |||
| <img src="{{ 'images/v23.1/dbeaver-01-select-cockroachdb.png' | relative_url }}" alt="DBeaver - Select CockroachDB" style="border:1px solid #eee;max-width:100%" /> | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please don't introduce more cross version links. The right fix is to link to images/v21.1/dbeaver-01-select-cockroachdb.png - if it's not there, please create that folder and copy that image there.
|
|
||
| ## Step 2. Update the connection settings | ||
|
|
||
| On the **Create new connection** dialog that appears, click **Network settings**. | ||
|
|
||
| <img src="{{ 'images/v2.1/dbeaver-02-cockroachdb-connection-settings.png' | relative_url }}" alt="DBeaver - CockroachDB connection settings" style="border:1px solid #eee;max-width:100%" /> | ||
| <img src="{{ 'images/v23.1/dbeaver-02-cockroachdb-connection-settings.png' | relative_url }}" alt="DBeaver - CockroachDB connection settings" style="border:1px solid #eee;max-width:100%" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same comment here re: please don't introduce new cross-version links. I don't want to leave a comment on every single instance in this PR, please check with grep or similar to make sure we're not adding more cross-version links to other versions. The right fix for these instances is in my previous comment
| @@ -32,17 +32,17 @@ To work through this tutorial, take the following steps: | |||
|
|
|||
| Start DBeaver, and select **Database > New Connection** from the menu. In the dialog that appears, select **CockroachDB** from the list. | |||
|
|
|||
| <img src="{{ 'images/v2.1/dbeaver-01-select-cockroachdb.png' | relative_url }}" alt="DBeaver - Select CockroachDB" style="border:1px solid #eee;max-width:100%" /> | |||
| <img src="{{ 'images/v23.1/dbeaver-01-select-cockroachdb.png' | relative_url }}" alt="DBeaver - Select CockroachDB" style="border:1px solid #eee;max-width:100%" /> | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same as above re: please don't introduce more cross-version linking, please fix by creating the v21.2 images directory and copying the images there. This stinks but it will make the future cross-version link removal PRs easier
Summary
Archives CockroachDB v2.1 documentation following the established v19.2 pattern. This removes
outdated documentation while preserving the release page with proper archived notice and
support policy links.
Changes Made
🗂️ Core Archival
/v2.1/folder with allversion-specific docs
/images/v2.1/directorysidebar-data-v2.1.jsonto disable sidebar onarchived page
_includes/v2.1/and_includes/releases/v2.1/directories
🔗 Cross-Version Reference Fixes (78+ references)
{% include v2.1/client-transaction-retry.md %}with inline transaction retry content
images/common/directoriesuse common directory
docs.cockroachlabs.com/docs/v2.1/links to point tostable docs
version
📄 Release Page Updates
releases/v2.1.mdfrom dynamic content to staticarchived notice
#v2-1-0through#v2-1-11, plus alpha/beta/RC variants) to prevent broken links from unsupported-versionspage
Validation
navigation
Impact
downloads