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

ExecutionPolicy default value is RemoteSigned #72993

Merged
merged 3 commits into from Jan 4, 2021
Merged

ExecutionPolicy default value is RemoteSigned #72993

merged 3 commits into from Jan 4, 2021

Conversation

jpmat296
Copy link
Contributor

SUMMARY

In Windows setup steps, settings ExecutionPolicy is restored to value Restricted. But actually default value for Windows server is not Restricted but RemoteSigned (source).

I have got issue with win_dsc because ExecutionPolicy was Restricted.

ISSUE TYPE
  • Docs Pull Request

+label: docsite_pr

##### SUMMARY
In Windows setup steps, settings `ExecutionPolicy` is restored to value `Restricted`. But actually default value for Windows server is not `Restricted` but `RemoteSigned` ([source](https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_execution_policies?view=powershell-7.1#default)).

I have got issue with `win_dsc` because `ExecutionPolicy` was `Restricted`.

##### ISSUE TYPE
- Docs Pull Request

+label: docsite_pr
@ansibot ansibot added affects_2.11 core_review In order to be merged, this PR must follow the core review workflow. docs This issue/PR relates to or includes documentation. docsite This issue/PR relates to the documentation website. needs_triage Needs a first human triage before being processed. new_contributor This PR is the first contribution by a new community member. small_patch support:core This issue/PR relates to code supported by the Ansible Engineering Team. windows Windows community labels Dec 16, 2020
@ansibot ansibot added support:community This issue/PR relates to code supported by the Ansible community. and removed small_patch labels Dec 16, 2020
@bcoca bcoca removed the needs_triage Needs a first human triage before being processed. label Dec 17, 2020
@samccann
Copy link
Contributor

Thanks @jpmat296 for your Ansible docs PR. The question I have - from the source you point to, the default is correct for a Windows client, but needs to be RemoteSigned for a Windows server. Reading that document, it seems like it is referring to both.

If you agree, I've added some suggested edits to make this clear.

@jpmat296
Copy link
Contributor Author

Dear @samccann I'm ok with your suggested edit. Thanks a lot.

@ansibot ansibot added the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Dec 29, 2020
@ansibot ansibot removed the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Jan 4, 2021
@samccann samccann merged commit 3074ede into ansible:devel Jan 4, 2021
@samccann
Copy link
Contributor

samccann commented Jan 4, 2021

Thanks @jpmat296 for the Ansible docs fix!

@ansible ansible locked and limited conversation to collaborators Feb 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects_2.11 core_review In order to be merged, this PR must follow the core review workflow. docs This issue/PR relates to or includes documentation. docsite This issue/PR relates to the documentation website. new_contributor This PR is the first contribution by a new community member. support:community This issue/PR relates to code supported by the Ansible community. support:core This issue/PR relates to code supported by the Ansible Engineering Team. windows Windows community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants