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

Fixing up Connection Permissions to upgrade grafana API version #288

Merged
merged 2 commits into from
Sep 3, 2024

Conversation

safaci2000
Copy link
Contributor

@safaci2000 safaci2000 commented Aug 31, 2024

Fixes #271

ChangeLog:

  • Enable Enterprise Grafana integration tests.
  • Removed enterprise config flag, in favor of API check
  • Update Connection Permission to use new API endpoints introduced with library updates
  • Change testing to be testcontainer driven rather than github actions driven. (ie. go test -v ./... is all you need)
  • Adding both an API and general debug flag
  • Removing the enterprise configuration flag, instead programmatically determining state
  • Migrated away from folder ID to folderUID for dashboard CRUD

@safaci2000 safaci2000 force-pushed the feature/apiUpgrade branch 4 times, most recently from 69327ec to 3a73837 Compare September 1, 2024 20:43
@safaci2000 safaci2000 changed the title Feature/api upgrade Fixing up Connection Permissions to upgrade grafana API version Sep 1, 2024
@safaci2000 safaci2000 force-pushed the feature/apiUpgrade branch 19 times, most recently from e2a20c5 to c3d195b Compare September 3, 2024 05:01
Fixes #271

ChangeLog:
  - Enable Enterprise Grafana integration tests.
  - Removed enterprise config flag, in favor of API check
  - Update Connection Permission to use new API endpoints introduced with library updates
  - Change testing to be testcontainer driven rather than github actions driven. (ie. go test -v ./... is all you need)
  - Adding both an API and general debug flag
  - Removing the enterprise configuration flag, instead programmatically determining state
  - Migrated away from folder ID to folderUID for dashboard CRUD
@safaci2000 safaci2000 merged commit b02a854 into master Sep 3, 2024
3 checks passed
@safaci2000 safaci2000 deleted the feature/apiUpgrade branch September 3, 2024 19:02
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.

[BUG] Update Connection Permission to use correct endpoint.
2 participants