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

KAFKA-15051: add missing GET plugin/config endpoint #13803

Merged
merged 2 commits into from
Jun 7, 2023

Conversation

jeqo
Copy link
Contributor

@jeqo jeqo commented Jun 2, 2023

No description provided.

docs/connect.html Outdated Show resolved Hide resolved
@vvcephei
Copy link
Contributor

vvcephei commented Jun 6, 2023

test this please

@C0urante
Copy link
Contributor

C0urante commented Jun 6, 2023

@vvcephei I kicked off another Jenkins build--I haven't seen GitHub comments working for a while to re-trigger them. Are you trying to test that out?

@vvcephei
Copy link
Contributor

vvcephei commented Jun 6, 2023

Yeah, sorry for the weird drive-bys. I was indeed trying to see if those commands still work. Apparently, they do not.

I think this is the plugin documentation, but it also looks like it was associated with security flaws, so maybe Infra disabled it: https://plugins.jenkins.io/ghprb/

I'll raise a ticket with them. At the least, if it doesn't work anymore, they should remove the docs for that role in asf.yaml.

Co-authored-by: Chris Egerton <[email protected]>
Copy link
Contributor

@C0urante C0urante left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @jeqo!

@C0urante C0urante merged commit 9cfc4b9 into apache:trunk Jun 7, 2023
1 check failed
@C0urante
Copy link
Contributor

C0urante commented Jun 7, 2023

@vvcephei haha, no worries! I'll leave Jenkins alone if I see you in the neighborhood testing again 🙏

@jeqo jeqo deleted the docs-add-connect-plugin-get-config branch June 8, 2023 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants