[mimir-rules-action] fix multiline output expression

[rasta-rocket]

Signed-off-by: rasta-rocket [email protected]

What this PR does

Hey folks 👋

Small PR that fix/simplify the way the multiline is rendered in the output of the action mimir-rules-action

With the following pipeline:

name: mimir_rules_ci
on:
  workflow_dispatch:
  pull_request:
jobs:
  mimir_rules:
    name: Check Mimir Rules
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v4
      - name: Check Mimir rules
        uses: rasta-rocket/mimir/operations/mimir-rules-action@action_mimirtool_fix_multiline
        env:
          MIMIR_ADDRESS: "XXXXX"
          MIMIR_TENANT_ID: XXXXX
          ACTION: check
          RULES_DIR: "./mimir/alert,./mimir/record"
      - name: Diff Mimir rules
        id: diff_rules
        uses: rasta-rocket/mimir/operations/mimir-rules-action@action_mimirtool_fix_multiline
        env:
          MIMIR_ADDRESS: "XXXXX"
          MIMIR_TENANT_ID: XXXXX
          ACTION: diff
          RULES_DIR: "./mimir/alert,./mimir/record"
      - name: Put diff in summary
        run: |
           echo "${{ steps.diff_rules.outputs.detailed }}"

           echo '# Diff summary' >> $GITHUB_STEP_SUMMARY
           echo '```diff' >> $GITHUB_STEP_SUMMARY
           echo "${{ steps.diff_rules.outputs.detailed }}" >> $GITHUB_STEP_SUMMARY
           echo '```' >> $GITHUB_STEP_SUMMARY

This has been done following : https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#example-of-a-multiline-string

Don't hesitate to ping me if any remarks/questions 😉 ?

Cheers ☀️

Which issue(s) this PR fixes or relates to

Fixes #

Checklist

• Tests updated.
• Documentation added.
• CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
• about-versioning.md updated with experimental features.

[CLAassistant]

All committers have signed the CLA.

[rasta-rocket]

@56quarters 👋

I'm trying my luck with you to get a review on this 😅

[colega]

Thank you! Sorry for such a long delay for the review.

[colega]

I understand that there can't be any collision between "$OUTPUT" and "EOF", as "$OUTPUT" would need to be exactly "EOF" and I can't see that being an option for the $OUTPUT.

[colega]

I'm trying to merge this but for some reason the CI won't run, can I ask you to rebase this on latest main and force-push? (Edit: figured out, I didn't see the CI approval button because I was trying "the new merge experience" feature in Github)

[rasta-rocket]

@colega sorry for the late answer and thanks a lot for the review 🚀

Any chance you can do a review on a similar PR: grafana/cortex-rules-action#27 ?

[colega]

Reviewed, thanks!