Fix app_tel_for_sok app.conf install stanza header#1724
Fix app_tel_for_sok app.conf install stanza header#1724vivekr-splunk merged 3 commits intosplunk:developfrom
Conversation
|
CLA Assistant Lite bot CLA Assistant Lite bot All contributors have signed the COC ✍️ ✅ |
There was a problem hiding this comment.
Pull request overview
Fixes the invalid app.conf generated for the telemetry app (app_tel_for_sok) by avoiding echo -e, which can be emitted literally as -e on some /bin/sh implementations.
Changes:
- Replace
echo -ewithprintf '%s'when writingdefault/app.confandmetadata/default.metafor the telemetry app. - Keep the rest of the telemetry app creation flow unchanged (directory creation + reload/bundle push).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
recheck |
|
I have read the CLA Document and I hereby sign the CLA |
|
I have read the Code of Conduct and I hereby sign the COC |
1 similar comment
|
I have read the Code of Conduct and I hereby sign the COC |
|
I have read the CLA Document and I hereby sign the CLA |
|
I have read the Code of Conduct and I hereby accept the Terms |
33eacf7 to
9a51fa5
Compare
…ll] on its first line instead of just [install]
…LS_TOKEN is not available in these workflow runs for GitHub security reasons)
a720d63 to
05aadae
Compare
|
This PR introduced incorrect syntax:
src: https://docs.github.com/en/actions/how-tos/write-workflows/choose-what-workflows-do/use-secrets |
Description
Fixes #1723
Key Changes
Fixes the first line of the app.conf within
app_tel_for_sokso it does not have a-eprefix.Testing and Verification
Locally built the operator and confirmed that app.conf is correct.
Related Issues
Jira tickets, GitHub issues, Support tickets...
PR Checklist