Skip to content

Whitelisting URLs for installation of the OTEL Collector on Windows and Linux #5255

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

amee-sumo
Copy link
Collaborator

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

Purpose of this pull request

Create a section for the list of repository URLs to be whitelisted for a smooth installation of the OTEL Collector on Windows and Linux.

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-756

@amee-sumo amee-sumo added doc:update Revisions and updates to content do-not-merge Pull requests that should not be merged labels Apr 8, 2025
@amee-sumo amee-sumo self-assigned this Apr 8, 2025
@amee-sumo amee-sumo requested review from kimsauce, jpipkin1, mafsumo, JV0812 and a team as code owners April 8, 2025 09:56
@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Apr 8, 2025
@amee-sumo amee-sumo marked this pull request as draft April 8, 2025 09:56
Comment on lines +38 to +39
Whitelist the following repository URLs to allow a seamless installation:

Copy link
Collaborator

Choose a reason for hiding this comment

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

We use "allowlist" and "denylist" instead of "whitelist" and "blacklist". https://help.sumologic.com/docs/contributing/style-guide/#cultural-neutrality

Suggested change
Whitelist the following repository URLs to allow a seamless installation:
Allowlist the following repository URLs to allow a seamless installation:

Comment on lines +34 to +35
Whitelist the following repository URLs to allow a seamless installation:

Copy link
Collaborator

Choose a reason for hiding this comment

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

We use "allowlist" and "denylist" instead of "whitelist" and "blacklist". https://help.sumologic.com/docs/contributing/style-guide/#cultural-neutrality

Suggested change
Whitelist the following repository URLs to allow a seamless installation:
Allowlist the following repository URLs to allow a seamless installation:

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.

one minor fix!

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:update Revisions and updates to content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants