Skip to content

Commit

Permalink
fix: fix correlation link and better hints (#2620)
Browse files Browse the repository at this point in the history
  • Loading branch information
35C4n0r authored Nov 25, 2024
1 parent af41180 commit 815f6f3
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ export const CorrelationForm = ({
<Button
className="cursor-default ml-2"
type="button"
tooltip="When the first alert arrives, Keep will start to calculate the timespan, any new alert within the timeframe will correlate into the same incident. Time cannot exceed 14 days"
tooltip="When the first alert arrives, Keep calculates the timespan. Any new alert within this timeframe will correlate into the same incident. The timeframe cannot exceed 14 days."
icon={QuestionMarkCircleIcon}
size="xs"
variant="light"
Expand Down Expand Up @@ -98,7 +98,7 @@ export const CorrelationForm = ({
<Button
className="cursor-default ml-2"
type="button"
tooltip="Keep will use these attributes to split between incidents. For example, with attribute host, Keep will correlate alert with hostX and alert with host hostY to different incidents. You cannot calculate attributes to group by without alerts"
tooltip="Attributes are used to distinguish between incidents. For example, grouping by 'host' will correlate alerts with hostX and hostY into separate incidents. Attributes cannot be calculated without alerts."
icon={QuestionMarkCircleIcon}
size="xs"
variant="light"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,7 @@ export const CorrelationGroups = () => {
<Button
className="cursor-default"
type="button"
tooltip="Any Rule consists of one or more Correlations. Each alert group is evaluated separately and the results are combined using AND combinator.
For example, if you want to group alerts that has a severity of 'critical' and another alert with a source of 'Kibana', you would create a rule with two alert groups.
The first alert group would have a rule with severity = 'critical' and the second alert group would have a rule with source = 'kibana'."
tooltip="A Rule contains one or more Correlations, each evaluating a separate alert group. Results are combined using an AND operator. For instance, to group alerts by severity 'critical' and source 'Kibana', create two alert groups: one with severity = 'critical' and another with source = 'Kibana'."
icon={QuestionMarkCircleIcon}
size="xs"
variant="light"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@ import { FormProvider, SubmitHandler, useForm } from "react-hook-form";
import { CorrelationForm } from "./CorrelationForm";
import { CorrelationGroups } from "./CorrelationGroups";
import { CorrelationSubmission } from "./CorrelationSubmission";
import { Link } from "@/components/ui";
import { ArrowUpRightIcon } from "@heroicons/react/24/outline";
import { useHydratedSession as useSession } from "@/shared/lib/hooks/useHydratedSession";
import { useRules } from "utils/hooks/useRules";
import { CorrelationForm as CorrelationFormType } from ".";
Expand Down Expand Up @@ -112,10 +114,16 @@ export const CorrelationSidebarBody = ({
color="teal"
>
A versatile tool for grouping and consolidating alerts. Read more in
our{" "}
<a href="https://docs.keephq.dev/overview/ruleengine" target="_blank">
docs.
</a>
our{" "}
<Link
icon={ArrowUpRightIcon}
iconPosition="right"
className="!text-orange-500 hover:!text-orange-700 ml-0.5"
target="_blank"
href="https://docs.keephq.dev/overview/ruleengine"
>
docs
</Link>
<Button
className="absolute top-0 right-0"
onClick={() => setIsCalloutShown(false)}
Expand Down

0 comments on commit 815f6f3

Please sign in to comment.