Forward service ports

[rberrelleza]

Fixes #577

Proposed changes

• forwardscan now take a string. if a string is given, the forward will go to a service in the namespace instead of to the local dev env

name: hello-world
image: okteto/golang:1
workdir: /okteto
command: ["bash"]
forward:
  - 8080:8080
  - 3306:mariadb:3306

@rlamana how do you think we should show this in the CLI's UI? Separate from the forward?

This is how it looks right now. It's sorted so the dev env ports are shown first

[golangcibot]

Error return value of (k8s.io/client-go/testing.ObjectTracker).Add is not checked (from errcheck)

[codecov]

Codecov Report

Merging #609 into master will decrease coverage by 2.91%.
The diff coverage is 47.5%.

@@            Coverage Diff             @@
##           master     #609      +/-   ##
==========================================
- Coverage      40%   37.08%   -2.92%     
==========================================
  Files          35       41       +6     
  Lines        3090     3559     +469     
==========================================
+ Hits         1236     1320      +84     
- Misses       1775     2158     +383     
- Partials       79       81       +2

Impacted Files	Coverage Δ
pkg/model/serializer.go	84.84% <ø> (+0.95%)	⬆️
cmd/up.go	6.52% <0%> (-0.06%)	⬇️
cmd/build.go	0% <0%> (ø)	⬆️
cmd/down.go	0% <0%> (ø)	⬆️
pkg/k8s/services/crud.go	6.06% <28.57%> (ø)
pkg/k8s/forward/manager.go	32.08% <31.96%> (ø)
pkg/model/dev.go	61.3% <50%> (-0.24%)	⬇️
pkg/k8s/pods/pod.go	7.1% <87.5%> (ø)
pkg/model/forward.go	94.87% <94.87%> (ø)
pkg/k8s/services/translate.go	0% <0%> (ø)
... and 5 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8f42326...6679b9f. Read the comment docs.

[pchico83]

I would make the remote port mandatory as kubectl does. I find it quite confusing to make it optional.
Also, could you document the new syntax?

[rberrelleza]

I would make the remote port mandatory as kubectl does. I find it quite confusing to make it optional.

Yeah, I was doubtful about that one as well. I'll remove it.

Also, could you document the new syntax?
Yes, I'll do it once we release the new version.