mgmt icon indicating copy to clipboard operation
mgmt copied to clipboard

[LOVE] exec resource needs a refreshonly-like option

Open ffrank opened this issue 6 years ago • 3 comments

Versions:

  • mgmt version (eg: mgmt --version): 0.0.17

  • operating system/distribution (eg: uname -a): any

  • golang version (eg: go version): 1.10

Description:

With the goal of enabling Puppet code to run through mgmt, it will be necessary to implement the refreshonly semantics in exec resources. This means that the resource will never sync on its own, and only take action when receiving refresh events. A parameter should toggle this mode for each resource.

ffrank avatar Feb 06 '19 19:02 ffrank

we also need a "time period" before doing the refresh.

it is not rare that a checkapply happens multiple times in very short time period.

I would like to see a "quiet_period" parameter added to the refreshonly.

roidelapluie avatar Feb 07 '19 08:02 roidelapluie

@ffrank In this case, I assume the idea is to never receive any events, so no Watch() behaviour then?

I'm okay with this patch, maybe we should call it Refresh?

@roidelapluie Can you expand on your time period idea please? Are you should you aren't interested in the already existing limit and burst metaparams?

purpleidea avatar Feb 07 '19 13:02 purpleidea

@peckpeck I think this is what you were interested in. Perhaps you want to describe the "don't run on first run without notification" feature.

purpleidea avatar Feb 26 '19 15:02 purpleidea