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

Create WindowsDesktops for matching DynamicWindowsDesktops #46991

Merged
merged 80 commits into from
Oct 23, 2024

Conversation

probakowski
Copy link
Contributor

This PR enables Windows Desktop Services to create WindowsDesktops based on matching DynamicWindowsDesktops to allow dynamic registration. Matching will be done based on resource matchers added in #46985

This is part of splitting #46738 into smaller, easier to review, PRs

Closes #37103

changelog: Support dynamic registration for Windows desktops

@probakowski probakowski marked this pull request as ready for review October 1, 2024 18:17
@github-actions github-actions bot requested review from greedy52 and hugoShaka October 1, 2024 18:17
Copy link
Contributor

@greedy52 greedy52 left a comment

Choose a reason for hiding this comment

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

LGTM. any tests will cover this logic?

@probakowski
Copy link
Contributor Author

LGTM. any tests will cover this logic?

Yes, I've added tests

@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from hugoShaka October 18, 2024 21:47
Base automatically changed from probakowski/register-resources-cache to master October 22, 2024 21:40
@probakowski probakowski enabled auto-merge October 23, 2024 17:59
@probakowski probakowski added this pull request to the merge queue Oct 23, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 23, 2024
@probakowski probakowski added this pull request to the merge queue Oct 23, 2024
@probakowski probakowski removed this pull request from the merge queue due to a manual request Oct 23, 2024
@probakowski probakowski added this pull request to the merge queue Oct 23, 2024
@probakowski probakowski removed this pull request from the merge queue due to a manual request Oct 23, 2024
@probakowski probakowski added this pull request to the merge queue Oct 23, 2024
Merged via the queue into master with commit f651ab1 Oct 23, 2024
40 checks passed
@probakowski probakowski deleted the probakowski/register-resources-discovery branch October 23, 2024 20:53
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