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(sumologicextension): add implementation of Sumo Logic Extension #31031

Merged
merged 10 commits into from
Feb 28, 2024

Conversation

sumo-drosiek
Copy link
Member

@sumo-drosiek sumo-drosiek commented Feb 5, 2024

Description:

Add functionality based on Sumo Logic Repo code

Link to tracking Issue:

#29601

Testing:

  • Unit tests
  • manual tests
  • in use by customers for long time already

Documentation:

  • code comments
  • README.md

@sumo-drosiek sumo-drosiek changed the title feat(sumologicexporter): add implementation of Sumo Logic Exporter feat(sumologicextension): add implementation of Sumo Logic Extension Feb 5, 2024
@sumo-drosiek sumo-drosiek force-pushed the drosiek-sumologicextension branch 3 times, most recently from 45c401a to b43337f Compare February 5, 2024 20:28
@sumo-drosiek sumo-drosiek marked this pull request as ready for review February 6, 2024 07:53
@sumo-drosiek sumo-drosiek requested a review from a team February 6, 2024 07:53
@andrzej-stencel
Copy link
Member

andrzej-stencel commented Feb 6, 2024

Oh, probably need to add metadata.yaml.

EDIT: Scratch that, the file already exists in the repo 🤦

@sumo-drosiek
Copy link
Member Author

cc: @swiatekm-sumo @dashpole for second pair of eyes

extension/sumologicextension/README.md Outdated Show resolved Hide resolved
@andrzej-stencel andrzej-stencel added the ready to merge Code review completed; ready to merge by maintainers label Feb 16, 2024
@sumo-drosiek
Copy link
Member Author

sumo-drosiek commented Feb 16, 2024

cc: @open-telemetry/collector-contrib-maintainer

I didn't touch the failing receiver test 🙈

@sumo-drosiek sumo-drosiek force-pushed the drosiek-sumologicextension branch 3 times, most recently from 1659c6f to c8435a2 Compare February 19, 2024 11:52
Dominik Rosiek added 3 commits February 19, 2024 12:54
Signed-off-by: Dominik Rosiek <[email protected]>
Signed-off-by: Dominik Rosiek <[email protected]>
@sumo-drosiek
Copy link
Member Author

I rebased changes on the top of main branch

@atoulme
Copy link
Contributor

atoulme commented Feb 26, 2024

Please resolve conflicts

@atoulme atoulme removed the ready to merge Code review completed; ready to merge by maintainers label Feb 26, 2024
@sumo-drosiek
Copy link
Member Author

cc: @open-telemetry/collector-contrib-maintainer

@andrzej-stencel andrzej-stencel added the ready to merge Code review completed; ready to merge by maintainers label Feb 27, 2024
@dmitryax
Copy link
Member

@sumo-drosiek sorry for the delay. This needs another rebase

@dmitryax dmitryax merged commit c7fbf38 into open-telemetry:main Feb 28, 2024
142 checks passed
@github-actions github-actions bot added this to the next release milestone Feb 28, 2024
mx-psi added a commit to mx-psi/opentelemetry-collector-contrib that referenced this pull request Mar 6, 2024
XinRanZhAWS pushed a commit to XinRanZhAWS/opentelemetry-collector-contrib that referenced this pull request Mar 13, 2024
…pen-telemetry#31031)

**Description:**

Add functionality based on Sumo Logic Repo code

**Link to tracking Issue:**

open-telemetry#29601

**Testing:**

* Unit tests
* manual tests
* in use by customers for long time already

**Documentation:**

* code comments
* README.md

---------

Signed-off-by: Dominik Rosiek <[email protected]>
Co-authored-by: Andrzej Stencel <[email protected]>
Co-authored-by: Mikołaj Świątek <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extension/sumologic ready to merge Code review completed; ready to merge by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants