-
Notifications
You must be signed in to change notification settings - Fork 30
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
Assures the current SDK version for Builders labs #297
Conversation
7832733
to
a104853
Compare
@@ -1,5 +1,5 @@ | |||
slug: avoiding-installation-pitfalls | |||
id: amjfqroa5bfk | |||
id: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this id be blank?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the id's are blanked out by the dev/prod workflow
@@ -1,5 +1,5 @@ | |||
slug: closing-information-gap | |||
id: hfb41xqkyx0q | |||
id: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
another blank id
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the id's are blanked out by the dev/prod workflow
@@ -1,5 +1,5 @@ | |||
slug: distributing-with-replicated | |||
id: evmpzv6ycpgs | |||
id: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
blank id
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the id's are blanked out by the dev/prod workflow
TL;DR
Uses GitHub API and agent variables to keep the SDK up-to-date
Details
It was getting painful to keep the version of the Replicated SDK up-to-date in
the labs built to support the Builders experience. Right now we don't have
releases for the SDK but we do tag each version. This change uses the GitHub
API to get the latest version of the SDK (based on the tags). It then sets that
variable into an agent variable and uses it in both lifecycle scripts and
track instructions.
There is one place that still has a hard-coded SDK values: the asset that is
used to populate the static URL for the support bundle in "Closing the
Information Gap." We still don't have continuous deliver of that file the the
GCP bucket, so I'm going to defer solving the versioning issue until we
implement that. I'm not terribly concerned with it falling out of date for now,
since it is checking image pull permissions and those are going to be set at
the repository level.