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

Why API300::Synergy::LogicalSwitchGroup ? #134

Closed
tmiotto opened this issue Jan 20, 2017 · 3 comments
Closed

Why API300::Synergy::LogicalSwitchGroup ? #134

tmiotto opened this issue Jan 20, 2017 · 3 comments

Comments

@tmiotto
Copy link
Contributor

tmiotto commented Jan 20, 2017

Scenario/Intent

I'm trying to understand why the SDK has the API300::Synergy::LogicalSwitchGroup if it won't do anything. If the resource cease to exists probably we don't want it to be there instead creating and making all methods unavailable.

Environment Details

  • OneView SDK Version: 3.1.0

Steps to Reproduce

Open lib/oneview-sdk/resource/api300/synergy/

Expected Result

Probably I won't need to have the logical_switch_group.rb file.

Actual Result

I would see the logical_switch_group.rb file implementing the LogicalSwitchGroup without any method available.

@ricardogpsf
Copy link
Member

ricardogpsf commented Feb 8, 2017

This thinking works for API300::Synergy::LogicalSwitch too.
The documentation has not logical_switch reference.

@jsmartt
Copy link
Collaborator

jsmartt commented Feb 8, 2017

Good point. Shall I remove it from my open PR too?

@ricardogpsf
Copy link
Member

ricardogpsf commented Feb 8, 2017

No. I was wrong. LogicalSwitch have actions for another API (for api of /rest/internal-link-sets).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants