-
Notifications
You must be signed in to change notification settings - Fork 7
fix/vector metrics #2058
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
Open
goastler
wants to merge
64
commits into
main
Choose a base branch
from
fix/vector-metrics
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
fix/vector metrics #2058
Conversation
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
Member
goastler
commented
Aug 20, 2025
- make vector throw error on missing env vars
- make vector collect metrics
- make docker containers export metrics
- make vector dockfile run vector directly, no need for env substitution anymore due to expected env vars in vector config
- make caddy expose http metrics on all interfaces under a given port, adjust vector to match
- docker in dockerfile
- tls off for caddy metrics
- use http 1 for caddy metrics, no ssl
- disable https
- tls off
- use http directly
- http 1 only
- https redir
- typo
- use redis password in vector connection str
- fix env var name
- err if env var missing
- vector depend on redis
- typo
- use redis-exporter
- use internal network for redis exporter
- vector binds
- remove binds
- use prom scrape
- use prom scrape endpoints - plural
- make vector log in json format
- add mongo, vector and redis logs
- fix vrl
- fix string parse in vrl
- fix parsing error when .message isn't defined in a log
- use mongo exporter container
- use env file
- pin version
- make mongo-exporter use env vars from the env file
- make vector config non-fail on string parse
- escape
- remove command
- fix vector warnings
- default to empty message in vector vrl
- command
- disable pronode1
- healthcheck fix
- fix mongo exporter version
- use default entrypoint for mongo-exporter
- test mongodburi
- typo
- revert back to vector based collection of mongo metrics
- add capabilities to vector
- shift tcp metrics to cadvisor instead of vector
- remove vector host matrics
- typo
- revert to host metrics
- move docker compose env to file
- lower scape interval
- bump vector img ver
- re-enable provider1
- use message_str representation if not logging json
- oo configs
…n anymore due to expected env vars in vector config
…adjust vector to match
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
forgetso
previously approved these changes
Aug 26, 2025
c3bc3c5 to
b9f10d3
Compare
1fa7d19 to
a7ec1d3
Compare
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.