Skip to content
This repository has been archived by the owner on Jan 24, 2023. It is now read-only.

internal agent default managenent host urls #14

Merged
merged 1 commit into from
Dec 22, 2020
Merged

internal agent default managenent host urls #14

merged 1 commit into from
Dec 22, 2020

Conversation

valeriano-manassero
Copy link
Contributor

@valeriano-manassero valeriano-manassero commented Dec 21, 2020

Since Agents deployed into the cluster with the charts are directly managed by k8s, I think it's better to go to internal service url.

With these urls there are no ingresses configuration involved and the communications are private between the nodes without the need of any extra layer resulting in easier setup.
Moreover communications between nodes are encrypted by default.

As final result the engineer deploying the cluster will not need to set any specific url since these one works by design.

@sapir-allegro
Copy link
Contributor

@valeriano-manassero this seems better for me as well, we'll merge it soon!

@clearml-bot clearml-bot merged commit 0992006 into clearml:master Dec 22, 2020
@valeriano-manassero valeriano-manassero deleted the agent-internal-references branch December 22, 2020 21:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants