Skip to content

Commit

Permalink
fix: load balancer variable naming
Browse files Browse the repository at this point in the history
Commit 45c7cb0 changed how load balancer groups are accessed. This was
a good change, but some of the load balancer variables were changed
accidentally. e.g.

load_balancers_nonmeza --> load_balancers_meza_nonmeza

This "_meza_nonmeza" ending (a) was applied inconsistently and (b)
doesn't make sense as a naming convention.

This commit fixes naming.
  • Loading branch information
jamesmontalvo3 committed Sep 19, 2023
1 parent 444b057 commit b289881
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
10 changes: 5 additions & 5 deletions config/defaults.yml
Original file line number Diff line number Diff line change
Expand Up @@ -240,24 +240,24 @@ load_balancers_all: >
( groups.get('load_balancers', [] ) ) +
( groups.get('load_balancers_meza_internal', [] ) ) +
( groups.get('load_balancers_meza_external', [] ) ) +
( groups.get('load_balancers_meza_nonmeza', [] ) ) +
( groups.get('load_balancers_meza_nonmeza_internal', [] ) ) +
( groups.get('load_balancers_meza_nonmeza_external', [] ) )
( groups.get('load_balancers_nonmeza', [] ) ) +
( groups.get('load_balancers_nonmeza_internal', [] ) ) +
( groups.get('load_balancers_nonmeza_external', [] ) )
}}
# Just load balancers for handling internal services
load_balancers_internal: >
{{
( groups.get('load_balancers_meza_internal', [] ) ) +
( groups.get('load_balancers_meza_nonmeza_internal', [] ) )
( groups.get('load_balancers_nonmeza_internal', [] ) )
}}
# Just load balancers for handling external traffic
load_balancers_external: >
{{
( groups.get('load_balancers_meza_external', [] ) ) +
( groups.get('load_balancers_meza_nonmeza_external', [] ) )
( groups.get('load_balancers_nonmeza_external', [] ) )
}}
# Just load balancers that handle internal and external
Expand Down
2 changes: 1 addition & 1 deletion src/scripts/getmeza.sh
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ if [ -f /etc/redhat-release ]; then
done

# if Debian support still desired, add else condition here
fi
fi

# make sure conf-meza exists and has good permissions
mkdir -p ${INSTALL_DIR}/conf-meza/secret
Expand Down

0 comments on commit b289881

Please sign in to comment.