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

[BUG] Consistency of resource parameters #68

Closed
1 task done
lechnerc77 opened this issue Apr 4, 2024 · 1 comment
Closed
1 task done

[BUG] Consistency of resource parameters #68

lechnerc77 opened this issue Apr 4, 2024 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@lechnerc77
Copy link
Member

Is there an existing issue for this?

  • I have searched the existing issues

What version of the Terraform provider are you using?

main-branch

What version of the Terraform CLI are you using?

1.7.3

What CF API version are you using?

3.153

What type of issue are you facing

bug report

Describe the bug

The parameter names of the resources are not consistent with regards to the aspect that an ID or a name should be provided especially when it comes to org and space.

Examples:

  • cloudfoundry_service_instance: The parameter is called space and the ID of the CF space must be provided
  • cloudfoundry_app: The parameter is called space and the name of the CF space must be provided

Expected Behavior

The fields representing IDs should be clearly distinguishable from the fields that should be provided with a name e.g., space for the name and space_id for the ID

Steps To Reproduce

Check the documentation of the above mentioned resources to check the semantic difference

Your Terraform Configuration

n/a

Roles

n/a

Add screenshots to help explain your problem

n/a

Additional context

n/a

@lechnerc77 lechnerc77 added the bug Something isn't working label Apr 4, 2024
@Gourab1998 Gourab1998 self-assigned this Apr 4, 2024
@Gourab1998
Copy link
Contributor

SInce in all other resources convention for space/org attribute means guid, we will change the app resource attribute only as space_name and org_name to make it distinct

Gourab1998 added a commit that referenced this issue Apr 4, 2024
Dray56 pushed a commit that referenced this issue May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants