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

SqlServerDsc: Remove old examples in favor of new resource specific examples #462

Closed
johlju opened this issue Apr 1, 2017 · 7 comments · Fixed by #976
Closed

SqlServerDsc: Remove old examples in favor of new resource specific examples #462

johlju opened this issue Apr 1, 2017 · 7 comments · Fixed by #976
Labels
enhancement The issue is an enhancement request.

Comments

@johlju
Copy link
Member

johlju commented Apr 1, 2017

I suggest that we remove the below examples due to the fact that we can't keep up with changing these examples when other modules change. For example, I'm thinking that the steps for setting up LCM is well documented in docs.microsoft.com, and setting up a cluster must be documented in those modules that knows best how to do that with there resources. We might miss a step, or the resource change to behave in another way, or the user has a different environment or requirment, and then the examples won't work any way.
Also the examples does not follow the template so that the common tests can test the examples. That can be fixed though.
I would rather point the user in our documentation (README.md or resource specific examples) to examples or/and documentation in other resources that would help setting up the prerequisites for using our resources.

I really want to here other views on this.

Examples suggested to be removed:
DSCClusterSqlBuild.ps1
DSCFCISqlBuild.ps1
DSCSqlBuild.ps1
DSCSQLBuildEncrypted.ps1
SQL-ClusterDB.ps1
SQL-Standalone.ps1
SQLPush_SingleServer.ps1

@johlju johlju added the discussion The issue is a discussion. label Apr 1, 2017
@johlju
Copy link
Member Author

johlju commented Apr 1, 2017

The examples here:
https://github.com/PowerShell/xSQLServer/tree/dev/Examples/Resources/xSQLServerSetup
I think they should replace examples here (remove these):

DSCFCISqlBuild
DSCSqlBuild
DSCSQLBuildEncrypted
SQL-ClusterDB
SQL-Standalone
SQLPush_SingleServer

And the examples here:
https://github.com/PowerShell/xSQLServer/tree/dev/Examples/Resources/xSQLServerAlwaysOnAvailabilityGroup
https://github.com/PowerShell/xSQLServer/tree/dev/Examples/Resources/xSQLServerAlwaysOnAvailabilityGroupReplica

I think they should replace this examples (remove these):
DSCClusterSqlBuild

@randomnote1
Copy link
Contributor

Could we provide links in the documentation to the items that we're removing? I think it would be helpful for the beginner to have as much copy/paste material as possible.

@johlju
Copy link
Member Author

johlju commented Apr 3, 2017

Yes, you are right. Maybe we could add these as .md files instead under the Examples folder? Maybe write it up like a blog and link to various example throughout the DSC Resource Kit? It would then be easier for contributors to add (text) to that so we collect more knowledge into one place.

Then that might fix this one as well #176
But it would be awesome if we could have that at docs.microsoft.com. Under PowerShell here maybe. https://docs.microsoft.com/sv-se/sql/sql-hub-menu?toc=%2fsql%2ftoc.json#pivot=sdkstools
Or maybe here https://github.com/PowerShell/PowerShell-Docs
I will try to remember to ask about this on the next community call.

@randomnote1
Copy link
Contributor

Yup, I think you're spot on with the blog-style examples.

@johlju
Copy link
Member Author

johlju commented Jul 31, 2017

All the examples in the root of /Examples should be rewritten, because they reference resources that no longer exist or has been been updated with breaking changes. My thought was that I would have time to fix this during the month that went by. But other stuff needed to be done
I'm looking at splitting these examples between xSQLServer and xFailOverCluster. Help is welcomed. :)

@johlju
Copy link
Member Author

johlju commented Oct 8, 2017

I think these examples should be implemented as configurations using the templates from the repository DscConfigurations instead. Then they are able to be integration tested by the users themselves with an already finished framework for production ready configurations. The configurations should be able to be linked to that DscConfigurations repository.

These exampled as they are today (linked in the issue description) is not production ready.

@johlju johlju added enhancement The issue is an enhancement request. in progress The issue is being actively worked on by someone. and removed discussion The issue is a discussion. labels Oct 8, 2017
johlju added a commit to johlju/SqlServerDsc that referenced this issue Oct 8, 2017
- The complete examples that no longer was accurate and referenced resources
  that no longer exist was removed. Accurate examples can be found in each
  specific resource example folder. Examples for installing Failover Cluster
  can be found in the resource examples folders in the xFailOverCluster
  resource module (issue dsccommunity#462).
- A README.md was created under the examples folder to be used as reference how
  to installed certain scenarios (issue dsccommunity#462).
- Removed the specific common test for examples in this repository and instead
  opt-in for the common test in the DscResource.Tests repository (issue dsccommunity#669).
@johlju
Copy link
Member Author

johlju commented Nov 10, 2017

This will resolve issue #373 as well.

johlju added a commit to johlju/SqlServerDsc that referenced this issue Dec 6, 2017
- The complete examples that no longer was accurate and referenced resources
  that no longer exist was removed. Accurate examples can be found in each
  specific resource example folder. Examples for installing Failover Cluster
  can be found in the resource examples folders in the xFailOverCluster
  resource module (issue dsccommunity#462).
- A README.md was created under the examples folder to be used as reference how
  to installed certain scenarios (issue dsccommunity#462).
- Removed the specific common test for examples in this repository and instead
  opt-in for the common test in the DscResource.Tests repository (issue dsccommunity#669).
johlju added a commit to johlju/SqlServerDsc that referenced this issue Dec 6, 2017
- The complete examples was removed. They no longer was accurate and some
  referenced resources that no longer existed. Accurate examples can be found
  in each specific resource example folder. Examples for installing Failover Cluster
  can be found in the resource examples folders in the xFailOverCluster
  resource module (issue dsccommunity#462).
@johlju johlju changed the title xSQLServer: Remove old examples in favor of new resource specific examples SqlServerDsc: Remove old examples in favor of new resource specific examples Dec 22, 2017
johlju added a commit to johlju/SqlServerDsc that referenced this issue Dec 23, 2017
- The complete examples that no longer was accurate and referenced resources
  that no longer exist was removed. Accurate examples can be found in each
  specific resource example folder. Examples for installing Failover Cluster
  can be found in the resource examples folders in the xFailOverCluster
  resource module (issue dsccommunity#462).
- A README.md was created under the examples folder to be used as reference how
  to installed certain scenarios (issue dsccommunity#462).
- Removed the specific common test for examples in this repository and instead
  opt-in for the common test in the DscResource.Tests repository (issue dsccommunity#669).
johlju added a commit to johlju/SqlServerDsc that referenced this issue Dec 23, 2017
- The complete examples was removed. They no longer was accurate and some
  referenced resources that no longer existed. Accurate examples can be found
  in each specific resource example folder. Examples for installing Failover Cluster
  can be found in the resource examples folders in the xFailOverCluster
  resource module (issue dsccommunity#462).
johlju added a commit to johlju/SqlServerDsc that referenced this issue Dec 28, 2017
- The complete examples that no longer was accurate and referenced resources
  that no longer exist was removed. Accurate examples can be found in each
  specific resource example folder. Examples for installing Failover Cluster
  can be found in the resource examples folders in the xFailOverCluster
  resource module (issue dsccommunity#462).
- A README.md was created under the examples folder to be used as reference how
  to installed certain scenarios (issue dsccommunity#462).
- Removed the specific common test for examples in this repository and instead
  opt-in for the common test in the DscResource.Tests repository (issue dsccommunity#669).
johlju added a commit to johlju/SqlServerDsc that referenced this issue Dec 28, 2017
- The complete examples was removed. They no longer was accurate and some
  referenced resources that no longer existed. Accurate examples can be found
  in each specific resource example folder. Examples for installing Failover Cluster
  can be found in the resource examples folders in the xFailOverCluster
  resource module (issue dsccommunity#462).
johlju added a commit to johlju/SqlServerDsc that referenced this issue Jan 1, 2018
- The complete examples that no longer was accurate and referenced resources
  that no longer exist was removed. Accurate examples can be found in each
  specific resource example folder. Examples for installing Failover Cluster
  can be found in the resource examples folders in the xFailOverCluster
  resource module (issue dsccommunity#462).
- A README.md was created under the examples folder to be used as reference how
  to installed certain scenarios (issue dsccommunity#462).
- Removed the specific common test for examples in this repository and instead
  opt-in for the common test in the DscResource.Tests repository (issue dsccommunity#669).
johlju added a commit to johlju/SqlServerDsc that referenced this issue Jan 1, 2018
- The complete examples was removed. They no longer was accurate and some
  referenced resources that no longer existed. Accurate examples can be found
  in each specific resource example folder. Examples for installing Failover Cluster
  can be found in the resource examples folders in the xFailOverCluster
  resource module (issue dsccommunity#462).
johlju added a commit to johlju/SqlServerDsc that referenced this issue Jan 4, 2018
- The complete examples that no longer was accurate and referenced resources
  that no longer exist was removed. Accurate examples can be found in each
  specific resource example folder. Examples for installing Failover Cluster
  can be found in the resource examples folders in the xFailOverCluster
  resource module (issue dsccommunity#462).
- A README.md was created under the examples folder to be used as reference how
  to installed certain scenarios (issue dsccommunity#462).
- Removed the specific common test for examples in this repository and instead
  opt-in for the common test in the DscResource.Tests repository (issue dsccommunity#669).
johlju added a commit to johlju/SqlServerDsc that referenced this issue Jan 4, 2018
- The complete examples was removed. They no longer was accurate and some
  referenced resources that no longer existed. Accurate examples can be found
  in each specific resource example folder. Examples for installing Failover Cluster
  can be found in the resource examples folders in the xFailOverCluster
  resource module (issue dsccommunity#462).
johlju added a commit that referenced this issue Jan 4, 2018
- Changes to SqlServerDsc
  - The complete examples were removed. They were no longer accurate and some
    referenced resources that no longer exist. Accurate examples can be found
    in each specific resource example folder. Examples for installing Failover Cluster
    can be found in the resource examples folders in the xFailOverCluster
    resource module (issue #462).
  - A README.md was created under the Examples folder to be used as reference how
    to install certain scenarios (issue #462).
  - Removed the local specific common test for compiling examples in this repository
    and instead opted-in for the common test in the 'DscResource.Tests' repository (issue #669).
@johlju johlju removed the in progress The issue is being actively worked on by someone. label Jan 4, 2018
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.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants