Fix "external_hostname" should not be required while using Route ingress

[TheRealHaoLiu]

SUMMARY

Fix "external_hostname" should not be required while using Route ingress

ISSUE TYPE

• Bug, Docs Fix or other nominal change

ADDITIONAL INFORMATION

[fosterseth]

we have a task that sets the external_hostname if ingress_type is "route"

- name: Set external_hostname
  set_fact:
    external_hostname: "{{ ingress.result.status.ingress[0].host }}"
  when: ingress_type | lower == 'route'

and in the jinja2 template it is only referenced under a {% if ingress_type|lower == "route" %} conditional

so not sure in which situation this wouldn't be defined when referenced.

[TheRealHaoLiu]

we have a task that sets the external_hostname if ingress_type is "route"

- name: Set external_hostname
  set_fact:
    external_hostname: "{{ ingress.result.status.ingress[0].host }}"
  when: ingress_type | lower == 'route'

and in the jinja2 template it is only referenced under a {% if ingress_type|lower == "route" %} conditional

so not sure in which situation this wouldn't be defined when referenced.

the task execute after the route is created

so bug affect all case when ingress_type=route

[fosterseth]

ah didn't realize the ordering there, got it