Skip to content

Commit

Permalink
Fix lints
Browse files Browse the repository at this point in the history
  • Loading branch information
Scott Davidson committed Feb 26, 2024
1 parent 0ed105f commit 7a5869a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions roles/os_networks/tasks/networks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -80,13 +80,13 @@

# Workaround bug https://bugs.launchpad.net/ansible-collections-openstack/+bug/2049658
# by looking up external network information using networks_info and then explicitly
# passing the network ID into the openstack.cloud.router. Remove this workaround and
# passing the network ID into the openstack.cloud.router. Remove this workaround and
# uncomment code above when bug is fixed.

- name: Ensure router is registered with neutron
# Can't loop over blocks in Ansible so have to
# include separate tasks file instead :(
include_tasks: router_workaround.yml
ansible.builtin.include_tasks: router_workaround.yml
with_items: "{{ os_networks_routers }}"
when: item.state | default('present') == 'present'

Expand Down
2 changes: 1 addition & 1 deletion roles/os_networks/tasks/router_workaround.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
# passing the network ID into the openstack.cloud.router's network field.

# NOTE: When the item.network parameter is an ID then we are effectively querying a
# network by ID just to extract it's ID... but since the 'name' field of
# network by ID just to extract it's ID... but since the 'name' field of
# openstack.cloud.networks_info makes no distinction between names and IDs we can't
# really avoid this.

Expand Down

0 comments on commit 7a5869a

Please sign in to comment.