[Tarantool-patches] [PATCH 1/2] ci: use sudo to install packages if not root
Sergey Bronnikov
sergeyb at tarantool.org
Fri Oct 2 16:18:36 MSK 2026
Hi, Alexander!
thanks for the patch! LGTM
Sergey
On 10/1/26 04:06, Alexander Turenko wrote:
> The setup-linux and setup-debuggers actions call `apt` without `sudo`,
> which works on our self-hosted runners: the jobs run there as root. The
> next commit moves the ARM64 jobs to GitHub-hosted runners, where the
> steps run as the runner's regular user, so we have to call `sudo` there.
>
> If the current user is already `root`, `sudo` is not called: this way we
> don't depend on its presence when we don't need it.
>
> Drafted by an LLM, conducted and corrected by the commit author.
> ---
> .github/actions/setup-debuggers/action.yml | 6 ++++--
> .github/actions/setup-linux/action.yml | 6 ++++--
> 2 files changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/.github/actions/setup-debuggers/action.yml b/.github/actions/setup-debuggers/action.yml
> index c34d4502..e2ac09db 100644
> --- a/.github/actions/setup-debuggers/action.yml
> +++ b/.github/actions/setup-debuggers/action.yml
> @@ -7,6 +7,8 @@ runs:
> uses: ./.github/actions/setup-linux
> - name: Install dependencies for the tests
> run: |
> - apt -y update
> - apt install -y python3 gdb lldb
> + # GitHub-hosted runners run the steps as a regular user.
> + SUDO=$([ "$(id -u)" = 0 ] || echo sudo)
> + ${SUDO} apt -y update
> + ${SUDO} apt install -y python3 gdb lldb
> shell: bash
> diff --git a/.github/actions/setup-linux/action.yml b/.github/actions/setup-linux/action.yml
> index f0171b83..8f13940a 100644
> --- a/.github/actions/setup-linux/action.yml
> +++ b/.github/actions/setup-linux/action.yml
> @@ -14,6 +14,8 @@ runs:
> shell: bash
> - name: Install build and test dependencies
> run: |
> - apt -y update
> - apt -y install cmake gcc make ninja-build perl
> + # GitHub-hosted runners run the steps as a regular user.
> + SUDO=$([ "$(id -u)" = 0 ] || echo sudo)
> + ${SUDO} apt -y update
> + ${SUDO} apt -y install cmake gcc make ninja-build perl
> shell: bash
More information about the Tarantool-patches
mailing list