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

Run on VM #142

Merged
merged 17 commits into from
Feb 24, 2024
Merged

Run on VM #142

merged 17 commits into from
Feb 24, 2024

Conversation

AlexAxthelm
Copy link
Collaborator

@AlexAxthelm AlexAxthelm commented Feb 17, 2024

Instructions for setting up a VM (including RMI-specific instructions).

Includes some helper scripts to facilitate mounting Azure File Shares.

Define the prerequisite steps prior to running data prep
Everything works up through creating and connecting to VM
Add a helper script to mount Azure File Shares
Mounting an Azure File Share to a linux OS via SMB defaults to
read/write access. The change to the mount script default to read-only
with file permissions (`0555`)

Update Docs accordingly.
Reflect actual Azure FIles structure
don't touch config in this PR.
This was referenced Feb 18, 2024
AlexAxthelm added a commit that referenced this pull request Feb 18, 2024
Change from /outputs to /mnt/outputs, to match structure defined in #142 and #145

No associated issue.
cjyetman added a commit that referenced this pull request Feb 23, 2024
Change from /outputs to /mnt/outputs, to match structure defined in #142
and #145

No associated issue.

Co-authored-by: CJ Yetman <[email protected]>
@AlexAxthelm AlexAxthelm marked this pull request as ready for review February 23, 2024 13:15
@AlexAxthelm AlexAxthelm mentioned this pull request Feb 24, 2024
3 tasks
@AlexAxthelm AlexAxthelm merged commit 32929ff into main Feb 24, 2024
@AlexAxthelm AlexAxthelm deleted the docs/how-to-run-on-vm branch February 24, 2024 15:34
cjyetman added a commit that referenced this pull request Feb 26, 2024
Update local docker config to match changes made to
[docker-compose.yml](https://github.com/RMI-PACTA/workflow.data.preparation/blob/main/docker-compose.yml)
in #142

Also set `2022Q4` config to inherit the `docker` config again (removed
in #145). @AlexAxthelm this should not affect the `2022Q4_azure` because
`2022Q4` will inherit the `docker` config, but when `2022Q4_azure`
inherits `2022Q4` it will overwrite those paths from the `docker` config
with the expected paths.
``` r
config::get(config = "docker")$factset_data_path
#> [1] "/mnt/factset-extracted"
config::get(config = "2022Q4")$factset_data_path
#> [1] "/mnt/factset-extracted"
config::get(config = "2022Q4_azure")$factset_data_path
#> [1] "/mnt/factset-extracted/factset-pacta_timestamp-20221231T000000Z_pulled-20240217T134528Z"
```
cjyetman added a commit that referenced this pull request Feb 26, 2024
…#166)

update Docker README section to reflect changes made to
docker-compose.yml in #142
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