Skip to content

feat(BA-4766): inject BACKENDAI_PERSISTENT_PATHS env var and show persistent paths on shell startup#9738

Merged
fregataa merged 1 commit intomainfrom
BA-4766
Apr 22, 2026
Merged

feat(BA-4766): inject BACKENDAI_PERSISTENT_PATHS env var and show persistent paths on shell startup#9738
fregataa merged 1 commit intomainfrom
BA-4766

Conversation

@seedspirit
Copy link
Copy Markdown
Contributor

@seedspirit seedspirit commented Mar 6, 2026 •
edited
Loading

Summary

  • Inject BACKENDAI_PERSISTENT_PATHS environment variable (colon-separated vfolder kernel paths) into containers at session creation time
  • Update .bashrc / .zshrc to display persistent vfolder mount paths once on shell startup, warning users that all other files are ephemeral
  • Prepare EnvironProvisioner with the same logic for when the stage pipeline is wired up
  • Ensure that the final mount path is displayed even when mounting with a modified path
mount image

Test plan

  • pants fmt fix lint check all pass
  • Manual verification: confirm BACKENDAI_PERSISTENT_PATHS is set inside container via echo $BACKENDAI_PERSISTENT_PATHS

Resolves BA-4766

Copilot AI review requested due to automatic review settings March 6, 2026 06:45
@github-actions github-actions Bot added the size:L 100~500 LoC label Mar 6, 2026
seedspirit added a commit that referenced this pull request Mar 6, 2026
@seedspirit
changelog: add news fragment for PR #9738
2970881
@github-actions github-actions Bot added comp:manager Related to Manager component comp:agent Related to Agent component labels Mar 6, 2026
Copilot AI reviewed Mar 6, 2026
View reviewed changes
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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@seedspirit seedspirit changed the title feat(BA-4766): add persistent mount path metadata for ephemeral file warning feat(BA-4766): Add persistent mount path metadata for ephemeral file warning Mar 11, 2026
@seedspirit seedspirit requested a review from a team March 11, 2026 07:48
jopemachine
jopemachine previously approved these changes Mar 12, 2026
View reviewed changes
fregataa
fregataa reviewed Mar 12, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@fregataa fregataa left a comment

Choose a reason for hiding this comment

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

Is there any Kernel Info data type to update?

@seedspirit
Copy link
Copy Markdown
Contributor Author

seedspirit commented Mar 12, 2026

Is there any Kernel Info data type to update?

@fregataa I've confirmed that Kernel Info data updates are also needed. I'll add them.

@github-actions github-actions Bot added the area:docs Documentations label Mar 12, 2026
seedspirit added a commit that referenced this pull request Mar 12, 2026
@seedspirit
changelog: add news fragment for PR #9738
5461833
seedspirit added a commit that referenced this pull request Mar 13, 2026
@seedspirit
changelog: add news fragment for PR #9738
aeedc88
seedspirit added a commit that referenced this pull request Mar 24, 2026
@seedspirit
changelog: add news fragment for PR #9738
938ae56
seedspirit added a commit that referenced this pull request Mar 25, 2026
@seedspirit
changelog: add news fragment for PR #9738
8f9cc72
@github-actions github-actions Bot added the comp:common Related to Common component label Mar 25, 2026
@seedspirit seedspirit requested a review from a team March 25, 2026 06:40
fregataa
fregataa reviewed Mar 25, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@fregataa fregataa left a comment

Choose a reason for hiding this comment

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

  1. Is mounted vfolder info not included in GET session/kernel API?
  2. Please check CI failure

jopemachine
jopemachine previously approved these changes Mar 25, 2026
View reviewed changes
seedspirit added a commit that referenced this pull request Mar 26, 2026
@seedspirit
changelog: add news fragment for PR #9738
eaa1bb5
@seedspirit
Copy link
Copy Markdown
Contributor Author

seedspirit commented Mar 26, 2026

  1. Is mounted vfolder info not included in GET session/kernel API?

@fregataa yes, but it returns vfolder id, not vfolder mounted path, so it is slightly different

fregataa
fregataa requested changes Mar 26, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@fregataa fregataa left a comment

Choose a reason for hiding this comment

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

Need version annotations

Comment thread docs/manager/graphql-reference/supergraph.graphql
Comment thread docs/manager/graphql-reference/supergraph.graphql Outdated
Comment thread docs/manager/graphql-reference/v2-schema.graphql
Comment thread docs/manager/graphql-reference/v2-schema.graphql Outdated
@seedspirit seedspirit requested a review from fregataa March 26, 2026 05:18
@seedspirit seedspirit added this to the 26.4 milestone Mar 26, 2026
fregataa
fregataa reviewed Mar 30, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@fregataa fregataa left a comment

Choose a reason for hiding this comment

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

Please resolve conflict. And it is good to add a new env but not sure to add persistentMountPaths field. Wouldn't be better to add VFolder node field?

@seedspirit seedspirit requested a review from fregataa April 15, 2026 02:14
@seedspirit @claude
feat(BA-4766): inject BACKENDAI_PERSISTENT_PATHS env var and show per…
3842808 
…sistent paths on shell startup

Inject BACKENDAI_PERSISTENT_PATHS environment variable (colon-separated vfolder
kernel paths) into containers at session creation time, and update .bashrc/.zshrc
to display persistent mount paths once on shell startup so users know which
directories survive session termination.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added size:M 30~100 LoC and removed size:L 100~500 LoC labels Apr 22, 2026
@seedspirit seedspirit changed the title feat(BA-4766): Add persistent mount path metadata for ephemeral file warning feat(BA-4766): inject BACKENDAI_PERSISTENT_PATHS env var and show persistent paths on shell startup Apr 22, 2026
@seedspirit seedspirit requested a review from a team April 22, 2026 05:24
@jopemachine
Copy link
Copy Markdown
Member

jopemachine commented Apr 22, 2026

Please check this PR @fregataa, which you requested change

@fregataa fregataa modified the milestones: 26.4, 26.5 Apr 22, 2026
@fregataa fregataa merged commit b871cdc into main Apr 22, 2026
34 checks passed
@fregataa fregataa deleted the BA-4766 branch April 22, 2026 05:46
jopemachine pushed a commit that referenced this pull request Apr 24, 2026
@seedspirit @claude @jopemachine
feat(BA-4766): inject BACKENDAI_PERSISTENT_PATHS env var and show per…
0eb48b1 
…sistent paths on shell startup (#9738)

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@jopemachine jopemachine jopemachine approved these changes

@fregataa fregataa fregataa approved these changes

Assignees

@seedspirit seedspirit

Labels

area:docs Documentations comp:agent Related to Agent component comp:common Related to Common component comp:manager Related to Manager component size:M 30~100 LoC

Projects

None yet

Milestone

26.5

Development

Successfully merging this pull request may close these issues.

4 participants

@seedspirit @jopemachine @fregataa