Skip to content

Update create_asset_mapping_spreadsheet.py#36

Open
robbfo wants to merge 1 commit into
masterfrom
robbfo-patch-1-asset-mapping-csv-field-limit
Open

Update create_asset_mapping_spreadsheet.py#36
robbfo wants to merge 1 commit into
masterfrom
robbfo-patch-1-asset-mapping-csv-field-limit

Conversation

@robbfo

@robbfo robbfo commented May 13, 2025

Copy link
Copy Markdown

Per CDAO PVC issue brought forward by a very large workspace, there is an exception which may occur when running the asset mapping for thousands of users and/or groups. Group membership field csv size can excede the default 130k characters. This patch should resolve the issue.

Per CDAO PVC issue brought forward by a very large workspace, there is an exception which may occur when running the asset mapping for thousands of users and/or groups. Group membership field csv size can excede the default 130k characters. This patch should resolve the issue.
@robbfo robbfo requested a review from a team May 13, 2025 17:16

@tejasnp163 tejasnp163 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good add

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.

2 participants