Skip to content

[9.0](backport #45100) [Heartbeat] Add base64 encoding option to inline monitors#45260

Merged
emilioalvap merged 2 commits into9.0from
mergify/bp/9.0/pr-45100
Jul 10, 2025
Merged

[9.0](backport #45100) [Heartbeat] Add base64 encoding option to inline monitors#45260
emilioalvap merged 2 commits into9.0from
mergify/bp/9.0/pr-45100

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Jul 9, 2025

Proposed commit message

Add encoding: base64 option to inline monitors. This change allows inline monitors to support characters that would otherwise break yaml format.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

How to test this PR locally

  1. Build heartbeat locally
  2. Specify a base64-encoded inline script monitor.
  3. Execute heartbeat and check monitor is decoded at setup.

Related issues

This is an automatic backport of pull request #45100 done by [Mergify](https://mergify.com).
@emilioalvap @mergify
[Heartbeat] Add base64 encoding option to inline monitors (#45100)
d34c291 
* Add base64 encoding option to inline monitors

* Fix linter

* Fix linter

* Remove unused Decode() method

* Fix linter

* Add changelog

(cherry picked from commit 2266395)
@mergify mergify bot requested a review from a team as a code owner July 9, 2025 15:05
@github-actions github-actions bot added bug Team:obs-ds-hosted-services Label for the Observability Hosted Services team labels Jul 9, 2025
@elasticmachine
Copy link
Contributor

elasticmachine commented Jul 9, 2025

Pinging @elastic/obs-ds-hosted-services (Team:obs-ds-hosted-services)
@mergify mergify bot mentioned this pull request Jul 9, 2025
Merged
6 tasks
mcapell
mcapell reviewed Jul 10, 2025
View reviewed changes
@emilioalvap @mcapell
Update CHANGELOG.next.asciidoc
3b4ea27 
Co-authored-by: Marc Capell <marc.capell@gmail.com>
@emilioalvap emilioalvap enabled auto-merge (squash) July 10, 2025 09:51
@emilioalvap emilioalvap merged commit 60c8989 into 9.0 Jul 10, 2025
28 checks passed
@emilioalvap emilioalvap deleted the mergify/bp/9.0/pr-45100 branch July 10, 2025 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Team:obs-ds-hosted-services Label for the Observability Hosted Services team

3 participants

@elasticmachine @mcapell @emilioalvap