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

feat!: Enable Helm Release to be Optional #5

Merged
merged 12 commits into from
Dec 18, 2024
Merged

Conversation

theishshah
Copy link
Contributor

Add additional boolean flag to enable helm release. Currently configured to set base count for operator and wandb if true, defaults to true.

@theishshah theishshah changed the title Enable Helm Release to be Optional feat: Enable Helm Release to be Optional Nov 27, 2024
@j7m4 j7m4 changed the title feat: Enable Helm Release to be Optional feat!: Enable Helm Release to be Optional Dec 18, 2024
Comment on lines +1 to +43
#!/bin/bash
set -e

echo "Starting Terraform state migration..."

# Function to check if a resource exists in terraform state
check_resource() {
terraform state list | grep -q "$1" && return 0 || return 1
}

# Function to migrate a single resource
migrate_resource() {
local OLD_RESOURCE="$1"
local NEW_RESOURCE="$2"

if check_resource "$OLD_RESOURCE"; then
echo "Migrating ${OLD_RESOURCE} to ${NEW_RESOURCE}"
terraform state mv -lock=false "$OLD_RESOURCE" "$NEW_RESOURCE" || {
echo "Failed to migrate ${OLD_RESOURCE}"
return 1
}
else
echo "Resource ${OLD_RESOURCE} not found in state, skipping..."
fi
}

# Backup the current state
echo "Creating state backup..."
terraform state pull > terraform.tfstate.backup.$(date +%Y%m%d-%H%M%S)

# Migrate each resource
echo "Migrating resources..."

# Find all helm_release resources and migrate them
terraform state list | grep "helm_release" | while read -r resource; do
# Skip if resource already has count index
if [[ "$resource" != *"[0]"* ]]; then
migrate_resource "$resource" "${resource}[0]"
fi
done

echo "Migration completed!"
echo "Please review the state and run 'terraform plan' to verify the migration"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need this checked in?

@j7m4 j7m4 merged commit f69c6ea into main Dec 18, 2024
4 checks passed
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.

3 participants