feat(CENG-462): Add manage_entitlements_privilege to repository resource#196
Merged
cloudsmith-iduffy merged 3 commits intomasterfrom Apr 24, 2026
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds support for configuring and reading manage_entitlements_privilege on Cloudsmith repositories, extending the provider’s repository resource and repository data source to expose Cloudsmith’s entitlement-token management privilege level.
Changes:
- Added
manage_entitlements_privilegetocloudsmith_repositoryschema and wired it through create/read/update. - Added
manage_entitlements_privilegetocloudsmith_repositorydata source read + schema. - Updated repository resource/data source docs and updated the repository resource acceptance test.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/resources/repository.md | Documents the new repository resource argument/attribute. |
| docs/data-sources/repository.md | Documents the new repository data source attribute. |
| cloudsmith/resource_repository.go | Adds schema + API mapping for manage_entitlements_privilege in CRUD. |
| cloudsmith/resource_repository_test.go | Extends acceptance coverage for the repository resource to include the new field. |
| cloudsmith/data_source_repository.go | Adds schema + read mapping for the new field on the repository data source. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
34faddc to
e990cdc
Compare
estebangarcia
approved these changes
Apr 24, 2026
…rce and data source Add the manage_entitlements_privilege attribute to the repository resource (create, read, update) and data source (read), with validation, docs, and acceptance tests for both. Also cleans up doc formatting: backtick-wraps enum values, fixes trailing whitespace, and removes stale go.sum entries from the API client upgrade. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Pre-install terraform via setup-terraform and set TF_ACC_TERRAFORM_PATH so the plugin test framework reuses a single binary instead of copying per test, eliminating the text file busy race condition. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
019c1e9 to
b0b0dca
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Add manage_entitlements_privilege field to the cloudsmith_repository resource and data source
Type of Change
Testing
Acceptance tests were updated but I also tested manually with:
Additional Notes