Skip to content

Merge dev.servicegraphs into main - #12

Merged
rfratto merged 6 commits into
mainfrom
dev.servicegraphs
Oct 14, 2021
Merged

Merge dev.servicegraphs into main#12
rfratto merged 6 commits into
mainfrom
dev.servicegraphs

Conversation

@rfratto

@rfratto rfratto commented Oct 14, 2021

Copy link
Copy Markdown
Owner

No description provided.

mapno and others added 6 commits September 2, 2021 18:12
* Bare implementation of the service graph processor

* Comment fixes

* Collect unpaired spans metric

* Add unpaired metric

* Implementation improvements

* Some more improvements

* No need to close

* Fix CI

* Improve test stability

* Some documentation

* Fix tests

* Truly fix it
…rage#891)

* Move port to load_balancing config as receiver_port

When extracting load balancing from tail sampling,
only the the load_balancing block was moved into its own block,
but the receiver's port was left in tail_sampling.

This means that the receiver port for load_balancing could not
be configured without tail_sampling.

Now its moved to load_balancing as receiver_port.

* Load balance without tail sampling

* Move port up
* Check span's error

* Also check http status code

* Update tests

* Inspect http and grpc status too

* Use map for performance

* Cleanup
@rfratto
rfratto merged commit d39ef04 into main Oct 14, 2021
@rfratto
rfratto deleted the dev.servicegraphs branch March 14, 2022 18:20
rfratto pushed a commit that referenced this pull request Jun 28, 2022
Promtail has had a longstanding issue with relative symlinks (see
grafana/loki/grafana-cold-storage#1240 , grafana/loki/grafana-cold-storage#4905 , and grafana/tail/#11 ). This was fixed
in grafana/tail/#12 but was never pulled into agent. In this commit we update
grafana/tail to pull in this fix.

This commit was created by updating the commit hash for tail and then running
`go mod tidy`.
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