-   Notifications  You must be signed in to change notification settings 
- Fork 89
Wdt 412 #616
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
Conversation
| ARG_VALIDATION_ERROR_EXIT_CODE = 98 | ||
| PROG_RESTART_REQUIRED = 103 | ||
| PROG_ROLLBACK_IF_RESTART_EXIT_CODE = 3 | ||
| PROG_ROLLBACK_IF_RESTART_EXIT_CODE = 104 | 
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.
Needs a message in shared.sh, shared.cmd in checkExitCode for this exit code
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.
fixed
| ARG_VALIDATION_ERROR_EXIT_CODE = 98 | ||
| PROG_RESTART_REQUIRED = 103 | ||
| PROG_ROLLBACK_IF_RESTART_EXIT_CODE = 3 | ||
| PROG_ROLLBACK_IF_RESTART_EXIT_CODE = 104 | 
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.
Just read the update document trying to find the command line argument to rollback. That argument isn't documented and also there are bad RCs in the documentation that need to be removed. 104 needs to be added.
 101 - The domain needs to be restarted and the Update Domain Tool needs to be re-invoked with the same arguments. 102 - The servers impacted by the update operation need to be restarted, in a rolling fashion, starting with the Administration Server, if applicable. 103 - The entire domain needs to be restarted. 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.
Added
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.
I created a simple JRF domain. Then ran update with a value that requires restart. I got a failure with the update that edit was in progress, even though I just created the domain in the previous step.
SEVERE Messages: 1. WLSDPLY-09015: updateDomain deployment failed: Domain has outstanding edit session weblogic, deploy cannot proceed and will exit Total: WARNING : 0 SEVERE : 1 updateDomain.cmd failed (exit code = 2) I tried it again with a brand new domain as I had been using the same name for several tests the last few days. This time I got
Problem invoking WLST - Traceback (innermost last): File "C:\Users\crountre\temp\weblogic-deploy\lib\python\update.py", line 328, in ? File "C:\Users\crountre\temp\weblogic-deploy\lib\python\update.py", line 313, in main File "C:\Users\crountre\temp\weblogic-deploy\lib\python\update.py", line 106, in __update File "C:\Users\crountre\temp\weblogic-deploy\lib\python\update.py", line 149, in __update_online File "C:\Users\crountre\temp\weblogic-deploy\lib\python\update.py", line 180, in __check_update_require_domain_restart File "C:\Users\crountre\temp\weblogic-deploy\lib\python\wlsdeploy\tool\util\wlst_helper.py", line 231, in cancel_edit File "<iostream>", line 591, in cancelEdit File "<iostream>", line 553, in raiseWLSTException WLSTException: Error occurred while performing cancelEdit : Cannot call Edit functions when you are not in the Edit tree. updateDomain.cmd completed but with some issues (exit code = 1) | Hmm, what's your test case looks like? and any logs? | 
| updateDomain.log.txt 
 
  | 
   site/update.md  Outdated    
 | When running the tool in WLST online mode, the update operation may require server restarts or a domain restart to pick up the changes. The update operation can also encounter situations where it cannot complete its operation until the domain is restarted. To communicate these conditions to scripts that may be calling the Update Domain Tool, the shell scripts have three special, non-zero exit codes to communicate these states: | ||
|  | ||
| - `101` - The domain needs to be restarted and the Update Domain Tool needs to be re-invoked with the same arguments. | ||
| - `102` - The servers impacted by the update operation need to be restarted, in a rolling fashion, starting with the Administration Server, if applicable. | 
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.
the rollback command arg is not documented. And the 101, 102 are not valid are they?
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.
The rollback command arg is for future operator use only originally, let me add the doc in it. I didn't change 101 and 102, no idea where they come from
   site/deploy.md  Outdated    
 |  | ||
| When running the tool in WLST online mode, the deploy operation may require server restarts or a domain restart to pick up the changes. The deploy operation can also encounter situations where it cannot complete its operation until the domain is restarted. To communicate these conditions to scripts that may be calling the Deploy Applications Tool, the shell scripts have three special, non-zero exit codes to communicate these states: | ||
|  | ||
| - `101` - The domain needs to be restarted and the Deploy Applications Tool needs to be re-invoked with the same arguments. | 
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.
101 and 102 are not valid are they? the rollback command line arg is not documented.
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.
I don't see anywhere returning 101, 102 from the source. I am removing them now.
| Added online system test also | 
This restore the return code of restart required if wlst isRestartRequired returns true during update operation.