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

Move saving of ems.api_version back to parser #136

Merged
merged 1 commit into from
Nov 21, 2017

Conversation

agrare
Copy link
Member

@agrare agrare commented Nov 14, 2017

When we moved saving the api_version to save_inventory we started
getting periodic errors in provisioning when checking
ext_management_system.api_version > '5.5' api_version would be nil.

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

When we moved saving the api_version to save_inventory we started
getting periodic errors in provisioning when checking
ext_management_system.api_version > '5.5' api_version would be nil.

https://bugzilla.redhat.com/show_bug.cgi?id=1503281
@miq-bot
Copy link
Member

miq-bot commented Nov 14, 2017

Checked commit agrare@4ed8b98 with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
5 files checked, 1 offense detected

**

  • 💣 💥 🔥 🚒 - Linter/Yaml - missing config files

@agrare
Copy link
Member Author

agrare commented Nov 15, 2017

@blomquisg tested this with @lkhomenk and he confirmed this resolves the issue

@agrare
Copy link
Member Author

agrare commented Nov 20, 2017

cc @blomquisg

@Ladas Ladas self-assigned this Nov 21, 2017
Copy link
Contributor

@Ladas Ladas left a comment

Choose a reason for hiding this comment

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

Looks great. 👍 I mean it's ugly, but we need more work to be able to fix it properly for next release. :-)

@Ladas Ladas merged commit bc7a70d into ManageIQ:master Nov 21, 2017
@Ladas Ladas added this to the Sprint 74 Ending Nov 27, 2017 milestone Nov 21, 2017
simaishi pushed a commit that referenced this pull request Nov 27, 2017
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 35401eee94a75a2522fbc6e8cab0695f684a4776
Author: Ladislav Smola <[email protected]>
Date:   Tue Nov 21 17:47:35 2017 +0100

    Merge pull request #136 from agrare/bz_1503281_fix_api_version_nil_error
    
    Move saving of ems.api_version back to parser
    (cherry picked from commit bc7a70de32f5f8383fff20636c7c63c5b0796ba0)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1517837

@agrare agrare deleted the bz_1503281_fix_api_version_nil_error branch April 25, 2018 12:44
agrare pushed a commit to agrare/manageiq-providers-vmware that referenced this pull request Apr 15, 2019
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.

5 participants