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

[chore] move basicauth extension to generated lifecycle tests #30424

Merged
merged 5 commits into from
Feb 1, 2024

Conversation

atoulme
Copy link
Contributor

@atoulme atoulme commented Jan 11, 2024

Relates to #27849

I have removed code blocking the creation of the extension as Config.Validate does the same checks.

@atoulme atoulme requested a review from jpkrohling as a code owner January 11, 2024 07:29
@atoulme atoulme requested a review from a team January 11, 2024 07:29
@github-actions github-actions bot requested a review from frzifus January 11, 2024 07:30
@atoulme atoulme force-pushed the basicauth_lifecycle branch 2 times, most recently from b8e5467 to 262f848 Compare January 12, 2024 00:56
@dmitryax
Copy link
Member

Can you please split to keep the git history clean?

@atoulme
Copy link
Contributor Author

atoulme commented Jan 12, 2024

#30470 is open to split

@atoulme atoulme force-pushed the basicauth_lifecycle branch from cae2477 to 670034e Compare January 31, 2024 22:48
@TylerHelmuth TylerHelmuth merged commit b5d2e4c into open-telemetry:main Feb 1, 2024
95 checks passed
@github-actions github-actions bot added this to the next release milestone Feb 1, 2024
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this pull request Feb 1, 2024
…elemetry#30424)

Relates to
open-telemetry#27849

I have removed code blocking the creation of the extension as
Config.Validate does the same checks.

---------

Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
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.

6 participants