Skip to content

Conversation

@ocofaigh
Copy link
Contributor

@ocofaigh ocofaigh commented Jul 1, 2025

Description

  • Bumped to latest v3.2.1 of cloud pak deployer
  • Helm upgrade is failing with below error. It seems its trying to change the value of VolumeName. In this PR I have set it to a static value to see if it will fix the upgrade. I also updated the upgrade test so it will test the apply on upgrae, so it should now catch this bug.
 2025/07/01 20:38:52 Terraform apply | Error: failed to replace object: PersistentVolumeClaim "cloud-pak-deployer-status" is invalid: spec: Forbidden: spec is immutable after creation except resources.requests and volumeAttributesClassName for bound claims
 2025/07/01 20:38:52 Terraform apply |   core.PersistentVolumeClaimSpec{
 2025/07/01 20:38:52 Terraform apply |   	AccessModes:      {"ReadWriteOnce"},
 2025/07/01 20:38:52 Terraform apply |   	Selector:         nil,
 2025/07/01 20:38:52 Terraform apply |   	Resources:        {Requests: {s"storage": {i: {...}, s: "10Gi", Format: "BinarySI"}}},
 2025/07/01 20:38:52 Terraform apply | - 	VolumeName:       "pvc-0c01d463-1ac9-4764-bb5e-25461e8574cc",
 2025/07/01 20:38:52 Terraform apply | + 	VolumeName:       "",
 2025/07/01 20:38:52 Terraform apply |   	StorageClassName: &"ibmc-vpc-block-10iops-tier",
 2025/07/01 20:38:52 Terraform apply |   	VolumeMode:       &"Filesystem",
 2025/07/01 20:38:52 Terraform apply |   	... // 3 identical fields
 2025/07/01 20:38:52 Terraform apply |   }

Release required?

  • No release
  • Patch release (x.x.X)
  • Minor release (x.X.x)
  • Major release (X.x.x)
Release notes content

Run the pipeline

If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.

Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:

/run pipeline

Checklist for reviewers

  • If relevant, a test for the change is included or updated with this PR.
  • If relevant, documentation for the change is included or updated with this PR.

For mergers

  • Use a conventional commit message to set the release level. Follow the guidelines.
  • Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
  • Use the Squash and merge option.

@ocofaigh ocofaigh requested a review from argeiger as a code owner July 1, 2025 21:04
@ocofaigh
Copy link
Contributor Author

ocofaigh commented Jul 1, 2025

/run pipeline

1 similar comment
@ocofaigh
Copy link
Contributor Author

ocofaigh commented Jul 2, 2025

/run pipeline

@ocofaigh
Copy link
Contributor Author

ocofaigh commented Jul 2, 2025

The "fix" for VolumeName does not work. It now fails initial deploy with:

 2025/07/02 09:07:32 Terraform apply | Error: cannot patch "cloud-pak-deployer-status" with kind PersistentVolumeClaim: PersistentVolumeClaim "cloud-pak-deployer-status" is invalid: spec: Forbidden: spec is immutable after creation except resources.requests and volumeAttributesClassName for bound claims
 2025/07/02 09:07:32 Terraform apply |   core.PersistentVolumeClaimSpec{
 2025/07/02 09:07:32 Terraform apply |   	AccessModes:      {"ReadWriteOnce"},
 2025/07/02 09:07:32 Terraform apply |   	Selector:         nil,
 2025/07/02 09:07:32 Terraform apply |   	Resources:        {Requests: {s"storage": {i: {...}, s: "10Gi", Format: "BinarySI"}}},
 2025/07/02 09:07:32 Terraform apply | - 	VolumeName:       "pvc-0c01d463-1ac9-4764-bb5e-25461e8574cc",
 2025/07/02 09:07:32 Terraform apply | + 	VolumeName:       "cloud-pak-deployer-status",
 2025/07/02 09:07:32 Terraform apply |   	StorageClassName: &"ibmc-vpc-block-10iops-tier",
 2025/07/02 09:07:32 Terraform apply |   	VolumeMode:       &"Filesystem",
 2025/07/02 09:07:32 Terraform apply |   	... // 3 identical fields
 2025/07/02 09:07:32 Terraform apply |   }

Will need to revisit

@argeiger
Copy link
Contributor

/run pipeline

@argeiger
Copy link
Contributor

/run pipeline

@argeiger
Copy link
Contributor

/run pipeline

@argeiger
Copy link
Contributor

/run pipeline

@vburckhardt vburckhardt self-requested a review July 14, 2025 09:27
@vburckhardt vburckhardt merged commit 79a7c22 into main Jul 14, 2025
2 checks passed
@vburckhardt vburckhardt deleted the day2 branch July 14, 2025 09:27
@terraform-ibm-modules-ops
Copy link
Contributor

🎉 This PR is included in version 2.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants