Revert "Fix app_tel_for_sok app.conf install stanza header (#1724)"#1728
Open
vivekr-splunk wants to merge 2 commits intodevelopfrom
Open
Revert "Fix app_tel_for_sok app.conf install stanza header (#1724)"#1728vivekr-splunk wants to merge 2 commits intodevelopfrom
vivekr-splunk wants to merge 2 commits intodevelopfrom
Conversation
This reverts commit 7fa8698.
fdf2dd3 to
4c7bf51
Compare
There was a problem hiding this comment.
Pull request overview
This PR reverts commit 7fa8698921a131ac7f120d8afb57a5e82d55f60c from PR #1724, which bundled two distinct changes together: a telemetry app.conf fix and workflow secret-guard conditions. The revert is necessary because the workflow changes caused push workflows on the develop branch to fail at startup (no jobs created). By reverting the entire PR, the codebase returns to a stable baseline, allowing the telemetry app fix to be reapplied separately in a follow-up PR.
Changes:
- Reverts telemetry app shell command changes in
pkg/splunk/enterprise/names.go(changing back fromprintf '%s'toecho -e) - Removes
ifconditionals that checked for empty secrets in 5 workflow files
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pkg/splunk/enterprise/names.go | Reverts telemetry app creation commands from printf '%s' back to echo -e for both SHC and non-SHC scenarios |
| .github/workflows/prodsec-workflow.yml | Removes the if: ${{ secrets.FOSSA_API_TOKEN != '' }} condition from FOSSA Test step |
| .github/workflows/distroless-build-test-push-workflow.yml | Removes the if: ${{ secrets.COVERALLS_TOKEN != '' }} condition from Code Coverage step |
| .github/workflows/build-test-push-workflow.yml | Removes the if: ${{ secrets.COVERALLS_TOKEN != '' }} condition from Code Coverage step |
| .github/workflows/arm-Ubuntu-build-test-push-workflow.yml | Removes the if: ${{ secrets.COVERALLS_TOKEN != '' }} condition from Code Coverage step |
| .github/workflows/arm-AL2023-build-test-push-workflow-AL2023.yml | Removes the if: ${{ secrets.COVERALLS_TOKEN != '' }} condition from Code Coverage step |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
gabrielm-splunk
approved these changes
Feb 26, 2026
kasiakoziol
approved these changes
Feb 26, 2026
rlieberman-splunk
approved these changes
Feb 26, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This reverts commit
7fa8698921a131ac7f120d8afb57a5e82d55f60cfrom PR #1724.Reason:
developstarted failing at workflow startup (no jobs created) after Fix app_tel_for_sok app.conf install stanza header #1724 merged.After this revert merges, I will recreate a clean PR with only the intended telemetry app fix in
pkg/splunk/enterprise/names.go.