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

Ingress section has fallen into disrepair #51

Open
1 of 7 tasks
DickChesterwood opened this issue Jul 8, 2024 · 2 comments
Open
1 of 7 tasks

Ingress section has fallen into disrepair #51

DickChesterwood opened this issue Jul 8, 2024 · 2 comments

Comments

@DickChesterwood
Copy link
Owner

DickChesterwood commented Jul 8, 2024

It's been an open issue for too long, multiple problems with this section suggest a total re-record from scratch.

  • course is now Desktop centred, ingress needs to be installed there.
  • yaml schemas have changed - code is up to date but the videos don't reflect this
  • Missing field "ingressClassName" now required - this should have been in the course yaml
  • Get rid of the old websocket implementation as it causes further trouble here
  • aws section needs reworking. Install should now be easier but the files need checking, especially the annotations. Do a full run on production cluster
  • There's a vague mention of ALB early on, possibly this should be expanded (maybe take this offline from the course as it's arguably getting advanced)
  • The TLS stuff (which is offline) is the most awkward part, the OP on the original Q&A was actually dealing with this rather than any of the above.
@DickChesterwood
Copy link
Owner Author

Pickup point for Tuesday, run up a cluster and get ingress.yaml working there.

I think the best way to handle the videos is to order a script and re-record, there are too many problems to patch up the existing.

@DickChesterwood
Copy link
Owner Author

DickChesterwood commented Jul 17, 2024

All running fine with the two changes as outlined on the original Q&A. Although the changes are actually minimal, it's still worth a full re-record. Studio booked for 18/7/24, 13:00.

This will then roll on to #52

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

No branches or pull requests

1 participant