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

Revised tutorial setup steps #869

Merged
merged 1 commit into from
Jun 17, 2016

Conversation

davidwrede
Copy link

@davidwrede davidwrede commented Jun 15, 2016

  • Revised and clarified when you need to run sudo hab subcommand in tutorial.
  • Also included minor formatting updates.

Signed-off-by: David Wrede [email protected]

@thesentinels
Copy link
Contributor

By analyzing the blame information on this pull request, we identified @smith and @cwebberOps to be potential reviewers


export PATH=$PATH:~/bin

To keep from typing this in for every terminal window you use, add the above statement to your shell profile.

4. Open a terminal window and run `hab setup`. If your host machine is running Linux, the `hab` CLI must run as root, so we suggest running it as root through the `sudo` command.
3. On Mac OS X, run `hab setup`. If your host machine is running Linux, we suggest running `sudo hab setup`. The `hab` CLI must run as root to place your [origin keys](/docs/concepts-keys#origin-keys) in the default location for the studio to use them (`/hab/cache/keys`).
Copy link
Contributor

Choose a reason for hiding this comment

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

Is sudo hab setup necessary? hab won't use preferences stored in $HOME/.hab?

Copy link
Author

@davidwrede davidwrede Jun 15, 2016

Choose a reason for hiding this comment

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

It's because when we call sudo hab studio enter, the keys are expected to be in /hab/cache/keys not $HOME/.hab/cache/keys. I guess we could override with an env var to look there.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes. It's best to minimize the scope of what needs sudo. Basically anything short of package building does not need sudo. If users want to re-use the keys they generated as their regular user, they can set HAB_CACHE_KEY_PATH before entering the studio.

Copy link
Author

Choose a reason for hiding this comment

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

I think there's an issue with getting keys imported from an overridden HAB_CACHE_KEY_PATH into the interactive studio. The only way I could get things to work was to have the keys in /hab/cache/keys on my Linux machine.

@davidwrede davidwrede force-pushed the dw/revise_linux_steps branch from 7b90642 to ba284c5 Compare June 16, 2016 19:23
@smith
Copy link
Contributor

smith commented Jun 16, 2016

@thesentinels r+

@thesentinels
Copy link
Contributor

📌 Commit ba284c5 has been approved by smith

@adamhjk
Copy link
Contributor

adamhjk commented Jun 16, 2016

@thesentinels retry

@thesentinels
Copy link
Contributor

⌛ Testing commit ba284c5 with merge b2ec523...

thesentinels pushed a commit that referenced this pull request Jun 16, 2016
Signed-off-by: David Wrede <[email protected]>

Pull request: #869
Approved by: smith
@smith
Copy link
Contributor

smith commented Jun 16, 2016

@thesentinels force

@smith
Copy link
Contributor

smith commented Jun 16, 2016

@thesentinels r+

@thesentinels
Copy link
Contributor

💡 This pull request was already approved, no need to approve it again.

  • This pull request is currently being tested. If there's no response from the continuous integration service, you may use retry to trigger a build again.

@thesentinels
Copy link
Contributor

📌 Commit ba284c5 has been approved by smith

@smith
Copy link
Contributor

smith commented Jun 16, 2016

@thesentinels retry

2 similar comments
@adamhjk
Copy link
Contributor

adamhjk commented Jun 17, 2016

@thesentinels retry

@adamhjk
Copy link
Contributor

adamhjk commented Jun 17, 2016

@thesentinels retry

@adamhjk
Copy link
Contributor

adamhjk commented Jun 17, 2016

@thesentinels clean

@adamhjk
Copy link
Contributor

adamhjk commented Jun 17, 2016

@thesentinels retry

@adamhjk adamhjk merged commit 7c5397f into habitat-sh:master Jun 17, 2016
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.

5 participants