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

BREAKING CHANGE: SqlAg & SqlReplica: Removes SQLServerNetName in favor of EndpointHostName #938

Merged
merged 5 commits into from
Dec 8, 2017

Conversation

johlju
Copy link
Member

@johlju johlju commented Dec 7, 2017

Pull Request (PR) description

This Pull Request (PR) fixes the following issues:
Fixes #924

Task list:

  • Change details added to Unreleased section of CHANGELOG.md?
  • Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
  • Examples appropriately updated?
  • New/changed code adheres to Style Guidelines?
  • Unit and (optional) Integration tests created/updated where possible?

This change is Reviewable

- The read-only property SQLServerNetName was removed in favor of EndpointHostName (issue dsccommunity#924).
  Get-TargetResource will return the value of property NetName as the default value if
  EndpointHostName parameter is not assigned a value.
- The read-only property SQLServerNetName was removed in favor of EndpointHostName (issue dsccommunity#924).
  Get-TargetResource will return the value of property NetName as the default value if
  EndpointHostName parameter is not assigned a value.
@johlju johlju added the needs review The pull request needs a code review. label Dec 7, 2017
@codecov-io
Copy link

codecov-io commented Dec 7, 2017

Codecov Report

Merging #938 into dev will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@         Coverage Diff         @@
##            dev   #938   +/-   ##
===================================
  Coverage    96%    96%           
===================================
  Files        32     32           
  Lines      3515   3515           
===================================
  Hits       3395   3395           
  Misses      120    120

@johlju johlju added the breaking change When used on an issue, the issue has been determined to be a breaking change. label Dec 7, 2017
@johlju
Copy link
Member Author

johlju commented Dec 8, 2017

Reviewed 6 of 7 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 2 unresolved discussions.


CHANGELOG.md, line 27 at r2 (raw file):

    of EndpointHostName ([issue #924](https://github.com/PowerShell/SqlServerDsc/issues/924)).
    Get-TargetResource will return the value of property [NetName](https://technet.microsoft.com/en-us/library/microsoft.sqlserver.management.smo.server.netname(v=sql.105).aspx)
    as the default value if EndpointHostName parameter is not assigned a value.

This is wrong, it will always return the NetName as the value.


CHANGELOG.md, line 47 at r2 (raw file):

    of EndpointHostName ([issue #924](https://github.com/PowerShell/SqlServerDsc/issues/924)).
    Get-TargetResource will return the value of property [NetName](https://technet.microsoft.com/en-us/library/microsoft.sqlserver.management.smo.server.netname(v=sql.105).aspx)
    as the default value if EndpointHostName parameter is not assigned a value.

This is wrong, it will always return the NetName as the value.


Comments from Reviewable

@johlju johlju added waiting for author response The pull request is waiting for the author to respond to comments in the pull request. and removed needs review The pull request needs a code review. labels Dec 8, 2017
@johlju johlju added needs review The pull request needs a code review. and removed waiting for author response The pull request is waiting for the author to respond to comments in the pull request. labels Dec 8, 2017
@johlju
Copy link
Member Author

johlju commented Dec 8, 2017

Review status: 6 of 7 files reviewed at latest revision, 2 unresolved discussions.


CHANGELOG.md, line 27 at r2 (raw file):

Previously, johlju (Johan Ljunggren) wrote…

This is wrong, it will always return the NetName as the value.

Done.


CHANGELOG.md, line 47 at r2 (raw file):

Previously, johlju (Johan Ljunggren) wrote…

This is wrong, it will always return the NetName as the value.

Done.


Comments from Reviewable

@johlju
Copy link
Member Author

johlju commented Dec 8, 2017

:lgtm:


Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@johlju johlju merged commit 972ddff into dsccommunity:dev Dec 8, 2017
@vors vors removed the needs review The pull request needs a code review. label Dec 8, 2017
@johlju johlju changed the title SqlAg & SqlReplica: Removes SQLServerNetName in favor of EndpointHostName BREAKING CHNAGE: SqlAg & SqlReplica: Removes SQLServerNetName in favor of EndpointHostName Dec 9, 2017
@johlju johlju changed the title BREAKING CHNAGE: SqlAg & SqlReplica: Removes SQLServerNetName in favor of EndpointHostName BREAKING CHANGE: SqlAg & SqlReplica: Removes SQLServerNetName in favor of EndpointHostName Dec 9, 2017
@johlju johlju deleted the fix-issue-924 branch December 9, 2017 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change When used on an issue, the issue has been determined to be a breaking change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SqlAG & SqlAGReplica: Remove parameter SQLServerNetName
3 participants