Added docker::stack refresh

[kwek]

Added 'refresh' possibility to docker::stack

[puppet-community-rangefinder]

docker::stack is a type

that may have no external impact to Forge modules.

This module is declared in 7 of 575 indexed public Puppetfiles.

---

These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.

Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.

[codecov-commenter]

Codecov Report

Merging #657 into main will decrease coverage by 3.27%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #657      +/-   ##
==========================================
- Coverage   25.64%   22.37%   -3.28%     
==========================================
  Files          20       20              
  Lines         741      751      +10     
==========================================
- Hits          190      168      -22     
- Misses        551      583      +32     

Impacted Files	Coverage Δ
lib/facter/docker.rb	42.66% <0.00%> (-40.42%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4510ab1...05be785. Read the comment docs.

[adrianiurca]

Hi @kwek, thank you for your contribution. The code looks good, but can you also add an acceptance test?
The acceptance test for stack refresh should be present in spec/acceptance/stack_spec.rb
Please let us know if you need any help.

kind regards,
@adrianiurca

[carabasdaniel]

Hi @kwek,

As @adrianiurca mentioned before we usually merge PRs that have at least an acceptance tests confirming the functionality implemented, however as the docker stack define type will be deprecated in a future release could you probably try the docker_stack type/provider if it covers the functionality you need ?

Thanks!

[carabasdaniel]

Closing this PR due to lack of activity. Please feel free to re-open if the docker_stack provider does not cover this.