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

Update cloud-linode.md #14348

Merged
merged 1 commit into from
Nov 11, 2021
Merged

Update cloud-linode.md #14348

merged 1 commit into from
Nov 11, 2021

Conversation

jdeighton
Copy link
Contributor

Update list of required permissions on the Linode Personal Access Token.

Added StackScripts read/write as a required permission on the Personal Access Token.

Description

Updated documentation for Linode to ensure correct permissions are set on the Personal Access Token.

Motivation and Context

If a Personal Access Token is created with just Linodes read/write permission, the Linode API will reject it with a 401: Your OAuth token is not authorized to use this endpoint. in the task: cloud-linode : Create a stackscript.

How Has This Been Tested?

  1. Create a Linode Personal Access Token that only has Linodes read/write permission.
  2. Run ./algo script to deploy server (selecting Linode as the provider and following remaining steps).
  3. Observe 401 error from Linode API.
  4. Create new Linode Personal Access Token with both Linodes and StackScripts read/write permissions
  5. Re-run ./algo script to deploy server (again, selecting Linode as the provider and answering remaining questions)
  6. Server is succesfully deployed.

Types of changes

  • Documentation update

Checklist:

  • I have read the CONTRIBUTING document.
  • [] My code follows the code style of this project.
  • [] My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • [] I have added tests to cover my changes.
  • [] All new and existing tests passed.

Update list of required permissions on the Linode Personal Access Token.
@CLAassistant
Copy link

CLAassistant commented Nov 6, 2021

CLA assistant check
All committers have signed the CLA.

@ghost
Copy link

ghost commented Dec 11, 2021

thx

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.

3 participants