-
Notifications
You must be signed in to change notification settings - Fork 21
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Modify for Amazon Linux 2023 #44
Conversation
I see, We are trying to merge this PR onto main branch, however, I have some questions.
|
@lufia |
- include_tasks: amazonlinux2023.yml | ||
when: ansible_distribution == "Amazon" and ansible_distribution_major_version == "2023" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i like it
@lufia |
@yuichisatake Thank you! I'm going to merge this patch. |
@@ -6,7 +6,10 @@ | |||
when: ansible_os_family == "RedHat" and ansible_distribution != "Amazon" | |||
|
|||
- include_tasks: amazonlinux.yml | |||
when: ansible_distribution == "Amazon" | |||
when: ansible_distribution == "Amazon" and ansible_distribution_major_version == "2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok
$ ansible localhost -m setup | grep ansible_distribution_major
"ansible_distribution_major_version": "2",
[ec2-user@ip-172-31-20-176 ~]$ cat /etc/os-release
NAME="Amazon Linux"
VERSION="2"
...
python3 is installed by default on amazon linux2023.
So, when I changed ansible's interpreter_python to "/usr/bin/python3" and executed the role, the following error occurred.
I created a pull request that works with the pyton3 interpreter, so please consider merging it.