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

Fix update and list for DynamicWindowsDesktops #48046

Merged
merged 11 commits into from
Oct 29, 2024

Conversation

probakowski
Copy link
Contributor

This PR adds UpsertDynamicWindowsDesktop method to the client and uses it in tctl create command.
It also fixes UpdateDynamicWindowsDesktop to conditionally update resource only if revision matches.
It also fixes problem with ListDynamicWindowsDesktops and cache

@github-actions github-actions bot added size/sm tctl tctl - Teleport admin tool labels Oct 28, 2024
@probakowski probakowski added the no-changelog Indicates that a PR does not require a changelog entry label Oct 28, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-48046.d3pp5qlev8mo18.amplifyapp.com

@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from fspmarshall October 29, 2024 18:43
@probakowski probakowski enabled auto-merge October 29, 2024 20:14
@probakowski probakowski added this pull request to the merge queue Oct 29, 2024
Merged via the queue into master with commit dddd9ea Oct 29, 2024
43 checks passed
@probakowski probakowski deleted the probakowski/registration-update branch October 29, 2024 22:37
@public-teleport-github-review-bot

@probakowski See the table below for backport results.

Branch Result
branch/v17 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/branch/v17 desktop-access no-changelog Indicates that a PR does not require a changelog entry size/sm tctl tctl - Teleport admin tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants