solvaholic / octodns-sync

GitHub Action to test and deploy DNS settings with OctoDNS
MIT License
28 stars 14 forks source link

Support multiple runs of the action in the same job #66

Closed travislikestocode closed 3 years ago

travislikestocode commented 3 years ago

Description

Somewhat naive attempt at allowing the action to run multiple times in the same job. By deleting the plan and log before running octodns-sync, we make the plan file available to later steps. I also added the plan output to the action for more flexibility. The ability to use multiple octodns_refs is not supported. The ref from the first run will be used for any subsequent runs.

Hope this adequately addresses the concerns in the issue. I'm very open to any change suggestions!

Motivation and Context

See #57

How Has This Been Tested?

Created a test workflow which runs the action twice in the same job, and then prints the plan output. The action run was successful

Screenshots (if appropriate):

Types of changes

Checklist:

solvaholic commented 3 years ago

I'll kick the linter, see if I can get it to sort itself out...

Worked around this initially by pushing refs/pull/66/head as the issue66 branch, then updated the linter workflow in #68.