fix issue #199

[roumano]

• Use a list with name and mode (optional) attribut
• Use a loop_contorl / loop_var for better read the playbook
• Change only the permission of /etc/mysql/debian.cnf from 0644 to 0600 on all debian but it's can easily update to other OS

Description

Actually the permission is hardcoded to 0644 for all files.

Related Issue

fix issue #199

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

• I have read the CONTRIBUTING document.
• I have run the pre-merge tests locally and they pass.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

[eRadical]

@roumano can you please try to resolve conflicts?

[roumano]

@roumano can you please try to resolve conflicts?

Yes no problem, it's done

[eRadical]

I still get "This branch cannot be rebased due to conflicts"

[roumano]

@eRadical , Hi, it's strange, on github it's say "This branch has no conflicts with the base branch"
are you sure, you trying to make the MR on master branch ?

but as i don't have write access, i also see this : "Only those with write access to this repository can merge pull requests."

[eRadical]

Found it: The master branch in your fork is 9 commits behind.

Sync the fork 1st.
Then rebase permission_file branch.

[roumano]

Found it: The master branch in your fork is 9 commits behind.

Sync the fork 1st. Then rebase permission_file branch.

Please retry now

[roumano]

@eRadical , the issue "This branch cannot be rebased due to conflicts" is fixed for your side or not ?

[eRadical]

@roumano I have » "This branch cannot be rebased due to conflicts"

[roumano]

@roumano I have » "This branch cannot be rebased due to conflicts"

Please try again, if it's failed again, i will create a complete new (and clean) branch , ...

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[eRadical]

@roumano - can you try again to rebase? As I think this is valuable.

[roumano]

Sync fork done, please try again

[eRadical]

In vars/debian.yml you didn't quote 0600 that's why it's failing for debian 10 and ubuntu 18.

[eRadical]

For debian 12... the failure is old and I believe is from somewhere else.

[roumano]

In vars/debian.yml you didn't quote 0600 that's why it's failing for debian 10 and ubuntu 18.

Thx for the information, i've fixed it.

[eRadical]

@elcomtik - LGTM, thoughts?

[elcomtik]

LGTM