Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add event query ids #207

Merged
merged 50 commits into from
Feb 12, 2024
Merged

Add event query ids #207

merged 50 commits into from
Feb 12, 2024

Conversation

danhurwit
Copy link
Contributor

@danhurwit danhurwit commented Feb 8, 2024

Adds event_query_ids field to dashboards as well as the event_query resource type.

Automates the generation of docs because that step always seems to be broken.

Copy link
Contributor

@MisterSquishy MisterSquishy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥳 🥳 🥳

.go-version Outdated
1.89.1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

additive changes get a minor bump, no?

Suggested change
1.89.1
1.90.0

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah just caught that

project_name = "` + testProject + `"
dashboard_name = "Acceptance Test Dashboard (TestAccDashboardServiceHealthPanel)"
dashboard_description = "Dashboard to test the service health panel"
event_query_ids = []
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ffti but id also be interested to document what happens when

  • you try to link a dashboard to a nonexistent event query id
  • you try to delete an event query that is still referenced by a dashboard

@danhurwit danhurwit merged commit 3ee2eaa into main Feb 12, 2024
@danhurwit danhurwit deleted the add-event-query-ids branch February 12, 2024 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants