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

CLOUDSTACK-7986 [F5 LB] Failed to execute IPAssocCommand due to com.clou... #47

Closed
wants to merge 1 commit into from

Conversation

sudhansu7
Copy link

added 3 new method to strip partition information from VirtualServer, LBPool, VLAN api response.

With BigIP V11.x VirtualServer, LBPool, VLAN api response has been modified.
Now BigIP returns resource name with user partition information
ex: if vlanname is vlan-100 then the get_list() will return /Common/vlan-100 (/Common -> Suer portition)
This method will strip the partition information and only returns a list with vlan name (vlan-100)

…loud.utils.exception.ExecutionException: Exception caught in Networking::urn:iControl:Networking/VLAN::create()

added 3 new method to strip partition information from VirtualServer, LBPool, VLAN api response.

With BigIP V11.x VirtualServer, LBPool, VLAN api response has been modified.
Now BigIP returns resource  name with user partition information
ex: if vlanname is vlan-100 then the get_list() will return /Common/vlan-100 (/Common -> Suer portition)
This method will strip the partition information and only returns a list with vlan name (vlan-100)
@rohityadavcloud
Copy link
Member

Our travis does not run noredist builds, can you confirm any build jobs (from Jenkins etc.) so I can help merge it?

@karuturi
Copy link
Member

karuturi commented Dec 8, 2014

I tested this with a noredist build. The "stripped" functions work fine ;)
I will apply the patch.

@rohityadavcloud
Copy link
Member

Thanks Rajani, go for it!

@karuturi
Copy link
Member

karuturi commented Dec 8, 2014

missed adding a link to the pr in commit message
4.3: 7d17e95
4.4: c8882b3
4.5 and master: 0347b2f

@sudhansu7
Copy link
Author

Thanks Rajani.

@sudhansu7 sudhansu7 closed this Dec 9, 2014
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

Successfully merging this pull request may close these issues.

None yet

3 participants