don't use namespaces for all-namespaces

[yehiyam]

This change is 

[nassiharel]

Reviewable status: 0 of 4 files reviewed, 3 unresolved discussions (waiting on @yehiyam)

---

lib/pods.js, line 4 at r1 (raw file):

podName

There is no use of podName if useNamespace is false?

---

lib/resourceQuotas.js, line 5 at r1 (raw file):

name

There is no use of name if useNamespace is false?

---

tests/test.js, line 168 at r1 (raw file):

················

[yehiyam]

Reviewable status: 0 of 4 files reviewed, 3 unresolved discussions (waiting on @nassiharel)

---

lib/pods.js, line 4 at r1 (raw file):

Previously, NassiHarel (Nassi Harel) wrote…

podName

There is no use of podName if useNamespace is true?

If useNamespace is false (== all namespaces) you can't specify a podName. It is the same as in `kubectl

---

lib/resourceQuotas.js, line 5 at r1 (raw file):

Previously, NassiHarel (Nassi Harel) wrote…

name

There is no use of name if useNamespace is true?

If useNamespace is false (== all namespaces) you can't specify a name. It is the same as in kubectl

---

tests/test.js, line 168 at r1 (raw file):

Previously, NassiHarel (Nassi Harel) wrote…

················

Done.
For the future, we need to either set up lint to catch these (here it is in unit-test) or ignore it

[nassiharel]

Reviewed 4 of 4 files at r1.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @yehiyam)