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

Allow PATCH in an API CORS setup #11700

Merged

Conversation

fabianofranz
Copy link
Member

Allows the PATCH method to be used in a REST API CORS setup.

Upstream: kubernetes/kubernetes#35978

@jwforres

@fabianofranz
Copy link
Member Author

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 531f408

@jwforres
Copy link
Member

jwforres commented Nov 1, 2016

@openshift/api-review any issues with this change?

@deads2k
Copy link
Contributor

deads2k commented Nov 1, 2016

@openshift/api-review any issues with this change?

Looks fine.

@smarterclayton
Copy link
Contributor

Same

@deads2k
Copy link
Contributor

deads2k commented Nov 1, 2016

[merge]

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/10930/) (Base Commit: 3b97ee7)

@fabianofranz
Copy link
Member Author

flaked on #10773 re[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 531f408

@openshift-bot
Copy link
Contributor

openshift-bot commented Nov 3, 2016

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/11093/) (Base Commit: da8083a) (Image: devenv-rhel7_5310)

@openshift-bot openshift-bot merged commit c5693d3 into openshift:master Nov 3, 2016
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.

5 participants