ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Nate Cole" <nc...@hortonworks.com>
Subject Re: Review Request 29547: Alerts: Allow Ability To Test An AlertTarget Before Creating It
Date Mon, 05 Jan 2015 14:27:14 GMT

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



ambari-server/src/main/java/org/apache/ambari/server/api/resources/AlertTargetValidationResourceDefinition.java
<https://reviews.apache.org/r/29547/#comment110208>

    Instead of this new class, just use SimpleResourceDefinition.



ambari-server/src/main/java/org/apache/ambari/server/api/resources/ResourceInstanceFactoryImpl.java
<https://reviews.apache.org/r/29547/#comment110209>

    Use a SimpleResourceDefinition.


- Nate Cole


On Jan. 2, 2015, 1:06 p.m., Yurii Shylov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/29547/
> -----------------------------------------------------------
> 
> (Updated Jan. 2, 2015, 1:06 p.m.)
> 
> 
> Review request for Ambari, Jonathan Hurley, Nate Cole, and Tom Beerbower.
> 
> 
> Bugs: AMBARI-8978
>     https://issues.apache.org/jira/browse/AMBARI-8978
> 
> 
> Repository: ambari
> 
> 
> Description
> -------
> 
> During the cluster installation, the web client would like to be able to have the administrator
configure an alert target for use with that cluster. However, because there are many properties
that are used to successfully create an AlertTarget, it's likely that the settings originally
provided may not work.
> 
> For example, when creating an AlertTarget for SMTP, if the security or port are not valid
(or the SMTP server is restricting access to certain IP addresses) then the target won't be
able to properly use it.
> 
> We need to be able to allow an AlertTarget to be "tested" before actually creating it
in the system. 
> 
> I propose a new endpoint off of targets that can be used to POST to. The POST can contain
all of the alert properties that would normally be found on an AlertTarget. The difference
is that no target is created; instead a status is returned about whether the target works
(and why it doesn't if it failed).
> 
> I would suggest also altering the dispatcher interface to support a new method; something
like {{Dispatcher.testAlertTarget(...)}} which will simply exercise the properties of the
target to ensure a good connection.
> 
> 
> Diffs
> -----
> 
>   ambari-server/src/main/java/org/apache/ambari/server/api/resources/AlertTargetValidationResourceDefinition.java
PRE-CREATION 
>   ambari-server/src/main/java/org/apache/ambari/server/api/resources/ResourceInstanceFactoryImpl.java
e55b2cb 
>   ambari-server/src/main/java/org/apache/ambari/server/api/services/AlertTargetValidationService.java
PRE-CREATION 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertTargetValidationResourceProvider.java
PRE-CREATION 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/internal/DefaultProviderModule.java
be2a9ad 
>   ambari-server/src/main/java/org/apache/ambari/server/controller/spi/Resource.java 89d6837

>   ambari-server/src/main/java/org/apache/ambari/server/notifications/NotificationDispatcher.java
10946be 
>   ambari-server/src/main/java/org/apache/ambari/server/notifications/dispatchers/EmailDispatcher.java
f979c03 
>   ambari-server/src/main/java/org/apache/ambari/server/notifications/dispatchers/SNMPDispatcher.java
0e75801 
>   ambari-server/src/main/java/org/apache/ambari/server/state/services/AlertNoticeDispatchService.java
b085112 
>   ambari-server/src/test/java/org/apache/ambari/server/controller/internal/AlertTargetValidationResourceProviderTest.java
PRE-CREATION 
>   ambari-server/src/test/java/org/apache/ambari/server/notifications/EmailDispatcherTest.java
1e7689f 
>   ambari-server/src/test/java/org/apache/ambari/server/notifications/MockDispatcher.java
616551f 
>   ambari-server/src/test/java/org/apache/ambari/server/notifications/dispatchers/EmailDispatcherTest.java
PRE-CREATION 
>   ambari-server/src/test/java/org/apache/ambari/server/notifications/dispatchers/SNMPDispatcherTest.java
db4af1c 
>   ambari-server/src/test/java/org/apache/ambari/server/state/services/AlertNoticeDispatchServiceTest.java
2e984bf 
> 
> Diff: https://reviews.apache.org/r/29547/diff/
> 
> 
> Testing
> -------
> 
> In progress
> 
> 
> Thanks,
> 
> Yurii Shylov
> 
>


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