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

[PL-42246] : Added more information in docs to have better clarity #766

Merged
merged 4 commits into from
Nov 21, 2023

Conversation

adiyaar24
Copy link
Collaborator

@adiyaar24 adiyaar24 commented Nov 20, 2023

Describe your changes

When setting up a new resource group using Terraform, if a user neglects to specify the included scopes, the backend (BE) takes charge by automatically populating the default scope.
However, during subsequent changes, Terraform attempts to make the scope null, assuming it was never defined by the user. The snag arises because the backend doesn't permit this operation; the scope is a required field in our database. To enhance clarity, we're updating our documentation to ensure a more transparent understanding of these intricacies.

Comment Triggers

PR Check triggers
  • Build: trigger build
  • Sub Category Field Check: trigger subcategoryfieldcheck
  • gitleaks: trigger gitleaks

@adiyaar24 adiyaar24 changed the title [PL-42246] -> Added more information in docs to have better clarity [PL-42246] : Added more information in docs to have better clarity Nov 20, 2023
Copy link

@Jimit09Gandhi Jimit09Gandhi left a comment

Choose a reason for hiding this comment

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

LGTM. Have it reviewed by @brian-f-harness

internal/service/platform/resource_group/resource_group.go Outdated Show resolved Hide resolved
docs/resources/platform_resource_group.md Outdated Show resolved Hide resolved
Co-authored-by: brian-f-harness <brian.fisher@harness.io>
Co-authored-by: brian-f-harness <brian.fisher@harness.io>
@adiyaar24 adiyaar24 merged commit 9bc9f08 into main Nov 21, 2023
3 checks passed
@adiyaar24 adiyaar24 deleted the fix/PL-42246 branch November 21, 2023 08:39
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.

None yet

5 participants