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.

Sign up for GitHub

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

Jump to bottom

Update ingress section files #27

Merged
merged 3 commits into from
Jan 17, 2022
Merged

Update ingress section files #27

merged 3 commits into from
Jan 17, 2022

Conversation

ivorscott
Copy link
Collaborator

@ivorscott ivorscott commented Jan 8, 2022 •
edited
Loading

Move ingress class logic to ingress nginx controller and create example for linux users too.

Tested on

  • Kubernetes 1.22
  • Apple Silicon Mac (Docker Desktop)
  • Ubuntu VM

#14

@BretFisher BretFisher self-requested a review January 17, 2022 17:02
@BretFisher BretFisher added bug Something isn't working deprecation update A fix for something that is outdated but not broken yet labels Jan 17, 2022
@BretFisher BretFisher merged commit 9035e67 into BretFisher:mastery Jan 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BretFisher BretFisher BretFisher approved these changes

Assignees

@ivorscott ivorscott

Labels
bug Something isn't working deprecation update A fix for something that is outdated but not broken yet
Projects
Bret Fisher's Course Updates and Fixes
Status: Done ✅
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ivorscott @BretFisher