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

Convert xHyper-V to HQRM standards #62

Closed
PlagueHO opened this issue Oct 18, 2016 · 5 comments · Fixed by #194
Closed

Convert xHyper-V to HQRM standards #62

PlagueHO opened this issue Oct 18, 2016 · 5 comments · Fixed by #194
Labels
enhancement The issue is an enhancement request. help wanted The issue is up for grabs for anyone in the community.

Comments

@PlagueHO
Copy link
Member

This does not necessarily require renaming this module to HQRM naming but this module should be updated to bring it inline with current HQRM standards.

This should include Integration tests (even if they can't be run on AppVeyor).

@kwirkykat kwirkykat added enhancement The issue is an enhancement request. help wanted The issue is up for grabs for anyone in the community. labels Oct 18, 2016
@rchaganti
Copy link
Contributor

Converting the resources in this module to HQRM isn't a very easy task. Also, I have many other resources in cHyper-V at the moment. So, there has to be a consolidated effort to make this module HQRM complaint. To that extent, I started a new repository in my account.

https://github.com/rchaganti/HyperVDsc

I have taken this path after carefully reviewing what we have here and the complexity in splitting the resources into more granular resources and make them HQRM compliant. Instead, it will be easy to write some of these from scratch.

I have added a list of resources to the README (my wishlist + what I already have). There are 27 so far and I already have a bunch of them in progress. I have a Kanban for this project at https://waffle.io/rchaganti/HyperVDsc.

I am going to seek community to come forward to create the resources that do not exist today.

Do let me know your thoughts on this. I will start pushing what I have starting next week. I will setup AppVeyor tests and build checks once I have the DSC resources being added to the repository.

@bgelens
Copy link
Contributor

bgelens commented Jun 21, 2017

Pinging @PlagueHO @iainbrighton

Would it be easier to create HyperVDsc (liking this on better personally) or Hyper-VDsc as a new module instead of converting xHyper-V?

And would you guys join @rchaganti as contributors if you like what he is proposing?

@iainbrighton
Copy link
Contributor

@bgelens @PlagueHO @rchaganti Happy to help out and starting again will certainly be easier with one or two of those resources. I think that getting a list of required resources listed in some form of priority order needs to be done first? Once that's done we can all agree on schemas. Thoughts?

@rchaganti
Copy link
Contributor

Yes. I have a wish list that I am working towards.
https://waffle.io/rchaganti/HyperVDsc

You can see good to have and must have resources as per my requirements at the moment. We can work on this ordering. I have a bunch of them already in progress including the resources from my cHyper-V.

@PlagueHO
Copy link
Member Author

I'm in favor of this resource being re-engineered to try and reduce complexity and increase flexibility (and bring in HQRM). I read a great article on this from @rchaganti (http://www.powershellmagazine.com/2017/05/23/psdsc-doing-it-right-resource-granularity/).

Naming wise HyperVDsc gets my vote because I'm never that comfortable with dashes in names (I guess because of variables not allowing them).

I'm happy to put some time in reviewing or copying missing resources across. I've got a few outstanding PR's and new resources (FileContentDsc, new xStorage) to complete, then I'll have more time to put in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement The issue is an enhancement request. help wanted The issue is up for grabs for anyone in the community.
Projects
None yet
5 participants