Skip to content

Add PWB sharedStorage.subPath parity with homeStorage#651

Merged
samcofer merged 8 commits intomainfrom
workbench-shared-subpath
Mar 28, 2025
Merged

Add PWB sharedStorage.subPath parity with homeStorage#651
samcofer merged 8 commits intomainfrom
workbench-shared-subpath

Conversation

@samcofer
Copy link
Contributor

Home storage currently supports a subPath value that can be used to select a subPath to mount home directories from. A customer has requested that we also support this functionality in the sharedStorage location.

@samcofer
Copy link
Contributor Author

Question for @GCRev and @zachhannum, for small changes like this, do you bundle them into a larger change and bump the Chart version there, or should I coordinate the version bump with @kmasiello from her PR here: #649

@GCRev
Copy link
Contributor

GCRev commented Mar 27, 2025

@samcofer we usually see any change bump the version, so talking to Katie is a good idea, or you could wait until her PR is merged and then come back here to update the minor version number.

@samcofer
Copy link
Contributor Author

I'll talk with her about merging and then bringing in her changes! Thanks!

@samcofer
Copy link
Contributor Author

@GCRev Alright, this should be ready now. Thanks!

name: rstudio-workbench
description: Official Helm chart for Posit Workbench
version: 0.8.13
version: 0.8.14
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@GCRev GCRev left a comment

Choose a reason for hiding this comment

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

That's how we do it. LGTM

@samcofer
Copy link
Contributor Author

I suspect that something may be wrong with the GHA checks, as they've been stuck for awhile. Do you know a good way to kick them, or merge in spite of them?

@samcofer samcofer merged commit 228a98e into main Mar 28, 2025
6 of 7 checks passed
@samcofer samcofer deleted the workbench-shared-subpath branch March 28, 2025 13:36
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