VMware Cloud Community
admin
Immortal
Immortal

hq r13175 - in trunk/src/org/hyperic/hq: appdef/server/session authz/server/session

Reply
0 Kudos
1 Reply
scottmf
Enthusiast
Enthusiast

Hi Patrick,
I don't understand this change. How does it fix the problem?

HHQ-3059
As an HQ Admin, I need HQ to suppress alerts for resources that use an
SNMP device, if the unavailable SNMP device is the root cause in the
hierarchical alerting evaluation. However, if an agent that is
responsible for metric collection for a remote SNMP device becomes
unavailable, HQ needs to verify whether the SNMP device is truly
unavailable through a secondary check before suppressing alerts in the
hierarchy.

Aren't we just removing valuable functionality?


On May 12, 2009, at 1:15 PM, pnguyen@hyperic.com wrote:

> Author: pnguyen
> Date: 2009-05-12 13:15:42 -0700 (Tue, 12 May 2009)
> New Revision: 13175
> URL: http://svn.hyperic.org/?view=rev&root=Hyperic+HQ&revision=13175
>
> Modified:
> trunk/src/org/hyperic/hq/appdef/server/session/PlatformDAO.java
> trunk/src/org/hyperic/hq/authz/server/session/ResourceEdgeDAO.java
> Log:
> HHQ-3059: remove name search on fqdn
>
> Modified: trunk/src/org/hyperic/hq/appdef/server/session/
> PlatformDAO.java
> ===================================================================
> --- trunk/src/org/hyperic/hq/appdef/server/session/PlatformDAO.java
> 2009-05-12 18:07:44 UTC (rev 13174)
> +++ trunk/src/org/hyperic/hq/appdef/server/session/PlatformDAO.java
> 2009-05-12 20:15:42 UTC (rev 13175)
> @@ -259,10 +259,10 @@
> if (platformName != null && platformName.trim().length() >
> 0) {
> HQDialect dialect = Util.getHQDialect();
> nameEx = dialect.getRegExSQL("r.sort_name", ":regex",
> true, false);
> - String fqdnEx = dialect.getRegExSQL("p.fqdn", ":regex",
> true, false);
>
> - sql += " and (" + fqdnEx + " or " + nameEx + ") ";
> + sql += " and (" + nameEx + ") ";
> }
> + sql += " order by r.sort_name ";
>
> Query query = getSession()
> .createSQLQuery(sql)
>
> Modified: trunk/src/org/hyperic/hq/authz/server/session/
> ResourceEdgeDAO.java
> ===================================================================
> --- trunk/src/org/hyperic/hq/authz/server/session/
> ResourceEdgeDAO.java 2009-05-12 18:07:44 UTC (rev 13174)
> +++ trunk/src/org/hyperic/hq/authz/server/session/
> ResourceEdgeDAO.java 2009-05-12 20:15:42 UTC (rev 13175)
> @@ -112,10 +112,10 @@
> if (platformName != null && platformName.trim().length() >
> 0) {
> HQDialect dialect = Util.getHQDialect();
> nameEx = dialect.getRegExSQL("r.sort_name", ":regex",
> true, false);
> - String fqdnEx = dialect.getRegExSQL("p.fqdn", ":regex",
> true, false);
>
> - sql += " and (" + fqdnEx + " or " + nameEx + ") ";
> + sql += " and (" + nameEx + ") ";
> }
> + sql += " order by r.sort_name ";
>
> Query query = getSession()
> .createSQLQuery(sql)
>



Reply
0 Kudos