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

fix logical interconnect group issues #544

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

adarobin
Copy link
Contributor

Description

The crashes I have experienced with oneview_logical_interconnect_group have been resolved, but I have still had some issues with the resource complaining about attributes being set that are read only or have default values defined and showing a change when there isn't any. There were also several attributes that were not being set on a read.

This also fixes the resource to read in uplink_set when doing an import. I haven't figured out a way to handle the ordering of uplink_set when doing an import though. The ideal solution would probably be for uplink_set to be TypeSet, but I'm not sure how feasible that is.

Issues Resolved

Check List

  • New functionality includes testing.
    • All tests pass for go 1.11 + gofmt checks.
  • New functionality has been documented in the README if applicable.
    • New functionality has been thoroughly documented in the examples (please include helpful comments).
    • New endpoints supported are updated in the endpoints-support.md file.
  • Changes are documented in the CHANGELOG.

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.

1 participant