Skip to content

Deprecated host_key_checking option#84276

Closed
Akasurde wants to merge 1 commit intoansible:develfrom
Akasurde:deprecate_host_key_checking_global
Closed

Deprecated host_key_checking option#84276
Akasurde wants to merge 1 commit intoansible:develfrom
Akasurde:deprecate_host_key_checking_global

Conversation

@Akasurde
Copy link
Copy Markdown
Member

@Akasurde Akasurde commented Nov 7, 2024

SUMMARY
  • User can set host_key_checking configuration from
    connection plugins such as ssh and paramiko_ssh itself.
    Deprecate host_key_checking from global configuration.

Signed-off-by: Abhijeet Kasurde akasurde@redhat.com

ISSUE TYPE
  • Bugfix Pull Request

@ansibot ansibot added bug This issue/PR relates to a bug. needs_triage Needs a first human triage before being processed. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. labels Nov 7, 2024
@ansibot
Copy link
Copy Markdown
Contributor

ansibot commented Nov 7, 2024

The test ansible-test sanity --test import --python 3.11 [explain] failed with 5 errors:

lib/ansible/utils/color.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself
lib/ansible/utils/display.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself
lib/ansible/utils/jsonrpc.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself
lib/ansible/utils/listify.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself
lib/ansible/utils/py3compat.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself

The test ansible-test sanity --test import --python 3.12 [explain] failed with 5 errors:

lib/ansible/utils/color.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself
lib/ansible/utils/display.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself
lib/ansible/utils/jsonrpc.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself
lib/ansible/utils/listify.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself
lib/ansible/utils/py3compat.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself

The test ansible-test sanity --test import --python 3.13 [explain] failed with 5 errors:

lib/ansible/utils/color.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself
lib/ansible/utils/display.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself
lib/ansible/utils/jsonrpc.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself
lib/ansible/utils/listify.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself
lib/ansible/utils/py3compat.py:0:0: stderr: [DEPRECATED] HOST_KEY_CHECKING option. Reason: This option was moved to the individual plugin itself

click here for bot help

@bcoca
Copy link
Copy Markdown
Member

bcoca commented Nov 7, 2024

guessing #81947 would fix the issue of just 'constant' access vs specific setting

@webknjaz webknjaz added the ci_verified Changes made in this PR are causing tests to fail. label Nov 8, 2024
@mkrizek mkrizek removed the needs_triage Needs a first human triage before being processed. label Nov 12, 2024
@ansibot ansibot added the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Nov 19, 2024
@Akasurde
Copy link
Copy Markdown
Member Author

Akasurde commented Apr 7, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

azure-pipelines bot commented Apr 7, 2025

Azure Pipelines successfully started running 1 pipeline(s).

@Akasurde Akasurde force-pushed the deprecate_host_key_checking_global branch from 0e2bd66 to 5aa2101 Compare April 7, 2025 17:31
@ansibot ansibot removed ci_verified Changes made in this PR are causing tests to fail. stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. labels Apr 7, 2025
@Akasurde
Deprecated host_key_checking option
41cc4f9 
* User can set host_key_checking configuration from
  connection plugins such as ssh and paramiko_ssh itself.
  Deprecate host_key_checking from global configuration.

Signed-off-by: Abhijeet Kasurde <Akasurde@redhat.com>
@Akasurde Akasurde force-pushed the deprecate_host_key_checking_global branch from 5aa2101 to 41cc4f9 Compare April 7, 2025 18:16
@webknjaz webknjaz added the ci_verified Changes made in this PR are causing tests to fail. label Apr 10, 2025
@ansibot ansibot added the stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested. label Apr 17, 2025
@Akasurde Akasurde closed this Mar 30, 2026
@ansible ansible locked as resolved and limited conversation to collaborators Apr 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

bug This issue/PR relates to a bug. ci_verified Changes made in this PR are causing tests to fail. needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR. stale_ci This PR has been tested by CI more than one week ago. Close and re-open this PR to get it retested.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Akasurde @ansibot @bcoca @webknjaz @mkrizek