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

Remove ECE 3.7 #2999

Merged
merged 2 commits into from
May 7, 2024
Merged

Conversation

florent-leborgne
Copy link
Contributor

This PR removes ECE 3.7 from the build as per ongoing incident around ECE https://elastic.slack.com/archives/C071N20PEF7

This PR can be reverted once ECE 3.7.1 version gets released, in the coming days

Copy link

github-actions bot commented May 7, 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.

@florent-leborgne
Copy link
Contributor Author

run docs-build rebuild

@florent-leborgne
Copy link
Contributor Author

Will re-run once links are fixed in cloud repo

@florent-leborgne
Copy link
Contributor Author

run docs-build rebuild

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! 🦒
Nice to see the CI checks passing now! maybe the ms-107 backport for the broken link fix doesn't matter.

@florent-leborgne
Copy link
Contributor Author

@kilfoyle it will matter on thursday when ms-107 releases, but right now ESS is still on ms-106, and that backport went through correctly

@florent-leborgne florent-leborgne merged commit a8a9f32 into elastic:master May 7, 2024
3 checks passed
This pull request was closed.
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.

2 participants