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 intomasterfrom
fix-gcloud-login
Open

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

Conversation

@RammusXu
Copy link
Contributor

@RammusXu RammusXu commented Jun 30, 2020

google-cloud-sdk need:

  • python2
  • bash

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

@allanlei allanlei left a comment

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


RUN apk update && \
apk add curl bash python git && \
RUN apk add curl bash python2 && \
Copy link
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