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

Upgrade armrest gem to 0.7.3 #74

Merged
merged 1 commit into from Jun 1, 2017
Merged

Upgrade armrest gem to 0.7.3 #74

merged 1 commit into from Jun 1, 2017

Conversation

djberg96
Copy link
Collaborator

@djberg96 djberg96 commented May 26, 2017

This bumps the azure-armrest dependency to 0.7.3, which added guards against ECONNREFUSED and TimeoutException for the StorageAccountService#get_private_images method.

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

@djberg96 djberg96 requested a review from bronaghs May 26, 2017 15:04
@miq-bot
Copy link
Member

miq-bot commented May 26, 2017

Checked commit https://github.com/djberg96/manageiq-providers-azure/commit/2ffcc658ea748608791743a6067a2ddc504d0f9f with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 🍪

@bronaghs bronaghs merged commit 9761bcb into ManageIQ:master Jun 1, 2017
@bronaghs bronaghs added this to the Sprint 62 Ending Jun 5, 2017 milestone Jun 1, 2017
@bronaghs bronaghs added the bug label Jun 1, 2017
@simaishi
Copy link
Contributor

simaishi commented Jun 7, 2017

Backported to Euwe via ManageIQ/manageiq#15239

simaishi pushed a commit that referenced this pull request Jun 9, 2017
@simaishi
Copy link
Contributor

simaishi commented Jun 9, 2017

Fine backport details:

$ git log -1
commit ba9fc3f9e10cb51699bea1ade3ed17b18af79d28
Author: Bronagh Sorota <bsorota@redhat.com>
Date:   Thu Jun 1 11:50:31 2017 -0400

    Merge pull request #74 from djberg96/gemspec
    
    Upgrade armrest gem to 0.7.3
    (cherry picked from commit 9761bcb41c5423b05c245367d08fcbaf0148b5f3)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1459318

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

Successfully merging this pull request may close these issues.

None yet

4 participants