Skip to content

Conversation

@amee-sumo
Copy link
Collaborator

@amee-sumo amee-sumo commented Apr 3, 2025

Purpose of this pull request

This pull request is to create a new app doc for Sumo Collection.

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

https://sumologic.atlassian.net/browse/DOCS-779

@amee-sumo amee-sumo added doc:new New section or topic to content do-not-merge Pull requests that should not be merged labels Apr 3, 2025
@amee-sumo amee-sumo self-assigned this Apr 3, 2025
@amee-sumo amee-sumo requested review from a team, JV0812, jpipkin1, kimsauce and mafsumo as code owners April 3, 2025 14:38
@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Apr 3, 2025
Copy link
Collaborator

@kimsauce kimsauce left a comment

Choose a reason for hiding this comment

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

Hi @amee-sumo ,

@amee-sumo
Copy link
Collaborator Author

Hi @amee-sumo ,

Thank you @kimsauce for noting this crucial point. I'll connect with @shivani-sumo on this and update the docs as needed.

@shivani-sumo
Copy link
Collaborator

@kimsauce @amee-sumo,

The app is used to visualize data fetched from Sumo's API. And given information Installed and Hosted Collectors are supported is part of the corresponding C2C's configuration.

Hence, this app is the same as other security apps. We do not need to show anything extra.

Also, regarding health events.
The doc below shows a list of health events' API., which is not for app customers.
https://help.sumologic.com/docs/api/health-events/

And below, one is informative for the customer.
https://help.sumologic.com/docs/manage/health-events/

@amee-sumo amee-sumo requested a review from kimsauce April 4, 2025 13:10
@kimsauce
Copy link
Collaborator

kimsauce commented Apr 5, 2025

@kimsauce @amee-sumo,

The app is used to visualize data fetched from Sumo's API. And given information Installed and Hosted Collectors are supported is part of the corresponding C2C's configuration.

Hence, this app is the same as other security apps. We do not need to show anything extra.

Also, regarding health events. The doc below shows a list of health events' API., which is not for app customers. https://help.sumologic.com/docs/api/health-events/

And below, one is informative for the customer. https://help.sumologic.com/docs/manage/health-events/

Thanks, @shivani-sumo ! That helps clarify the internal implementation. Just to confirm, are we saying the source doc is only applicable to C2C environments? If it applies more broadly (e.g., any environment using Installed/Hosted Collectors), it may still make sense to move it up a level in the folder structure.

And while the app isn't a Health Events interface per se, would you be okay with adding a cross-link like "For related info on collector health events, see this doc." to help orient users and improve discoverability of related resources?

@shivani-sumo
Copy link
Collaborator

@kimsauce,

Just to confirm, are we saying the source doc is only applicable to C2C environments? If it applies more broadly (e.g., any environment using Installed/Hosted Collectors), it may still make sense to move it up a level in the folder structure.

The given source doc is only applicable to C2C environments.

And while the app isn't a Health Events interface per se, would you be okay with adding a cross-link like "For related info on collector health events, see this doc." to help orient users and improve discoverability of related resources?

Yes, we can add this health events-related reference doc.

@kimsauce
Copy link
Collaborator

kimsauce commented Apr 8, 2025

@kimsauce,

Just to confirm, are we saying the source doc is only applicable to C2C environments? If it applies more broadly (e.g., any environment using Installed/Hosted Collectors), it may still make sense to move it up a level in the folder structure.

The given source doc is only applicable to C2C environments.

And while the app isn't a Health Events interface per se, would you be okay with adding a cross-link like "For related info on collector health events, see this doc." to help orient users and improve discoverability of related resources?

Yes, we can add this health events-related reference doc.

Thanks for clarifying! Can you please see this suggested edit? #5250

@ankitgoelcmu ankitgoelcmu added this pull request to the merge queue Apr 21, 2025
Merged via the queue into main with commit 9364aa9 Apr 21, 2025
5 checks passed
@ankitgoelcmu ankitgoelcmu deleted the Sumo-Collection branch April 21, 2025 17:28
@JV0812 JV0812 mentioned this pull request Apr 22, 2025
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed Contributor approved, listed in .clabot file do-not-merge Pull requests that should not be merged doc:new New section or topic to content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants