Skip to content

feat(current-user): expose customer info in sysdig_current_user data source#699

Merged
airadier merged 6 commits intomasterfrom
customer-info-in-current-user
Feb 9, 2026
Merged

feat(current-user): expose customer info in sysdig_current_user data source#699
airadier merged 6 commits intomasterfrom
customer-info-in-current-user

Conversation

@airadier
Copy link
Copy Markdown
Collaborator

@airadier airadier commented Feb 6, 2026

  • Adds customer_id, customer_name, and customer_external_id computed attributes to sysdig_current_user
  • Clarifies sysdig_secure_tenant_external_id docs to distinguish cloud onboarding external ID from customer-level external ID

…source

Add customer_id, customer_name, and customer_external_id computed
attributes to the sysdig_current_user data source, sourced from the
user's customer object in the API response. Clarify documentation for
sysdig_secure_tenant_external_id to distinguish the cloud onboarding
external ID from the customer-level external ID.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@airadier airadier requested a review from a team as a code owner February 6, 2026 15:15
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Expose customer metadata on the sysdig_current_user data source and clarify the difference between tenant cloud-onboarding external IDs vs customer external IDs in documentation.

Changes:

  • Add customer_id, customer_name, and customer_external_id computed attributes to sysdig_current_user.
  • Extend the v2 client user model to deserialize customer information.
  • Update docs to clarify sysdig_secure_tenant_external_id meaning and document new sysdig_current_user attributes.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
website/docs/d/secure_tenant_external_id.md Clarifies the tenant external ID is for cloud onboarding and points users to sysdig_current_user for customer external ID.
website/docs/d/current_user.md Documents new customer-related computed attributes for the current user data source.
sysdig/internal/client/v2/model.go Adds Customer model and includes it in User for API deserialization.
sysdig/data_source_sysdig_current_user_test.go Extends acceptance test to assert new attributes are present in state.
sysdig/data_source_sysdig_current_user.go Adds schema + read mapping for customer attributes on the data source.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread sysdig/data_source_sysdig_current_user.go
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings February 6, 2026 15:36
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread sysdig/data_source_sysdig_current_user.go Outdated
Comment thread sysdig/data_source_sysdig_current_user_test.go Outdated
@airadier airadier changed the title feat(current-user): expose customer info in sysdig_current_user data source feat(current-user): [ET-723] expose customer info in sysdig_current_user data source Feb 6, 2026
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings February 6, 2026 16:20
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread sysdig/data_source_sysdig_current_user.go
Comment thread website/docs/d/secure_tenant_external_id.md Outdated
customer_name and customer_external_id may be empty depending on the
account/environment, making TestCheckResourceAttrSet brittle. The
customer_id check alone is sufficient to verify the customer object
mapping works correctly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@airadier airadier requested a review from Copilot February 6, 2026 16:26
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread sysdig/data_source_sysdig_current_user_test.go
@airadier airadier requested a review from Copilot February 9, 2026 09:51
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread sysdig/data_source_sysdig_current_user_test.go
Comment thread website/docs/d/secure_tenant_external_id.md Outdated
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator Author

@airadier airadier left a comment

Choose a reason for hiding this comment

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

Addressing Copilot review comments.

Comment thread sysdig/data_source_sysdig_current_user_test.go Outdated
@airadier airadier changed the title feat(current-user): [ET-723] expose customer info in sysdig_current_user data source feat(current-user): expose customer info in sysdig_current_user data source Feb 9, 2026
@airadier airadier added this pull request to the merge queue Feb 9, 2026
Merged via the queue into master with commit 77f1bed Feb 9, 2026
171 checks passed
@airadier airadier deleted the customer-info-in-current-user branch February 9, 2026 16:12
fcracker79 pushed a commit that referenced this pull request Mar 5, 2026
…source (#699)

- Adds customer_id, customer_name, and customer_external_id computed
attributes to sysdig_current_user
- Clarifies sysdig_secure_tenant_external_id docs to distinguish cloud
onboarding external ID from customer-level external ID

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

3 participants