Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scale nodes #45

Open
wants to merge 2 commits into
base: v3.10
Choose a base branch
from
Open

Scale nodes #45

wants to merge 2 commits into from

Conversation

shwetha-h-p
Copy link

No description provided.

shwethahp added 2 commits October 11, 2018 23:13
(i.e deploy_cns: False) on v3.10

1. hosts.j2 to define variables related to logging, metrics, grafana
    when deploy_cns is False

2. If sp_app_id is defined in vars.yml, sp_app variable will not be populated
    when executing the task azure_infra/tasks/cloudprovider.yml. Hence
    adding the sp_app_id check before checking for sp_app in hosts.j2

3. Define sections for glusterfs, glusterfs_registry even when
    deploy_cns is False. This is because we check for
    'inventory_hostname in groups.glusterfs' while registering the
    nodes to RHSM in roles/ocp_pre/tasks/subscribe.yml

4. Fixing the typos in vars.yml.example
    - define deploy_grafana which will be used in hosts.j2
    - modified app_subnet_cidr: "192.168.0.128/25" to
    app_subnet_cidr: "192.168.0.128/27"

5. renamed vars deploy_grafan to deploy_grafana

Signed-off-by: shwethahp <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant