-
Notifications
You must be signed in to change notification settings - Fork 47
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
Missing modules #8
Comments
…at the same time Changed the SessionHost parameter to SessionHosts Added the following resources from the azure-quickstart-templates/rds-deployment/ repo to Fix dsccommunity#8 : xRDGatewayConfiguration, xRDLicenseConfiguration, xRDServer
All mentioned above, both in Greg's and Jordan's is needed for this resource to be usefull in bigger automation scenarios, or even small for that matter... |
@kwirkykat Do you see any licensing / copyright implications from taking the latest version from the quickstart gallery and merging it back into this repository? I've checked the module and it still ships with an MIT license and looks like it was a direct fork at some point in the past. |
@mgreenegit @kwirkykat can you give a quick answer on @ld0614 question? We are about doing something similar in xFailOverCluster, moving code from a quick start template. |
As per @scottpas issue #31 these are also located here https://github.com/Azure/RDS-Templates/tree/master/DSC/Configuration/xRemoteDesktopSessionHost/DSCResources one option is to move what is missing there instead. Here seems to be the location of the qucikstart template; https://github.com/Azure/azure-quickstart-templates/tree/master/rds-deployment |
Yes, let's merge work from the quickstart templates to the DSC resources. Both use the MIT license so it is safe to do, and I will let their team know. It would be nice to give credit in the readme. |
Then we have a go, so If someone in the community like start adding these resource here, feel free. |
I've just made public the work that I've been doing to merge the quickstart DSC and upgrade the module to support High Availability. This work can be found here. I believe the code to be working (its already in production with some clients) however there are a couple of linting failures and a complete lack of unit testing for the PR. I hope to fix these issues over the next few weeks prior to submitting a formal PR. Obviously I'm more that happy to receive feedback on the changes, please let me know if anyone has any comments, suggestions or concerns. @kwirkykat This work is split across the quickstart template work, work completed personally (I've already signed a personal CLA) and work completed on behalf of the company I work for (PowerON Platforms). I've got documented approval to submit the corporate work into the Resource kit however as I've only signed a personal CLA, do I need to sign anything else prior to submitting a PR? |
@ld0614 If you click the 'Details' link after the CLA check you can read the CLA. I see it says (but please read the entire CLA and the section Your Employer):
|
@johlju when I went to the CLA bot I wasn't able to change my declaration from personal to on behalf of a business, while this may not be an issue I just wanted to be completely upfront with the provenance of the code to ensure that there are no issues in the future |
I have only seen one type of CLA, the one CLA which you personally sign. So I think the last phrase that I quoted says that when you sign it you also have made sure that you have permission to sign it for your employer (if your work was made as an employee). If you have a document saying that you are allowed to submission your work you made as an employee and have signed the CLA I think your good to go. |
@ld0614 I see now what you mean about “behalf of a business”. In this link https://cla.opensource.microsoft.com there is a sample pdf that has the “behalf of a business”, but the CLA in the checks section in the PR’s does not? 🤔 |
@ld0614 I will try to sort this out next week so we get a clear answer. |
@ld0614 I have a contact now, and I will send you an e-mail. |
I think this was resolved in PR #33. |
We've been working with the xRemoteDesktopSessionHost DSC module that is included with azure-quickstart-templates/rds-deployment/. This module includes several other resources for configuring a RDS environment that are not included in this published version. I assume these were built by the RDS team but it would be nice if they were either combined into this module or a new DSC module created (something like xRemoteDesktopConfiguration) to include the resources that are missing from this module. Those resources are:
xRDGatewayConfiguration
xRDLicenseConfiguration
xRDServer
Let us know if there are any plans to provide/maintain these in the PowerShell Gallery.
Thanks...Greg
The text was updated successfully, but these errors were encountered: