Skip to content

Add servicelinker enrichment step to experimental import pipeline#775

Merged
henrik242 merged 5 commits intomasterfrom
henrik/servicelinker-in-pipeline
Feb 24, 2026
Merged

Add servicelinker enrichment step to experimental import pipeline#775
henrik242 merged 5 commits intomasterfrom
henrik/servicelinker-in-pipeline

Conversation

@henrik242
Copy link
Contributor

Insert servicelinker between Antu pre-validation and Ashur filtering. On success, the enriched NeTEx file replaces the original in the internal bucket before Ashur proceeds. On failure, the pipeline continues with the original file (graceful degradation). Enabled in dev only for now.

Insert servicelinker between Antu pre-validation and Ashur filtering.
On success, the enriched NeTEx file replaces the original in the internal
bucket before Ashur proceeds. On failure, the pipeline continues with the
original file (graceful degradation). Enabled in dev only for now.
@henrik242 henrik242 requested a review from erlendev February 20, 2026 14:05
Store the enriched file at the servicelinker exchange path in the
internal bucket instead of overwriting the original. This ensures
the nightly job always re-enriches from the clean operator-provided
data and picks up any changes in stop coordinates or provider config.
Read generateMissingServiceLinksForModes from ChouetteInfo and serialise
it as a comma-separated ServiceLinkModes PubSub attribute before triggering
servicelinker. Empty set signals no generation; null omits the header so
servicelinker falls back to generating for all modes (backward compat).
…ink modes

When generateMissingServiceLinksForModes is an explicitly empty set, bypass
the servicelinker pubsub round-trip and route directly to ashur. Avoids
unnecessary GCS copies and network calls for providers that should never
have service links generated.
@sonarqubecloud
Copy link

Copy link
Contributor

@erlendev erlendev left a comment

Choose a reason for hiding this comment

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

LGTM

@henrik242 henrik242 merged commit c83cc9e into master Feb 24, 2026
9 checks passed
@henrik242 henrik242 deleted the henrik/servicelinker-in-pipeline branch February 24, 2026 12:18
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