Skip to content

feat: add Terraform module for lustre-server - #52

Merged
NucciTheBoss merged 1 commit into
canonical:mainfrom
jedel1043:terraform-lustre-server
Aug 13, 2026
Merged

NucciTheBoss merged 1 commit into
canonical:mainfrom
jedel1043:terraform-lustre-server

Conversation

@jedel1043

Copy link
Copy Markdown
Collaborator

Pre-submission checklist

  • I read and followed the CONTRIBUTING guidelines.
  • I have ensured that lint, typecheck, and unit tests complete successfully.

Summary of changes

Adds a Terraform module for deploying the Lustre server, facilitating the setup of a Lustre cluster with Terraform.

Docs

  • I have created a pull request to add or update relevant documentation in canonical/charmed-hpc-docs or another documentation location.

A README has been added with all the information on how to use the Terraform module.

Signed-off-by: José Julián Espina <julian.espina@canonical.com>
@jedel1043
jedel1043 requested a review from a team as a code owner August 13, 2026 15:52
@jedel1043
jedel1043 requested review from NucciTheBoss and benschwaller and removed request for a team August 13, 2026 15:52
@jedel1043 jedel1043 added C-filesystem Component: Filesystem feature This pull request adds a new feature to the project labels Aug 13, 2026

@NucciTheBoss NucciTheBoss left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great! Glad to see that my AI skill was useful!

Left a foresight comment about potential default channels in the future, but this is something that will come after we spin Lustre out into its own repository.

variable "channel" {
description = "Charm channel to deploy from"
type = string
default = "latest/edge"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

[foresight]: eventually we'll want to switch this over to the stable channel for Lustre. I'm expecting the channel name to be something like 2/stable to represent the latest major version of Lustre.

@NucciTheBoss
NucciTheBoss merged commit aaaa517 into canonical:main Aug 13, 2026
6 checks passed
@jedel1043
jedel1043 deleted the terraform-lustre-server branch August 13, 2026 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-filesystem Component: Filesystem feature This pull request adds a new feature to the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants