Expose `applications` and `exports` fields in `InventoryTarget` and Jsonnet `inventory()` callback

kapicorp / kapitan

Generic templated configuration management for Kubernetes, Terraform and other things

https://kapitan.dev

Apache License 2.0

1.81k stars 199 forks source link

Expose `applications` and `exports` fields in `InventoryTarget` and Jsonnet `inventory()` callback #1142

Closed simu closed 7 months ago

simu commented 7 months ago

Proposed Changes

Restore the Kapitan <= 0.32 behavior of exposing the Reclass applications (which Commodore depends on) and exports fields in the inventory

Docs and Tests

[x] Tests added
[ ] Updated documentation

simu commented 7 months ago

Thank you for the fix. Do you know of any more special keys in reclass, that perhaps need to be exposed?

Reclass also has top-level key exports, we could probably expose that field as well, but I'm not aware that anyone actually uses that Reclass functionality (and it's not yet implemented in reclass-rs either).

simu commented 7 months ago

Thank you for the fix. Do you know of any more special keys in reclass, that perhaps need to be exposed?

Reclass also has top-level key exports, we could probably expose that field as well, but I'm not aware that anyone actually uses that Reclass functionality (and it's not yet implemented in reclass-rs either).

I've updated the PR to also expose the exports field and have switched the implementation to dataclasses.asdict()