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

Fix: cli destroy incomplete setup #48

Merged
merged 3 commits into from
Feb 12, 2024
Merged

Conversation

mturetskii
Copy link
Member

@mturetskii mturetskii commented Feb 9, 2024

Description of the Change

  • Fallback to Git Clone on Local GitOps Repo Initialization Failure: In scenarios where local initialization of a GitOps repository fails, the system now automatically attempts to clone the repository from Git. This ensures continuity and reduces setup failures for users.

  • Enhanced Error Handling in Terraform Wrapper: Corrected the error output mechanism within the Terraform wrapper. Errors are now more accurately displayed to the user, enhancing the clarity and immediacy of troubleshooting.

  • Avoiding Unnecessary Initialization: The process has been refined to prevent the attempt to initialize a repository if certain prerequisites, such as the completion of the "gitops-vcs" step, are not met. This optimization prevents errors and inefficiencies in the setup process.

@mturetskii mturetskii requested a review from all4code February 9, 2024 16:31
@mturetskii mturetskii self-assigned this Feb 9, 2024
@mturetskii mturetskii merged commit 6b375eb into main Feb 12, 2024
@mturetskii mturetskii deleted the fix/cli-destroy-incomplete-setup branch February 12, 2024 13:42
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.

2 participants