Difference between revisions of "ReadDeletedYesOrOnly"
m (Text replace - "__NOTOC__" to "") |
|||
(5 intermediate revisions by 2 users not shown) | |||
Line 1: | Line 1: | ||
− | + | ||
+ | == Introduction == | ||
+ | |||
This is a list of places in the Nova code where we check for read_deleted being set to "yes" (which means to include both deleted and non-deleted rows) or "only" (which means to include only deleted rows). | This is a list of places in the Nova code where we check for read_deleted being set to "yes" (which means to include both deleted and non-deleted rows) or "only" (which means to include only deleted rows). | ||
If we're going to change the way we track deleted rows, we first need to fix all the places that currently rely on read_deleted. | If we're going to change the way we track deleted rows, we first need to fix all the places that currently rely on read_deleted. | ||
+ | |||
+ | For unit tests I only list the file, since the exact method will change when the corresponding tested module does. | ||
= nova/compute/instance_types.py = | = nova/compute/instance_types.py = | ||
Line 11: | Line 15: | ||
= nova/compute/manager.py = | = nova/compute/manager.py = | ||
− | |||
− | =nova/context.py = | + | ComputeManager._cleanup_running_deleted_instances read_deleted="yes" |
− | + | ||
− | + | = nova/context.py = | |
− | + | ||
− | + | RequestContext.__init__ has read_deleted param, default "no" | |
+ | RequestContext.read_deleted property | ||
+ | RequestContext.elevated takes param read_deleted | ||
+ | |||
get_admin_context takes param read_deleted default "no" | get_admin_context takes param read_deleted default "no" | ||
= nova/db/sqlalchemy/api.py = | = nova/db/sqlalchemy/api.py = | ||
+ | |||
model_query takes kw param read_deleted, default "no" | model_query takes kw param read_deleted, default "no" | ||
+ | |||
fixed_ip_get "yes" | fixed_ip_get "yes" | ||
+ | |||
fixed_ip_get_all "yes" | fixed_ip_get_all "yes" | ||
+ | |||
fixed_ip_get_by_address "yes" | fixed_ip_get_by_address "yes" | ||
+ | |||
(other fixed_ip functions do not have read_deleted "yes", which is inconsistent. Bug?) | (other fixed_ip functions do not have read_deleted "yes", which is inconsistent. Bug?) | ||
+ | |||
_virtual_interface_query param read_deleted, default "yes" | _virtual_interface_query param read_deleted, default "yes" | ||
+ | |||
_ec2_volume_get_query "yes" | _ec2_volume_get_query "yes" | ||
+ | |||
_ec2_snapshot_get_query "yes" | _ec2_snapshot_get_query "yes" | ||
+ | |||
volume_get_iscsi_target_num "yes" | volume_get_iscsi_target_num "yes" | ||
+ | |||
migration_get "yes" | migration_get "yes" | ||
+ | |||
migration_get_by_instance_and_status "yes" | migration_get_by_instance_and_status "yes" | ||
+ | |||
migration_get_unconfirmed_by_dest_compute "yes" | migration_get_unconfirmed_by_dest_compute "yes" | ||
+ | |||
console_get_by_pool_instance "yes" | console_get_by_pool_instance "yes" | ||
+ | |||
console_get_all_by_instance "yes" | console_get_all_by_instance "yes" | ||
+ | |||
console_get "yes" | console_get "yes" | ||
+ | |||
instance_type_get_all "yes" if param inactive is True | instance_type_get_all "yes" if param inactive is True | ||
+ | |||
_instance_type)_access_query "yes" | _instance_type)_access_query "yes" | ||
+ | |||
agent_build_destroy "yes" | agent_build_destroy "yes" | ||
+ | |||
agent_build_update "yes" | agent_build_update "yes" | ||
+ | |||
bw_usage_get "yes" | bw_usage_get "yes" | ||
+ | |||
bw_usage_get_by_uuids "yes" | bw_usage_get_by_uuids "yes" | ||
+ | |||
bw_usage_update "yes" | bw_usage_update "yes" | ||
+ | |||
s3_image_get "yes" | s3_image_get "yes" | ||
+ | |||
s3_image_get_by_uuid "yes" | s3_image_get_by_uuid "yes" | ||
+ | |||
aggregate_metadata_get_item "yes" | aggregate_metadata_get_item "yes" | ||
+ | |||
aggregate-host_add "yes" | aggregate-host_add "yes" | ||
+ | |||
_ec2_instance_get_query "yes" | _ec2_instance_get_query "yes" | ||
+ | |||
_security_group_get_query "only" | _security_group_get_query "only" | ||
− | nova/network/manager.py | + | = nova/network/manager.py = |
− | nova/network/quantum/nova_ipam_lib.py | + | |
− | nova/notifications.py | + | FloatingIP.deallocate_for_instance "yes" |
− | nova/openstack/common/rpc/common.py | + | |
− | nova/tests/api/ec2/test_cinder_cloud.py | + | NetworkManager._do_trigger_security_group_members_refresh_for_instance "yes" |
− | nova/tests/api/ec2/test_cloud.py | + | |
− | nova/tests/api/openstack/compute/contrib/test_flavor_manage.py | + | NetworkManager.deallocate_fixed_ip "yes" |
− | nova/tests/compute/test_compute.py | + | |
− | nova/tests/compute/test_compute_utils.py | + | = nova/network/quantum/nova_ipam_lib.py = |
− | nova/tests/network/test_manager.py | + | |
− | nova/tests/test_context.py | + | QuantumNovaIPAMLib.deallocate_ips_by_vif "yes" |
− | nova/tests/test_db_api.py | + | |
− | nova/tests/test_instance_types.py | + | = nova/notifications.py = |
− | + | ||
− | nova/virt/baremetal/db/sqlalchemy/api.py | + | bandwidth_usage "yes" |
− | tools/xenserver/vm_vdi_cleaner.py | + | |
+ | = nova/openstack/common/rpc/common.py = | ||
+ | |||
+ | CommmonRpcContext.elevated read_deleted param | ||
+ | |||
+ | = nova/tests/api/ec2/test_cinder_cloud.py = | ||
+ | |||
+ | = nova/tests/api/ec2/test_cloud.py = | ||
+ | |||
+ | = nova/tests/api/openstack/compute/contrib/test_flavor_manage.py = | ||
+ | |||
+ | = nova/tests/compute/test_compute.py = | ||
+ | |||
+ | = nova/tests/compute/test_compute_utils.py = | ||
+ | |||
+ | = nova/tests/network/test_manager.py = | ||
+ | |||
+ | = nova/tests/test_context.py = | ||
+ | |||
+ | = nova/tests/test_db_api.py = | ||
+ | |||
+ | = nova/tests/test_instance_types.py = | ||
+ | |||
+ | = nova/virt/baremetal/db/sqlalchemy/api.py = | ||
+ | |||
+ | model_query kw param read_deleted | ||
+ | |||
+ | = tools/xenserver/vm_vdi_cleaner.py = | ||
+ | |||
+ | find_orphaned_instances "only" |
Latest revision as of 23:30, 17 February 2013
Contents
- 1 Introduction
- 2 nova/compute/instance_types.py
- 3 nova/compute/manager.py
- 4 nova/context.py
- 5 nova/db/sqlalchemy/api.py
- 6 nova/network/manager.py
- 7 nova/network/quantum/nova_ipam_lib.py
- 8 nova/notifications.py
- 9 nova/openstack/common/rpc/common.py
- 10 nova/tests/api/ec2/test_cinder_cloud.py
- 11 nova/tests/api/ec2/test_cloud.py
- 12 nova/tests/api/openstack/compute/contrib/test_flavor_manage.py
- 13 nova/tests/compute/test_compute.py
- 14 nova/tests/compute/test_compute_utils.py
- 15 nova/tests/network/test_manager.py
- 16 nova/tests/test_context.py
- 17 nova/tests/test_db_api.py
- 18 nova/tests/test_instance_types.py
- 19 nova/virt/baremetal/db/sqlalchemy/api.py
- 20 tools/xenserver/vm_vdi_cleaner.py
Introduction
This is a list of places in the Nova code where we check for read_deleted being set to "yes" (which means to include both deleted and non-deleted rows) or "only" (which means to include only deleted rows).
If we're going to change the way we track deleted rows, we first need to fix all the places that currently rely on read_deleted.
For unit tests I only list the file, since the exact method will change when the corresponding tested module does.
nova/compute/instance_types.py
get_instance_type sets read_deleted="yes" if param inactive is True
get_instance_type_by_flavor_id has read_deleted param, default "yes"
nova/compute/manager.py
ComputeManager._cleanup_running_deleted_instances read_deleted="yes"
nova/context.py
RequestContext.__init__ has read_deleted param, default "no" RequestContext.read_deleted property RequestContext.elevated takes param read_deleted
get_admin_context takes param read_deleted default "no"
nova/db/sqlalchemy/api.py
model_query takes kw param read_deleted, default "no"
fixed_ip_get "yes"
fixed_ip_get_all "yes"
fixed_ip_get_by_address "yes"
(other fixed_ip functions do not have read_deleted "yes", which is inconsistent. Bug?)
_virtual_interface_query param read_deleted, default "yes"
_ec2_volume_get_query "yes"
_ec2_snapshot_get_query "yes"
volume_get_iscsi_target_num "yes"
migration_get "yes"
migration_get_by_instance_and_status "yes"
migration_get_unconfirmed_by_dest_compute "yes"
console_get_by_pool_instance "yes"
console_get_all_by_instance "yes"
console_get "yes"
instance_type_get_all "yes" if param inactive is True
_instance_type)_access_query "yes"
agent_build_destroy "yes"
agent_build_update "yes"
bw_usage_get "yes"
bw_usage_get_by_uuids "yes"
bw_usage_update "yes"
s3_image_get "yes"
s3_image_get_by_uuid "yes"
aggregate_metadata_get_item "yes"
aggregate-host_add "yes"
_ec2_instance_get_query "yes"
_security_group_get_query "only"
nova/network/manager.py
FloatingIP.deallocate_for_instance "yes"
NetworkManager._do_trigger_security_group_members_refresh_for_instance "yes"
NetworkManager.deallocate_fixed_ip "yes"
nova/network/quantum/nova_ipam_lib.py
QuantumNovaIPAMLib.deallocate_ips_by_vif "yes"
nova/notifications.py
bandwidth_usage "yes"
nova/openstack/common/rpc/common.py
CommmonRpcContext.elevated read_deleted param
nova/tests/api/ec2/test_cinder_cloud.py
nova/tests/api/ec2/test_cloud.py
nova/tests/api/openstack/compute/contrib/test_flavor_manage.py
nova/tests/compute/test_compute.py
nova/tests/compute/test_compute_utils.py
nova/tests/network/test_manager.py
nova/tests/test_context.py
nova/tests/test_db_api.py
nova/tests/test_instance_types.py
nova/virt/baremetal/db/sqlalchemy/api.py
model_query kw param read_deleted
tools/xenserver/vm_vdi_cleaner.py
find_orphaned_instances "only"