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.

Sign up for GitHub

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

Jump to bottom

Tests failing: New-SelfSignedCertificateEx from Script Center is no longer available #250

Closed
PlagueHO opened this issue Feb 5, 2021 · 0 comments · Fixed by #251
Closed

Tests failing: New-SelfSignedCertificateEx from Script Center is no longer available #250

PlagueHO opened this issue Feb 5, 2021 · 0 comments · Fixed by #251
Assignees
@PlagueHO
Labels
blocking release The issue or pull request is blocking the next release. Higher priority than label 'High priority'. bug The issue is a bug. in progress The issue is being actively worked on by someone.

Comments

@PlagueHO
Copy link
Member

PlagueHO commented Feb 5, 2021

The Microsoft Technet Script Center Gallery has been retired: https://docs.microsoft.com/en-us/teamblog/technet-gallery-retirement

The Test Helper function Install-NewSelfSignedCertificateExScript used this script to create self-signed certificates for testing on Windows Server 2012 R2 machines because the built in version was not adequate.

Rather than trying to recreate this script, we should remove this dependency and note that the tests can not be run on Windows Server 2012 R2. We don't run these on Windows Server 2012 R2 in the pipeline.

Note: This won't stop the resource from working on Windows Server 2012 R2 - it will just drop support for running the tests.

I'll submit this change shortly so that we can get PRs through.
@PlagueHO PlagueHO added bug The issue is a bug. in progress The issue is being actively worked on by someone. blocking release The issue or pull request is blocking the next release. Higher priority than label 'High priority'. labels Feb 5, 2021
@PlagueHO PlagueHO self-assigned this Feb 5, 2021
@PlagueHO PlagueHO mentioned this issue Feb 5, 2021
Merged
9 tasks
PlagueHO added a commit to PlagueHO/CertificateDsc that referenced this issue Feb 5, 2021
@PlagueHO
Fix Issue dsccommunity#250
3ea6ef2
@PlagueHO PlagueHO mentioned this issue Feb 7, 2021
Merged
9 tasks
PlagueHO added a commit that referenced this issue Feb 11, 2021
@PlagueHO
Remove Test Dependency on New-SelfSignedCertificateEx - Fixes #221, #250
57f817a 
 (#251)

* Remove Windows Server 2012 R2 test support

* Correct support for using New-SelfSignedCertificate

* Fix unit tests and FIPS support on PS7

* Fix CertExport Integration Tests

* Improve performance of resources

* Changes as per PR comments. Cleaned up comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PlagueHO PlagueHO

Labels
blocking release The issue or pull request is blocking the next release. Higher priority than label 'High priority'. bug The issue is a bug. in progress The issue is being actively worked on by someone.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove Test Dependency on New-SelfSignedCertificateEx - Fixes #221, #250 PlagueHO/CertificateDsc
1 participant
@PlagueHO