Skip to content

Conversation

@colleenmcginnis
Copy link
Contributor

⚠️ Wait until elastic/beats#44682 + backport are merged

Update assembler config to publish Beats documentation from the 9.0 branch.

cc @belimawr

@colleenmcginnis colleenmcginnis self-assigned this Jun 5, 2025
@colleenmcginnis colleenmcginnis added the automation packaging, ci/cd. label Jun 5, 2025
@colleenmcginnis colleenmcginnis changed the title publish beats docs from 9.0 Publish beats docs from the 9.0 branch Jun 5, 2025
apm-k8s-attacher:
beats:
current: "9.0"
next: main
Copy link

Choose a reason for hiding this comment

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

Can we have next as 9.1? That would ensure we don't publish main again by accident and that once v9.1.0 is released, there won't be any unreleased features in our docs.

Copy link
Contributor Author

@colleenmcginnis colleenmcginnis Jun 5, 2025

Choose a reason for hiding this comment

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

next only refers to which branch will be published to our internal staging site. There's no automation in place that will replace the current value with the next value.

As more repos start using different current / next branches (and as development in this repo continues), we'll likely replace the value of current with a variable like stack_version and the docs release manager will be responsible for updating the latest Stack version just like we did with AsciiDoc.

(@Mpdreamz please correct me if I got anything wrong!)

Copy link
Member

Choose a reason for hiding this comment

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

You didn't :)

@Mpdreamz
Copy link
Member

Mpdreamz commented Jun 6, 2025

This LGTM we just need to wait for

elastic/beats#44682 (review)

And its backport to 9.0 so that beats publishes its links from the 9.0 branch.

After that ci / validate-assembler (pull_request)` can be rerun and should go green.

@colleenmcginnis colleenmcginnis marked this pull request as ready for review June 6, 2025 13:17
@colleenmcginnis colleenmcginnis requested a review from a team as a code owner June 6, 2025 13:17
@colleenmcginnis colleenmcginnis enabled auto-merge (squash) June 6, 2025 13:17
Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

https://elastic-docs-link-index.s3.us-east-2.amazonaws.com/link-index.json looking good and checks are happy.

LGTM, can I wait merging this till Monday? We need to do a new release to get the assembler.yml to production.

@colleenmcginnis colleenmcginnis merged commit 1187fa2 into main Jun 6, 2025
16 checks passed
@colleenmcginnis colleenmcginnis deleted the cmcg-beats-9.0 branch June 6, 2025 13:22
@Mpdreamz
Copy link
Member

Mpdreamz commented Jun 6, 2025

Ahh automerge! :) Will wait with releasing this and ensuring production reflects assembler.yml till monday.

@colleenmcginnis
Copy link
Contributor Author

Sorry! 🙈

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation packaging, ci/cd.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants