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

Kubeflow ACLs sync is failing with 422 error #747

Open
andreyvelich opened this issue Feb 10, 2025 · 3 comments
Open

Kubeflow ACLs sync is failing with 422 error #747

andreyvelich opened this issue Feb 10, 2025 · 3 comments

Comments

@andreyvelich
Copy link
Member

Our regular ACLs sync is failing:

{"component":"peribolos","error":"status code 422 not one of [200], body: {\"message\":\"The request could not be processed.\",\"documentation_url\":\"https://docs.github.com/rest/orgs/members#set-organization-membership-for-a-user\",\"status\":\"422\"}","file":"k8s.io/test-infra/prow/cmd/peribolos/main.go:477","func":"main.configureOrgMembers.func1","level":"warning","msg":"UpdateOrgMembership(kubeflow, prodonjs, false) failed","severity":"warning","time":"2025-02-10T20:30:42Z"}

{"component":"peribolos","file":"k8s.io/test-infra/prow/cmd/peribolos/main.go:194","func":"main.main","level":"fatal","msg":"Configuration failed: failed to configure kubeflow members: status code 422 not one of [200], body: {\"message\":\"The request could not be processed.\",\"documentation_url\":\"https://docs.github.com/rest/orgs/members#set-organization-membership-for-a-user\",\"status\":\"422\"}","severity":"fatal","time":"2025-02-10T20:30:42Z"}

However, this GitHub handler still exists (@prodonjs) https://github.com/prodonjs

Any thoughts on this error @petr-muller @matthyx @droslean @krzyzacy @smg247 @cblecker @cjwagner @MadhavJivrajani @nikhita @Priyankasaggu11929 ?

@jlewi Did you see similar errors before with peribolos other than 404: kubernetes/test-infra#14321 ?

cc @kubeflow/kubeflow-steering-committee @franciscojavierarceo @juliusvonkohout

@jlewi
Copy link
Contributor

jlewi commented Feb 10, 2025

I don't recall any problems.

@petr-muller
Copy link

petr-muller commented Feb 11, 2025

Hrm, the GH API message is not really helpful for that 422. I recall we had some issues with inviting people who removed themselves from the org previously (I guess as an anti-harassment protection in GH) but I dug and the symptom was different back then (GH responded with 403, not 422):

kubernetes/test-infra#22723

But back then GH was apparently showing a better message if you used its UI, so trying to add them manually would be the thing I would try.

@cjwagner
Copy link
Member

I don't recall seeing this error. Petr's advice sounds good though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants