influxdata / ui

UI for InfluxDB
94 stars 42 forks source link

Notebooks(Alerts): Notification records in notification history suggest alert instances no longer exist #3708

Open karel-rehor opened 2 years ago

karel-rehor commented 2 years ago

About the bug

Steps to reproduce: List the minimal actions needed to reproduce the behavior.

  1. In AWS EC2 set up a micro instance with a simple HTTP listener that echos messages back to the console
  2. Open Influxdbv2 (note that during this test the feature flag 'notebooksNewEndpoints') was set
  3. Add test data to a bucket with data points that will trigger a critical threshold alert
  4. Create a new Alert Notebook
  5. Create a query based on the test data
  6. In the alerts section define a threshold designed to trigger an alert notification
  7. In the endpoints section select HTTP and set the URL to the EC2 instance. In this case plain HTTP with no tokens or authentication.
  8. Generate more test data with timestamps in the future and values to trigger the notification
  9. Export the Notebook as a task
  10. Monitor the EC2 instance console for messages.
  11. Once messages are received verify them in Alerts > Alert History > Notifications

Expected behavior:

Expected the records in notification history to provide correct and helpful information about sent notifications and the artefacts that generated them.

Perhaps notifications from Notebook Generated tasks should be listed separately, with links to the notebook and the task?

Actual behavior:

Notifications listed under the "Check" column as "Notebook Generated Check" have no links. On hover the tooltip suggests that "The check that created this no longer exists". However, the notebook and the task do exist.

Visual Proof:

NotificationHistory02

About your environment

Environment info:

Testing in K8S-IDPE remocal.

last UI commit

commit a918e664f19a21d33b21fe2c19c7cac8aa01c573 (upstream/master, origin/master, origin/HEAD, master)
Author: Palakp41 <palak@influxdata.com>
Date:   Tue Jan 25 17:16:27 2022 -0800

last K8S-IDPE commit

commit 84c1d4fd7f71ca41759a5f31e3c245c59f3bd4da (HEAD -> master, origin/master, origin/HEAD)
Author: Hercules Mango Churchill <alchemy@influxdata.com>
Date:   Wed Jan 26 02:00:44 2022 +0000

Config:

Logs:

Performance: Generate profiles with the following commands for bugs related to performance, locking, out of memory (OOM), etc.

# Commands should be run when the bug is actively.
# Note: This command will run for at least 30 seconds.
curl -o profiles.tar.gz "http://localhost:8086/debug/pprof/all?cpu=true"
curl -o vars.txt "http://localhost:8086/debug/vars"
iostat -xd 1 30 > iostat.txt
# Attach the `profiles.tar.gz`, `vars.txt`, and `iostat.txt` output files.
jzahnd commented 2 years ago

@jzahnd will review with Bonitoo team on why we don't want to do this