Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DAB does not support root_path not nested under user home directory or repo #575

Closed
connorbrown-8451 opened this issue Jul 12, 2023 · 1 comment · Fixed by #576
Closed
Labels
Bug Something isn't working DABs DABs related issues

Comments

@connorbrown-8451
Copy link

When trying to change the workspace.root_path parameter in a DAB bundle.yml file, I receive the following error:

Starting upload of bundle files
Error: path must be nested under /Users/<my_user_name>@8451.com or /Repos/<my_user_name>@8451.com

This is when trying to use a bundle file with the following contents:

variables:
  root_directory:
    description: "The directory that should be used for this bundle"
workspace:
  root_path: ${var.root_directory}/cb/.bundle/${bundle.name}

I have access to the directory, but it seems that the DAB is forcing me to use my user directory or repo. It would be useful for us if we could change the root_directory to a directory that could be shared by more than one user. Please let me know if there is another setting I should try.

@pietern
Copy link
Contributor

pietern commented Jul 13, 2023

Thanks for reporting the issue. This limitation has been in place for the sync functionality to avoid inadvertently clobbering directories outside the users home directory. We have since update the sync code to not delete files it's not already tracking, so it's safe to remove this limitation. This is done in #576.

@pietern pietern linked a pull request Jul 13, 2023 that will close this issue
@andrewnester andrewnester added Bug Something isn't working DABs DABs related issues labels Jul 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working DABs DABs related issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants