Skip to content

Conversation

@tkajtoch
Copy link
Member

Summary

Resolves https://github.com/elastic/eui-private/issues/286

This PR updates the new eui-deploy-docs pipeline to run when a release tag is created and publishes the built files accordingly. It also includes a few cleanup items:

  1. Disables the eui-release pipeline since it's not used now and is constantly timing out
  2. Removes eui-pull-request-deploy-docs pipeline that was handling building the old PR preview documentation website and is not needed anymore
  3. Removes eui-release-deploy-docs pipeline that was handling building the old production documentation site

QA

Unfortunately, there's no way to test these changes before merging. Please verify that the updated catalog-info.yaml is valid and the updated bash script looks correct.

@tkajtoch tkajtoch added the skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation) label Apr 16, 2025
@tkajtoch tkajtoch self-assigned this Apr 16, 2025
@tkajtoch tkajtoch requested a review from a team as a code owner April 16, 2025 22:02
Copy link
Contributor

@weronikaolejniczak weronikaolejniczak left a comment

Choose a reason for hiding this comment

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

Looking good! I checked the catalog-info.yaml validity:

Screenshot 2025-04-17 at 10 03 56

I have some questions, doubts though.

@tkajtoch tkajtoch force-pushed the build/publish-new-docs-site branch from db20d9b to e37ffa0 Compare April 21, 2025 08:10
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @tkajtoch

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

History

cc @tkajtoch

Copy link
Contributor

@weronikaolejniczak weronikaolejniczak left a comment

Choose a reason for hiding this comment

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

Looking good 💪🏻

@tkajtoch tkajtoch merged commit a685b47 into elastic:main Apr 22, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants