MitocGroup / recink

REciNK - Rethink Continuous Integration for JavaScript Applications
https://www.npmjs.org/package/recink
Mozilla Public License 2.0
14 stars 5 forks source link

Global resource config does not override custom module options #157

Closed avozicov closed 6 years ago

avozicov commented 6 years ago

We need to fix the behavior for defining and applying global resource configuration Actual Result:

Please use the following config and commands to reproduce the issue:

ddimitrioglo commented 6 years ago

@avozicov could not reproduce with the following config:

---
$:
  preprocess:
    '$.terraform.vars.access_key': 'eval'
    '$.terraform.vars.secret_key': 'eval'
    '$.terraform.vars.region': 'eval'
  emit:
    pattern:
      - /.+\.js.*$/i
    ignore:
      - /^(.*\/)?node_modules(\/?$)?/i
  terraform:
    resource: '.resource'
    plan: false
    apply: false
    destroy: true
    vars:
      access_key: 'process.env.AWS_ACCESS_KEY_ID'
      secret_key: 'process.env.AWS_SECRET_ACCESS_KEY'
      region: 'process.env.AWS_DEFAULT_REGION'

iam2:
  root: './iam'
  terraform:
    cache: false

Please let me know if I'm missing something

avozicov commented 6 years ago

@ddimitrioglo I can confirm that the issue is not reproduced anymore on my side as well. I have successfully executed the plan for my resource using plan: true option defined in the global terraform config

screen shot 2018-04-11 at 17 36 47
eistrati commented 6 years ago

Checked.