Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

draft: Add API to Telepresence #3568

Draft
wants to merge 18 commits into
base: release/v2
Choose a base branch
from
Draft

Conversation

thallgren
Copy link
Contributor

No description provided.

@thallgren thallgren marked this pull request as draft April 9, 2024 05:33
@thallgren thallgren force-pushed the thallgren/cli-api branch 4 times, most recently from 58207fd to 9f182f3 Compare April 9, 2024 07:21
@thallgren thallgren added the ok to test Applied by maintainers when a PR is ready to have tests run on it label Apr 9, 2024
@github-actions github-actions bot removed the ok to test Applied by maintainers when a PR is ready to have tests run on it label Apr 9, 2024
@thallgren thallgren added the ok to test Applied by maintainers when a PR is ready to have tests run on it label Apr 9, 2024
@github-actions github-actions bot removed the ok to test Applied by maintainers when a PR is ready to have tests run on it label Apr 9, 2024
@thallgren thallgren force-pushed the thallgren/cli-api branch 3 times, most recently from c931c96 to 6a3d1ed Compare April 9, 2024 14:52
@thallgren thallgren added the ok to test Applied by maintainers when a PR is ready to have tests run on it label Apr 9, 2024
@github-actions github-actions bot removed the ok to test Applied by maintainers when a PR is ready to have tests run on it label Apr 9, 2024
@thallgren thallgren added the ok to test Applied by maintainers when a PR is ready to have tests run on it label Apr 9, 2024
@github-actions github-actions bot removed the ok to test Applied by maintainers when a PR is ready to have tests run on it label Apr 9, 2024
@thallgren thallgren force-pushed the thallgren/cli-api branch 4 times, most recently from 4661cc4 to 79c48f3 Compare April 12, 2024 11:21
@thallgren thallgren force-pushed the thallgren/cli-api branch 5 times, most recently from 3b83371 to 6d5efa9 Compare April 24, 2024 07:45
The logic for detecting if a cluster is a local Kind cluster, and
therefore needs some special attention when using `telepresence connect
--docker`, relied on the presence of `Aliases` in the Docker network
that a Kind cluster sets up. In Docker versions from 26 and up, this
value is no longer used, but the corresponding info can instead be found
in the new `DNSNames` field.

Signed-off-by: Thomas Hallgren <thomas@datawire.io>
Signed-off-by: Thomas Hallgren <thomas@datawire.io>
The time spent by a connection must be accumulated, and time spent when
the connection is dormant (no remain calls arrive from the client) must
not be counted.

This commit ensures that all remain-calls add the proper time and that
sequences of more than 4 missed remain calls are counted as one. This
means when a user closes the laptop lid, the connection will be
considered dormant.

Signed-off-by: Thomas Hallgren <thomas@datawire.io>
The traffic-manager considers some events as fatal when waiting for
traffic-agent to arrive after an injection has been initiated. This
logic would trigger on events like "Warning FailedScheduling 0/63 nodes
are available" although those events indicate a recoverable condition
and kill the wait. This is now fixed so that the events are logged but
the wait continues.

Signed-off-by: Thomas Hallgren <thomas@datawire.io>
Signed-off-by: Thomas Hallgren <thomas@datawire.io>
Signed-off-by: Thomas Hallgren <thomas@datawire.io>
This message may vary depending on what the container is doing. If it
started a debugger, then the message should state that we're waiting
for a debugger frontend to attach. If not, the message should probably
be "type <ctrl>-C to exit...".

Signed-off-by: Thomas Hallgren <thomas@datawire.io>
Signed-off-by: Thomas Hallgren <thomas@datawire.io>
Signed-off-by: Thomas Hallgren <thomas@datawire.io>
Signed-off-by: Thomas Hallgren <thomas@datawire.io>
Signed-off-by: Thomas Hallgren <thomas@datawire.io>
Signed-off-by: Thomas Hallgren <thomas@datawire.io>
Signed-off-by: Thomas Hallgren <thomas@datawire.io>
Adds a client identifier to the `ConnectRequest` structure used when
connecting via the API. If set, this identifier will override the
default `<user>@<host>` value of the `client` property in the
`ClientInfo` structure that is passed to the manager's `ArriveAsClient`
method.

Signed-off-by: Thomas Hallgren <thomas@datawire.io>
The new attribute is included when the workload is intercepted. The
entries contain the client-id of the currently intercepting users.

Signed-off-by: Thomas Hallgren <thomas@datawire.io>
Signed-off-by: Thomas Hallgren <thomas@datawire.io>
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.

None yet

1 participant