Skip to content

Update doc/ci/ssh_keys/README.md

Waldek Maleska requested to merge (removed):weakcamel-master-patch-93112 into master

What does this MR do?

Resolves #220169 (closed) which postulates using command -v over which command to check whether ssh-agent is available.

Screenshots

n/a

Does this MR meet the acceptance criteria?

Conformity

I can't really tell, this is a community submission - you're welcome to do with it what you will

Tested by:

$ docker run -it ubuntu:xenial
# command -v ssh-agent >/dev/null || ( apt-get update -y && apt-get install openssh-client -y )
... <installation happens>

# command -v ssh-agent >/dev/null || ( apt-get update -y && apt-get install openssh-client -y )
<no effect - already installed>

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Waldek Maleska

Merge request reports