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

Re-Record Azure Managed Disks SSA Tests #25

Merged

Conversation

jerryk55
Copy link
Member

@jerryk55 jerryk55 commented Sep 5, 2017

Due to an as yet undiagnosed dependency on a change in another gem/repo,
these test recordings were no longer working even after working correctly at the time
of the previous PR's merge. They have been re-recorded to get the tests back up
and running.

The tests were initially recorded as part of #23.

@roliveri please review and merge (so we can get going on the rest of the required PRs for
Blocker bug https://bugzilla.redhat.com/show_bug.cgi?id=1475540) Thanks.

Due to an as yet undiagnosed dependency on a change in another gem/repo,
these test recordings were no longer working even after working correctly at the time
of the previous PR's merge.  They have been re-recorded to get the tests back up
and running.
@jerryk55
Copy link
Member Author

jerryk55 commented Sep 5, 2017

@roliveri these tests now pass - I'd like to make sure that #24 includes the updated tests so if you can merge this one first....

@miq-bot
Copy link
Member

miq-bot commented Sep 5, 2017

Checked commits jerryk55/manageiq-smartstate@0c88470~...a70475e with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks fine. ⭐

@roliveri roliveri self-assigned this Sep 5, 2017
@roliveri roliveri merged commit 140baa0 into ManageIQ:master Sep 5, 2017
jerryk55 added a commit to jerryk55/manageiq-gems-pending that referenced this pull request Sep 6, 2017
ManageIQ/manageiq-smartstate#23

and

ManageIQ/manageiq-smartstate#25

were added as part of the work required for blocker bug

https://bugzilla.redhat.com/show_bug.cgi?id=1475540.

This commit and subsequent PR track the backport of the changes into the Fine release.
@simaishi
Copy link
Contributor

simaishi commented Sep 6, 2017

Backported to Fine via ManageIQ/manageiq-gems-pending#267

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.

4 participants