Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adds more scripts for post-setup #108

Merged
merged 2 commits into from
Oct 24, 2024
Merged

Adds more scripts for post-setup #108

merged 2 commits into from
Oct 24, 2024

Conversation

jasonrhodes
Copy link
Member

Changes

  • Removes unnecessary prerequisites from scripts/rca-demo/README.md
  • Adds post-setup script to set things up in Kibana and ES after the initial app setup
  • Moves some shared logic from the initial setup script into a shared.sh file that can be sourced by all scripts

dgieselaar
dgieselaar reviewed Oct 24, 2024
View reviewed changes
scripts/rca-demo/post-setup
create_rule 9055220c-8fb1-4f9f-be7c-0a33eb2bafc5 rule-01.json

# Update mappints in ES
curl -X PUT http://localhost:9200/logs-*otel*/_mapping -H "Authorization: ApiKey $OTEL_DEMO_ES_API_KEY" -H "Content-Type: application/json" -d @$SCRIPTS_DIR/data/mapping-message-alias.json
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this will fail if the data stream doesn't exist, in that case you need to update the component template as well.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I'll add the component update in a follow up and just make this fail silently

@jasonrhodes jasonrhodes marked this pull request as ready for review October 24, 2024 17:24
@jasonrhodes jasonrhodes merged commit cefde56 into elastic:rca_ingress_obs Oct 24, 2024
3 checks passed
@jasonrhodes jasonrhodes deleted the more-scripting branch October 24, 2024 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dgieselaar dgieselaar dgieselaar left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jasonrhodes @dgieselaar