search

linux-system-roles / nbde_client

Ansible role for configuring Network Bound Disk Encryption clients (e.g. clevis)
https://linux-system-roles.github.io/nbde_client/
MIT License
14 stars 24 forks source link

Add test for the include variable problem #13

Closed pcahyna closed 4 years ago

pcahyna commented 4 years ago

tests the problem that #9 fixed - if a caller role has the same variable files as this role expects, they can take precedence and yield unexpected results.

richm commented 4 years ago

looks good - just waiting for the test results

nhosoi commented 4 years ago

lgtm (I applied your patch to logging and confirmed it passes with "All assertions passed". :)

sergio-correia commented 4 years ago

lgtm. What about #12?

pcahyna commented 4 years ago

12's dead, baby.