Skip to content

Commit

Permalink
containers/teuthology-dev: Remove access token
Browse files Browse the repository at this point in the history
This container is built and pushed via GitHub Actions. GHA likes to provision a
personal access token for each job that gives tightly-scoped access to the git
repository to the job. When we build our container, we end up including
`.git/config`, which contains the token. Later, in ceph-dev-stack's CI, an
`ls-remote` is run against ceph.git, which ends up causing git to prompt for
credentials even though the repo is public. Removing the token should allow
reading all the relevant repos from the built container image.

Signed-off-by: Zack Cerza <[email protected]>
  • Loading branch information
zmc committed Jan 6, 2025
1 parent 6f07bcc commit 7cc6ba4
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions containers/teuthology-dev/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ RUN \
PIP_INSTALL_FLAGS="-r requirements.txt" ./bootstrap
COPY . /teuthology
RUN \
git config -f ./.git/config --unset 'http.https://github.com/.extraheader' && \
./bootstrap
COPY containers/teuthology-dev/containerized_node.yaml /teuthology
COPY containers/teuthology-dev/.teuthology.yaml /root
Expand Down

0 comments on commit 7cc6ba4

Please sign in to comment.