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

Added fleet-server known issue 7.17.20 #1006

Merged
merged 8 commits into from
Apr 9, 2024
Merged

Conversation

juliaElastic
Copy link
Contributor

@juliaElastic juliaElastic commented Apr 9, 2024

@juliaElastic juliaElastic requested review from davidkyle and a team April 9, 2024 07:09
@juliaElastic
Copy link
Contributor Author

Build fails with this issue:

INFO:build_docs:All cross-document links OK
INFO:build_docs:Saving branch tracker
INFO:build_docs:Building sitemap
INFO:build_docs:Commiting changes
INFO:build_docs:Pushing changes
INFO:build_docs:Error executing: git push origin ingest-docs_bk_1006 in GIT_DIR /docs_build/.repos/target_repo.git
INFO:build_docs:---out---
INFO:build_docs:
INFO:build_docs:---err---
INFO:build_docs:Failed to add the ECDSA host key for IP address '140.82.113.3' to the list of known hosts (/tmp/.ssh/known_hosts).
INFO:build_docs:To github.com:elastic/built-docs
INFO:build_docs: ! [rejected]                    ingest-docs_bk_1006 -> ingest-docs_bk_1006 (fetch first)
INFO:build_docs:error: failed to push some refs to 'git@github.com:elastic/built-docs'
INFO:build_docs:hint: Updates were rejected because the remote contains work that you do
INFO:build_docs:hint: not have locally. This is usually caused by another repository pushing
INFO:build_docs:hint: to the same ref. You may want to first integrate the remote changes
INFO:build_docs:hint: (e.g., 'git pull ...') before pushing again.
INFO:build_docs:hint: See the 'Note about fast-forwards' in 'git push --help' for details.
INFO:build_docs:
INFO:build_docs:---------
🚨 Error: The command exited with status 1
user command error: exit status 1

Copy link
Contributor

@jlind23 jlind23 left a comment

Choose a reason for hiding this comment

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

Code change looks good to me.

@leemthompo
Copy link

@juliaElastic CI is now green after rebuild :)

…iidoc

Co-authored-by: David Kilfoyle <41695641+kilfoyle@users.noreply.github.com>
Copy link
Contributor

@kilfoyle kilfoyle left a comment

Choose a reason for hiding this comment

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

LGTM! Just two small suggestions.
Thanks for adding this @juliaElastic!

@juliaElastic
Copy link
Contributor Author

Thanks @kilfoyle for the review! I added some more details that the issue impacts ESS too, as discussed with Julien and the team.

juliaElastic and others added 2 commits April 9, 2024 14:58
…iidoc

Co-authored-by: David Kilfoyle <41695641+kilfoyle@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants