Skip to content
This repository was archived by the owner on Feb 6, 2024. It is now read-only.

Fix gcloud add python2 - #13

Open
RammusXu wants to merge 4 commits into
masterfrom
fix-gcloud-login
Open

Fix gcloud add python2#13
RammusXu wants to merge 4 commits into
masterfrom
fix-gcloud-login

Conversation

@RammusXu

@RammusXu RammusXu commented Jun 30, 2020

Copy link
Copy Markdown
Contributor

google-cloud-sdk need:

  • python2
  • bash

@RammusXu
RammusXu requested a review from allanlei June 30, 2020 09:51

@allanlei allanlei left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The docker login to https://asia.gcr.io should be dynamic

Comment thread gcloud/login/Dockerfile

RUN apk update && \
apk add curl bash python git && \
RUN apk add curl bash python2 && \

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should just inherit from google/cloud-sdk image

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants