Skip to content

Conversation

@Neel050302
Copy link
Collaborator

What does this pull request do?

Test evidence

Signed-off-by: Neel050302 <[email protected]>
Signed-off-by: Neel050302 <[email protected]>
add github secrect


Signed-off-by: Neel050302 <[email protected]>
Signed-off-by: Neel050302 <[email protected]>
Signed-off-by: Neel050302 <[email protected]>
@@ -0,0 +1,21 @@
provider "google" {
Copy link
Contributor

Choose a reason for hiding this comment

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

what is this file? doesn't appear necessary

- name: 'Configure GCP Region'
run: gcloud config set compute/region australia-southeast2

# No need to change directory since Terraform files are in the root directory
Copy link
Contributor

Choose a reason for hiding this comment

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

it's not a good check of the root modules if they aren't being checked

run: gcloud config set compute/region australia-southeast2

# No need to change directory since Terraform files are in the root directory
- name: Terraform Init
Copy link
Contributor

@theraginggoblin theraginggoblin Aug 24, 2024

Choose a reason for hiding this comment

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

terraform init is a good choice for a terraform ci check

- name: Terraform Init
run: terraform init

- name: Terraform Format
Copy link
Contributor

@theraginggoblin theraginggoblin Aug 24, 2024

Choose a reason for hiding this comment

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

terraform format checking is a good choice for a terraform ci check

- name: Terraform Plan
run: terraform plan -input=false

- name: Terraform Apply
Copy link
Contributor

Choose a reason for hiding this comment

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

apply should not be ran as a basic terraform ci check

Copy link
Contributor

@theraginggoblin theraginggoblin left a comment

Choose a reason for hiding this comment

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

lets have a conversation about what it should check and how it checks it

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