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.

Sign up for GitHub

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

Jump to bottom

Add ITRS Geneos as vendor #2473

Merged
merged 1 commit into from
Mar 27, 2023
Merged

Add ITRS Geneos as vendor #2473

merged 1 commit into from
Mar 27, 2023

Conversation

jbarossitrs
Copy link
Contributor

ITRS provides an Open Telemetry plugin that works with our Geneos product, see https://devdocs.itrsgroup.com/docs/geneos/data-collection/opentelemetry/current/opentelemetry.html

@jbarossitrs jbarossitrs requested a review from a team March 9, 2023 15:18
@jbarossitrs
Copy link
Contributor Author

I'm not sure if the conversation in #2207 applies to ITRS, please let me know if there's anything more you need to know.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Mar 15, 2023 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

@svrnm
Copy link
Member

svrnm commented Mar 15, 2023

@jbarossitrs please make sure that you sign the CLA

@jbarossitrs
Copy link
Contributor Author

@svrnm I'm just waiting to hear from our legal department to confirm who our CLA Manager is and then I will sign the CLA.

@jbarossitrs
Copy link
Contributor Author

I now have internal confirmation but have encountered a problem with easyCLA so waiting for a ticket with lxf to resolve https://jira.linuxfoundation.org/plugins/servlet/desk/portal/4/SUPPORT-16896

@jbarossitrs
Copy link
Contributor Author

@svrnm should all be resolved now I think

@svrnm svrnm added the vendor-list PR that adds to the vendor list label Mar 21, 2023
@cartermp
Copy link
Contributor

/fix:format

chalin
chalin reviewed Mar 27, 2023
View reviewed changes
Copy link
Contributor

@chalin chalin left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was going to offer to fix the formatting and refcache, but the PR has conflicts because the Vednors page was updated since this PR was created.

@svrnm @cartermp - I think we should switch to a data-driven approach for the vendors table so that we can avoid formatting issue, either as the part of #2178 or another issue. WDYT?

@chalin
Copy link
Contributor

chalin commented Mar 27, 2023

I thought that I might not be able to update the branch, but somewhat surprisingly I was.
Rebase done and checks fixed! 😅

@cartermp
Copy link
Contributor

@chalin agreed.

@cartermp cartermp merged commit ab25b75 into open-telemetry:main Mar 27, 2023
@chalin chalin mentioned this pull request Mar 27, 2023
Closed
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chalin chalin chalin left review comments

@cartermp cartermp cartermp approved these changes

Assignees
No one assigned
Labels
vendor-list PR that adds to the vendor list
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jbarossitrs @svrnm @cartermp @chalin