Skip to content

Commit 0e84b7f

Browse files
greggdonovanclaude
andcommitted
Make rules_grafana 2.0.0 Bazel 8+ only
- Update bazel_compatibility to >=8.0.0 - Remove Bazel 7.x from CI matrix and BCR presubmit - Remove E2E Testing section from README (moved to playwright-e2e branch) - Enable grafana_with_plugins_docker_test (no longer manual since Bazel 8 only) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent f4fc668 commit 0e84b7f

File tree

5 files changed

+5
-24
lines changed

5 files changed

+5
-24
lines changed

.bcr/presubmit.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ bcr_test_module:
55
- ubuntu-22.04
66
- macos-15
77
bazel:
8-
- 7.x
98
- 8.x
109
tasks:
1110
run_tests:
@@ -24,7 +23,6 @@ bcr_test_module_linux:
2423
platform:
2524
- ubuntu-22.04
2625
bazel:
27-
- 7.x
2826
- 8.x
2927
tasks:
3028
build_oci:

.github/workflows/ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ jobs:
1212
strategy:
1313
fail-fast: false
1414
matrix:
15-
bazel: ["latest-7", "latest"]
15+
bazel: ["latest"]
1616
steps:
1717
- uses: actions/checkout@v4
1818

@@ -38,7 +38,7 @@ jobs:
3838
strategy:
3939
fail-fast: false
4040
matrix:
41-
bazel: ["latest-7", "latest"]
41+
bazel: ["latest"]
4242
steps:
4343
- uses: actions/checkout@v4
4444

MODULE.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
module(
22
name = "rules_grafana",
33
version = "2.0.0",
4-
bazel_compatibility = [">=7.0.0"],
4+
bazel_compatibility = [">=8.0.0"],
55
compatibility_level = 1,
66
)
77

README.md

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ use_repo(grafana, "grafana_oci")
5757

5858
## Bazel compatibility
5959

60-
Requires Bazel 7.0.0 or later with bzlmod enabled.
60+
Requires Bazel 8.0.0 or later with bzlmod enabled.
6161

6262
`rules_grafana` depends on [`rules_python`](https://github.com/bazelbuild/rules_python) and
6363
[`rules_oci`](https://github.com/bazel-contrib/rules_oci), but these are automatically managed
@@ -202,15 +202,6 @@ Then pass the plugin to the image rule's `plugins` list as `@grafana_plotly_plug
202202
The default version of Grafana (12.0) may not suit your needs.
203203
You can override the container by modifying the grafana extension in your MODULE.bazel.
204204

205-
## E2E Testing
206-
207-
The repository includes Playwright-based E2E tests to validate the Grafana deployment:
208-
209-
```bash
210-
# Run the full E2E test suite
211-
./e2e/run-e2e.sh
212-
```
213-
214205
## API Reference
215206

216207
### `json_dashboards`

test/BUILD.bazel

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -84,13 +84,5 @@ container_structure_test(
8484
name = "grafana_with_plugins_docker_test",
8585
configs = ["docker_with_plugins_tests.yaml"],
8686
image = ":grafana_with_plugins",
87-
# TODO: Plugin install path contains the canonical repo name which differs:
88-
# - Bazel 7.x: _main~grafana~grafana_plotly_plugin
89-
# - Bazel 8.x: +grafana+grafana_plotly_plugin
90-
# The test config hardcodes one format, so it fails on the other version.
91-
# Fix: use a glob pattern or make the path version-agnostic.
92-
tags = [
93-
"docker",
94-
"manual",
95-
],
87+
tags = ["docker"],
9688
)

0 commit comments

Comments
 (0)