hamlet-io / executor-bash

Executor for hamlet based on bash
GNU General Public License v3.0
0 stars 4 forks source link

fix: resetting of temporary file stack #263

Closed ml019 closed 3 years ago

ml019 commented 3 years ago

Intent of Change

Description

Previously each invocation of utility.sh would empty the temporary file stack. With the adding of a call to setCredentials.sh (which sources utility.sh) in createTemplate.sh when running a script, the previously established temporary directory was lost resulting in the failure of subsequent template passes.

Along the way, a couple of other things were noted and fixed;

Motivation and Context

Template generation is failing when trying to generate a template for an api gateway which uses a pregeneration script and triggers the bug.

How Has This Been Tested?

Will be tested on automation server once merged

Related Changes

Prerequisite PRs:

Dependent PRs:

Consumer Actions:

ml019 commented 3 years ago

@roleyfoley some time pressure to get this one in for HA. Going to merge it but still would appreciate your eyes over it. I'll be testing it in HA once its in the edge docker container - I'll need to manually trigger the tram and docker builds.