search

yipeeio / yipee

An opensource version of the Yipee.io editor.
Apache License 2.0
22 stars 7 forks source link

NodePort not really optional on NodePort Service #15

Open palemtnrider opened 5 years ago

palemtnrider commented 5 years ago

(Found on SaaS yipee: staging2) Supplying an empty Node port in a service port where the service is of type LoadBalancer or likely NodePort generates a "Teapot" error in the backend. This is because it tries to parseInt a value of "null." Looks like the same bug exists in the yipee-tiyay version of the converter as well.

palemtnrider commented 5 years ago

Reassigning to UI as the value being passed in for nodePort is the string "null". Either an actual null or an empty string works correctly.