Skip to content
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

186049138 - Update awscli image to include openssh #309

Merged
merged 1 commit into from
Oct 18, 2023

Conversation

nimalank7
Copy link
Contributor

@nimalank7 nimalank7 commented Oct 17, 2023

Description:

  • paas-bootstrap's create-bosh-concourse pipeline installs openssh using apk but it should be bundled as part of the docker image
  • Bundle openssh into the awscli docker image

How to review

Description:
- paas-bootstrap's create-bosh-concourse [pipeline](https://github.com/alphagov/paas-bootstrap/blob/9061827f4c35ac844d1e4dfdefb2eb3e167cf13f/concourse/pipelines/create-bosh-concourse.yml#L645) installs openssh using apk but it should be bundled as part of the docker image
- Bundle openssh into the awscli docker image
Copy link
Contributor

@malcgds malcgds left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nimalank7 nimalank7 merged commit ac97869 into main Oct 18, 2023
28 checks passed
@nimalank7 nimalank7 deleted the 186049138-Update-awscli-image-to-include-openssh branch October 18, 2023 16:34
@@ -7,6 +7,7 @@ RUN apk add --no-cache \
less>=590-r0 \
python3~3.10 \
py3-pip~22 \
openssh \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know you are just moving things from paas-bootstrap. However this feels like it should be 'openssh-client'. I don't think we need the server packages.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants