CI: update matrix

[Andersson007]

SUMMARY

stable-2.16 has been created, adding it to the matrix

postgresql_db

A task containing an empty variable fails now

116     lc_collate: "pt_BR{{ locale_latin_suffix }}"

with

with `[ERROR]: Task failed: Module failed: Database query failed: invalid LC_COLLATE locale name: "pt_BRNone"

the var is defined as empty in another file, so Ansible since 2.19 substitutes it with None. In the earlier versions it seems to substitute it with an empty string.

Adding the following for the variable has helped for postgresql_db test target:

- set_fact:
    locale_latin_suffix: "{{ (locale_latin_suffix is not none) | ternary(locale_latin_suffix, '') }}"

postgresql_info

the failures were partly fixed. The following one was reported in ansible/ansible#85036

213       - result.databases is defined
214
215   - name: postgresql_info - test return publication info
        ^ column 5

fatal: [testhost]: FAILED! => {
    "changed": false,
    "msg": "Task failed: Module failed: Key of type 'int' is not JSON serializable by the 'module_legacy_m2c' profile."
}

[Andersson007]

(pinging @oraNod here as he asked)

[andreasscherbaum]

s/until/as long/

[Andersson007]

@andreasscherbaum thanks! If you "suggest" the changes i could press the commit button so that they'll be technically counted as your code contributions

[andreasscherbaum]

@Andersson007 I don't see an option to do that :-(

My suggestion is:

# Keep this as long as the tests work.

[andreasscherbaum]

I'd rephrase this, along the lines of: When the tests stop working, remove this target and update the documentation with the fact.

As long as this is tested and works, the documentation can show that.

[Andersson007]

Folks, fyi, the changes i've made for the _info module (see the commit) are not breaking. the core doesn't implicitly convert keys in return dicts to strings anymore, so we must convert ints to strs explicitly now, see ansible/ansible#85036