Skip to content

fix: tune Azure NetApp workbench-shared volumes and enable backup enforcement#285

Merged
timtalbot merged 1 commit into
mainfrom
netapp-workbench-shared-tuning
May 5, 2026
Merged

fix: tune Azure NetApp workbench-shared volumes and enable backup enforcement#285
timtalbot merged 1 commit into
mainfrom
netapp-workbench-shared-tuning

Conversation

@timtalbot
Copy link
Copy Markdown
Contributor

Summary

The workbench-shared NetApp volume is used for inter-process communication between Workbench sessions (e.g. rserver socket coordination), not for storing user data. It doesn't need large capacity or backup/snapshot protection.

  • Revert workbench-shared default capacity from 200 GiB to 50 GiB — the previous increase was applied uniformly to all volume types, but workbench-shared doesn't need it. Keeping it at 50 GiB avoids pushing the capacity pool over its size limit.
  • Disable data protection for workbench-shared volumes — snapshots and backups are unnecessary for ephemeral IPC data and waste capacity pool quota.
  • Set policy_enforced=True on volume backup properties — without this flag, the backup policy is attached but not actually enforced. This fixes backups being in a disabled state on volumes that do need protection (connect, workbench).

Test plan

  • just test-python-pulumi passes (187 tests)
  • ptd ensure <target> --only-steps persistent --dry-run shows workbench-shared volumes unchanged at 50 GiB with no dataProtection block
  • ptd ensure <target> --only-steps persistent --dry-run shows connect/workbench volumes gain policyEnforced: true

…orcement

The workbench-shared volume is used for inter-process communication
between Workbench sessions, not for storing user data. It does not
need large capacity or backup/snapshot protection.

- Revert workbench-shared default capacity from 200 GiB to 50 GiB
- Skip data_protection (backups and snapshots) for workbench-shared volumes
- Set policy_enforced=True on volume backup properties so the backup
  policy is actually active on volumes that do need protection
@timtalbot timtalbot marked this pull request as ready for review May 4, 2026 20:28
@timtalbot timtalbot requested a review from a team as a code owner May 4, 2026 20:28
@timtalbot timtalbot requested a review from Lytol May 4, 2026 20:29
@timtalbot timtalbot assigned amdove and unassigned amdove May 4, 2026
@timtalbot timtalbot requested a review from amdove May 4, 2026 20:32
@timtalbot timtalbot added this pull request to the merge queue May 5, 2026
Merged via the queue into main with commit 41c063b May 5, 2026
5 checks passed
@timtalbot timtalbot deleted the netapp-workbench-shared-tuning branch May 5, 2026 14:44
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