Skip to content

Commit

Permalink
Credentials file does not need to be mounted
Browse files Browse the repository at this point in the history
  • Loading branch information
rhiroe committed Oct 2, 2024
1 parent dc592a8 commit 12c249e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 10 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ jobs:
- uses: actions/checkout@v4

- name: create dummy mount file
run: mkdir -p ~/.aws/config && touch ~/.aws/credentials && touch ~/.saml2aws
run: mkdir ~/.aws && touch ~/.aws/config && touch ~/.saml2aws

- name: "Install latest devcontainer CLI"
run: npm install -g @devcontainers/cli
Expand All @@ -44,7 +44,7 @@ jobs:
- uses: actions/checkout@v4

- name: create dummy mount file
run: mkdir -p ~/.aws/config && touch ~/.aws/credentials && touch ~/.saml2aws
run: mkdir ~/.aws && touch ~/.aws/config && touch ~/.saml2aws

- name: "Install latest devcontainer CLI"
run: npm install -g @devcontainers/cli
Expand All @@ -59,7 +59,7 @@ jobs:
- uses: actions/checkout@v4

- name: create dummy mount file
run: mkdir -p ~/.aws/config && touch ~/.aws/credentials && touch ~/.saml2aws
run: mkdir ~/.aws && touch ~/.aws/config && touch ~/.saml2aws

- name: "Install latest devcontainer CLI"
run: npm install -g @devcontainers/cli
Expand Down
7 changes: 0 additions & 7 deletions src/ecspresso/devcontainer-feature.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,13 +16,6 @@
"description": "Version of ecspresso to install."
}
},
"mounts": [
{
"source": "${localEnv:HOME}/.aws/credentials",
"target": "/root/.aws/credentials",
"type": "bind"
}
],
"installsAfter": [
"ghcr.io/devcontainers/features/aws-cli"
]
Expand Down

0 comments on commit 12c249e

Please sign in to comment.