8a98228251
For rolling upgrades, we added a 'version' column to all the DB object tables. However, we forgot to save the version value for conductor objects. This patch addresses that. Change-Id: Ic366f771491774f9708c9a81ab76dc13757d852b Partial-Bug: #1526283
148 lines
6.7 KiB
Python
148 lines
6.7 KiB
Python
# coding=utf-8
|
|
#
|
|
# Copyright 2013 Hewlett-Packard Development Company, L.P.
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
|
# not use this file except in compliance with the License. You may obtain
|
|
# a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
# License for the specific language governing permissions and limitations
|
|
# under the License.
|
|
|
|
from oslo_versionedobjects import base as object_base
|
|
|
|
from ironic.common.i18n import _
|
|
from ironic.db import api as db_api
|
|
from ironic.objects import base
|
|
from ironic.objects import fields as object_fields
|
|
|
|
|
|
@base.IronicObjectRegistry.register
|
|
class Conductor(base.IronicObject, object_base.VersionedObjectDictCompat):
|
|
# Version 1.0: Initial version
|
|
# Version 1.1: Add register() and unregister(), make the context parameter
|
|
# to touch() optional.
|
|
# Version 1.2: Add register_hardware_interfaces() and
|
|
# unregister_all_hardware_interfaces()
|
|
VERSION = '1.2'
|
|
|
|
dbapi = db_api.get_instance()
|
|
|
|
fields = {
|
|
'id': object_fields.IntegerField(),
|
|
'drivers': object_fields.ListOfStringsField(nullable=True),
|
|
'hostname': object_fields.StringField(),
|
|
}
|
|
|
|
# NOTE(xek): We don't want to enable RPC on this call just yet. Remotable
|
|
# methods can be used in the future to replace current explicit RPC calls.
|
|
# Implications of calling new remote procedures should be thought through.
|
|
# @object_base.remotable_classmethod
|
|
@classmethod
|
|
def get_by_hostname(cls, context, hostname):
|
|
"""Get a Conductor record by its hostname.
|
|
|
|
:param cls: the :class:`Conductor`
|
|
:param context: Security context
|
|
:param hostname: the hostname on which a Conductor is running
|
|
:returns: a :class:`Conductor` object.
|
|
"""
|
|
db_obj = cls.dbapi.get_conductor(hostname)
|
|
conductor = cls._from_db_object(context, cls(), db_obj)
|
|
return conductor
|
|
|
|
def save(self, context):
|
|
"""Save is not supported by Conductor objects."""
|
|
raise NotImplementedError(
|
|
_('Cannot update a conductor record directly.'))
|
|
|
|
# NOTE(xek): We don't want to enable RPC on this call just yet. Remotable
|
|
# methods can be used in the future to replace current explicit RPC calls.
|
|
# Implications of calling new remote procedures should be thought through.
|
|
# @object_base.remotable
|
|
def refresh(self, context=None):
|
|
"""Loads and applies updates for this Conductor.
|
|
|
|
Loads a :class:`Conductor` with the same uuid from the database and
|
|
checks for updated attributes. Updates are applied from
|
|
the loaded chassis column by column, if there are any updates.
|
|
|
|
:param context: Security context. NOTE: This should only
|
|
be used internally by the indirection_api.
|
|
Unfortunately, RPC requires context as the first
|
|
argument, even though we don't use it.
|
|
A context should be set when instantiating the
|
|
object, e.g.: Conductor(context)
|
|
"""
|
|
current = self.get_by_hostname(self._context, hostname=self.hostname)
|
|
self.obj_refresh(current)
|
|
|
|
# NOTE(xek): We don't want to enable RPC on this call just yet. Remotable
|
|
# methods can be used in the future to replace current explicit RPC calls.
|
|
# Implications of calling new remote procedures should be thought through.
|
|
# @object_base.remotable
|
|
def touch(self, context=None):
|
|
"""Touch this conductor's DB record, marking it as up-to-date."""
|
|
self.dbapi.touch_conductor(self.hostname)
|
|
|
|
# NOTE(xek): We don't want to enable RPC on this call just yet. Remotable
|
|
# methods can be used in the future to replace current explicit RPC calls.
|
|
# Implications of calling new remote procedures should be thought through.
|
|
# @object_base.remotable
|
|
@classmethod
|
|
def register(cls, context, hostname, drivers, update_existing=False):
|
|
"""Register an active conductor with the cluster.
|
|
|
|
:param cls: the :class:`Conductor`
|
|
:param context: Security context
|
|
:param hostname: the hostname on which the conductor will run
|
|
:param drivers: the list of drivers enabled in the conductor
|
|
:param update_existing: When false, registration will raise an
|
|
exception when a conflicting online record
|
|
is found. When true, will overwrite the
|
|
existing record. Default: False.
|
|
:raises: ConductorAlreadyRegistered
|
|
:returns: a :class:`Conductor` object.
|
|
|
|
"""
|
|
db_cond = cls.dbapi.register_conductor(
|
|
{'hostname': hostname,
|
|
'drivers': drivers,
|
|
'version': cls.get_target_version()},
|
|
update_existing=update_existing)
|
|
return cls._from_db_object(context, cls(), db_cond)
|
|
|
|
# NOTE(xek): We don't want to enable RPC on this call just yet. Remotable
|
|
# methods can be used in the future to replace current explicit RPC calls.
|
|
# Implications of calling new remote procedures should be thought through.
|
|
# @object_base.remotable
|
|
def unregister(self, context=None):
|
|
"""Remove this conductor from the service registry."""
|
|
self.unregister_all_hardware_interfaces()
|
|
self.dbapi.unregister_conductor(self.hostname)
|
|
|
|
def register_hardware_interfaces(self, hardware_type, interface_type,
|
|
interfaces, default_interface):
|
|
"""Register hardware interfaces with the conductor.
|
|
|
|
:param hardware_type: Name of hardware type for the interfaces.
|
|
:param interface_type: Type of interfaces, e.g. 'deploy' or 'boot'.
|
|
:param interfaces: List of interface names to register.
|
|
:param default_interface: String, the default interface for this
|
|
hardware type and interface type.
|
|
"""
|
|
self.dbapi.register_conductor_hardware_interfaces(self.id,
|
|
hardware_type,
|
|
interface_type,
|
|
interfaces,
|
|
default_interface)
|
|
|
|
def unregister_all_hardware_interfaces(self):
|
|
"""Unregister all hardware interfaces for this conductor."""
|
|
self.dbapi.unregister_conductor_hardware_interfaces(self.id)
|