[PUP-6919] Puppet::Util::Windows::Process.execute should accept a working directory Created: 2016/11/15 Updated: 2019/09/04 Resolved: 2018/09/26 |
|
Status: | Closed |
Project: | Puppet |
Component/s: | None |
Affects Version/s: | None |
Fix Version/s: | PUP 5.5.7, PUP 6.0.1 |
Type: | Bug | Priority: | Normal |
Reporter: | Ethan Brown | Assignee: | Enis Inan |
Resolution: | Fixed | Votes: | 0 |
Labels: | package, type_and_provider, windows | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | Not Specified | ||
Original Estimate: | Not Specified |
Issue Links: |
|
||||||||||||||||||||||||
Template: | PUP Bug Template customfield_10700 161235 | ||||||||||||||||||||||||
Acceptance Criteria: |
|
||||||||||||||||||||||||
Epic Link: | WINning | ||||||||||||||||||||||||
Team: | Platform OS | ||||||||||||||||||||||||
Story Points: | 3 | ||||||||||||||||||||||||
Sprint: | Platform OS Kanban | ||||||||||||||||||||||||
Release Notes: | New Feature | ||||||||||||||||||||||||
Release Notes Summary: | Puppet::Util::Execution.execute now supports a `cwd` option to specify the current working directory that the command will run in. This option is only available on the agent. It cannot be used on the master, meaning it cannot be used in, for example, regular functions, hiera backends, or report processors. |
Description |
Puppet::Util::Execution.execute should take in a `cwd` option that lets the user specify the new process' working directory. |
Comments |
Comment by Ethan Brown [ 2017/02/24 ] |
I've temporarily added PUP 5.0.0 as a fixVersion, though I'd like to see this fixed for the Glisan agent if possible. |
Comment by Enis Inan [ 2018/09/05 ] |
Ended up reverting this change – see https://github.com/puppetlabs/puppet/pull/7045. Note that the POSIX issue with cd'ing to the cwd when the user does not have permissions to access it would have been resolved in https://github.com/puppetlabs/puppet/pull/7034; however, that was not merged due to some other issues with the existing implementation. These were:
For reference, the original implementation of this ticket was merged here: https://github.com/puppetlabs/puppet/pull/7016 |
Comment by Kris Bosland [ 2018/09/26 ] |
Passed CI in e2f7ccfb7 |
Comment by Michelle Fredette [ 2018/10/01 ] |
added release note for 6.0.1 |