Skip to content

Change docs/ references from Terraform to OpenTofu #544

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

Merged
merged 5 commits into from
Jan 23, 2025
Merged

Conversation

bertiethorpe
Copy link
Member

No description provided.

@bertiethorpe bertiethorpe requested a review from a team as a code owner January 21, 2025 15:33
@bertiethorpe bertiethorpe requested a review from priteau January 21, 2025 15:35
@priteau
Copy link
Member

priteau commented Jan 21, 2025

There are also references in the README file. And unrelated to docs, what about ansible/ci/retrieve_inventory.yml?

@sjpb sjpb changed the title Change /docs references from Terraform to OpenTofu Change docs/ references from Terraform to OpenTofu Jan 21, 2025
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

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

Couple of bits which should have been tweaked regardless of tofu/terraform. Thanks.

bertiethorpe and others added 2 commits January 23, 2025 16:05
@bertiethorpe bertiethorpe requested a review from sjpb January 23, 2025 16:06
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

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

LGTM

@sjpb sjpb merged commit d6efcb6 into main Jan 23, 2025
1 of 2 checks passed
@sjpb sjpb deleted the docs/terraform-to-tofu branch January 23, 2025 16:07
bertiethorpe added a commit that referenced this pull request Jan 29, 2025
* change terraform references to opentofu in docs

* remove wider reference to terraform

* Update environments/README.md

Co-authored-by: Steve Brasier <[email protected]>

* Update environments/common/README.md

Co-authored-by: Steve Brasier <[email protected]>

---------

Co-authored-by: Steve Brasier <[email protected]>
sjpb added a commit that referenced this pull request Feb 11, 2025
* add rebuild role to appliance and modify groupvars

* improve readability of group_vars

* Define login nodes using an opentofu module (#547)

* define login nodes using tf module

* Apply suggestions from code review

Co-authored-by: Matt Anson <[email protected]>

* tweak README to explain compute groups

* try to clarify login/compute groups

---------

Co-authored-by: Matt Anson <[email protected]>

* Change docs/ references from Terraform to OpenTofu (#544)

* change terraform references to opentofu in docs

* remove wider reference to terraform

* Update environments/README.md

Co-authored-by: Steve Brasier <[email protected]>

* Update environments/common/README.md

Co-authored-by: Steve Brasier <[email protected]>

---------

Co-authored-by: Steve Brasier <[email protected]>

* fix instance_id in compute inventory to be target image, not deployed image

* review all roles for compute_init_enable

* fix permissions to /exports/cluster

* make openhpc_config more greppable

* Set ResumeTimeout and ReturnToService overrides in group_vars

* CI tests for reboot via slurm (without rebuild)

* fpinrocky 8 pythoolsvenv version

* refining comments and task names

* rebuild role readme

---------

Co-authored-by: Steve Brasier <[email protected]>
Co-authored-by: Matt Anson <[email protected]>
Co-authored-by: Steve Brasier <[email protected]>
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