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.

Sign up for GitHub

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

Jump to bottom

[GOCA] fix reserve virtual network method #5663

Closed
treywelsh opened this issue Dec 6, 2021 · 0 comments
Closed

[GOCA] fix reserve virtual network method #5663

treywelsh opened this issue Dec 6, 2021 · 0 comments

Comments

@treywelsh
Copy link
Contributor

Description
The virtual network reserve method panic in case of an error:
panic: runtime error: invalid memory address or nil pointer dereference
It's due to a small mistake in the error management.

To Reproduce

vnID, err := controller.VirtualNetwork(XXX).Reserve("")

Expected behavior
Return the error
@treywelsh treywelsh changed the title [GOCA] fix [GOCA] fix reserve virtual network method Dec 6, 2021
treywelsh added a commit to treywelsh/one that referenced this issue Dec 6, 2021
@treywelsh
B OpenNebula#5663: GOCA - fix vnet reserve method
76746f7
treywelsh added a commit to treywelsh/one that referenced this issue Dec 6, 2021
@treywelsh
B OpenNebula#5663: GOCA - fix vnet reserve method
c4c83c6 
Signed-off-by: Pierre Lafievre <pierre.lafievre@iguanesolutions.com>
rsmontero pushed a commit that referenced this issue Dec 14, 2021
@treywelsh
B #5663: GOCA - fix vnet reserve method (#5664)
0b269ff 
Signed-off-by: Pierre Lafievre <pierre.lafievre@iguanesolutions.com>
rsmontero pushed a commit that referenced this issue Dec 14, 2021
@treywelsh @rsmontero
B #5663: GOCA - fix vnet reserve method (#5664)
b94dad5 
Signed-off-by: Pierre Lafievre <pierre.lafievre@iguanesolutions.com>
(cherry picked from commit 0b269ff)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@treywelsh