Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix disk controller sorting #1032

Merged
merged 1 commit into from
Apr 13, 2020
Merged

Fix disk controller sorting #1032

merged 1 commit into from
Apr 13, 2020

Conversation

dhekimian
Copy link
Contributor

Description

Acceptance tests

  • Have you added an acceptance test for the functionality being added?
  • Have you run the acceptance tests on this branch? (If so, please include the test log in a gist)

References

@ghost ghost added the size/xs Relative Sizing: Extra-Small label Apr 10, 2020
@dhekimian
Copy link
Contributor Author

@bill-rich In the sort function, it was comparing incorrectly.
This should go along with #1029 , #1031 and resolve #997.

@dhekimian
Copy link
Contributor Author

@bill-rich Over the weekend, we built the provider using this branch and was able to successfully test that it resolves #997

What's needed to get this merged?

Copy link
Contributor

@bill-rich bill-rich left a comment

Choose a reason for hiding this comment

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

Nice catch!

@bill-rich bill-rich merged commit 4da38ad into hashicorp:master Apr 13, 2020
@ghost
Copy link

ghost commented May 14, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks!

@ghost ghost locked and limited conversation to collaborators May 14, 2020
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
size/xs Relative Sizing: Extra-Small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to clone VCSA 7 template with multiple scsi controllers and disks
2 participants