Skip to content

Import the host description when discovering desktops#57962

Merged
zmb3 merged 1 commit intomasterfrom
zmb3/desktop-ldap-description
Aug 19, 2025
Merged

Import the host description when discovering desktops#57962
zmb3 merged 1 commit intomasterfrom
zmb3/desktop-ldap-description

Conversation

@zmb3
Copy link
Copy Markdown
Collaborator

@zmb3 zmb3 commented Aug 16, 2025

Include the description attribute in LDAP queries, and set the resulting description as the metadata description on any Teleport desktop resources we create.

This is not surfaced anywhere in the UI (though that would be a nice addition in the future).

Updates #55148

Changelog: Windows desktop LDAP discovery now auto-populates the resource's description field.

Include the decription attribute in LDAP queries, and set the
resulting description as the metdata description on any Teleport
desktop resources we create.

This is not surfaced anywhere in the UI (though that would be a
nice addition in the future).

Updates #55148
@zmb3
Copy link
Copy Markdown
Collaborator Author

zmb3 commented Aug 16, 2025

teleport on zmb3/desktop-ldap-description [$] ➜ tctl get windows_desktop/EC2AMAZ-H7C7QIS-child-root-zac-local-b6692126 | grep desc
  description: this is a description for child

@zmb3 zmb3 added this pull request to the merge queue Aug 19, 2025
Merged via the queue into master with commit 6a64549 Aug 19, 2025
45 checks passed
@zmb3 zmb3 deleted the zmb3/desktop-ldap-description branch August 19, 2025 21:31
@backport-bot-workflows
Copy link
Copy Markdown
Contributor

@zmb3 See the table below for backport results.

Branch Result
branch/v16 Create PR
branch/v17 Create PR
branch/v18 Create PR

mmcallister pushed a commit that referenced this pull request Sep 22, 2025
Include the decription attribute in LDAP queries, and set the
resulting description as the metdata description on any Teleport
desktop resources we create.

This is not surfaced anywhere in the UI (though that would be a
nice addition in the future).

Updates #55148
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants