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

feat: adjust event collection configuration #2407

Merged
merged 8 commits into from
Jul 6, 2022

Commits on Jul 5, 2022

  1. Configuration menu
    Copy the full SHA
    94a11ac View commit details
    Browse the repository at this point in the history

Commits on Jul 6, 2022

  1. feat!: use sumologic.events.enabled property

    This replaces the `otelevents.enabled` property.
    With this change, every property does exactly one thing:
    - `sumologic.events.enabled` enables or disables events collection
    - `sumologic.events.provider` switches between Fluentd and Otelcol
    We also still support the (now deprecated) `fluentd.events.enabled`
    property to disable events collection with Fluentd.
    andrzej-stencel committed Jul 6, 2022
    Configuration menu
    Copy the full SHA
    089ba42 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    a191e83 View commit details
    Browse the repository at this point in the history
  3. test: fix terraform tests

    The tests were using the undocumented and nonexisting `sumologic.events.enabled` property
    in a way that collided with the changes to events configuration.
    andrzej-stencel committed Jul 6, 2022
    Configuration menu
    Copy the full SHA
    d46e4d7 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    3875712 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    7653ace View commit details
    Browse the repository at this point in the history
  6. test(integration): fix otelcol logs test

    Changing the number of expected endpoints created from 2 to 1,
    because the events endpoint is no longer being created
    when events are disabled.
    andrzej-stencel committed Jul 6, 2022
    Configuration menu
    Copy the full SHA
    c17e16d View commit details
    Browse the repository at this point in the history
  7. test(integration): make tests more verbose

    It took me a good while to find the cause of the failures.
    Hopefully the additional comments will make it easier.
    andrzej-stencel committed Jul 6, 2022
    Configuration menu
    Copy the full SHA
    9074623 View commit details
    Browse the repository at this point in the history