Skip to content
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

Removal of fqcn[action] rule from ansible-lint skip_list #378

Merged
merged 5 commits into from
Jun 11, 2023
Merged

Removal of fqcn[action] rule from ansible-lint skip_list #378

merged 5 commits into from
Jun 11, 2023

Conversation

ThomasSanson
Copy link
Contributor

This pull request introduces several updates and improvements to the project:

  1. The fqcn[action] rule has been removed from the skip_list as it is no longer necessary.

  2. The Ansible modules used in the playbook have been updated to use the community.general and community.docker collections. These modules have been moved to these collections and are no longer available in the core Ansible distribution. This change ensures that the playbook remains compatible with the latest version of Ansible.

  3. Deprecated Ansible modules have been replaced with their updated counterparts. This change improves the maintainability of the roles, as the updated modules are more reliable and have better support.

  4. The Ansible module names used in the roles have been updated to use fully qualified names. This helps to avoid any potential conflicts with other modules

The fqcn[action] rule was removed from the skip_list as it is no longer necessary.
… community.docker collections

The Ansible modules used in the playbook have been updated to use the community.general and community.docker collections. This is because the modules have been moved to these collections and are no longer available in the core Ansible distribution. This change ensures that the playbook remains compatible with the latest version of Ansible.

🐛 fix(roles): replace deprecated Ansible modules with their updated counterparts
✨ feat(roles): improve maintainability by using updated Ansible modules
The deprecated Ansible modules have been replaced with their updated counterparts to ensure that the roles continue to work as expected. This improves the maintainability of the roles as the updated modules are more reliable and have better support.

🚚 chore(roles): update Ansible module names to use fully qualified names
The Ansible modules used in the roles have been updated to use fully qualified names. This is to ensure that the correct modules are used and to avoid any potential conflicts with other modules. The updated modules are as follows:
- community.postgresql.postgresql_schema
- community.postgresql.postgresql_user
- ansible.posix.authorized_key
- ansible.posix.mount
- ansible.posix.sysctl
- community.general.timezone
- community.general.make
@vitabaks
Copy link
Owner

I think within the framework of this PR, it is now necessary to set the fqdn for all modules throughout the code, which of course is a lot of time.

@ThomasSanson Do you need my help with this?

@ThomasSanson
Copy link
Contributor Author

@vitabaks,

Thank you for your offer, but I'll handle this and gradually address the skip_list of ansible-lint.

Regardless, this pull request will be approved. However, I must first change the minimum version of Ansible.

🔨 chore(config_pgcluster.yml): update minimal Ansible version to 2.11.0
🔨 chore(deploy_pgcluster.yml): update minimal Ansible version to 2.11.0
🔨 chore(meta/main.yml): update minimum supported Ansible version to 2.11.0
The minimum supported Ansible version has been updated to 2.11.0 in all relevant files. This is done to ensure compatibility with the latest version of Ansible and to take advantage of its new features and improvements.
…rement

This commit adds a new file named runtime.yml which specifies the minimum version of Ansible required to run the application. This file is important as it helps ensure that the application is run on a compatible version of Ansible.
This commit adds a newline at the end of the runtime.yml file to improve readability and ensure consistency with other files in the repository.
@ThomasSanson
Copy link
Contributor Author

ThomasSanson commented Jun 10, 2023

@vitabaks,

Could you please restart the GitHub centos8 action ?

See #374

@ThomasSanson
Copy link
Contributor Author

pull request is ready 😇

@vitabaks
Copy link
Owner

vitabaks commented Jun 11, 2023

I skimmed through the code and found that there are still a lot of modules without fqdn. Please tell me why you translated only part of the code to fqdn and not the whole code?

I understand that if we decided to use this style, then it should be followed throughout the code.

For example
instead of apt should be ansible.builtin.apt
instead of debug should be ansible.builtin.debug

and etc

Or do I understand something wrong?

@ThomasSanson
Copy link
Contributor Author

I activated only fqcn[action] and not fqcn[action-core] because it allows me to make progress gradually.

I will handle the other part in a separate pull request.

This way, it reduces the workload for your reviews too.

@vitabaks
Copy link
Owner

thank you for the clarification.

I completely agree with you, it is much more convenient to go gradually in small steps.

@vitabaks vitabaks merged commit b29734f into vitabaks:master Jun 11, 2023
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.

2 participants