spoons.technology/core-infra
4
0
Fork 1
Code Issues 9 Pull requests 1 Projects Releases Packages Wiki Activity Actions

WIP: Allow deployment on incus #17

Draft
tc424 wants to merge 11 commits from tc424/core-infra:incus-test into main
pull from: tc424/core-infra:incus-test
merge into: spoons.technology:main
Conversation 3 Commits 11 Files changed 20 +435 -141
tc424 commented 2026-01-07 21:51:54 +00:00
Owner
Copy link
No description provided.
tom commented 2026-01-08 08:03:23 +00:00
Owner
Copy link

Looks sensible. Only comment I'd have is perhaps renaming the "image_names" module to "provider_metadata", to make it easy to reuse for other provider-specific stuff (like DNS servers?).
If adding more stuff, it might also be less annoying to read if it had a variable for "provider", rather than returning all of the data and doing the filter on use.

Looks sensible. Only comment I'd have is perhaps renaming the "image_names" module to "provider_metadata", to make it easy to reuse for other provider-specific stuff (like DNS servers?). If adding more stuff, it might also be less annoying to read if it had a variable for "provider", rather than returning all of the data and doing the filter on use.
tc424 commented 2026-01-08 08:47:40 +00:00
Author
Owner
Copy link

Yeah, I was thinking about renaming it. The other option that occurred to me was just to inline the per-provider data into the modules for each provider using locals? I assume that's possible in tf ..

Yeah, I was thinking about renaming it. The other option that occurred to me was just to inline the per-provider data into the modules for each provider using locals? I assume that's possible in tf ..
tom commented 2026-01-08 10:57:02 +00:00
Owner
Copy link

You mean put it in the "scaleway_instance" module etc. ? That'd work, though it might make it easy to forget to add equivs for other providers (if we care about that).

You mean put it in the "scaleway_instance" module etc. ? That'd work, though it might make it easy to forget to add equivs for other providers (if we care about that).
which is being deployed potentially to one of a selection of
different cloud providers (or similar local environments, e.g.
incus.) This is an otherwise missing feature in terraform:
https://github.com/hashicorp/terraform/issues/1439
separate file; this means only this file needs to be converted
to terranix to allow templating, making life easier for people
who prefer to read HCL :)
selectable (somehow)
This pull request has changes conflicting with the target branch.
  • flake.lock
  • flake.nix
  • identity-services/config_scripts/identity.tf
  • terraform/environments.auto.tfvars
  • terraform/variables.tf
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u incus-test:tc424-incus-test
git switch tc424-incus-test
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
spoons.technology/core-infra!17
No description provided.