Skip to content

Add LICENSE and package manifests, use tofu plan -concise - #58

Merged
3uzbcqje merged 1 commit into
mainfrom
add-license-and-package-manifests
Sep 5, 2026
Merged

3uzbcqje merged 1 commit into
mainfrom
add-license-and-package-manifests

Conversation

@3uzbcqje

@3uzbcqje 3uzbcqje commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

Ported back from the upstream template (devopscoop/azure-aks-template), which had tooling files this repo lacked. Found while comparing versions across the two repos.

Changes

  • LICENSE — MIT. This repo is public and had no license at all.
  • Brewfile / pkglist.txt — every CLI tool this repo uses, for macOS and Arch respectively. Extended beyond the template's four (az, tenv, git, kubectl) with:
    • tflint — this repo has .tflint.hcl and AGENTS.md documents the lint step
    • ghtag-pvc-disks.yml uses it to open its PR
  • tofu plan -concise — suppresses the per-resource Refreshing state... progress lines, which currently dominate the plan output posted to PRs. Supported since OpenTofu 1.10.0, so the pinned 1.12.6 is fine.

Please confirm before merging

The LICENSE copyright line reads Copyright (c) 2026 Equal Vote Coalition. I picked that entity name and year — please correct it if the copyright should sit with a different entity.

Verification

tofu fmt -recursive -check, tofu validate, and tflint all pass. No .tf files changed, so infrastructure is unaffected — tofu plan reported no changes before this branch.

🤖 Generated with Claude Code

https://claude.ai/code/session_015sgVLhPxfudCHBuHMHXFu3

Ported back from the upstream template (devopscoop/azure-aks-template),
which had tooling files this repo lacked:

- LICENSE: MIT. This repo is public and had no license.
- Brewfile / pkglist.txt: every CLI tool this repo uses. Extended beyond
  the template's four with tflint (this repo has .tflint.hcl and AGENTS.md
  documents the lint step) and gh (tag-pvc-disks.yml opens a PR with it).
- tofu plan -concise: suppresses per-resource "Refreshing state..."
  progress lines, which dominate the plan output posted to PRs. Supported
  since OpenTofu 1.10.0, so the pinned 1.12.6 is fine.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015sgVLhPxfudCHBuHMHXFu3
@3uzbcqje
3uzbcqje requested a review from ArendPeter as a code owner September 5, 2026 05:08
@coderabbitai

coderabbitai Bot commented Sep 5, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: ae03a9af-d2aa-434e-96a6-64136f9df2cd


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

OpenTofu Format and Style success

OpenTofu Initialization ⚙️success

OpenTofu Validation success

Validation Output

Success! The configuration is valid.

OpenTofu Plan success

Show Plan


No changes. Your infrastructure matches the configuration.

OpenTofu has compared your real infrastructure against your configuration and
found no differences, so no changes are needed.



Pusher: @3uzbcqje, Action: pull_request, Working Directory: ``, Workflow: .github/workflows/opentofu.yml

@3uzbcqje
3uzbcqje merged commit 239912a into main Sep 5, 2026
2 checks passed
@3uzbcqje
3uzbcqje deleted the add-license-and-package-manifests branch September 5, 2026 05:13
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.

1 participant