ansible-collections / community.hashi_vault

Ansible collection for managing and working with HashiCorp Vault.
https://docs.ansible.com/ansible/devel/collections/community/hashi_vault/index.html
GNU General Public License v3.0
80 stars 59 forks source link

KV get fixups #264

Closed briantist closed 2 years ago

briantist commented 2 years ago
SUMMARY

Fixes #258 Fixes https://github.com/ansible-collections/community.hashi_vault/pull/246#discussion_r862379062

ISSUE TYPE
COMPONENT NAME

vault_kv1_get vault_kv2_get vault_pki_generate_certificate

ADDITIONAL INFORMATION

The new plugins are not released yet, so there is no breaking change or changelog frragment needed.

github-actions[bot] commented 2 years ago

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and the docs are now incorporated into main: https://ansible-collections.github.io/community.hashi_vault/branch/main

codecov[bot] commented 2 years ago

Codecov Report

Merging #264 (6aa57ba) into main (54c81ef) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #264   +/-   ##
=======================================
  Coverage   98.34%   98.34%           
=======================================
  Files          66       66           
  Lines        3208     3208           
  Branches      267      267           
=======================================
  Hits         3155     3155           
  Misses         44       44           
  Partials        9        9           
Flag Coverage Δ
env_docker-default 98.34% <100.00%> (ø)
integration 81.03% <100.00%> (ø)
sanity 39.88% <0.00%> (ø)
target_ansible-doc 100.00% <ø> (ø)
target_auth_approle 89.47% <ø> (ø)
target_auth_aws_iam 50.00% <ø> (ø)
target_auth_cert 86.36% <ø> (ø)
target_auth_jwt 91.30% <ø> (ø)
target_auth_ldap 89.47% <ø> (ø)
target_auth_none 100.00% <ø> (ø)
target_auth_token 71.42% <ø> (ø)
target_auth_userpass 85.71% <ø> (ø)
target_connection_options 74.76% <ø> (ø)
target_controller 80.84% <100.00%> (ø)
target_filter_vault_login_token 77.77% <ø> (ø)
target_import 39.88% <0.00%> (ø)
target_lookup_hashi_vault 81.33% <ø> (ø)
target_lookup_vault_kv1_get 91.30% <100.00%> (ø)
target_lookup_vault_kv2_get 91.11% <100.00%> (ø)
target_lookup_vault_login 88.57% <ø> (ø)
target_lookup_vault_read 90.00% <ø> (ø)
target_lookup_vault_token_create 78.18% <ø> (ø)
target_lookup_vault_write 59.14% <ø> (ø)
target_module_utils 97.24% <ø> (ø)
target_module_vault_kv1_get 87.23% <100.00%> (ø)
target_module_vault_kv2_get 86.95% <100.00%> (ø)
target_module_vault_login 83.33% <ø> (ø)
target_module_vault_pki_generate_certificate 78.26% <ø> (ø)
target_module_vault_read 85.36% <ø> (ø)
target_module_vault_token_create 90.00% <ø> (ø)
target_module_vault_write 57.53% <ø> (ø)
target_modules 79.08% <100.00%> (ø)
units 95.26% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
plugins/doc_fragments/engine_mount.py 100.00% <ø> (ø)
plugins/modules/vault_pki_generate_certificate.py 95.65% <ø> (ø)
plugins/lookup/vault_kv1_get.py 100.00% <100.00%> (ø)
plugins/lookup/vault_kv2_get.py 100.00% <100.00%> (ø)
plugins/modules/vault_kv1_get.py 100.00% <100.00%> (ø)
plugins/modules/vault_kv2_get.py 100.00% <100.00%> (ø)
tests/unit/plugins/lookup/test_vault_kv1_get.py 100.00% <100.00%> (ø)
tests/unit/plugins/lookup/test_vault_kv2_get.py 100.00% <100.00%> (ø)
tests/unit/plugins/modules/test_vault_kv1_get.py 100.00% <100.00%> (ø)
tests/unit/plugins/modules/test_vault_kv2_get.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 54c81ef...6aa57ba. Read the comment docs.

briantist commented 2 years ago

@felixfontein thank you for reviewing!