Skip to content

fix: create jaeger ExternalName services in train-ticket setup job#11

Open
somyas25 wants to merge 2 commits intoxlab-uiuc:masterfrom
somyas25:jaeger-issue-somyas3
Open

fix: create jaeger ExternalName services in train-ticket setup job#11
somyas25 wants to merge 2 commits intoxlab-uiuc:masterfrom
somyas25:jaeger-issue-somyas3

Conversation

@somyas25
Copy link
Copy Markdown

Problem

The train-ticket pods crash on startup if the Jaeger ExternalName services don't exist in the train-ticket namespace before deployment. The conductor's create_external_name_service() runs after app.deploy(), which is too late for train-ticket.
More details about the problem are described in this issue #558.

Fix

I added an initContainer to the existing train-ticket-deploy job that creates the Jaeger ExternalName services before the main deploy container starts. The initContainer checks if each service already exists before creating it so that it works correctly regardless of whether the services were already created by the conductor or didn't exist yet.

Testing

I ran problem 17 containing train-ticket application deployment to check if it correctly works.
image

Previous Attempt

I tried adding ExternalName services directly to the Helm chart template: caused Helm ownership conflicts.

Jaeger not functional in train-ticket fix 2

Jaeger not functional in train-ticket fix 3
@somyas25 somyas25 marked this pull request as draft March 20, 2026 00:15
@somyas25 somyas25 marked this pull request as ready for review March 20, 2026 00:16
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.

1 participant