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

Remove iSCSIInitiator IsPersistent parameter #14

Open
PlagueHO opened this issue May 4, 2016 · 2 comments
Open

Remove iSCSIInitiator IsPersistent parameter #14

PlagueHO opened this issue May 4, 2016 · 2 comments
Assignees
Labels
enhancement The issue is an enhancement request.

Comments

@PlagueHO
Copy link
Member

PlagueHO commented May 4, 2016

I don't think this is useful when used in a DSC environment. The parameter could be removed from the MOF file but internally just default to $True.

@PlagueHO PlagueHO added the enhancement The issue is an enhancement request. label May 4, 2016
@PlagueHO PlagueHO self-assigned this Aug 8, 2017
@MarkPerry24
Copy link

You may want to consider that DSC may not be only used to keep a server at a given configuration e.g. monitor/refresh mode may not be used and DSC can be deployed via the Start-DSCConfiguration cmdlet and could also be delivered via a DevOps Orchestrator. E.g. I may want to add a temp connection to a SAN for whatever reason and I can quickly add to DSC config. compile and run Start-DSCConfiguration against the remote machine. I'll get my lun and it'll be gone on reboot. LCM refresh would need to be disabled in this case I suspect (as DSC will re-add it as a non persistent device) but it is a valid configuration. Another use case would be to use Invoke-DSCResource and call a single DSC Method as per https://docs.microsoft.com/en-us/powershell/module/psdesiredstateconfiguration/invoke-dscresource?view=powershell-5.1
which requires the refresh mode is disabled.

@PlagueHO
Copy link
Member Author

PlagueHO commented Jun 3, 2018

Thanks @Someone24github - sounds like it is best to leave this parameter in for the use cases you mention.

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

No branches or pull requests

2 participants