ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alejandro Fernandez <afernan...@hortonworks.com>
Subject Review Request 44492: [DRAFT] AMBARI-15330. Bubble up errors during RU/EU
Date Tue, 08 Mar 2016 02:11:15 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/44492/
-----------------------------------------------------------

Review request for Ambari, Dmytro Grinenko, Dmitro Lisnichenko, Jonathan Hurley, Nate Cole,
and Sid Wagle.


Bugs: AMBARI-15330
    https://issues.apache.org/jira/browse/AMBARI-15330


Repository: ambari


Description
-------

During RU/EU, need a way to bubble up an error of the current item that failed. This is useful
to quickly get a human-readable error that others UIs can quickly retrieve.
It can print a human-readable error, plus stdout and stderr.
This would become part of the upgrade endpoint. e.g,
api/v1/clusters/$name/upgrade_summary/$request_id

```
{

cluster_name: "c1",
request_id: 1,
fail_reason: "Failed calling RESTART ZOOKEEPER/ZOOKEEPER_SERVER on host c6401.ambari.apache.org",

// Notice that the rest are inherited from the failed task if it exists.
attempt_cnt: 1,
command: "CUSTOM_COMMAND",
command_detail: "RESTART ZOOKEEPER/ZOOKEEPER_SERVER",
custom_command_name: "RESTART",
end_time: -1,
error_log: "/var/lib/ambari-agent/data/errors-1234.txt",
exit_code: 1,
host_name: "c6401.ambari.apache.org",
id: 1234,
output_log: "/var/lib/ambari-agent/data/output-1234.txt",
role: "ZOOKEEPER_SERVER",
stage_id: 1,
start_time: 123456789,
status: "HOLDING_FAILED",
stdout: "",
stderr: ""
}
```


Diffs
-----

  ambari-server/src/main/java/org/apache/ambari/server/api/resources/ResourceInstanceFactoryImpl.java
3526e23 
  ambari-server/src/main/java/org/apache/ambari/server/api/services/ClusterService.java 7200b83

  ambari-server/src/main/java/org/apache/ambari/server/api/services/UpgradeSummaryService.java
PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/DefaultProviderModule.java
d1d3fe6 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/TaskResourceProvider.java
510d6fb 
  ambari-server/src/main/java/org/apache/ambari/server/controller/internal/UpgradeSummary.java
PRE-CREATION 
  ambari-server/src/main/java/org/apache/ambari/server/controller/spi/Resource.java e79f300

  ambari-server/src/main/java/org/apache/ambari/server/orm/dao/HostRoleCommandDAO.java b48ffa8

  ambari-server/src/main/java/org/apache/ambari/server/state/UpgradeHelper.java 05d49c4 

Diff: https://reviews.apache.org/r/44492/diff/


Testing
-------

Verified on RU and EU by introducing a failure.

Still need to write unit tests.
======


Thanks,

Alejandro Fernandez


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message