Skip to content
This repository has been archived by the owner on Aug 29, 2018. It is now read-only.

Conversation

Miciah
Copy link
Contributor

@Miciah Miciah commented Dec 16, 2016

F5IControlRestLoadBalancerModel#add_pool_member: Specify the partition of the pool member.

Sometimes the F5 iControl REST API returns a bogus HTTP 404 response, even if the pool member is added successfully, if the partition is not specified. See https://devcentral.f5.com/questions/icontrol-rest-404-despite-success-when-adding-pool-member.

This commit is related to bug 1227472.

https://bugzilla.redhat.com/show_bug.cgi?id=1227472

F5IControlRestLoadBalancerModel#add_pool_member: Specify the partition of
the pool member.

Sometimes the F5 iControl REST API returns a bogus HTTP 404 response, even
if the pool member is added successfully, if the partition is not
specified. See
<https://devcentral.f5.com/questions/icontrol-rest-404-despite-success-when-adding-pool-member>.

This commit is related to bug 1227472.

https://bugzilla.redhat.com/show_bug.cgi?id=1227472
@Miciah Miciah force-pushed the bug-1227472-routing-daemon-f5-specify-partition-for-pool-members branch from 83d5477 to c48bd6c Compare December 16, 2016 09:23
@thrasher-redhat
Copy link
Contributor

LGTM

@tiwillia
Copy link
Member

LGTM 🍦

@Miciah
Copy link
Contributor Author

Miciah commented Dec 16, 2016

Thanks! openshift-bot, please [merge]!

@openshift-bot
Copy link

Evaluated for online merge up to c48bd6c

@openshift-bot
Copy link

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link

Evaluated for online test up to c48bd6c

@openshift-bot
Copy link

Online Test Results: SUCCESS (https://ci.dev.openshift.redhat.com/jenkins/job/test_pull_requests/9448/) (Base Commit: 2fbcf34)

@openshift-bot
Copy link

openshift-bot commented Dec 17, 2016

Online Merge Results: SUCCESS (https://ci.dev.openshift.redhat.com/jenkins/job/test_pull_requests/9453/) (Base Commit: 692c819) (Image: devenv_5836)

@openshift-bot openshift-bot merged commit 694c7e8 into openshift:master Dec 17, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants