search

ManageIQ / manageiq

ManageIQ Open-Source Management Platform
https://manageiq.org
Apache License 2.0
1.35k stars 898 forks source link

Drop legacy refresh #19888

Closed agrare closed 4 years ago

agrare commented 4 years ago

With the vast majority of providers using graph refresh as the only options now we can start to remove the old legacy EmsRefresh and save_inventory refresh code.

Two manager types do not have Persister definitions at all:

Some providers still are not on graph refresh:

The following providers have to have their legacy refresh parser code deleted

A number of provider still have inventory_object_refresh in config/settings.yml even though it is the only option

Cleanup tasks:

d-m-u commented 4 years ago

A number of provider still have inventory_object_refresh in config/settings.yml even though it is the only option addressed in the following:

chessbyte commented 4 years ago

This is a nice-to-have for Jansa, can slip into a future release.

d-m-u commented 4 years ago

cleanup task of deleting all the inventory_object_refresh? methods from the rest of the providers:

agrare commented 4 years ago

Going to mark this as complete now, next step is to remove some of the callers of save_inventory directly (e.g. https://github.com/ManageIQ/manageiq/issues/19985) and delete the app/models/ems_refresh/save_inventory*rb code.