Fix Molecule links in inventory

[hcherukuri]

Fix Molecule links in inventory overrides inline group_vars/host_vars so all inline variable definitions were silently ignored.

[hcherukuri]

@istraka has found an issue with the molecule tests vars. Please see the details below:

I found an problem in wildfly molecule tests:
    links:
      group_vars: ../group_varshttps://github.com/ansible-middleware/wildfly/blob/main/molecule/default/molecule.yml#L17
will override everything else.

For example
    group_vars:
      all:
        molecule_scenario_requires_app: truehttps://github.com/ansible-middleware/wildfly/blob/main/molecule/default/molecule.yml#L18
will have no effect

 instance-rhel9:
        wildfly_java_version: 17https://github.com/ansible-middleware/wildfly/blob/main/.config/molecule/config.yml#L39
will have no effect

I tested it and the values are not propagated.see. You can use either hosts/group_vars/host_vars sections of inventory OR links. If you use both, links will win.in https://docs.ansible.com/projects/molecule/configuration/ (edited) docs.ansible.comConfiguration - Ansible Molecule

[RanabirChakraborty]

@hcherukuri this will work fine for this moment but when we'll upgrade the rhel version to rhel10 then we need jdk21 as default version as rhel10 doesn't have jdk17 and it'll cause issues. With this fix in future we'll try to pass for example jdk17 for rhel8,9,10 but for rhel10 this will work and then we need to come back to this logic or something separate to run diff rhel versions with diff jdk versions.

[hcherukuri]

Thanks @RanabirChakraborty, i agree it might be a issue when we add RHEL 10. I will add the RHEL 10 scenario and test it out to see if this works.

[RanabirChakraborty]

@hcherukuri thanks seems like it's working fine now with rhel10 , you can comment out the [rhel10 ](name: instance-rhel10) part and then we'll merge this PR.

[hcherukuri]

@RanabirChakraborty Done

[RanabirChakraborty]

LGTM