Skip to content

Allow data/<module_name>.yaml in modules

Steve Traylen requested to merge moduledata into master

Currently we fail e.g on

  • it-puppet-module-batch_support/data/batch_support.yaml

ideally these files in modules would not exist:

  • They are needed little since module level data
  • when and when not this data is loaded is very confusing.
    • Only loaded with explicit lookup of data from the same module.

Allow to pass for now and then worry about this in some future campaign when it matters - unclear when that might be.

Resolves:

$ if [ -d data ] ; then ruby ci/extra-yaml-checks.rb data hostgroup ; fi
Found YAML errors.
ERROR:YAML file "data/batch_support.yaml" in an unexpected tree (neither "fqdns/" nor "hostgroup/")

test being run as a hostgroup despite the fact that it is not.

Edited by Steve Traylen

Merge request reports