Skip to content

Removing OS Compute Hypervisors list request #12

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
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Removing OS Compute Hypervisors list request #12

wants to merge 4 commits into from

Conversation

mah88
Copy link

@mah88 mah88 commented Dec 18, 2017

No description provided.

gc4rella
gc4rella requested changes Dec 18, 2017
View reviewed changes
Copy link
Member

@gc4rella gc4rella left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no need to keep that method when the call is not executed. Maybe good to remove it completely.

mah88 added 3 commits December 18, 2017 17:52
@mah88
Update CoreModule.java
40ff7dc
@mah88
Removing ComputeNodeMap
9aa2d8d 
removing computeNodeMap since it depends on the OS compute hupervisors list request
@mah88
Update CoreModule.java
ad870e1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gc4rella gc4rella gc4rella requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mah88 @gc4rella