Skip to content
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.

Sign up for GitHub

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

Jump to bottom

bumps current version to 8.10 #2756

Merged
merged 2 commits into from
Sep 12, 2023
Merged

bumps current version to 8.10 #2756

merged 2 commits into from
Sep 12, 2023

Conversation

benironside
Copy link
Contributor

This PR changes the "current" version of the documentation to 8.9 and changes that set of docs from "unreleased" to "released".

Do not merge until release day.

Based on #2722

@benironside benironside self-assigned this Sep 10, 2023
@benironside
Copy link
Contributor Author

The following build error is happening:

10:16:57 INFO:build_docs:Bad cross-document links:
10:16:57 INFO:build_docs:  /tmp/docsbuild/target_repo/html/en/cloud-enterprise/3.4/ece-configure-as-remote-clusters.html contains broken links to:
10:16:57 INFO:build_docs:   - en/elasticsearch/reference/8.10/remote-clusters-connect.html#configure-remote-clusters-dynamic
10:16:57 INFO:build_docs:  /tmp/docsbuild/target_repo/html/en/cloud-enterprise/3.5/ece-configure-as-remote-clusters.html contains broken links to:
10:16:57 INFO:build_docs:   - en/elasticsearch/reference/8.10/remote-clusters-connect.html#configure-remote-clusters-dynamic
10:16:57 INFO:build_docs:  /tmp/docsbuild/target_repo/html/en/cloud-enterprise/3.6/ece-configure-as-remote-clusters.html contains broken links to:
10:16:57 INFO:build_docs:   - en/elasticsearch/reference/8.10/remote-clusters-connect.html#configure-remote-clusters-dynamic
10:16:57 INFO:build_docs:  /tmp/docsbuild/target_repo/html/en/cloud-enterprise/current/ece-configure-as-remote-clusters.html contains broken links to:
10:16:57 INFO:build_docs:   - en/elasticsearch/reference/8.10/remote-clusters-connect.html#configure-remote-clusters-dynamic
10:16:57 INFO:build_docs:  /tmp/docsbuild/target_repo/html/en/cloud/current/ec-configure-as-remote-clusters.html contains broken links to:
10:16:57 INFO:build_docs:   - en/elasticsearch/reference/8.10/remote-clusters-connect.html#configure-remote-clusters-dynamic
10:16:57 INFO:build_docs:  /tmp/docsbuild/target_repo/html/en/cloud/latest/ec-configure-as-remote-clusters.html contains broken links to:
10:16:57 INFO:build_docs:   - en/elasticsearch/reference/8.10/remote-clusters-connect.html#configure-remote-clusters-dynamic
10:17:28 runbld>>> <<<<<<<<<<<< SCRIPT EXECUTION END <<<<<<<<<<<<

@abdonpijpelink abdonpijpelink mentioned this pull request Sep 11, 2023
Merged
@abdonpijpelink
Copy link
Contributor

The build error should be resolved by elastic/elasticsearch#99401

@abdonpijpelink
Copy link
Contributor

@elasticmachine run elasticsearch-ci/docs

lcawl
lcawl approved these changes Sep 11, 2023
View reviewed changes
Copy link
Contributor

@lcawl lcawl left a comment

Choose a reason for hiding this comment

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

LGTM!

@jrodewig
Copy link
Contributor

jrodewig commented Sep 11, 2023 •
edited
Loading

Rerunning the docs check since I discovered elastic/elasticsearch#99371 broke the build during backports. Should be fixed now, but I want to ensure the release goes smoothly.

@jrodewig
Copy link
Contributor

@elasticmachine run elasticsearch-ci/docs

@jrodewig jrodewig merged commit 92c8cbb into master Sep 12, 2023
@jrodewig jrodewig deleted the bgoldstein-8.10.0-release branch September 12, 2023 13:56
jrodewig added a commit that referenced this pull request Sep 15, 2023
@jrodewig @benironside
Revert "bumps current version to 8.10 (#2756)" (#2762)
9b85e77 
Co-authored-by: Benjamin Ironside Goldstein <[email protected]>
kilfoyle pushed a commit that referenced this pull request Sep 18, 2023
@jrodewig @benironside
Bump docs version to 8.10.1 (#2761)
c0a57c3 
* Bump docs version to 8.10.1

* bumps current version to 8.10 (#2756)

---------

Co-authored-by: Benjamin Ironside Goldstein <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lcawl lcawl lcawl approved these changes

@jrodewig jrodewig Awaiting requested review from jrodewig

Assignees

@benironside benironside

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@benironside @abdonpijpelink @jrodewig @lcawl