headlamp
headlamp copied to clipboard
Allow Port forwarding from the headlamp UI
[Title: describe the change in one sentence]
In the container info section of pod details we now have a button to allow port forwarding to the target port this button is situated right beside the port option
Testing done

- [x] Allow to port forward to a pod from desktop app UI
@ashu8912 , this is indeed a good start. I think there's still some things to consider which hopefully I covered in my review. Any questions, let me know.
@joaquimrocha can you mention the steps you followed to run the port forward??
(FTR, responded today in a call)




@ashu8912 , I tried the PR and it works fine for the Pod's port forwarding. The Service forwarding was still not working correctly when I tried that named port (non-numeric).
In order to make progress and be more efficient about the feature, I am proposing we drop the Service's port forwarding for now.
I also added a fix up commit that I'd like you to check. This makes the style closer to the latest mockups.
This is reviewed and tested. We should merge after merging its base PR (#843 ).
This was deleted automatically after I merged + deleted the base branch, but it should have been automatically rebased on main... Maybe this is a bug on Github. Reopening so we merge this.