Skip to content

Conversation

@Mpdreamz
Copy link
Member

@Mpdreamz Mpdreamz commented Oct 23, 2025

Added a lot more logging too:

content-source match elastic/elasticsearch 9.1

info ::e.d.c.a.a.tSourceMatch:: NO speculative build 9.1 is lte product current '9.2.0'
info ::e.d.a.c.atchingService:: 'elastic/elasticsearch' '9.1' combination not found in configuration.

::set-output name=content-source-match::false
::set-output name=content-source-next::false
::set-output name=content-source-edge::false
::set-output name=content-source-current::false
::set-output name=content-source-speculative::false

assembler content-source match elastic/elasticsearch 9.2

nfo ::e.d.c.a.a.tSourceMatch:: Current is not using versioned branches checking product info
info ::e.d.c.a.a.tSourceMatch:: Speculative build 9.2 is gte product current '9.2.0'

::set-output name=content-source-match::true
::set-output name=content-source-next::false
::set-output name=content-source-current::false
::set-output name=content-source-edge::false
::set-output name=content-source-speculative::true

assembler content-source match elastic/docs-content 9.2

info ::e.d.c.a.a.tSourceMatch:: No versioning system found for elastic/docs-content on 9.2
info ::e.d.a.c.atchingService:: 'elastic/docs-content' '9.2' combination not found in configuration.

::set-output name=content-source-match::false
::set-output name=content-source-next::false
::set-output name=content-source-edge::false
::set-output name=content-source-current::false
::set-output name=content-source-speculative::false

New repositories will always build for integration branches:

assembler content-source match elastic/unknown 9.2

info ::e.d.c.a.a.tSourceMatch:: Repository elastic/unknown has not yet been onboarded into assembler.yml
info ::e.d.c.a.a.tSourceMatch:: Speculatively building elastic/unknown since it looks like an integration branch

::set-output name=content-source-match::true
::set-output name=content-source-next::false
::set-output name=content-source-current::false
::set-output name=content-source-edge::false
::set-output name=content-source-speculative::true

@Mpdreamz Mpdreamz requested a review from a team as a code owner October 23, 2025 10:42
@Mpdreamz Mpdreamz requested a review from cotti October 23, 2025 10:42
@Mpdreamz Mpdreamz added the fix label Oct 23, 2025
@Mpdreamz Mpdreamz self-assigned this Oct 23, 2025
Copy link
Member

Choose a reason for hiding this comment

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

Hmmmm.. I also noticed this in one of my PRs. hmmm

@Mpdreamz Mpdreamz enabled auto-merge (squash) October 23, 2025 10:53
@Mpdreamz Mpdreamz merged commit cb8c7df into main Oct 23, 2025
22 checks passed
@Mpdreamz Mpdreamz deleted the fix/match-checks branch October 23, 2025 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants