-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39952/
-----------------------------------------------------------
Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, Jayush Luniya,
Mahadev Konar, and Nate Cole.
Bugs: AMBARI-13722
https://issues.apache.org/jira/browse/AMBARI-13722
Repository: ambari
Description
-------
When failures are skipped in the Upgrade Wizard, there's a manual confirmation
step that is injected.
{code}
"Manual steps required"
Verifying Skipped Failures
[ ] I have performed the manual steps above.
[Pause Upgrade] [Proceed] <- [Downgrade] was missing, but that's a different
issue
{code}
This doesn't really make sense to the end user.
So the full scope if this jira becomes changes to structuredOut to incorporate
auto-skipped detail (hosts and services/components) and failures (hosts and
services/components)
{code}
{
"failures":{
"service_check":[
"ZOOKEEPER"
],
"host_component":{
"host1.vm":[
{
"component":"DATANODE"
}
],
"host2.vm":[
{
"component":"DATANODE"
}
]
}
},
"skipped":[
"service_check",
"host_component"
]
}
{code}
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/metadata/ActionMetadata.java
f5642a0
ambari-server/src/main/java/org/apache/ambari/server/serveraction/upgrades/AutoSkipFailedSummaryAction.java
9a84e38
ambari-server/src/test/java/org/apache/ambari/server/serveraction/upgrades/AutoSkipFailedSummaryActionTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/39952/diff/
Testing
-------
running full unit tests, should be good
Thanks,
Dmitro Lisnichenko
