manageiq: List migrations that don't use migration stub (good migration gem) and other migration issues
Migrations:
- 20131107000917_expand_dialog_field_default_value_size.rb -
ReservedMixinmodule in app/ - 20140201040548_add_update_repo_name_to_miq_database.rb -
ReservedMixinmodule in app/ - 20140214191729_enhance_firewall_rules_for_neutron_networking.rb -
ReservedMixinmodule in app/ - 20140421150958_create_miq_groups_users_join_table.rb -
ReservedMixinmodule in app/ - 20150206150955_migrate_miq_database_registration_organization_display_name_out_of_reserves.rb
ReservedMixinmodule in app/ - 20150407144345_add_kerberos_to_ext_management_system.rb -
ReservedMixinmodule in app/ - 20160119125950_add_created_on_for_container_entities.rb -access to models from
CONTAINER_TABLES - 20160317194215_remove_miq_user_role_from_miq_groups.rb (Entitlement)
- 20160418141210_add_read_only_to_miq_alert.rb (MiqAlert, YAML ??)
Specs:
- 20150522161336_add_container_entities_type_spec.rb(Container*,…)
-
spec_helperis loaded in spec migrations (EmsRefresh) 20141121200153_migrate_ems_attributes_to_endpoints_spec.rb 20151204143045_set_correct_sti_type_on_openstack_cloud_volume_spec.rb 20151222103721_migrate_provider_attributes_to_endpoints_spec.rb 20160120151642_migrate_url_from_provider_to_endpoints_spec.rb - 20140201040548_add_update_repo_name_to_miq_database_spec (Reserve, MiqDatabase)
- 20140409134713_move_log_collection_depot_settings_to_file_depot_spec (Authentication, …)
- 20140410132430_subclass_file_depot_by_protocol_spec (FileDepot)
- 20150206150955_migrate_miq_database_registration_organization_display_name_out_of_reserves (Reserve, MiqDatabase)
- 20151221134925_remove_sat5_repo_config_spec (Authentication)
- 20160418141210_add_read_only_to_miq_alert.rb
Need to convert to use active-record:
- https://github.com/ManageIQ/manageiq/blob/master/db/migrate/20151021093644_set_correct_sti_type_on_cloud_network.rb
- https://github.com/ManageIQ/manageiq/blob/master/db/migrate/20151021095831_set_correct_sti_type_on_cloud_subnet.rb
- https://github.com/ManageIQ/manageiq/blob/master/db/migrate/20160406195810_add_id_primary_key_to_join_tables.rb and remove ApplicationRecord
Need to add migration spec:
- 20160119125950_add_created_on_for_container_entities.rb
- 20140218232357_add_group_and_user_columns_to_miq_widget_contents.rb
- 20150206150955_migrate_miq_database_registration_organization_display_name_out_of_reserves.rb
- 20150311181430_add_draft_to_orchestration_templates.rb
- 20150822102141_fix_more_foreman_types.rb
- 20150823120001_namespace_ems_openstack_availability_zones_null.rb
- 20150907163347_confirm_all_class_renames.rb
- 20150909023532_namespace_ems_azure.rb
- 20160203101130_add_orderable_to_orchestration_templates.rb
- 20151021095831_set_correct_sti_type_on_cloud_subnet.rb
- 20151021093644_set_correct_sti_type_on_cloud_network.rb
Other issues:
- 20160120151045_add_url_to_endpoints_spec.rb is empty -> remove (spec is not needed)
- remove
self.inheritance_column = :_type_disabledfrom stub classes: HostsStorage, MiqSet,MiqGroupsUsers, MiqDatabase, MiqAlert, MiqDialog, MiqEventDefinition, ScanItem, MiqWidgetContent, TimeProfile, MiqServer, Zone, User, MiqAeNamespace, MiqGroup, PxeServer, MiqSearch, Endpoint, OperatingSystemFlavor, MiqSchedule, ContainerService, ContainerServicePortConfig, ContainerCondition, MiqReportResult, BinaryBlob, Relationship, MiqEventDefinition, BlacklistedEvent, Tenant, TenantQuota, MiqUserRole, ContainerImage, MiqProductFeature, ServiceTemplateCatalog, MiqRegion in all migrations
About this issue
- Original URL
- State: closed
- Created 8 years ago
- Comments: 18 (18 by maintainers)
Commits related to this issue
- Use the migration_stub to avoid loading the model in tests. Related #6739 — committed to jrafanie/manageiq by jrafanie 8 years ago
- Require ReservedMixin and find using migration stubs. Related #6739 — committed to jrafanie/manageiq by jrafanie 8 years ago
- spec_helper is automatically required See: 489a17d351df Related #6739 — committed to jrafanie/manageiq by jrafanie 8 years ago
- Rename spec file so it's actually run with other specs. Related #6739 — committed to jrafanie/manageiq by jrafanie 8 years ago
- Require the modules and use migration stubs for finds. Related #6739 — committed to jrafanie/manageiq by jrafanie 8 years ago
( Re-opening on @lpichler 's request … the checklist is not done yet 😃 )