Skip to content

Conversation

@DevITJAX
Copy link

@DevITJAX DevITJAX commented Dec 4, 2025

This PR fixes a small typo in Day 64 of the 90DaysOfDevOps Ansible section.

Changes:

  1. Corrected the command ansible Linux -m ping to ansible linux -m ping

    • Inventory group names in Ansible are lowercase, and the examples earlier in the page
      use [linux], so the command needed correction.
  2. Added a troubleshooting section explaining the SSH host key checking error:
    "Using a SSH password instead of a key is not possible because Host Key checking is enabled…"

    • This error commonly happens in Vagrant/VirtualBox lab environments.
    • Included the fix:
      export ANSIBLE_HOST_KEY_CHECKING=False

Reasoning:

  • Improves accuracy and avoids confusion for beginners.
  • Provides a practical solution for a common issue people encounter when following the tutorial.

Thank you for the great learning material happy to contribute!

Copilot AI review requested due to automatic review settings December 4, 2025 19:37
@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025

Our team will soon review your PR. Thanks @DevITJAX :)

@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025

Hello there, thanks for opening your first Pull Request. Someone will review it soon. Welcome to the #90DaysOfDevOps community!

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR improves Day 64 of the 90DaysOfDevOps Ansible tutorial by fixing command case sensitivity and adding helpful troubleshooting guidance for a common SSH error encountered in lab environments.

Key Changes:

  • Corrected ansible Linux -m ping to ansible linux -m ping for proper case consistency with Ansible inventory group naming conventions
  • Added troubleshooting section explaining SSH host key checking errors and providing the solution using export ANSIBLE_HOST_KEY_CHECKING=False

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

![](Images/Day64_config6.png)

I have now also started adding some additional hosts to our inventory, another name for this file as this is where you are going to define all of your devices, could be network devices, switches and routers for example also would be added here and grouped. In our hosts file though I have also added my credentials for accessing the Linux group of systems.
Troubleshooting: SSH Host Key Checking Error
Copy link

Copilot AI Dec 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The troubleshooting section needs proper markdown formatting:

  1. Add a blank line between line 60 and the new section
  2. Format the title as a heading (e.g., ### Troubleshooting: SSH Host Key Checking Error)

This will ensure consistency with the rest of the document structure and improve readability.

Suggested change
Troubleshooting: SSH Host Key Checking Error
### Troubleshooting: SSH Host Key Checking Error

Copilot uses AI. Check for mistakes.
Troubleshooting: SSH Host Key Checking Error
If Ansible fails to connect and you see an error like:

`Using a SSH password instead of a key is not possible because Host Key checking is enabled and sshpass does not support this.
Copy link

Copilot AI Dec 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Grammar error: "a SSH" should be "an SSH" since SSH is pronounced with a vowel sound (ess-ess-aitch).

Suggested change
`Using a SSH password instead of a key is not possible because Host Key checking is enabled and sshpass does not support this.
`Using an SSH password instead of a key is not possible because Host Key checking is enabled and sshpass does not support this.

Copilot uses AI. Check for mistakes.
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