- 
                Notifications
    You must be signed in to change notification settings 
- Fork 54
Pre-flight checks for zowe init and startup #4276
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
base: v3.x/staging
Are you sure you want to change the base?
Conversation
| build 7423 SUCCEEDED. | 
| Test workflow 6413 is started. | 
| build 7440 SUCCEEDED. | 
|  | ||
| common.printDebug(`Java ${javaVersionShort} is supported.`); | ||
| if (tooHigh) { | ||
| common.printMessage(`WARNING: Java ${javaVersionShort} is higher than the latest known working version for Zowe, v${JAVA_TESTED_VERSION}.`); | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should probably omit warning when startupcheck for dependency is set false
| Copyright Contributors to the Zowe Project. | ||
| */ | ||
|  | ||
| import * as common from '../../../../libs/common'; | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unsure if port-available is worth keeping if port-bind does the same and more.
| Test workflow 6423 is started. | 
| build 7772 SUCCEEDED. | 
| Test workflow 6692 is started. | 
| build 7779 SUCCEEDED. | 
| Test workflow 6698 is started. | 
| build 7780 SUCCEEDED. | 
| Test workflow 6699 is started. | 
| build 7781 SUCCEEDED. | 
| Test workflow 6700 is started. | 
| build 7782 SUCCEEDED. | 
| Test workflow 6701 is started. | 
| build 7797 SUCCEEDED. | 
| Test workflow 6707 is started. | 
| Test workflow 6707 is started. | 
| build 7798 SUCCEEDED. | 
| Test workflow 6708 is started. | 
| build 7810 FAILED. | 
| build 7815 SUCCEEDED. | 
| Test workflow 6723 is started. | 
Signed-off-by: 1000TurquoisePogs <[email protected]>
… bin/utils folder Signed-off-by: 1000TurquoisePogs <[email protected]>
Signed-off-by: 1000TurquoisePogs <[email protected]>
…icate validation. Use validators on zwe internal start prepare operation. Signed-off-by: 1000TurquoisePogs <[email protected]>
Signed-off-by: 1000TurquoisePogs <[email protected]>
…EnabledComponents for zwe internal start prepare. Seemed to be a prior bug? Signed-off-by: 1000TurquoisePogs <[email protected]>
…ty about when to pass/fail on unknown errors Signed-off-by: 1000TurquoisePogs <[email protected]>
Signed-off-by: 1000TurquoisePogs <[email protected]>
Signed-off-by: 1000TurquoisePogs <[email protected]>
Signed-off-by: 1000TurquoisePogs <[email protected]>
Signed-off-by: 1000TurquoisePogs <[email protected]>
4b0c645    to
    c031913      
    Compare
  
    | build 8416 SUCCEEDED. | 
| Test workflow 7169 is started. | 
I am prototyping a set of utilities that can be invoked by
zwe validateby a user or zwe itself in order to check if a Zowe install is likely to run successfully or not prior to starting up all the servers and getting complex errors that users may not understand the meaning of.At this point nothing is set in stone. It is rather a place to dump ideas to return to later.
If this PR becomes more official, this description will be updated to reflect what it can do.
So far, the following exists:
zwe internal start preparealready does, so the code could be moved here to be reusableIt could be helpful to invoke these at the last step of
zwe initas well as at each startup, to catch as much user error as possible.I am intending to write a bypass, set in the YAML, so that if any validation fails it can be ignored rather than to prevent some execution, in case our validation logic itself is an error.