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

[extension/filestorage] enable feature gate extension.filestorage.replaceUnsafeCharacters #30755

Merged

Conversation

andrzej-stencel
Copy link
Member

@andrzej-stencel andrzej-stencel commented Jan 24, 2024

Description:

Moves the feature gate extension.filestorage.replaceUnsafeCharacters to Beta stage, which means it is now enabled by default. This is according to the schedule for this feature gate described in the File Storage extension's README.

Link to tracking Issue:

Testing:

Updated tests to not require enabling of the feature gate.

Documentation:

Updated version in which feature gate was moved to Beta.

@andrzej-stencel andrzej-stencel force-pushed the file-storage-feature-beta branch from d6a16e6 to 293725c Compare January 24, 2024 16:37
@djaglowski djaglowski merged commit d316a95 into open-telemetry:main Jan 24, 2024
84 of 85 checks passed
@github-actions github-actions bot added this to the next release milestone Jan 24, 2024
@andrzej-stencel andrzej-stencel deleted the file-storage-feature-beta branch January 24, 2024 17:57
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Feb 1, 2024
…placeUnsafeCharacters` (open-telemetry#30755)

**Description:**

Moves the feature gate `extension.filestorage.replaceUnsafeCharacters`
to Beta stage, which means it is now enabled by default. This is
according to the schedule for this feature gate described in the File
Storage extension's
[README](https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/v0.92.0/extension/storage/filestorage/README.md#extensionfilestoragereplaceunsafecharacters).

**Link to tracking Issue:**

-
open-telemetry#3148

**Testing:**

Updated tests to not require enabling of the feature gate.

**Documentation:**

Updated version in which feature gate was moved to Beta.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants