Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Refactor pool external object #1660

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Refactor pool external object #1660

wants to merge 2 commits into from

Commits on Jul 15, 2021

  1. fix: PoolTangoObject full_name discovery. 

    The ways of retrieving the tango database host between PyTango and TangoNameValidators are inconsistent. For instance, inside a docker container PyTango returns <docker_network>.<docker_container_name>:<port> while using a Taurus validator returns the host and port specified by the TANGO_HOST environment variable.

Use Taurus validators instead of accessing PyTango database host and port API for discovery the full name of a PoolTangoObject.
    @catunlock
    catunlock committed Jul 15, 2021
    Configuration menu
    Copy the full SHA
    48b207d View commit details
    Browse the repository at this point in the history

  2. refactor: PoolExternalObject API. 

    PoolExternalObject requires many kwargs describing the object identity,
this forces the API users to discover those kwargs using Taurus validators.
Since PoolTangoObject can do this discovery by itself, we decided to
change the PoolExternalObject API and we adapted its usage in
Sardana code base.
    @catunlock
    catunlock committed Jul 15, 2021
    Configuration menu
    Copy the full SHA
    6bcefe0 View commit details
    Browse the repository at this point in the history