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

(CAT-1869) - Add configurable dsc_timeout param for dsc type #213

Merged
merged 1 commit into from
Jun 7, 2024

Conversation

jordanbreen28
Copy link
Contributor

@jordanbreen28 jordanbreen28 commented Jun 7, 2024

Prior to this commit, we set a default timeout of 20 minutes for the dsc type, and this was not configurable.

This change adds a new "dsc_timeout" param to the dsc type, which accepts an integer value for the number of seconds execution should wait before exiting.

Users can now configure this timeout through their manifests, same as you would with any other property see below:

dsc { "Running Script":
  resource_name => 'Script',
  module        => 'PSDesiredStateConfiguration',
  dsc_timeout   => 10,
  properties    => {
    getscript  => 'Start-Sleep -Seconds 50',
    testscript => '$false',
    setscript  => 'Start-Sleep -Seconds 50',
  },
}

@jordanbreen28 jordanbreen28 force-pushed the cat-1869-add_configurable_timeout branch 9 times, most recently from ade0932 to 6e32015 Compare June 7, 2024 11:33
@jordanbreen28 jordanbreen28 marked this pull request as ready for review June 7, 2024 11:55
@jordanbreen28 jordanbreen28 requested a review from a team as a code owner June 7, 2024 11:55
Prior to this commit, we set a default timeout of 20 minutes for the dsc
type, and this was not configurable.

This change adds a new "dsc_timeout" param to the dsc type, which
accepts an integer value for the number of seconds execution should wait
before exiting.

Users can now configure this timeout through their manifests, same as
you would with any other property see below:
```
dsc { "Running Script":
  resource_name => 'Script',
  module        => 'PSDesiredStateConfiguration',
  dsc_timeout   => 10,
  properties    => {
    getscript  => 'Start-Sleep -Seconds 5',
    testscript => '$false',
    setscript  => 'Start-Sleep -Seconds 5',
  },
}
```
@jordanbreen28 jordanbreen28 force-pushed the cat-1869-add_configurable_timeout branch from 6e32015 to fad6643 Compare June 7, 2024 12:12
@pmcmaw pmcmaw merged commit 89d4f6d into main Jun 7, 2024
12 checks passed
@pmcmaw pmcmaw deleted the cat-1869-add_configurable_timeout branch June 7, 2024 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants