Skip to content

feat: Orchestrator - Launchers - Kubernetes - Start labelling created Pods, Jobs and Services with a Tangle label - #341

Open
Ark-kun wants to merge 1 commit into
masterfrom
feat--Orchestrator---Launchers---Kubernetes---Start-labelling-created-Pods,-Jobs-and-Services-with-a-Tangle-label
Open

feat: Orchestrator - Launchers - Kubernetes - Start labelling created Pods, Jobs and Services with a Tangle label#341
Ark-kun wants to merge 1 commit into
masterfrom
feat--Orchestrator---Launchers---Kubernetes---Start-labelling-created-Pods,-Jobs-and-Services-with-a-Tangle-label

Conversation

@Ark-kun

@Ark-kun Ark-kun commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

This is useful to quickly identify objects created by Tangle

… Pods, Jobs and Services with a Tangle label

This is useful to quickly identify objects created by Tangle
@Ark-kun
Ark-kun requested a review from a team August 20, 2026 03:54

# Kubernetes annotation keys. (Has strict naming policy. Single slash only etc.)
_CLOUD_PIPELINES_KUBERNETES_ANNOTATION_KEY = "cloud-pipelines.net"
_TANGLE_KUBERNETES_LABEL_KEY = "tangle.tangleml.com"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Question: Would you like me to refactor https://github.com/Shopify/oasis-backend/pull/458 to use this label instead of tangleml.com or refactor this PR to use tangleml.com without the subdomain?

@morgan-wowk morgan-wowk Aug 20, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The default namespace is already created for staging but not yet prod on e09.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

TBH, I do not have an opinion here.
The main reason I added the tangle. prefix was to distinguish Tangle pods from Tangent pods.
I also thought about recent pushes to make Tangent "bigger" than Tangle (although those were scaled down by senior leadership).

I'd say, just proceed with Shopify/oasis-backend#458 as-is. We are already using the tangleml.com in launchers.py. We can refactor all of that later.

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