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

zwe diagnose: shell only and no doc #3627

Closed
Martin-Zeithaml opened this issue Oct 24, 2023 · 3 comments · Fixed by #3637
Closed

zwe diagnose: shell only and no doc #3627

Martin-Zeithaml opened this issue Oct 24, 2023 · 3 comments · Fixed by #3637
Labels
bug Something isn't working Config Manager Related to the config manager component Priority - Low

Comments

@Martin-Zeithaml
Copy link
Contributor

  1. Shell version is working only
  2. There is no doc update for this feature

Add a text to shell version of diagnose command:
zowe/runtime/2120/bin/commands/diagnose: echo "echo 'hello, world'" >> index.sh

Call diagnose with --configmrg
zowe/runtime/2120/bin: ./zwe diagnose -e "ZWEL0111E" --configmgr

Shell version is executed:

This code corresponds to the errors related to the Zowe Launcher and ZWE
To find the description of this error code, refer to the Zowe documentation for the Launcher at https://docs.zowe.org/stable/troubleshoot/launcher/launcher-error-codes and https://github.com/zowe/launcher/blob/v2.x/master/src/msg.h, and for ZWE at https://docs.zowe.org/stable/appendix/zwe_server_command_reference/zwe/
You may also explore reports from other users experiencing the same error by searching here https://github.com/search?q=org%3Azowe+ZWEL0111E&type=discussions
hello, world
@Martin-Zeithaml Martin-Zeithaml changed the title zwe dignose: shell only and no doc zwe diagnose: shell only and no doc Oct 24, 2023
@1000TurquoisePogs
Copy link
Member

no doc update because doc automation is broken. i need to fix it. there was some code conflict which broke the bot.

@1000TurquoisePogs
Copy link
Member

what is the reason to make the shell version? its fine but currently we have both TS & shell because we were moving away from shell. personally I wasn't going to write more shell, but maybe someone else would & maybe there is a good reason to?

@Martin-Zeithaml
Copy link
Contributor Author

Martin-Zeithaml commented Oct 25, 2023

Probably the wrong description of a problem, diagnose is already in shell and TS.
But no matter what, only shell is executed.

There is missing in the index.sh: IF USE_CONFIG -> run _CEE_RUNOPTS... configmgr -script ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Config Manager Related to the config manager component Priority - Low
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants