This repository has been archived by the owner on Jan 15, 2024. It is now read-only.
SLO: CRUD API Functionality #145
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
elainevuong
requested review from
joeblubaugh,
ellisda,
Leo-DiCara,
julienduchesne and
cesararmada
|
We're going to explore generating the SLO structure mechanically, but these types look good! |
ellisda
reviewed
Apr 27, 2023
ellisda
approved these changes
Apr 27, 2023
julienduchesne
approved these changes
May 1, 2023
flkhndlr
pushed a commit
to flkhndlr/grafana-api-golang-client
that referenced
this pull request
May 9, 2023
* ListSLOs functionality retrives all SLOs * CreateSLO and GetSLO Functionality implemented * DeleteSLO Functionality implemented * UpdateSLO Functionality implemented * UpdateSLO Functionality corrected * Update Linting Errors * Removes Unnecessary Comments * Update Variable Naming * Lint Checker * Update Tests for SLOs * Updating Slo Types with a comment to original source files * Updated SLO Types and Tests
JanSF
added a commit
to JanSF/grafana-api-golang-client
that referenced
this pull request
Sep 19, 2023
Squashed commit of the following: commit b384168 Merge: bc342a8 3f84a92 Author: flkhndlr <50197097+flkhndlr@users.noreply.github.com> Date: Tue May 9 08:44:32 2023 +0200 Merge branch 'grafana:master' into master commit bc342a8 Author: Philip Penquitt <30809710+PhilipPenquitt@users.noreply.github.com> Date: Thu Apr 6 16:49:11 2023 +0200 Trigger Build commit 2a88c53 Author: phpe <30809710+PhilipPenquitt@users.noreply.github.com> Date: Thu Apr 6 16:46:06 2023 +0200 Trigger Build commit 2471267 Author: Elaine Vuong <elaine.yt.vuong@gmail.com> Date: Thu May 4 14:57:04 2023 -0400 SLO: CRUD API Functionality (grafana#145) * ListSLOs functionality retrives all SLOs * CreateSLO and GetSLO Functionality implemented * DeleteSLO Functionality implemented * UpdateSLO Functionality implemented * UpdateSLO Functionality corrected * Update Linting Errors * Removes Unnecessary Comments * Update Variable Naming * Lint Checker * Update Tests for SLOs * Updating Slo Types with a comment to original source files * Updated SLO Types and Tests commit 836a3db Author: Julien Duchesne <julien.duchesne@grafana.com> Date: Thu Apr 27 09:15:11 2023 -0400 Folder UID: Wrong field type (grafana#148) Doh! commit ca95730 Author: Julien Duchesne <julien.duchesne@grafana.com> Date: Thu Apr 27 08:35:50 2023 -0400 Dashboard: Get folder UID from reads (grafana#147) It's easier to manage folders with UIDs than IDs. This will be used to add that functionality to the Terraform provider commit fcca53a Author: Julien Duchesne <julien.duchesne@grafana.com> Date: Tue Apr 25 13:02:16 2023 -0400 Folder delete: Fix panic when params are set (grafana#146) commit 65edf1b Author: Alex Weaver <weaver.alex.d@gmail.com> Date: Tue May 2 15:02:11 2023 -0500 Add dependabot commit 3a31f6f Merge: 0252eaa 6b31ab5 Author: flkhndlr <50197097+flkhndlr@users.noreply.github.com> Date: Tue Apr 18 10:58:34 2023 +0200 Merge branch 'master' of https://github.com/flkhndlr/grafana-api-golang-client commit 0252eaa Author: flkhndlr <50197097+flkhndlr@users.noreply.github.com> Date: Tue Apr 18 10:57:39 2023 +0200 fix: change name of variable commit 6b31ab5 Author: flkhndlr <50197097+flkhndlr@users.noreply.github.com> Date: Tue Apr 18 10:44:29 2023 +0200 adding back whitespaces commit b1311f0 Author: flkhndlr <50197097+flkhndlr@users.noreply.github.com> Date: Tue Apr 18 10:41:25 2023 +0200 clear whitespaces commit 17398b1 Author: flkhndlr <50197097+flkhndlr@users.noreply.github.com> Date: Tue Apr 18 10:29:34 2023 +0200 remove whitespace commit 10b4067 Author: Julien Duchesne <julien.duchesne@grafana.com> Date: Mon Apr 10 18:58:06 2023 -0400 Folder: Add option to force delete folder (grafana#142) grafana/terraform-provider-grafana#855 commit e73fa1f Author: Julien Duchesne <julien.duchesne@grafana.com> Date: Mon Apr 10 18:57:49 2023 -0400 Add missing report fields (grafana#143) grafana/terraform-provider-grafana#862 commit 62aa037 Author: Philip Penquitt <30809710+PhilipPenquitt@users.noreply.github.com> Date: Thu Apr 6 16:49:11 2023 +0200 Trigger Build commit 2cb0ad5 Author: flkhndlr <50197097+flkhndlr@users.noreply.github.com> Date: Thu Apr 6 15:01:20 2023 +0200 adding alert_rule fetch function (grafana#1) Co-authored-by: fkhe <Falk.Haendler@t-systems.com> commit c27c52e Author: flkhndlr <50197097+flkhndlr@users.noreply.github.com> Date: Thu Apr 6 14:59:04 2023 +0200 correcting testcases commit 4732c3e Author: flkhndlr <50197097+flkhndlr@users.noreply.github.com> Date: Thu Apr 6 14:49:25 2023 +0200 correcting testcases, add documentation commit 1921ba9 Author: flkhndlr <50197097+flkhndlr@users.noreply.github.com> Date: Thu Apr 6 13:23:20 2023 +0200 adding alert_rule fetch function commit 64ddabb Author: phpe <30809710+PhilipPenquitt@users.noreply.github.com> Date: Thu Apr 6 16:46:06 2023 +0200 Trigger Build commit e009637 Author: Philip Penquitt <30809710+PhilipPenquitt@users.noreply.github.com> Date: Thu Apr 6 14:37:21 2023 +0200 fix: formating fixed
julienduchesne
added a commit
that referenced
this pull request
Jan 11, 2024
* fix: formating fixed * Trigger Build * adding alert_rule fetch function * correcting testcases, add documentation * correcting testcases * adding alert_rule fetch function (#1) Co-authored-by: fkhe <Falk.Haendler@t-systems.com> * Trigger Build * Add missing report fields (#143) grafana/terraform-provider-grafana#862 * Folder: Add option to force delete folder (#142) grafana/terraform-provider-grafana#855 * remove whitespace * clear whitespaces * adding back whitespaces * fix: change name of variable * Add dependabot * Folder delete: Fix panic when params are set (#146) * Dashboard: Get folder UID from reads (#147) It's easier to manage folders with UIDs than IDs. This will be used to add that functionality to the Terraform provider * Folder UID: Wrong field type (#148) Doh! * SLO: CRUD API Functionality (#145) * ListSLOs functionality retrives all SLOs * CreateSLO and GetSLO Functionality implemented * DeleteSLO Functionality implemented * UpdateSLO Functionality implemented * UpdateSLO Functionality corrected * Update Linting Errors * Removes Unnecessary Comments * Update Variable Naming * Lint Checker * Update Tests for SLOs * Updating Slo Types with a comment to original source files * Updated SLO Types and Tests * Trigger Build * Trigger Build --------- Co-authored-by: Philip Penquitt <30809710+PhilipPenquitt@users.noreply.github.com> Co-authored-by: fkhe <Falk.Haendler@t-systems.com> Co-authored-by: Julien Duchesne <julien.duchesne@grafana.com> Co-authored-by: Alex Weaver <weaver.alex.d@gmail.com> Co-authored-by: Elaine Vuong <elaine.yt.vuong@gmail.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This Draft PR implements a interaction with the Grafana SLO API, mainly intended for usage with the Grafana Terraform Provider.
Needs to be reviewed in conjunction with this PR on the Grafana Terraform Provider
Testing TBD, but I'm happily taking any feedback!