The node terminal should use the currently selected namespace in console (if one is selected). Today it always is put in the `default` project.
Note that the Node page is not a namespaced page, but console should still be tracking an active namespace in Redux.
Sam and I discussed this and determined the best solution is to probably allow the user to select a namespace for the debug pod as the user will otherwise be unintentionally creating debug pods in whatever the console's active namespace is since 4.x does not display the namespace picker for nodes.
PR opened, awaiting approval.
Created attachment 1699079 [details]
After User Selects Namespace for debug pod
Now before opening Terminal for a node, user will be asked to select a namespace for debug pod, after he choose a namespace, the namespace selector will disappear. Why not always show the selected namespace? Since after the project selector disappear, user may forgot which namespace the debug pod is in. This is checked against 4.6.0-0.nightly-2020-06-26-035408
Assigning back for confirmation
(In reply to Yadan Pei from comment #7)
> Now before opening Terminal for a node, user will be asked to select a
> namespace for debug pod, after he choose a namespace, the namespace selector
> will disappear. Why not always show the selected namespace? Since after the
> project selector disappear, user may forgot which namespace the debug pod is
Prior to this change, the debug pod for the Nodes Terminal tab was always being created in the `default` namespace. Adding the namespace picker forces the user to choose a namespace for the debug pod each time the Terminal tab is visited. Note the debug pod is deleted once the user leaves the Terminal tab regardless of namespace. The intention with adding the picker is to make the user aware the debug pod is being created as a result of visiting the Terminal tab so they aren't surprised to see an additional pod running on the cluster. Leaving the picker would help serve as a visual cue which namespace the debug pod was created in, but it's not of great importance since the namespace the debug pod can be created anywhere. Reloading/revisiting the tab deletes an existing debug pod and forces the user to choose the namespace again, so doing so would be one way to account for a "mistake" in namespace selection. In other words, leaving the namespace picker present could encourage changes to the namespace that aren't really necessary?
The CLI uses the active project/namespace for this purpose since you're always within the context of a project. We couldn't do that in the console because Nodes pages do not have the namespace selector present.
Thank you Robb for you detail explanation, it sounds good to me. Moving to VERIFIED
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.
For information on the advisory (OpenShift Container Platform 4.6 GA Images), and where to find the updated
files, follow the link below.
If the solution does not work for you, open a new bug report.