Tim Skirvin

Results 18 comments of Tim Skirvin

https://github.com/voxpupuli/puppet-vault_lookup/pull/118 There you go, new pull request with what *looks* like the right merge. I don't know how well it works, I'm a bit suspicious that I missed something.

The static validations stuff - I'm not sure if that's a problem with the test suite in general or in the code I merged in.

That's why I included the yaml snippet; it was closer than I expected! The right fields look to be there.

I'm possibly willing to go at least test my "this is fairly easy" theory, but only if there's somebody that *can* take my pull commit if necessary. Is there anybody?

I did make a run at it, and it was (unsurprisingly) not as simple as I'd like. I have to admit that switching over to deviceID whole-sale might work better,...

Yes, that worked for me too (once I updated to a more recent version of HA). Thanks!

Note: I'm not closing this because it's still a bug, even if @rewardone's patch does what it's supposed to do.

This looks pretty good if it works!