Skip to content

need to pass in environment arg to command to make ipfs init work#521

Merged
alabdao merged 1 commit intomainfrom
as/ipfs-env-update
Jul 19, 2023
Merged

need to pass in environment arg to command to make ipfs init work#521
alabdao merged 1 commit intomainfrom
as/ipfs-env-update

Conversation

@alabdao
Copy link
Copy Markdown
Contributor

@alabdao alabdao commented Jul 18, 2023

without this option, ipfs is initialized at default location which isn't ipfs_path. This of course happens at first install where /etc/environment is updated but havent read yet.

Also, unarchive has support for get_url which allows it to directly download archives without having to first download, and then unarchive.

without this option, ipfs is initialized at default location which isn't `ipfs_path`. This of course happens at first install where `/etc/environment` is updated but havent read yet.

Also, `unarchive` has support for `get_url` which allows it to directly download archives without having to first download, and then unarchive.
@alabdao alabdao requested a review from hevans66 July 18, 2023 22:48
@vercel
Copy link
Copy Markdown

vercel bot commented Jul 18, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 18, 2023 10:48pm

Copy link
Copy Markdown
Contributor

@hevans66 hevans66 left a comment

Choose a reason for hiding this comment

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

Works for me. We can probably remove the environment block form ansible/provision_compute_only.yaml once this is in.

@alabdao alabdao merged commit 69c0ee0 into main Jul 19, 2023
@alabdao alabdao deleted the as/ipfs-env-update branch July 19, 2023 12:55
@alabdao alabdao added the ops For Ops/Infrastructure related items label Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ops For Ops/Infrastructure related items

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants