Skip to content
This repository has been archived by the owner on Oct 29, 2021. It is now read-only.

Corrected '0. Authentication' command #2252

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

juangascon
Copy link
Contributor

Description

The command shown was 'google auth login' but it should be 'gcloud auth login'

Motivation and Context

The command was not correct.
It solves a possible block following the tutorial for newbies as me on GCP.
This change protects us from searching in internet how to install the 'google' command that does not exist.

How Has This Been Tested?

  • Covered by existing integration testing
  • Added integration testing to cover
  • Tested locally
  • Have not tested

Testing should not be necessary. It is a small change in the documentation. No code involved. It should pass or not the test as before the proposed change.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@edwarnicke
Copy link
Member

@juangascon Could you add a signed off by line for DCO using

git commit -s --amend --no-edit;git push -f

The command shown was 'google auth login' but it should be 'gcloud auth login'

Signed-off-by: Gascon Juan <[email protected]>
@juangascon
Copy link
Contributor Author

Done. As with PR #2251 :-)

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.

None yet

2 participants