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.

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

Removed a trailing "`" in the xpack.ml.model_repository parameter #2779

Closed
wants to merge 18 commits into from

Conversation

ivssh
Copy link
Contributor

@ivssh ivssh commented Sep 3, 2024

Removed a trailing "`" in the documentation. This was causing errors in starting the elasticsearch node with bootstrap checks failing.

mergify bot and others added 12 commits July 5, 2024 18:42
…ment. (elastic#2744) (elastic#2745)

(cherry picked from commit b065d61)

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
…SER and E5 are recommended. (elastic#2750) (elastic#2751)

(cherry picked from commit 6625cb1)

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
(cherry picked from commit ffe6cd0)

Co-authored-by: Valeriy Khakhutskyy <1292899+valeriy42@users.noreply.github.com>
(cherry picked from commit dfe0bc4)

Co-authored-by: Quentin Pradet <quentin.pradet@elastic.co>
…stic#2763) (elastic#2764)

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
… and E5 (elastic#2765) (elastic#2767)

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
Typos from traned to trained
* Adds categorization job how to guide.

* Corrects image name.

(cherry picked from commit e132d9b)

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
…#2776)

(cherry picked from commit 5b42824)

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
(cherry picked from commit 437db55)

Co-authored-by: Liam Thompson <32779855+leemthompo@users.noreply.github.com>
Removed a trailing "`" in the documentation. This was causing errors in starting the elasticsearch node with bootstrap checks failing.
@ivssh ivssh requested a review from a team as a code owner September 3, 2024 12:50
Copy link

github-actions bot commented Sep 3, 2024

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

ivssh and others added 6 commits September 3, 2024 18:24
Removed a misstyped "le" from a previous commit
Removed the trailing quotes from xpack.ml.model_path, this was causing the bootstrap checks to fail
…c#2782)

(cherry picked from commit 52f8272)

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
…tic#2786)

(cherry picked from commit 10226bf)

Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
@szabosteve szabosteve added :ml backport-8.17 Automated backport with mergify backport-8.x Automated backport with mergify v8.17 v8.18 v9.0 labels Jan 29, 2025
szabosteve
szabosteve previously approved these changes Jan 29, 2025
@szabosteve szabosteve changed the base branch from 8.15 to main January 29, 2025 09:08
@szabosteve szabosteve dismissed their stale review January 29, 2025 09:08

The base branch was changed.

@szabosteve
Copy link
Contributor

@elasticmachine update branch

Copy link
Contributor

mergify bot commented Jan 29, 2025

This pull request is now in conflict. Could you fix it @ivssh? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch <remote-repo>
git checkout -b patch-1 <remote-repo>/patch-1
git merge <remote-repo>/main
git push <remote-repo> patch-1

@szabosteve
Copy link
Contributor

I'm closing this now because I don't have permission to push against the remote of this PR and the docs freeze almost here. Sorry about the inconvenience and thank you for the contribution. I'll add these changes in a separate PR.

@szabosteve szabosteve closed this Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-8.x Automated backport with mergify backport-8.17 Automated backport with mergify :ml v8.17 v8.18 v9.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants