search

dgkanatsios / CKAD-exercises

A set of exercises to prepare for Certified Kubernetes Application Developer exam by Cloud Native Computing Foundation
MIT License
8.62k stars 5.6k forks source link

refactor: simplify explanation for exposed ports for httpGet in probes #312

Open BoboTheBarbar opened 1 year ago

BoboTheBarbar commented 1 year ago

I think there is no reason to illustrate an exposed containerport 80, since it would obfuscate that there is no reason for this port to be exposed.

dgkanatsios commented 1 year ago

sorry, what's wrong with exposing the default nginx port?

BoboTheBarbar commented 1 year ago

sorry, what's wrong with exposing the default nginx port?

There is nothing wrong with exposing it. But the task requires to run a readyness probe and verify the port 80 is in use. It is an educational detail to show a student, that port 80 doesn't have to be exposed, since the readiness probe runs within the container. If we would expose port 80 on the pod, this detail would be obfuscated. I will clarify that in the comment a bit more in another commit.

BoboTheBarbar commented 1 year ago

New configuration has been tested manually grafik