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

Fix issues found by ansible-test and linters - enable all tests on all repos - remove suppressions #44

Closed nhosoi closed 3 years ago

nhosoi commented 3 years ago
richm commented 3 years ago

please also fix the lint error in tests/tests_passphrase_temporary_keyfile.yml you can probably use mode: preserve there unless it requires a stricter permission

nhosoi commented 3 years ago

please also fix the lint error in tests/tests_passphrase_temporary_keyfile.yml you can probably use mode: preserve there unless it requires a stricter permission

Thank you, @richm. I think this is what you suggest. I'm updating the pr.

--- a/tests/tests_passphrase_temporary_keyfile.yml
+++ b/tests/tests_passphrase_temporary_keyfile.yml
@@ -35,6 +35,7 @@
           copy:
             src: "{{ nbde_client_test_encryption_key_src }}"
             dest: "{{ nbde_client_test_encryption_key_src }}"
+            mode: preserve

Without the fix I was getting congratulations from the full tox run. Do you know why the mode check is skipped for me?

___________________________________ summary ____________________________________
  flake8: commands succeeded
  py38: commands succeeded
  py37: commands succeeded
  py27: commands succeeded
  pylint: commands succeeded
  custom: commands succeeded
  black: commands succeeded
  shellcheck: commands succeeded
SKIPPED:  py26: InterpreterNotFound: python2.6
  ansible-lint: commands succeeded
  yamllint: commands succeeded
  py36: commands succeeded
  collection: commands succeeded
  ansible-test: commands succeeded
  congratulations :)
richm commented 3 years ago

please also fix the lint error in tests/tests_passphrase_temporary_keyfile.yml you can probably use mode: preserve there unless it requires a stricter permission

Thank you, @richm. I think this is what you suggest. I'm updating the pr.

--- a/tests/tests_passphrase_temporary_keyfile.yml
+++ b/tests/tests_passphrase_temporary_keyfile.yml
@@ -35,6 +35,7 @@
           copy:
             src: "{{ nbde_client_test_encryption_key_src }}"
             dest: "{{ nbde_client_test_encryption_key_src }}"
+            mode: preserve

Without the fix I was getting congratulations from the full tox run. Do you know why the mode check is skipped for me?

No, I don't know.

___________________________________ summary ____________________________________
  flake8: commands succeeded
  py38: commands succeeded
  py37: commands succeeded
  py27: commands succeeded
  pylint: commands succeeded
  custom: commands succeeded
  black: commands succeeded
  shellcheck: commands succeeded
SKIPPED:  py26: InterpreterNotFound: python2.6
  ansible-lint: commands succeeded
  yamllint: commands succeeded
  py36: commands succeeded
  collection: commands succeeded
  ansible-test: commands succeeded
  congratulations :)