kubernetes-101 icon indicating copy to clipboard operation
kubernetes-101 copied to clipboard

Episode 6 - Update API version for ingress

Open porkbrain opened this issue 4 years ago • 6 comments

Updating apiVersion for networking in episode-06 in drupal-ingress resource to v1 from v1beta1.

Followed answers in these resources to achieve equivalent behaviour:

  • https://stackoverflow.com/a/64126069/5093093
  • https://forum.linuxfoundation.org/discussion/858146/lab10-1-error-creating-ingress-rule-pathtype-must-be-specified
  • https://docs.openshift.com/container-platform/4.6/rest_api/network_apis/ingress-networking-k8s-io-v1.html

I tested this change with my local cluster (albeit not running drupal). I selected pathType: ImplementationSpecific because that seems to forward the URI to the app where router can decide what to do.


Closes https://github.com/geerlingguy/kubernetes-101/issues/24

porkbrain avatar Apr 11 '21 19:04 porkbrain

Thanks! I'll be taking a look at this in my next pass through the text (working on the transcription into book form currently).

geerlingguy avatar Apr 12 '21 03:04 geerlingguy

This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark pull requests as stale.

stale[bot] avatar Jul 11 '21 04:07 stale[bot]

This pull request is no longer marked for closure.

stale[bot] avatar Jul 13 '21 16:07 stale[bot]

Tested this patch and it worked well.

The ingress is currently broken on a default install of drocker desktop or minikube on macos without this fix.

frob avatar Jan 07 '22 21:01 frob

Let me know if there's something you'd like to me to update before merging this PR.

porkbrain avatar Jan 28 '22 14:01 porkbrain

I'll get to it soon™, don't worry :)

If it's 'planned' then I definitely plan on having at it next go-round, but I have to sync the changes with the book text too, so I tend to batch these fixes together every few months.

geerlingguy avatar Jan 29 '22 04:01 geerlingguy