Skip to content

Conversation

sm-release-app[bot]
Copy link
Contributor

@sm-release-app sm-release-app bot commented Sep 19, 2025

🤖 I have created a release beep boop

1.35.2 (2025-09-25)

Fixes

Miscellaneous Chores


This PR was generated with Release Please. See documentation.

Copy link

github-actions bot commented Sep 19, 2025

Script size changes

Name +/- Main This PR Outcome
[module.js] = 2,384.55 kB 2,384.55 kB
[datasource/module.js] = 24.68 kB 24.68 kB

Totals

Name +/- Main This PR Outcome
[Scripts] = 2,409.23 kB 2,409.23 kB
[Non-script Assets] +0.05% 2,585.95 kB 2,587.24 kB
[All] +0.03% 4,995.18 kB 4,996.47 kB

Generated by 🚫 dangerJS against f5ec0a1

@sm-release-app sm-release-app bot force-pushed the release-please--branches--main--components--synthetic-monitoring-app branch 2 times, most recently from 3691ca2 to d2c51aa Compare September 24, 2025 08:09
Copy link

github-actions bot commented Sep 24, 2025

Probe API Server Mappings Verification Passed

The probeAPIServerMappings.json file is up to date and matches the Grafana documentation. 🎉

@sm-release-app sm-release-app bot force-pushed the release-please--branches--main--components--synthetic-monitoring-app branch from d2c51aa to fec009b Compare September 24, 2025 09:11
@sm-release-app sm-release-app bot force-pushed the release-please--branches--main--components--synthetic-monitoring-app branch from fec009b to f5ec0a1 Compare September 25, 2025 13:44
Copy link

github-actions bot commented Sep 25, 2025

Terraform Configuration Validation Failed

The generated terraform configurations have schema compatibility issues with the Grafana provider.

⚠️ These issues would cause terraform operations to fail in production.

Validation Errors

Error: Incorrect attribute value type
on testTerraformConfig.tf.json line 438, in resource.grafana_synthetic_monitoring_check_alerts.basic_http:
438:         "alerts": [
439:           {
440:             "name": "ProbeFailedExecutionsTooHigh",
441:             "threshold": 70,
442:             "period": "1h"
443:           },
444:           {
445:             "name": "TLSTargetCertificateCloseToExpiring",
446:             "threshold": 267,
447:             "period": "30m"
448:           },
449:           {
450:             "name": "HTTPRequestDurationTooHighAvg",
451:             "threshold": 62,
452:             "period": "10m"
453:           },
454:           {
455:             "name": "PingRequestDurationTooHighAvg",
456:             "threshold": 488,
457:             "period": "15m"
458:           },
459:           {
460:             "name": "DNSRequestDurationTooHighAvg",
461:             "threshold": 269,
462:             "period": "15m"
463:           }
464:         ]
Inappropriate value for attribute "alerts": element 0: attribute
"runbook_url" is required.::error::Terraform exited with code 1.error Command failed with exit code 1.info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

🔧 How to Fix

  1. Check field naming and requirements according to the Grafana Terraform Provider Documentation
  2. Ensure all required fields are present for each resource type
  3. Verify data types match the provider schema expectations
  4. Update the transformation functions in src/components/TerraformConfig/

🧪 To test locally

yarn build:generate-terraform-test-config
yarn verify:terraform-test-config

@The-9880 The-9880 marked this pull request as ready for review September 25, 2025 14:27
@The-9880 The-9880 requested review from a team as code owners September 25, 2025 14:27
@The-9880 The-9880 requested a review from ckbedwell September 25, 2025 14:27
Copy link

@The-9880 The-9880 left a comment

Choose a reason for hiding this comment

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

TF validation is failing: https://github.com/grafana/synthetic-monitoring-app/actions/runs/18009517947/job/51239171954?pr=1268

This is a bug in the TF provider and is unrelated to the SM app, so I am going to bypass CI as all other jobs have succeeded.

@The-9880 The-9880 merged commit bc9e613 into main Sep 25, 2025
33 of 36 checks passed
@The-9880 The-9880 deleted the release-please--branches--main--components--synthetic-monitoring-app branch September 25, 2025 14:29
@sm-release-app
Copy link
Contributor Author

🤖 Created releases:

🌻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant