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

Updates #36

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@
# this repository, with the Development group able to author and push pull requests.

# In a true org structure, this line would be something like: "app-fqdn-rules.tf @<secops>"
app-fqdn-rules.tf @<secops>
app-fqdn-rules.tf @NidaTech

# In a true org structure, these files should have the Network Operators account as the owner:
main.tf @<netops>
provider.tf @<netops>
variables.tf @<netops>
versions.tf @<netops>
main.tf @NidaRe
provider.tf @NidaRe
variables.tf @NidaRe
versions.tf @NidaRe
24 changes: 0 additions & 24 deletions .github/workflows/terraform.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,31 +45,7 @@ jobs:
run: terraform plan -no-color
continue-on-error: true

# Add terraform plan output back into the pull request
- name: Update Pull Request
uses: actions/[email protected]
if: github.event_name == 'pull_request'
env:
PLAN: "terraform\n${{ steps.plan.outputs.stdout }}"
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
script: |
const output = `#### Terraform Format and Style 🖌\`${{ steps.fmt.outcome }}\`
#### Terraform Initialization ⚙️\`${{ steps.init.outcome }}\`
#### Terraform Plan 📖\`${{ steps.plan.outcome }}\`
<details><summary>Show Plan</summary>
\`\`\`
${process.env.PLAN}
\`\`\`
</details>
*Pusher: @${{ github.actor }}, Action: \`${{ github.event_name }}\`*`;

github.issues.createComment({
issue_number: context.issue.number,
owner: context.repo.owner,
repo: context.repo.repo,
body: output
})

# Exit on Plan failure
- name: Terraform Plan Status
Expand Down
2 changes: 2 additions & 0 deletions app-fqdn-rules.tf
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@ locals {
tcp = {
"*.aviatrix.com" = "443"
"aviatrix.com" = "80"
"*.ubuntu.com" = "80"
"*.ubuntu.com" = "443"
}
udp = {
"dns.google.com" = "53"
Expand Down
2 changes: 1 addition & 1 deletion backend.tf
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
terraform {
required_version = ">= 1.1.0"
backend "remote" {
# organization = "<replace-with-your-Terraform-Cloud-organization-and-uncomment>"
organization = "Aviatrix-IaaC"
workspaces {
name = "ace-iac-day-two"
}
Expand Down