Skip to content

updated terraform schema#10

Open
hurryingauto3 wants to merge 1 commit intomainfrom
luhadiagarvit-IAC
Open

updated terraform schema#10
hurryingauto3 wants to merge 1 commit intomainfrom
luhadiagarvit-IAC

Conversation

@hurryingauto3
Copy link
Owner

This pull request introduces significant updates to the chameleon_devops/terraform configuration, focusing on simplifying the codebase, consolidating resources, and improving security group rules. The changes include removing redundant configurations, standardizing resource definitions, and adding new ingress rules for various services. Below is a summary of the most important changes:

Removal of Redundant Configurations

  • Removed the clouds.yaml file, which contained sensitive credentials and redundant cloud configurations. This file is no longer necessary for the Terraform setup.

Simplification and Consolidation of Resources

  • Consolidated SSH keypair imports by replacing multiple keypair resources (keypair_kvm, keypair_chi) with a single keypair resource for consistency.
  • Standardized GPU and staging node configurations by removing region-specific logic (e.g., use_chi_tacc, use_chi_uc) and simplifying resource definitions. [1] [2]

Updates to Security Group Rules

  • Replaced project-specific security groups (e.g., mlops_secgrp_proj4) with a single mlops_secgrp security group for all MLOps VMs.
  • Added new ingress rules to allow traffic for services such as MLflow UI (port 5000), Grafana (port 3000), Prometheus (port 9090), MinIO (ports 9000-9001), FastAPI (port 8000), Ray Dashboard (port 8265), and Ray Client Server (port 10001).

Improvements to Networking Configurations

  • Reintroduced routers and subnets for CHI@TACC and CHI@UC with clearer naming conventions and optional configurations for private networks. [1] [2]

Enhancements to Persistent Storage

  • Updated persistent storage configurations for the services and GPU nodes, including adjustments to volume sizes and resource names for clarity. [1] [2]

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