-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Recent update to v4.1.4 has incompatibilities with older versions of git #1708
Comments
FYI @jww3 |
Hi Jacob, Thank you for the bug report. I have investigated and determined that
As evidence, please have a look at https://github.com/jww3/checkout-test/actions/runs/8877937861 which is a pared-down version of your example workflow that focuses only on discovering the pre-installed git version. Thanks, |
@jww3 that is...interesting. When I first encountered this, it was with an image extended from I will do some more digging and get back to you. |
Okay, so follow up. I minimized my image that is built off of FROM ghcr.io/actions/actions-runner:2.316.0@sha256:1a9c7d1bb0896c8a23572452174f517709199e327ca967081936a950a52e9ec1
ENV RUNNER_TOOL_CACHE=/home/runner/_tool
# Switch to root to install dependencies
USER root
# Install common packages
RUN apt update -y \
&& apt install -y --no-install-recommends \
ca-certificates \
wget \
&& rm -rf /var/lib/apt/lists/*
# Install GitHub CLI (gh)
RUN mkdir -p -m 755 /etc/apt/keyrings \
&& wget -qO- https://cli.github.com/packages/githubcli-archive-keyring.gpg | tee /etc/apt/keyrings/githubcli-archive-keyring.gpg > /dev/null \
&& chmod go+r /etc/apt/keyrings/githubcli-archive-keyring.gpg \
&& echo "deb [arch=$(dpkg --print-architecture) signed-by=/etc/apt/keyrings/githubcli-archive-keyring.gpg] https://cli.github.com/packages stable main" | tee /etc/apt/sources.list.d/github-cli.list > /dev/null \
&& apt update -y \
&& apt install gh -y --no-install-recommends \
&& rm -rf /var/lib/apt/lists/* If I don't install the GitHub CLI, then Regardless, the update to I apologize that this observation is directly on From my perspective, I would think that follow up like these may be appropriate:
|
I attempted to reproduce this. My investigation resulted in Git version I'll talk to the GitHub CLI team about minimum Git version requirements. |
@jww3 For reproduction of what I'm seeing, use
|
Ah, I see the issue on It looks like the latest guidance from
in order to get the latest version of git. Here's what it took to get the |
I've proposed the following PR: actions/runner#3273 |
actions/runner#3273 has been merged. The experience here should be improved in upcoming actions/runner releases. Specifically, |
With the recent release of v4.1.4, which was to address #1689, I am continuing to find issues with the behavior from
actions/checkout
, specifically for older versions of git.To provide my details, analysis, and suggestions for resolution, I've created this minimal example: https://github.com/hicksjacobp/checkout-test
For summary,
actions/checkout@v4.1.4
still causes arepositoryformatversion
of1
to be configured in the repository due to callingsparse-checkout disable
. This appears to have been fixed in some version ofgit
after2.34.1
.Theactions/runner
image, which is the default image foractions-runner-controller
, happens to NOT have a newer version of git than2.34.1
, and further requires adding of theapt
feed to even install newer versions than2.34.1
.The
actions/runner
image, which is the default image foractions-runner-controller
, does not have git installed, nor the currentapt
feed for obtaining git versions newer than2.34.1
. If you install the GitHub CLI, thengit@v2.34.1
is installed, using a Dockerfile definition like so:My intention with this issue is to raise transparency of the behavior change with
actions/checkout@v4.1.4
which is different thanactions/checkout@v4.1.1
for users that may not find this very obvious (the previous version which was pullable withv4
). There are certainly some workarounds available as I put in the README of my minimal example.Other References
The text was updated successfully, but these errors were encountered: