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

Live debugging front-end #892

Closed
wants to merge 69 commits into from
Closed

Live debugging front-end #892

wants to merge 69 commits into from

Conversation

wildum
Copy link
Contributor

@wildum wildum commented May 21, 2024

THIS PR WILL BE MERGED TO A FEATURE BRANCH WHICH WILL RECEIVE A FEW OTHER PRs BEFORE BEING READY TO BE MERGED ON MAIN

This PR is the second part of the live debugging functionality.

It adds a live debugging page in the front-end:

Screen.Recording.2024-05-28.at.12.10.51.mov

The live debugging uses a 5000 lines buffer to display the feed. Once the buffer is full, old lines are removed.
The user:

  • can filter the buffer
  • use a sample probability to reduce the throughput
  • copy the content of the buffer to the clipboard
  • pause/resume/clear the page
  • enable/disable auto-scrolling

It supports modules.

ptodev and others added 30 commits May 8, 2024 12:19
Users looking at the Dockerfile may not understand that non-root support
is both undocumented and experimental. This commit makes that status
explicit with a comment.
* Upgrade Go to 1.22.3

* Sign .drone.yml
* Update build image to v0.1.1

* Update changelog

* Update changelog

* Update docker image bases and regenerate .drone.yml
* Attempt to restart docker target for loki source at a fixed interval to support containers restart

* add test

* add missing target cleanup

* improve test readability

* rename config option
* feat(helm): Refactoring Kubernetes recommended labels

Signed-off-by: Nicolas Lamirault <[email protected]>

* add changelog entry

Signed-off-by: Nicolas Lamirault <[email protected]>

* bump Helm chart version

Signed-off-by: Nicolas Lamirault <[email protected]>

* move Helm change entry to Helm chart's CHANGELOG.md

* add Unreleased section to Helm chart CHANGELOG.md

* `make generate-helm-tests generate-helm-docs`

---------

Signed-off-by: Nicolas Lamirault <[email protected]>
Co-authored-by: Robert Fratto <[email protected]>
* git checkin changes for crypto

* Check in changes to support windows cngcrypto

* Disable cngcrypto containers

* Fix changelog message.

* Remove windows messages

* Remove generate-winmanifest

* Remove more windows specific code

* rollback to how it was before

* rollback to how it was before

* Update CHANGELOG.md

Co-authored-by: Robert Fratto <[email protected]>

* Remove reference to CNGCrypto

---------

Co-authored-by: Robert Fratto <[email protected]>
Add an otelcol.receiver.file_stats component which wraps around the
upstream filestatsreceiver component from the -contrib distribution.

Because of an upstream bug, this component does not currently work 
on FreeBSD.

Closes #238.
actions/create-github-app-token by default only generates a token for
the current repository. We need a token for grafana/helm-charts.
* ci: use app token for pushing helm-chart tags

* ci: add alloy to repository list when generating a Helm release token
* Refactor out DistributedTargets and clean up.

* Add MovedAway method to DistributedTargets and rename existing methods.

* distributed_targets.go tests and benchmarks and refactor

* tests and refactor

* fix race condition in pyroscope test

* workaround for data race in prometheus

* changelog

* fix changelog

* feedback

* use a better named prom fork branch
…config` (#748)

* alloy-remote-config instead of agent-remote-config dependency

Signed-off-by: erikbaranowski <[email protected]>

* clean up go.mod

Signed-off-by: erikbaranowski <[email protected]>

---------

Signed-off-by: erikbaranowski <[email protected]>
* Fail if we see the port is not available

* Update changelog

* cleanup message

* Update CHANGELOG.md

Co-authored-by: Erik Baranowski <[email protected]>

---------

Co-authored-by: Erik Baranowski <[email protected]>
* Update Loki dependency.

* Clean up metrics generated from logs after a config reload.

Synced from:
grafana/loki#12938
* expose stability level to components

* update comment to clarify how the minStability can be used
* docs(prometheus/scrape): add example for K8s auth

Signed-off-by: hainenber <[email protected]>

* Apply suggestions from Clayton's code review

Co-authored-by: Clayton Cornell <[email protected]>

* Apply suggestions from 2nd Clayton's code review

Co-authored-by: Clayton Cornell <[email protected]>

---------

Signed-off-by: hainenber <[email protected]>
Co-authored-by: Clayton Cornell <[email protected]>
…#837)

* Add info to step 4 about order of arguments

* Update advice about arguments
…835)

* copy code from opentelemetry v0.99.0 for the prometheus receiver

* enable native histograms

* add integration test

* changelog

* enable native histogram support to otelcol.receiver.pormetheus for public preview

* integration tests use experimental feature gate

* update doc with commit SHA
Bumps [aquasecurity/trivy-action](https://github.com/aquasecurity/trivy-action) from 0.19.0 to 0.20.0.
- [Release notes](https://github.com/aquasecurity/trivy-action/releases)
- [Commits](aquasecurity/trivy-action@d710430...b2933f5)

---
updated-dependencies:
- dependency-name: aquasecurity/trivy-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…prehensive `internal/runtime` (#867)

Signed-off-by: hainenber <[email protected]>
* Implement yaml_decode with thanks to @djcode for most of the work.

* Update docs/sources/reference/stdlib/yaml_decode.md

Co-authored-by: Clayton Cornell <[email protected]>

---------

Co-authored-by: Clayton Cornell <[email protected]>
@wildum wildum marked this pull request as draft May 27, 2024 14:48
@wildum wildum force-pushed the xray-front branch 2 times, most recently from deff697 to af98b0e Compare May 28, 2024 10:06
…components (#825)

* feat(otelcol): allow overriding debug metric level for all components except `auth.* due to cyclic imports

Signed-off-by: hainenber <[email protected]>

* refactor(otelcol): decouple debug_metrics block from otelcol package

Signed-off-by: hainenber <[email protected]>

* feat(otelcol/auth): allow overriding debug metric level for `otelcol.auth` components

Signed-off-by: hainenber <[email protected]>

* fix(otelcol): set correct test expectation for default debug metrics level

Signed-off-by: hainenber <[email protected]>

* fix(ci): add missing cfg conversion for debug metrics in `otelcolconvert`

Signed-off-by: hainenber <[email protected]>

* docs: add ref to `debug_metrics` block for all `otelcol.*` components

Signed-off-by: hainenber <[email protected]>

* Apply suggestions from Clayton's code review

Co-authored-by: Clayton Cornell <[email protected]>

* Apply suggestions from code review

Co-authored-by: Paulin Todev <[email protected]>
Co-authored-by: Clayton Cornell <[email protected]>

* fix(otelcol): return error for failed conversion of debug metric level

Signed-off-by: hainenber <[email protected]>

* doc(otelcol/debug-metrics): move possible values for `level` outside of arg table

Signed-off-by: hainenber <[email protected]>

* fix(otelcol): set default debug metric config for all missing OtelCol components

Signed-off-by: hainenber <[email protected]>

* doc: display possible values for debug metrics level with double quotation

Signed-off-by: hainenber <[email protected]>

* doc: reduce padding space for `debug_metrics` table entry

Signed-off-by: hainenber <[email protected]>

---------

Signed-off-by: hainenber <[email protected]>
Co-authored-by: Clayton Cornell <[email protected]>
Co-authored-by: Paulin Todev <[email protected]>
StefanKurek and others added 17 commits May 28, 2024 15:26
* Updates OTEL version to v0.101.0.

Updates vcenterreceiver and its many bug fixes and updates for use by Alloy.
Updates probabilisticsampleprocessor to use new failed_closed config.

* Update CHANGELOG.md

Co-authored-by: Clayton Cornell <[email protected]>

* updates default for FailClosed to true for probabilistic_samplier processor

* fixed probabilistic_sampler converter test

* fixes probabilistic_sampler unit tests expecting failures when invalid sample conditions

* Fix CHANGELOG.md

---------

Co-authored-by: Clayton Cornell <[email protected]>
#808 introduced dependencies for the mixin, which
accidentally broke the example environment; as the mixin originally
didn't have dependencies, the example environment didn't install them
before provisioning dashboards.

This commit runs jb to locally install dependencies before provisioning
dashboards.
Include /loki/ at loki.write endpoint
…#939)

* updates with latest snowflake prometheus exporter (fixes null issues)

* Update CHANGELOG.md

Co-authored-by: William Dumont <[email protected]>

---------

Co-authored-by: William Dumont <[email protected]>
…perience (#943)

* feat(vcs): bubble up SSH key conversion error for better debugging experience

Signed-off-by: hainenber <[email protected]>

* chore: refactor code to be more succinct

Signed-off-by: hainenber <[email protected]>

---------

Signed-off-by: hainenber <[email protected]>
This includes all bugfixes found in main to date except for #703, which
is a more involved change that should probably wait for a minor release.
* bump node version to 18 in build image

* update comment
@wildum wildum closed this Jun 3, 2024
@wildum wildum mentioned this pull request Jun 3, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport-to-agent:no PR should NOT be backported to the agent repo. frozen-due-to-age
Projects
None yet
Development

Successfully merging this pull request may close these issues.