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

feat: add BuildSpnFunc to GSSAPIConfig for allow custom spn #2807

Merged
merged 1 commit into from Feb 22, 2024

Conversation

fooofei
Copy link
Contributor

@fooofei fooofei commented Feb 19, 2024

what this pr is

The new pr is a custom config enhancement that allows the creation of custom spn build.

After allow custom spn build, this also will satisfy #2437

how to implement

add a function type type BuildSpnFunc func(serviceName, host string) string

add a field BuildSpn BuildSpnFunc to type GSSAPIConfig struct

then user can set a custom BuildSpn

tests result

?       github.com/IBM/sarama/internal/toxiproxy        [no test files]
?       github.com/IBM/sarama/tools/kafka-console-consumer      [no test files]
?       github.com/IBM/sarama/tools/kafka-console-partitionconsumer     [no test files]
?       github.com/IBM/sarama/tools/kafka-console-producer      [no test files]
?       github.com/IBM/sarama/tools/kafka-producer-performance  [no test files]
?       github.com/IBM/sarama/tools/tls [no test files]
ok      github.com/IBM/sarama   31.257s
ok      github.com/IBM/sarama/mocks     (cached)

Signed-off-by: fooofei <aihujianfei@qq.com>
@fooofei
Copy link
Contributor Author

fooofei commented Feb 19, 2024

hi @dnwe @hindessm can you accept this while not accept #2773

@fooofei
Copy link
Contributor Author

fooofei commented Feb 21, 2024

hi @dnwe @hindessm

@dnwe
Copy link
Collaborator

dnwe commented Feb 21, 2024

@fooofei thanks for the PRs, we have seen them and don’t worry we will get around to reviewing both, it isn’t necessary to keep notifying us. If you need to use the capability right away you can point your app code at your branch

@fooofei
Copy link
Contributor Author

fooofei commented Feb 22, 2024

@dnwe
sorry about keep notifying .
i am working at a company, the company used the main repo code, not allow my forked private repo code, so I respect the pull request merged to main repo and pulish a new version.
I have 40 days to reach that goal.

Have passed 30 days after first pr #2773.

I have left about 10 days, so I have another pr #2807 to reach that goal

@dnwe dnwe added the feat label Feb 22, 2024
Copy link
Collaborator

@dnwe dnwe left a comment

Choose a reason for hiding this comment

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

Thanks for working on these changes, I had a chat with @hindessm earlier today and we agreed that this more-targetted PR makes the most sense to go forward with

@dnwe dnwe merged commit 38f2d15 into IBM:main Feb 22, 2024
13 checks passed
@dnwe
Copy link
Collaborator

dnwe commented Feb 22, 2024

Released in v1.43.0

dmitryax pushed a commit to open-telemetry/opentelemetry-collector-contrib that referenced this pull request Feb 27, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [github.com/IBM/sarama](https://togithub.com/IBM/sarama) | `v1.42.2`
-> `v1.43.0` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fIBM%2fsarama/v1.43.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fIBM%2fsarama/v1.43.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fIBM%2fsarama/v1.42.2/v1.43.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fIBM%2fsarama/v1.42.2/v1.43.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>IBM/sarama (github.com/IBM/sarama)</summary>

### [`v1.43.0`](https://togithub.com/IBM/sarama/releases/tag/v1.43.0):
Version 1.43.0 (2024-02-22)

[Compare
Source](https://togithub.com/IBM/sarama/compare/v1.42.2...v1.43.0)

<!-- Release notes generated using configuration in .github/release.yaml
at main -->

> \[!NOTE]\
> The go.mod directive has been bumped to 1.19 as the minimum version of
Go required for the module. This was necessary to continue to receive
updates from some of the third party dependencies that Sarama makes use
of for compression.

#### What's Changed

##### 🎉 New Features / Improvements

- feat: update go directive to 1.19 by
[@&#8203;dnwe](https://togithub.com/dnwe) in
[IBM/sarama#2795
- feat: add BuildSpnFunc to GSSAPIConfig for allow custom spn by
[@&#8203;fooofei](https://togithub.com/fooofei) in
[IBM/sarama#2807

##### 🐛 Fixes

- Use %v formatting words and remove unnecessary newline by
[@&#8203;puellanivis](https://togithub.com/puellanivis) in
[IBM/sarama#2802

##### 📦 Dependency updates

- chore(deps): bump github.com/klauspost/compress from 1.16.7 to 1.17.6
by [@&#8203;dependabot](https://togithub.com/dependabot) in
[IBM/sarama#2784
- chore(deps): bump github.com/eapache/go-resiliency from 1.5.0 to 1.6.0
by [@&#8203;dependabot](https://togithub.com/dependabot) in
[IBM/sarama#2810
- chore(deps): bump github.com/klauspost/compress from 1.17.6 to 1.17.7
by [@&#8203;dependabot](https://togithub.com/dependabot) in
[IBM/sarama#2811

##### 🔧 Maintenance

- chore(doc): add v1.42.2 to CHANGELOG.md by
[@&#8203;dnwe](https://togithub.com/dnwe) in
[IBM/sarama#2796

#### New Contributors

- [@&#8203;puellanivis](https://togithub.com/puellanivis) made their
first contribution in
[IBM/sarama#2802
- [@&#8203;fooofei](https://togithub.com/fooofei) made their first
contribution in
[IBM/sarama#2807

**Full Changelog**:
IBM/sarama@v1.42.2...v1.43.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any
time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yMTIuMCIsInVwZGF0ZWRJblZlciI6IjM3LjIxMi4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
XinRanZhAWS pushed a commit to XinRanZhAWS/opentelemetry-collector-contrib that referenced this pull request Mar 13, 2024
…etry#31427)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [github.com/IBM/sarama](https://togithub.com/IBM/sarama) | `v1.42.2`
-> `v1.43.0` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fIBM%2fsarama/v1.43.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fIBM%2fsarama/v1.43.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fIBM%2fsarama/v1.42.2/v1.43.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fIBM%2fsarama/v1.42.2/v1.43.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>IBM/sarama (github.com/IBM/sarama)</summary>

### [`v1.43.0`](https://togithub.com/IBM/sarama/releases/tag/v1.43.0):
Version 1.43.0 (2024-02-22)

[Compare
Source](https://togithub.com/IBM/sarama/compare/v1.42.2...v1.43.0)

<!-- Release notes generated using configuration in .github/release.yaml
at main -->

> \[!NOTE]\
> The go.mod directive has been bumped to 1.19 as the minimum version of
Go required for the module. This was necessary to continue to receive
updates from some of the third party dependencies that Sarama makes use
of for compression.

#### What's Changed

##### 🎉 New Features / Improvements

- feat: update go directive to 1.19 by
[@&open-telemetry#8203;dnwe](https://togithub.com/dnwe) in
[IBM/sarama#2795
- feat: add BuildSpnFunc to GSSAPIConfig for allow custom spn by
[@&open-telemetry#8203;fooofei](https://togithub.com/fooofei) in
[IBM/sarama#2807

##### 🐛 Fixes

- Use %v formatting words and remove unnecessary newline by
[@&open-telemetry#8203;puellanivis](https://togithub.com/puellanivis) in
[IBM/sarama#2802

##### 📦 Dependency updates

- chore(deps): bump github.com/klauspost/compress from 1.16.7 to 1.17.6
by [@&open-telemetry#8203;dependabot](https://togithub.com/dependabot) in
[IBM/sarama#2784
- chore(deps): bump github.com/eapache/go-resiliency from 1.5.0 to 1.6.0
by [@&open-telemetry#8203;dependabot](https://togithub.com/dependabot) in
[IBM/sarama#2810
- chore(deps): bump github.com/klauspost/compress from 1.17.6 to 1.17.7
by [@&open-telemetry#8203;dependabot](https://togithub.com/dependabot) in
[IBM/sarama#2811

##### 🔧 Maintenance

- chore(doc): add v1.42.2 to CHANGELOG.md by
[@&open-telemetry#8203;dnwe](https://togithub.com/dnwe) in
[IBM/sarama#2796

#### New Contributors

- [@&open-telemetry#8203;puellanivis](https://togithub.com/puellanivis) made their
first contribution in
[IBM/sarama#2802
- [@&open-telemetry#8203;fooofei](https://togithub.com/fooofei) made their first
contribution in
[IBM/sarama#2807

**Full Changelog**:
IBM/sarama@v1.42.2...v1.43.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any
time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yMTIuMCIsInVwZGF0ZWRJblZlciI6IjM3LjIxMi4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants