Skip to content
This repository has been archived by the owner on Feb 24, 2021. It is now read-only.

New resource module: GroupPolicy #101

Open
corydwood opened this issue Feb 17, 2016 · 33 comments · May be fixed by #478
Open

New resource module: GroupPolicy #101

corydwood opened this issue Feb 17, 2016 · 33 comments · May be fixed by #478
Labels
needs review The pull request needs a code review. new module submission A request that the community evaluate a resource module for submission into the DSC Resource Kit.

Comments

@corydwood
Copy link

I have resources for setting GP inheritance, importing GPOs from backups, and linking existing GPOs. These are MOF based resources.

I'm working on updating these to meet the contribution guidelines and would like to contribute them as a new module once they're updated.

Do we still need to name them with an x (ex: xGroupPolicy)?

@corydwood
Copy link
Author

Anyone have a recommendation on what to do about parameter sets? For example, I'm thinking of having two resources for importing GPOs - one for importing by GUID and one for importing by Name because TargetGuid and TargetName are in separate parameter sets for Import-GPO.

Is this the recommended approach?

@Justin-DynamicD
Copy link

If they ultimately both import a resource, I'd instead simply build a single resource and have two parameters: Name and NameFormat. Make "NameFormat" optional and default to "DisplayName" but accept "DisplayName" or "GUID".

That's assuming there's no other difference between the two resources.

@KarolKaczmarek
Copy link
Contributor

HI @corydwood
Glad to hear you are planning on contributing your module. Please still include "x" in your new resources. We will be dropping "x" across all modules in a consistent manner.

Regarding the second question, I agree with @Justin-DynamicD single resource sounds like the way to go here.

All the best!

@KarolKaczmarek KarolKaczmarek added the new module submission A request that the community evaluate a resource module for submission into the DSC Resource Kit. label Feb 19, 2016
@corydwood
Copy link
Author

@Justin-DynamicD - Thanks for the idea! I'll work on coding it that way. @KarolKaczmarek - Thanks for the info. I'll plan on keeping the "x" in the resource name.

@Justin-DynamicD
Copy link

Not to hijack the thread ... but should we still be using "c" for community resources that we publish to places like PSGallery directly or has differentiating between (x)perimental and (c)ommunity kind of fallen to the wayside?

@KarolKaczmarek
Copy link
Contributor

"c" no longer makes sense after we've open sourced ResourceKit on GitHub. It's all community now 👍
Also, we encourage everyone to contribute new resources under PowerShell org.

@Justin-DynamicD
Copy link

we should update the naming standards to reflect that.

@KarolKaczmarek
Copy link
Contributor

What specifically are you referring to?

@Justin-DynamicD
Copy link

its been so long ... i no longer remember. I just remember seeing a naming standards published somewhere that said "x" was ms reserved for experimental, "c" was community, and no prefix were modules officially supported by MS.

Am I dating myself?

@KarolKaczmarek
Copy link
Contributor

You remember correctly, that was our original message regarding naming prefixes in order to allow clear distinction between modules shipped by Microsoft and their forks/modules developed by community. After moving development of DSC Resource Kit on GitHub it's no longer valid and information about it has been removed from all repositories/PSGallery pages.

@Justin-DynamicD
Copy link

Looks like i better update some of my now dated modules/dsc resources then!

@corydwood
Copy link
Author

corydwood commented Feb 23, 2016

I have the xGroupPolicy module ready to submit.

How do I complete the option, "Allow the PowerShell organization to fork your repository and use that fork as a submodule of DscResources"?

Here's the module: https://github.com/corydwood/xGroupPolicy

@PlagueHO
Copy link
Contributor

This tells you what you need to know about transferring the module.

@corydwood
Copy link
Author

@PlagueHO - I saw that. It just tells you the two options for contributing a new module, but not how to do it. It says to follow up in the issue with the option you'd like to use, which I'm doing.

How do I allow the PowerShell organization to fork my repository and use that fork as a submodule of DscResources?

Can you just take the link to the module I included above and just fork it, or do I need to do anything else on my end?

@PlagueHO
Copy link
Contributor

Oh right - no I think the MSFT guys can just review the module with the link you've provided. Pretty sure they don't need anything else. Although only the MSFT team can confirm that 😄

@KarolKaczmarek KarolKaczmarek added the needs review The pull request needs a code review. label May 11, 2016
@kwirkykat
Copy link
Member

@corydwood I just posted a new guide for submitting new resource modules to the DSC Resource Kit.

I cannot access this link you provided to your repo: https://github.com/corydwood/xGroupPolicy

@kwirkykat kwirkykat self-assigned this Jul 25, 2016
@corydwood
Copy link
Author

@kwirkykat I've updated my module with the items from the new guide, except the Examples folder and creating a new, empty branch.

All of the examples are contained in the README.md file. If I put them in an Examples folder as well, there will be duplication of information, which isn't recommended. Should the examples be moved to an Examples folder instead of being in the README.md file?

Can you please expand on the steps to create an empty branch and creating a pull request to that branch? I haven't had any luck doing this by following steps I've found online.

I just tested accessing https://github.com/corydwood/xGroupPolicy without being signed into my GitHub account and was able to access it, so I'm not sure why you're unable to access it.

@PlagueHO
Copy link
Contributor

@corydwood - glad it isn't just me 😄 I haven't been able to figure out the empty branch thing yet either!

@kwirkykat
Copy link
Member

@corydwood If I copy and paste the link it works, but the clickable hyperlink doesn't lead to the same URL.

Yes, you will need the examples as runnable scripts in an Examples folder instead of the examples in the README. Also you will need to rename the repo/resource module to GroupPolicyDsc. Otherwise looks pretty good 👍

This blog seems to have the right process for the empty branch problem. Apparently you will need to create 2 new branches. I will update the documentation.

@kwirkykat
Copy link
Member

@corydwood Updated the new module guide with better instructions for setting up the review PR.

@kwirkykat kwirkykat added waiting for the author and removed needs review The pull request needs a code review. labels Jul 28, 2016
@corydwood
Copy link
Author

@kwirkykat I've updated the module with the new guidelines. I've also created the PR and it is ready for review.

@kwirkykat kwirkykat added needs review The pull request needs a code review. and removed waiting for the author labels Aug 3, 2016
@kwirkykat
Copy link
Member

kwirkykat commented Aug 12, 2016

Module submission progess:

  • 1. Separate public repository created
  • 2. Review pull request open
  • 3. 1st reviewer approved (Katie)
  • 4. 2nd reviewer approved (Travis)
  • 5. 3rd reviewer approved
  • 6. Tests passing
  • 7. Repository transferred
  • 8. Module released

@lwajswaj
Copy link

Is this module still under review?

@PlagueHO
Copy link
Contributor

Hi @lwajswaj - new community created modules are no longer being added to the DSC Resource Kit. Instead there is now a program where community created & maintained modules can be submitted to be reviewed by the community and will be linked to within this Repo (you can hear more of the discussion about this in this community call: https://github.com/PowerShell/DscResources/blob/master/CommunityCalls/2017-09-27/notes.md).

But you can still find this excellent and high-quality module here:
https://github.com/corydwood/GroupPolicyDsc

@corydwood - are you still wanting to get your resource linked over here? It would be good to get your module linked.

@johlju
Copy link
Contributor

johlju commented Apr 16, 2018

Are there still interest of adding the GroupPolicy resource module to the DSC Resource Kit?
If so I like to help getting this reviewed according to the new process of getting a resource module added to the DSC Resource Kit.
I have opened an issue to update the text on How to do a new resource module submission,
but basically if we gonna submit this resource module to DSC Resource Kit we need to make sure it follows the Quality Governance.

@lwajswaj
Copy link

I would say "yes" just for the fact I'm currently building automation around Domain Controllers and a step would be to add GPOs for user accounts, among others, where this module would be really helpful. Otherwise, I can just consume it from the source; but current DSC Toolkit is missing a part on Active Directory then.

@johlju
Copy link
Contributor

johlju commented Apr 16, 2018

@lwajswaj the resource would be linked from the Dsc Resource Kit to the owners (@corydwood) repository, so you would still need to get it at the "source". The difference to link it to the DSC Resource Kit is that it would mean the resource module follow the guidelines and quality of a resource module that DSC Resource Kit try to up hold.
Antoher option would be to add the resources to the existing xActiveDirectory resource module that is under the PowerShell DSC Team umbrella of the DSC Resource Kit.

@corydwood
Copy link
Author

@PlagueHO - Yes. I am still interested in getting this module linked to the Microsoft repo.

@johlju
Copy link
Contributor

johlju commented Apr 24, 2018

@corydwood great! Please send in a PR adding your resource module as a submodule which will link your repo from this repo. See how Michael did it here https://github.com/PowerShell/DscResources/pull/346/files.

It's a new process so I think we have to take it step by step and learn as we go.

@johlju
Copy link
Contributor

johlju commented May 31, 2018

@johlju
Copy link
Contributor

johlju commented Sep 22, 2018

@corydwood have you thought anything more around this? Are you still interested so we should keep the issue open?

@corydwood
Copy link
Author

I'm still interested. I'm planning on following the steps you provided. I just haven't gotten around to it yet.

@johlju
Copy link
Contributor

johlju commented Oct 5, 2018

Cool. We should leave this issue open then.

corydwood added a commit to corydwood/DscResources that referenced this issue Jan 15, 2019
@corydwood corydwood linked a pull request Jan 15, 2019 that will close this issue
24 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs review The pull request needs a code review. new module submission A request that the community evaluate a resource module for submission into the DSC Resource Kit.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants