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

Adds Embedded Cluster lab #346

Merged
merged 162 commits into from
Sep 13, 2024
Merged

Conversation

crdant
Copy link
Member

@crdant crdant commented Aug 14, 2024

TL;DR

Provides "Delivering Your Application as a Kubernetes Appliance" lab

Details

Implements a lab named "Delivering Your Application as a Kubernetes
Appliance" that shows how to package and install with Slackernews with the
Replicated Embedded Cluster.

The lab takes the user through iteratively building the Embedded Cluster
implementation of Slackernews based on the Helm chart. Once they have a
working implementation they install it as a customer then review the new
instance in the Vendor Portal.

@crdant
Copy link
Member Author

crdant commented Aug 14, 2024

@crdant
Copy link
Member Author

crdant commented Sep 3, 2024

@ajp-io @paigecalvert I'd really like to get this PR squared away before I start my vacation tomorrow evening. Any chance one of you can give it at approve?

Alternatively, @xavpaice or @chris-sanders , could I entice you into doing a review?

Copy link
Contributor

@paigecalvert paigecalvert left a comment

Choose a reason for hiding this comment

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

Just did another end-to-end run through @crdant and got stuck at the install step with the following error:
Screenshot 2024-09-03 at 8 56 34 PM

Looks like it wasn't wanting to download the install assets. I tried copy and pasting the commands from the lab and directly from the Vendor Portal just to make sure there wasn't some discrepancy there, but they look to be the same.

This of course meant that I couldn't test the last step in the Admin Console, but all the content looked good to me otherwise.

@ajp-io
Copy link
Member

ajp-io commented Sep 4, 2024

I've gotten stuck each time I've tried, so I haven't made it all the way through yet. But I will try to finish it today.

Co-authored-by: Paige Calvert <paige@replicated.com>
@crdant
Copy link
Member Author

crdant commented Sep 4, 2024

Just did another end-to-end run through @crdant and got stuck at the install step with the following error: Screenshot 2024-09-03 at 8 56 34 PM

Looks like it wasn't wanting to download the install assets. I tried copy and pasting the commands from the lab and directly from the Vendor Portal just to make sure there wasn't some discrepancy there, but they look to be the same.

This of course meant that I couldn't test the last step in the Admin Console, but all the content looked good to me otherwise.

@paigecalvert This generally means there was an error in the release or the download, if you see it again can you try running cat slackernews*.tgz and see what is shows? If it shows you a warning about a binary file then you can cancel, otherwise post the content here.

@crdant
Copy link
Member Author

crdant commented Sep 11, 2024

@ajp-io and @paigecalvert I found it! There was a --chart flag (and reference to the chart) where there should have been a --yaml-dir. Fix is testing now.

@crdant
Copy link
Member Author

crdant commented Sep 11, 2024

That was it. Fix is live and in this PR.

@crdant
Copy link
Member Author

crdant commented Sep 13, 2024

@ajp-io @paigecalvert You should be able to skip to "Using the Configuration to Install and Upgrade" and complete the lab from there. That's where the bad instruction was.

@paigecalvert
Copy link
Contributor

Sounds good @crdant I'll do that today

@crdant
Copy link
Member Author

crdant commented Sep 13, 2024

Thank you @paigecalvert

Copy link
Contributor

@paigecalvert paigecalvert left a comment

Choose a reason for hiding this comment

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

@crdant Success! Made it through the rest of the lab without errors. Just added a couple suggestions for some mismatches I saw with the config screen fields and the lab content

@crdant crdant merged commit 21c98e7 into main Sep 13, 2024
2 of 7 checks passed
@crdant crdant deleted the feature/crdant/delivering-as-an-appliance branch September 13, 2024 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type::feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants