Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Display a reason why the CLI prompts for host, port, etc #2115

Closed
Tracked by #2070
gejohnston opened this issue Apr 10, 2024 · 2 comments
Closed
Tracked by #2070

Display a reason why the CLI prompts for host, port, etc #2115

gejohnston opened this issue Apr 10, 2024 · 2 comments
Labels
enhancement New feature or request priority-medium Not functioning - next quarter if capacity permits v3 prospective changes for v3
Milestone

Comments

@gejohnston
Copy link
Member

Is your feature or enhancement request related to a problem or limitation? Please describe

When no configuration is found, the current CLI behavior is to prompt for connection properties as shown below:

zowe zos-files list data-set "SYS1.PARMLIB*"
Enter the host name of your service:

This is a necessary and appropriate prompt, but the user may have no idea why they are being prompted.

Describe your enhancement idea

A brief explanation before such a prompt would be beneficial. For example:

zowe zos-files list data-set "SYS1.PARMLIB*"
No Zowe client configuration exists.
Therefore, you will be asked for the connection properties
that are required to complete your command.

Enter the host name of your service:

If the user only had V1 profiles in place during V2, the V1 profiles would automatically be used.
In V3, V1 profiles are ignored. In V3, if the user only has V1 profiles in place, the user will get the same un-described prompt as shown below. This could be even more confusing for customers who were used to V1 profiles automatically working.

zowe zos-files list data-set "SYS1.PARMLIB*"
Enter the host name of your service:

This situation could also be addressed with a slightly modified description when only V1 profiles exist, as follows:

zowe zos-files list data-set "SYS1.PARMLIB*"
No Zowe client configuration exists.
Only V1 profiles exist. V1 profiles are no longer supported.
You should convert your V1 profiles to a newer Zowe client configuration.
Therefore, you will be asked for the connection properties
that are required to complete your command.

Enter the host name of your service:

Describe alternatives you've considered

Provide any additional context

Both of the 2 new prompts are already implemented in ConnectionPropsForSessCfg. However, the changes are currently comment-out because the changes cause 219 unit test failures and 144 system tests failures due to differences in output. Thus, the implementation effort is trivial, but the test updates will take much more work.

The current behavior would not be considered an error, so the priority of this enhancement would be considered low. The only urgency for this enhancement is that it would be a breaking change and will have to be implemented before V3 is released.

@gejohnston gejohnston added enhancement New feature or request new The issue wasn't triaged yet labels Apr 10, 2024
@github-project-automation github-project-automation bot moved this to New Issues in Zowe CLI Squad Apr 10, 2024
@gejohnston gejohnston added the priority-low Legit issue but cosmetic or nice-to-have label Apr 10, 2024
Copy link

Thank you for raising this enhancement request.
The community has 90 days to vote on it.
If the enhancement receives at least 5 upvotes, it is added to our development backlog.
If it receives fewer votes, the issue is closed.

@zowe-robot zowe-robot moved this from New Issues to Low Priority in Zowe CLI Squad Apr 10, 2024
@JTonda JTonda added priority-medium Not functioning - next quarter if capacity permits v3 prospective changes for v3 and removed new The issue wasn't triaged yet priority-low Legit issue but cosmetic or nice-to-have labels Apr 10, 2024
@JTonda JTonda added this to the Zowe V3 milestone Apr 10, 2024
@zowe-robot zowe-robot moved this from Low Priority to Medium Priority in Zowe CLI Squad Apr 10, 2024
@adam-wolfe adam-wolfe mentioned this issue Apr 10, 2024
23 tasks
@gejohnston
Copy link
Member Author

Implemented by pull request "Display messages before prompting for properties" #2123

@github-project-automation github-project-automation bot moved this from Medium Priority to Closed in Zowe CLI Squad Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority-medium Not functioning - next quarter if capacity permits v3 prospective changes for v3
Projects
None yet
Development

No branches or pull requests

2 participants