Skip to content

Commit

Permalink
Merge branch 'main' into feature/better-celpy-perf
Browse files Browse the repository at this point in the history
  • Loading branch information
shahargl authored Sep 25, 2024
2 parents 0ee4975 + 4689b42 commit 365d8c6
Show file tree
Hide file tree
Showing 10 changed files with 24 additions and 162 deletions.
4 changes: 0 additions & 4 deletions docs/mint.json
Original file line number Diff line number Diff line change
Expand Up @@ -86,12 +86,9 @@
"group": "Platform",
"pages": [
"platform/overview",
"platform/providers",
"platform/alerts",
"platform/alertseverityandstatus",
"platform/workflows",
"platform/settings",
"platform/metrics",
"platform/support"
]
},
Expand Down Expand Up @@ -188,7 +185,6 @@
"group": "Workflows",
"pages": [
"workflows/overview",
"workflows/getting-started",
{
"group": "Syntax",
"pages": [
Expand Down
2 changes: 1 addition & 1 deletion docs/overview/examples.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ In this example we will utilze:

This example consists of two steps:
1. Connect your tools - Datadog, OpsGenie and Postgres.
2. Create a workflow that is triggered by the alert, runs an SQL query, and decides whether to create an incident. Once the workflow is created, you can upload it via the [Workflows](/platform/workflows) page.
2. Create a workflow that is triggered by the alert, runs an SQL query, and decides whether to create an incident. Once the workflow is created, you can upload it via the [Workflows](https://docs.keephq.dev/workflows/overview) page.
```yaml
alert:
id: enterprise-tier-alerts
Expand Down
2 changes: 1 addition & 1 deletion docs/overview/introduction.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -23,4 +23,4 @@ Keep helps with every step of the alert lifecycle:
5. Summarization - Keep summarizes incidents based on past incidents and a knowledge base (Keep Enterprise only).

## How does Keep integrate with alerts?
Alerts can either be [pulled](/platform/alerts#pulled-alerts) by Keep or [pushed](/platform/alerts#pushed-alerts) into it. Keep also offers zero-click alert instrumentation through [webhook installation](/platform/providers#webhook-integration).
Alerts can either be [pulled](/platform/alerts#pulled-alerts) by Keep or [pushed](/platform/alerts#pushed-alerts) into it. Keep also offers zero-click alert instrumentation through [webhook installation](/providers/overview).
2 changes: 1 addition & 1 deletion docs/overview/keyconcepts.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ For example, consider Promethues. If you want to push alerts from Promethues to
'https://api.keephq.dev/alerts/event/prometheus' using API key authentication. Each Provider implements Push mechanism and is documented under the specific Provider page.

#### Push alerts to Keep (Automatic)
In compatible tools, Keep can automatically integrate with the alerting policy of the source tool and add itself as an alert destination. You can learn more about Webhook Integration [here](/platform/providers#webhook-integration).
In compatible tools, Keep can automatically integrate with the alerting policy of the source tool and add itself as an alert destination. You can learn more about Webhook Integration [here](/providers/overview).
Please note that this will slightly modify your monitors/notification policy.

### Pull alerts by Keep
Expand Down
4 changes: 2 additions & 2 deletions docs/platform/overview.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ The platform is accessible on https://platform.keephq.dev and let you start the

The platform is currently built on top of:

1. [Providers](/platform/providers) - connect your stack to Keep.
1. [Providers](/providers/overview) - connect your stack to Keep.
2. [Alerts](/platform/alerts) - single pane of glass for your alerts.
3. [Workflows](/platform/workflows) - create automations on top of your alerts (or regardless).
3. [Workflows](/workflows/overview) - create automations on top of your alerts (or regardless).
4. [Settings](/platform/settings) - the settings page (add users, etc).
48 changes: 0 additions & 48 deletions docs/platform/providers.mdx

This file was deleted.

74 changes: 0 additions & 74 deletions docs/platform/workflows.mdx

This file was deleted.

26 changes: 0 additions & 26 deletions docs/workflows/getting-started.mdx

This file was deleted.

4 changes: 2 additions & 2 deletions keep-ui/next.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -41,8 +41,8 @@ const nextConfig = {
: [
{
source: "/",
destination: "/providers",
permanent: true,
destination: "/incidents",
permanent: process.env.ENV === "production",
},
];
},
Expand Down
20 changes: 17 additions & 3 deletions tests/e2e_tests/test_end_to_end.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,9 +45,23 @@


def test_sanity(browser):
browser.goto("http://localhost:3000/providers")
browser.wait_for_url("http://localhost:3000/providers")
assert "Keep" in browser.title()
try:
browser.goto("http://localhost:3000/")
browser.wait_for_url("http://localhost:3000/incidents")
assert "Keep" in browser.title()
except Exception:
# Current file + test name for unique html and png dump.
current_test_name = (
"playwright_dump_"
+ os.path.basename(__file__)[:-3]
+ "_"
+ sys._getframe().f_code.co_name
)

browser.screenshot(path=current_test_name + ".png")
with open(current_test_name + ".html", "w") as f:
f.write(browser.content())
raise


def test_insert_new_alert(browser):
Expand Down

0 comments on commit 365d8c6

Please sign in to comment.