Missing modify_client_match and get_export_policy_rule

[zeten30]

SUMMARY

• Added 'modify_client_match' function. It was not defined at all
• Fixing 'get_export_policy_rule'. Client match was not included in query, so it always matched the first rule. Module was not able to create/modify multiple rules inside one policy

ISSUE TYPE

• Bugfix Pull Request

COMPONENT NAME

modules/storage/netapp/na_ontap_export_policy_rule.py

ANSIBLE VERSION

ansible 2.6.2
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/home/brq/mzink/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python2.7/site-packages/ansible
  executable location = /usr/bin/ansible
  python version = 2.7.15 (default, May 16 2018, 17:50:09) [GCC 8.1.1 20180502 (Red Hat 8.1.1-1)]

ADDITIONAL INFORMATION

# Missing function ->
fatal: [localhost]: FAILED! => {"changed": false, "module_stderr": "Traceback (most recent call last):\n  File \"/tmp/ansible_aRMrZU/ansible_module_na_ontap_export_policy_rule.py\", line 528, in <module>\n    main()\n  File \"/tmp/ansible_aRMrZU/ansible_module_na_ontap_export_policy_rule.py\", line 524, in main\n    rule_obj.apply()\n  File \"/tmp/ansible_aRMrZU/ansible_module_na_ontap_export_policy_rule.py\", line 512, in apply\n    self.modify_client_match(rule_index)\nAttributeError: 'NetAppontapExportRule' object has no attribute 'modify_client_match'\n", "module_stdout": "", "msg": "MODULE FAILURE", "rc": 1


# Single rule problem
Inventory
            "test20": # Policy
              "na-vsim.brq.redhat.com": # Client-match
                protocol:              nfs # Settings fro cline-match ^
                ro_rule:               sys
                rw_rule:               sys
                super_user_security:   sys
              ".brq.redhat.com":
                protocol:              nfs4
                rw_rule:               krb5p
                super_user_security:   krb5p

Module exited with success, but only the last rule ".brq.redhat.com" was present, not both!
Fixed by changing  'get_export_policy_rule' function.

[ansibot]

cc @amit0701 @carchi8py @hulquest @lmprice @ndswartz @schmots1
click here for bot help

[schmots1]

Will pull for internal test, and push out via process. Thank you for submission.

[zeten30]

Perfect. You are welcome.