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 terraform destroy for ec2 integration tests #1535

Merged
merged 1 commit into from
Feb 10, 2025
Merged

Fix terraform destroy for ec2 integration tests #1535

merged 1 commit into from
Feb 10, 2025

Conversation

musa-asad
Copy link
Contributor

@musa-asad musa-asad commented Feb 9, 2025

Description of the issue

Whenever we cancel our EC2 integration tests that don't use the terraform/ec2/linux directory, we get the following error:

This module is not yet installed. Run "terraform init" to install all
modules required by this configuration.

This is caused by not going into the correct directory when we run terraform destroy.

Description of changes

  • Set to correct directory for destroy.

License

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Tests

https://github.com/aws/amazon-cloudwatch-agent/actions/runs/13232946491/job/36933626388

Requirements

Before commit the code, please do the following steps.

  1. Run make fmt and make fmt-sh
  2. Run make lint

@musa-asad musa-asad self-assigned this Feb 9, 2025
@musa-asad musa-asad requested review from jefchien, dricross and phugay and removed request for jefchien February 10, 2025 04:42
@musa-asad musa-asad marked this pull request as ready for review February 10, 2025 04:44
@musa-asad musa-asad requested a review from a team as a code owner February 10, 2025 04:44
@musa-asad musa-asad merged commit 5e3b831 into main Feb 10, 2025
305 of 360 checks passed
@musa-asad musa-asad deleted the fix-ec2 branch February 10, 2025 15:09
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