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.

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

Revive the logstash container for testing #15654

Merged
merged 2 commits into from
Nov 21, 2024

Conversation

AlanCoding
Copy link
Member

SUMMARY

Fast demo:

$ docker exec -i -t $(docker ps -aqf "name=tools_logstash_1") tail -n 2 /logstash.log
{"headers":{"remote_user":"awx_logger","http_accept":"*/*","content_type":"application/json; charset=utf-8","request_path":"/","http_version":"HTTP/1.1","http_authorization":"Basic YXd4X2xvZ2dlcjp3b3JrZmxvd3M=","request_method":"POST","http_host":"logstash:8085","request_uri":"/","content_length":"360"},"@timestamp":"2024-11-21T19:13:14.561Z","level":"WARNING","cluster_host_id":"awx-1","@version":"1","host":"awx-1","stack_info":null,"guid":"7ff29201","logger_name":"awx.api.generics","message":"status 404 received by user alan attempting to access /api/v2/workflow_jobs/4/workflow_nodes/ from 172.18.0.1","tower_uuid":"00000000-0000-0000-0000-000000000000"}
{"headers":{"remote_user":"awx_logger","http_accept":"*/*","content_type":"application/json; charset=utf-8","request_path":"/","http_version":"HTTP/1.1","http_authorization":"Basic YXd4X2xvZ2dlcjp3b3JrZmxvd3M=","request_method":"POST","http_host":"logstash:8085","request_uri":"/","content_length":"360"},"@timestamp":"2024-11-21T19:13:21.941Z","level":"WARNING","cluster_host_id":"awx-1","@version":"1","host":"awx-1","stack_info":null,"guid":"5492b3be","logger_name":"awx.api.generics","message":"status 404 received by user alan attempting to access /api/v2/workflow_jobs/4/workflow_nodes/ from 172.18.0.1","tower_uuid":"00000000-0000-0000-0000-000000000000"}

This already existed, but it wasn't working by the prior instructions. Needed container name & network. Logstash container predated other changes that made these necessary.

I also wanted to demonstrate our future direction, with the files broken out better. You could hypothetically run this completely on its own. If we could keep each compose file a full independent package, then that should be easier in the long run.

Also, the Makefile is nearing the point of maintenance failure, so I'm trying to dodge around changes to that. The ELK stack sure doesn't work.

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • API

@AlanCoding AlanCoding enabled auto-merge (squash) November 21, 2024 20:04
@AlanCoding AlanCoding merged commit ce7911e into ansible:devel Nov 21, 2024
22 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants