search

ManageIQ / manageiq-providers-azure

ManageIQ plugin for the Microsoft Azure provider.
https://azure.microsoft.com
Apache License 2.0
8 stars 60 forks source link

Skip instances if we cannot find the flavor #491

Closed agrare closed 2 years ago

agrare commented 2 years ago

The flavor (aka series) is required for setting the instance hardware and operating information.

Without the flavor we don't have enough information to build the vm record and we should skip it.

miq-bot commented 2 years ago

Checked commit https://github.com/agrare/manageiq-providers-azure/commit/51f314f516355720e47cd845f910951ad6126359 with ruby 2.6.3, rubocop 1.13.0, haml-lint 0.35.0, and yamllint 1 file checked, 0 offenses detected Everything looks fine. :+1:

Fryguy commented 2 years ago

Backported to morphy in commit 1aa8ad70c2ce61d96a15ea28c3a48b7d9f27813c.

commit 1aa8ad70c2ce61d96a15ea28c3a48b7d9f27813c
Author: Keenan Brock <keenan@thebrocks.net>
Date:   Tue Feb 8 18:39:59 2022 -0500

    Merge pull request #491 from agrare/skip_instances_if_flavor_missing

    Skip instances if we cannot find the flavor

    (cherry picked from commit 511bde290a2b352c2a69b8d8338e8d9f8539069a)