Once rebased, you'll notice we're trying to use `-` instead of `_` for these attribute names. The best practice, we think, is to accept both by adding this to the `conform!` method:
```
    data['node-metadata'] = data.delete('node_metadata') if data['node_metadata']
```
And add this to the `run` method:
```
    data["node_metadata"] = data.delete("node-metadata") if data["node-metadata"]
```
The tedium of these few extra lines will vanish when [this issue](https://tickets.puppetlabs.com/browse/RAZOR-239) is handled.