Skip to content
This repository has been archived by the owner on Apr 18, 2022. It is now read-only.

Bump Teleport version so base AMI will build #42

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

Conversation

dcmcand
Copy link

@dcmcand dcmcand commented Jan 7, 2021

Fixes #41

Description of the Change

  • fix teleport install to use teleports recommended install method
  • bump teleport version to a supported version

@dcmcand dcmcand added the ready for review 👀 Ready for code review and approval from maintainers label Jan 7, 2021
Copy link

@orenfromberg orenfromberg left a comment

Choose a reason for hiding this comment

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

lgtm!

cd -
# # Download info on the teleport release we are targeting
TELEPORT_VERSION="4.4.6"
# TELEPORT_INFO=$(curl -sSf https://dashboard.gravitational.com/webapi/releases-oss?product=teleport | jq ".items | map(select(.version == \"${TELEPORT_VERSION}\")) | .[].downloads | map(select(.name == \"teleport-${TELEPORT_VERSION}-linux-amd64-bin.tar.gz\")) | .[]")

Choose a reason for hiding this comment

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

please remove the commented out script.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ready for review 👀 Ready for code review and approval from maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Packer won't build base AMI
2 participants