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

Assures the current SDK version for Builders labs #297

Merged
merged 17 commits into from
Aug 25, 2023
Merged

Conversation

crdant
Copy link
Member

@crdant crdant commented Jul 20, 2023

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.

@crdant crdant requested a review from a user July 20, 2023 02:24
@crdant crdant requested a review from vsklevko as a code owner July 20, 2023 02:24
@@ -1,5 +1,5 @@
slug: avoiding-installation-pitfalls
id: amjfqroa5bfk
id:
Copy link
Contributor

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?

Copy link
Member Author

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:
Copy link
Contributor

Choose a reason for hiding this comment

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

another blank id

Copy link
Member Author

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:
Copy link
Contributor

Choose a reason for hiding this comment

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

blank id

Copy link
Member Author

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

@crdant crdant merged commit 4217a09 into main Aug 25, 2023
14 checks passed
@crdant crdant deleted the chore/keep-sdk-updated branch May 3, 2024 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants