Skip to content

[8.18] [Oblt Onboarding][Auto Detect] Filter out httpjson inputs and fix accidental config backup file (#216978)#217497

Merged
mykolaharmash merged 2 commits intoelastic:8.18from
mykolaharmash:backport/8.18/pr-216978
Apr 9, 2025
Merged

[8.18] [Oblt Onboarding][Auto Detect] Filter out httpjson inputs and fix accidental config backup file (#216978)#217497
mykolaharmash merged 2 commits intoelastic:8.18from
mykolaharmash:backport/8.18/pr-216978

Conversation

@mykolaharmash
Copy link
Contributor

Backport

This will backport the following commits from main to 8.18:

Questions ?

Please refer to the Backport tool documentation

@elasticmachine
Copy link
Contributor

elasticmachine commented Apr 8, 2025

💔 Build Failed

Failed CI Steps

History

mykolaharmash and others added 2 commits April 8, 2025 15:09
…idental config backup file (elastic#216978)

Closes elastic#199744

* Adds a separate `sed` commands for Linux and macOS when replacing API
key within the Agent config. GNU and BSD versions of `sed` treat `-i`
(in-place editing) argument differently, GNU version allows `-i` without
a value while BSD version requires a backup file extension even when
it's empty 🫠
* Adds filtering of unsupported input types inside the integration
policies. For now it only filters out `httpjson`.

## How to test
1. Go through the auto-detect flow
2. Make sure there is no `'elastic-agent.yml='` file in the Agent
directory, or any other weird artifacts
3. Inspect individual integration config files, make sure they don't
have `httpjson` inputs

Co-authored-by: Joe Reuter <johannes.reuter@elastic.co>
(cherry picked from commit ec72d4a)

# Conflicts:
#	x-pack/solutions/observability/plugins/observability_onboarding/server/routes/flow/route.ts
@mykolaharmash mykolaharmash force-pushed the backport/8.18/pr-216978 branch from 089bb0a to 8de1f0c Compare April 8, 2025 13:10
@mykolaharmash mykolaharmash merged commit b0906ca into elastic:8.18 Apr 9, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants