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.

Sign up for GitHub

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

Jump to bottom

Missing definitions for targeted refresh for containers #17846

Merged
merged 1 commit into from
Aug 13, 2018
Merged

Missing definitions for targeted refresh for containers #17846

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
61 changes: 46 additions & 15 deletions app/models/manager_refresh/inventory_collection/builder/container_manager.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,14 +21,16 @@ def container_quotas

def container_quota_scopes
add_properties(
:manager_ref => %i(container_quota scope)
:manager_ref => %i(container_quota scope),
:parent_inventory_collections => %i(container_quotas)
)
end

def container_quota_items
add_properties(
:manager_ref => %i(container_quota resource quota_desired quota_enforced quota_observed),
:delete_method => :disconnect_inv
:manager_ref => %i(container_quota resource quota_desired quota_enforced quota_observed),
:delete_method => :disconnect_inv,
:parent_inventory_collections => %i(container_quotas)
)
end

Expand All @@ -41,7 +43,8 @@ def container_limits

def container_limit_items
add_properties(
:manager_ref => %i(container_limit resource item_type)
:manager_ref => %i(container_limit resource item_type),
:parent_inventory_collections => %i(container_limits)
)
end

Expand All @@ -55,20 +58,29 @@ def container_nodes
end

def computer_systems
add_properties(:manager_ref => %i(managed_entity))
add_properties(
:manager_ref => %i(managed_entity),
# TODO(lsmola) can we introspect this from the relation? Basically, the
# :parent_inventory_collections are needed only if :association goes :through other association. Then the
# parent is actually the root association (if we chain several :through associations). We should be able to
# create a tree of :through associations of ems and infer the parent_inventory_collections from that?
:parent_inventory_collections => %i(container_nodes),
)
end

def computer_system_hardwares
add_properties(
:model_class => ::Hardware,
:manager_ref => %i(computer_system)
:model_class => ::Hardware,
:manager_ref => %i(computer_system),
:parent_inventory_collections => %i(container_nodes),
)
end

def computer_system_operating_systems
add_properties(
:model_class => ::OperatingSystem,
:manager_ref => %i(computer_system)
:model_class => ::OperatingSystem,
:manager_ref => %i(computer_system),
:parent_inventory_collections => %i(container_nodes),
)
end

Expand Down Expand Up @@ -103,7 +115,10 @@ def container_groups
end

def container_volumes
add_properties(:manager_ref => %i(parent name))
add_properties(
:manager_ref => %i(parent name),
:parent_inventory_collections => %i(container_groups),
)
end

def containers
Expand All @@ -118,17 +133,24 @@ def containers

def container_port_configs
# parser sets :ems_ref => "#{pod_id}_#{container_name}_#{port_config.containerPort}_#{port_config.hostPort}_#{port_config.protocol}"
add_properties(
:parent_inventory_collections => %i(containers)
)
end

def container_env_vars
add_properties(
# TODO: (agrare) old save matches on all :name, :value, :field_path - does this matter?
:manager_ref => %i(container name)
:manager_ref => %i(container name),
:parent_inventory_collections => %i(containers)
)
end

def security_contexts
add_properties(:manager_ref => %i(resource))
add_properties(
:manager_ref => %i(resource),
:parent_inventory_collections => %i(containers)
)
end

def container_replicators
Expand All @@ -149,11 +171,17 @@ def container_services
end

def container_service_port_configs
add_properties(:manager_ref => %i(ems_ref protocol)) # TODO: (lsmola) make protocol part of the ems_ref?)
add_properties(
:manager_ref => %i(ems_ref protocol), # TODO: (lsmola) make protocol part of the ems_ref?)
:parent_inventory_collections => %i(container_services)
)
end

def container_routes
add_properties(:attributes_blacklist => %i(namespace))
add_properties(
:attributes_blacklist => %i(namespace),
:parent_inventory_collections => %i(container_services)
)
add_common_default_values
end

Expand All @@ -166,7 +194,10 @@ def container_templates
end

def container_template_parameters
add_properties(:manager_ref => %i(container_template name))
add_properties(
:manager_ref => %i(container_template name),
:parent_inventory_collections => %i(container_templates)
)
end

def container_builds
Expand Down