Skip to content

Conversation

@felixfontein
Copy link
Contributor

SUMMARY

This PR uses https://github.com/felixfontein/ansible/commits/show-deprecations/ to run the tests with ansible-test from ansible-core devel. This should not hide all deprecations, as ansible-test usually does (ansible/ansible#85436).

Ref: ansible-collections/community.general#10540

ISSUE TYPE
  • Test Pull Request
COMPONENT NAME

CI

…to not suppress deprecations in ansible-test devel.

ci_complete
@felixfontein
Copy link
Contributor Author

This mainly exposes one deprecation in ansible-core used in the integration tests:

2025-08-02T14:29:19.1917711Z 01:29 TASK [luks_device : Create with keyfile3] **************************************
2025-08-02T14:29:22.5569075Z 01:32 �[1;35m[WARNING]: Deprecation warnings can be disabled by setting `deprecation_warnings=False` in ansible.cfg.�[0m
2025-08-02T14:29:22.5570615Z 01:32 �[0;35m[DEPRECATION WARNING]: File '/home/alpine/ansible_collections/community/crypto/tests/output/.tmp/integration/luks_device-8m_bd2_g-ÅÑŚÌβŁÈ/tests/integration/targets/luks_device/files/keyfile3' could not be decoded as 'utf-8'. Invalid content has been escaped. This feature will be removed from ansible-core version 2.23. This file must be UTF-8 encoded.�[0m
2025-08-02T14:29:22.5601713Z 01:32 �[0;33mchanged: [testhost] => {"changed": true, "name": null}�[0m

Using the ansible.builtin.file lookup for binary files has never been a good idea, and is now officially deprecated.

I think we need a binary_file lookup that allows to read binary files (and base64 encodes them). Maybe something for community.general?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant