Skip to content

Commit 70efbac

Browse files
authored
add note re: sudo to Heroku deployment info
@keybits added `sudo` to the Heroku script back in September, most likely because many customers use our convenience Docker images, where the user is `circleci` rather than `root`, however `sudo` was taken back out a month ago in a community-submitted PR, most likely because most vanilla Docker images *do* have `root` as their user. Hopefully this is a good compromise?
1 parent eecf1bd commit 70efbac

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

jekyll/_cci2/deployment_integrations.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,8 @@ The built-in Heroku integration through the CircleCI UI is not implemented for C
112112
StrictHostKeyChecking no
113113
EOF
114114
```
115+
***Note:*** *this script will need to be executed with `sudo` if running in a Docker container where the user is not `root` (for example, CircleCI's [convenience images](hub.docker.com/r/circleci)).*
116+
115117
This file runs on CircleCI and configures everything Heroku needs to deploy the app. The second part creates a `.netrc` file and populates it with the API key and login details set previously.
116118
117119
2. Install and authorize Heroku for the CircleCI account that owns the project.

0 commit comments

Comments
 (0)