rpm_key: Added handler for ftp keys

[pvcasillasg]

Status code for fetch_url for ftp:// urls is None, this conditional change will allow to rpm_key for ftp urls usage

SUMMARY

Changed the conditional check when fetch a key, since fetch_url function sets a status == None for ftp urls.
fetch_keys uses the function fetch_url to retrieve the target file.
This conditional change will allow to check the url and when the url is ftp, will allow the return code none in order to retrieve the key, without disrupting the http/s retrieves

Fixes #83321

ISSUE TYPE

• Bugfix Pull Request

ADDITIONAL INFORMATION

Issue refer to ansible 2.15, but still persisting in latest

[pvcasillasg]

It's the docu ok now? @Akasurde

[bcoca]

Also there are no tests

[ansibot]

The test ansible-test sanity --test pep8 [explain] failed with 1 error:

lib/ansible/modules/rpm_key.py:156:95: E202: whitespace before ')'

The test ansible-test sanity --test boilerplate [explain] failed with 3 errors:

lib/ansible/modules/rpm_key.py:0:0: invalid: __metaclass__ = type
lib/ansible/modules/rpm_key.py:0:0: invalid: from __future__ import absolute_import, division, print_function
lib/ansible/modules/rpm_key.py:0:0: missing: from __future__ import annotations

click here for bot help

[pvcasillasg]

Attached test in screenshots:

[ansibot]

The test ansible-test sanity --test pep8 [explain] failed with 1 error:

lib/ansible/modules/rpm_key.py:12:1: E303: too many blank lines (3)

click here for bot help

[pvcasillasg]

I made several commits, i paste my rpm_key.py file into GitHub, some sanity errors and missed to erase the file:/ check that I used for testing purposes.
Sorry for the commit mess

Please, review it and in the case something more is needed, please, ask for it and I will happy to help @Akasurde @bcoca
If you need the test in a format specific, please just explain to me how to do it, since is my first contribution.
Thanks!

[pvcasillasg]

@Akasurde @bcoca is anything more needed to do?

[pvcasillasg]

@bcoca can you review it? Can't merge since the status is still need_revisions

[Akasurde]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[pvcasillasg]

How can I merge it?