Skip to content
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

r/instance: breaking change for 4.x support #208

Closed
4 tasks done
pradeep17j opened this issue Jul 30, 2024 · 4 comments
Closed
4 tasks done

r/instance: breaking change for 4.x support #208

pradeep17j opened this issue Jul 30, 2024 · 4 comments
Labels
user-error User Error

Comments

@pradeep17j
Copy link

pradeep17j commented Jul 30, 2024

Code of Conduct

  • I have read and agree to the Code of Conduct.
  • Vote on this issue by adding a 👍 reaction to the original issue initial description to help the maintainers prioritize.
  • Do not leave "+1" or other comments that do not add relevant information or questions.
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Terraform

v1.7.5

Terraform Provider

v0.10.0

VMware Cloud Foundation

4.x

Description

As described in the fix for #189.

r/vcf_instance will stop working for 4.x.

Please use cloud builder API to find if the version of VCF is 4.x or 5.x .
Accordingly make decision to not include cluster_image_enabled flag in the spec if it is 4.x.

A fix for #189 should not introduce a breaking change for 4.x (without a major semver.)
(spacegospod) - the provider is still under development and backwards compatibility and support is not guaranteed.
The statement that breaking changes are only allowed on major releases is not true, at least not before we hit v1.0

Affected Resources or Data Sources

r/vcf_instance

Terraform Configuration

resource vcf_instance "example"
{
}

Debug Output

Based on comment in #189 about breaking change in r/vcf_instance for 4.x.

Panic Output

No response

Expected Behavior

Creation of r/vcf_instance on 4.x should succeed.

Actual Behavior

terraform apply with or without cluster_image_enabled can fail for 4.x.

Steps to Reproduce

Submitting bug based on comment in #189.

Will run this workflow and update the bug.

Environment Details

No response

Screenshots

No response

References

No response

@pradeep17j pradeep17j added bug Bug needs-triage Needs Triage labels Jul 30, 2024
@github-actions github-actions bot added the pending-review Pending Review label Jul 30, 2024
@tenthirtyam tenthirtyam removed the pending-review Pending Review label Jul 30, 2024
@tenthirtyam tenthirtyam added this to the Backlog milestone Jul 30, 2024
@tenthirtyam tenthirtyam removed the needs-triage Needs Triage label Jul 30, 2024
@tenthirtyam
Copy link
Contributor

Formatting applied.

@tenthirtyam tenthirtyam added the breaking-change Breaking Change label Jul 30, 2024
@tenthirtyam tenthirtyam changed the title resource vcf_instance for 4.x r/instance: breaking change for 4.x support Jul 31, 2024
@spacegospod
Copy link
Contributor

@tenthirtyam The breaking change was concious and intentional.
There are other changes in the provider that are likely to break 4.X support and going forward I believe we should only support 1 major version at a time.

I propose that I update the docs to reflect that we only fully support VCF 5.X

@tenthirtyam
Copy link
Contributor

Closing as the version support is clearly provided in the README and the registry.

@tenthirtyam tenthirtyam closed this as not planned Won't fix, can't repro, duplicate, stale Jul 31, 2024
@tenthirtyam tenthirtyam added user-error User Error and removed bug Bug breaking-change Breaking Change labels Jul 31, 2024
@tenthirtyam tenthirtyam removed this from the Backlog milestone Aug 22, 2024
Copy link

I'm going to lock this issue because it has been closed for 30 days. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
user-error User Error
Projects
None yet
Development

No branches or pull requests

3 participants