Skip to content
This repository has been archived by the owner on Mar 8, 2024. It is now read-only.

fix: fix failing test #14

Merged
merged 2 commits into from
Sep 21, 2023
Merged

fix: fix failing test #14

merged 2 commits into from
Sep 21, 2023

Conversation

RothAndrew
Copy link
Member

@RothAndrew RothAndrew commented Sep 21, 2023 •
edited
Loading

  • run terraform destroy again if the first one fails to destroy everything
  • trim newlines in one of the makefiles

@RothAndrew RothAndrew self-assigned this Sep 21, 2023
@RothAndrew
Copy link
Member Author

See https://github.com/defenseunicorns/terraform-aws-uds-bastion/issues/47

@RothAndrew
Copy link
Member Author

RothAndrew commented Sep 21, 2023 •
edited
Loading

/test all
🤖 View pipeline run

@RothAndrew
Copy link
Member Author

RothAndrew commented Sep 21, 2023 •
edited
Loading

/test all
🤖 View pipeline run

@RothAndrew RothAndrew changed the title fix: fix failing test by potentially running terraform destroy twice fix: fix failing test Sep 21, 2023
@RothAndrew RothAndrew merged commit 32ac8d4 into main Sep 21, 2023
4 checks passed
@RothAndrew RothAndrew deleted the feature/fix-failing-test branch September 21, 2023 17:17
@RothAndrew RothAndrew mentioned this pull request Sep 19, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@RothAndrew RothAndrew

Labels
broken-test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@RothAndrew