Skip to content

Fix inconsistent plan on Slurm cluster reconfigure#4538

Merged
samskillman merged 3 commits into
GoogleCloudPlatform:developfrom
wiktorn:fix_inconsistent_plan
Sep 25, 2025
Merged

Fix inconsistent plan on Slurm cluster reconfigure#4538
samskillman merged 3 commits into
GoogleCloudPlatform:developfrom
wiktorn:fix_inconsistent_plan

Conversation

@wiktorn

@wiktorn wiktorn commented Aug 18, 2025

Copy link
Copy Markdown
Contributor

When changing startup script content or changing partitions, following error may occur:

╷
│ Error: Provider produced inconsistent final plan │
│ When expanding the plan for module.slurm_controller.module.slurm_files.google_storage_bucket_object.nodeset_config["flexnodeset"] to include new values learned so far during apply, provider "registry.terraform.io/hashicorp/google" produced an invalid new value for .md5hash: was known, but now unknown. │
│ This is a bug in the provider, which should be reported in the provider's own issue tracker.

By adding source_md5hash attribute to google_storage_bucket_object this problem is no longer reproducible.

Bumped version of the provider to v6.41 as this is when source_md5hash was introduced.

Submission Checklist

NOTE: Community submissions can take up to 2 weeks to be reviewed.

Please take the following actions before submitting this pull request.

  • Fork your PR branch from the Toolkit "develop" branch (not main)
  • Test all changes with pre-commit in a local branch #
  • Confirm that "make tests" passes all tests
  • Add or modify unit tests to cover code changes
  • Ensure that unit test coverage remains above 80%
  • Update all applicable documentation
  • Follow Cluster Toolkit Contribution guidelines #

b/398227662

@wiktorn wiktorn requested review from a team and samskillman as code owners August 18, 2025 12:07
When changing startup script content or changing partitions, following
error may occur:
╷
│ Error: Provider produced inconsistent final plan
│
│ When expanding the plan for module.slurm_controller.module.slurm_files.google_storage_bucket_object.nodeset_config["flexnodeset"] to include new values learned so far during apply, provider "registry.terraform.io/hashicorp/google" produced an invalid new value for .md5hash: was known, but now unknown.
│
│ This is a bug in the provider, which should be reported in the provider's own issue tracker.

With this change this is no longer the case
@wiktorn wiktorn force-pushed the fix_inconsistent_plan branch from af5f50e to dcc74ab Compare September 22, 2025 07:44
@wiktorn wiktorn changed the base branch from main to develop September 22, 2025 07:46
@samskillman samskillman added the release-bugfix Added to release notes under the "Bug fixes" heading. label Sep 25, 2025
@samskillman samskillman self-assigned this Sep 25, 2025
@samskillman

Copy link
Copy Markdown
Collaborator

/gcbrun

@samskillman

Copy link
Copy Markdown
Collaborator

/gcbrun

@samskillman

Copy link
Copy Markdown
Collaborator

/gcbrun(9c1caf4)

@samskillman

Copy link
Copy Markdown
Collaborator

Most tests above passed, other are due to unrelated issues, stockout, etc. Merging.

@samskillman samskillman merged commit fdd04fb into GoogleCloudPlatform:develop Sep 25, 2025
28 of 67 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-bugfix Added to release notes under the "Bug fixes" heading.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants