Skip to content
This repository was archived by the owner on Jul 8, 2022. It is now read-only.

Conversation

@mliszcz
Copy link
Collaborator

@mliszcz mliszcz commented Nov 28, 2019

The PR removes second database instance as proposed in #630. It impacts #608 test scenario.

Without #608 fix, with second database the failure was:

Tango exception
Severity = ERROR 
Error reason = DB_DeviceNotDefined
Desc : device test/debian8/10 not defined in the database !
Origin : DataBase::ImportDevice()

Without second database, the failure is:

25: Tango exception
25: Severity = ERROR
25: Error reason = API_TangoHostNotSet
25: Desc : TANGO_HOST env. variable not set, set it and retry (e.g. TANGO_HOST=<host>:<port>)
25: Origin : Database::Database

@mliszcz mliszcz requested review from Ingvord, bourtemb and t-b November 28, 2019 14:10
Copy link
Member

@bourtemb bourtemb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Good initiative.
Thanks @mliszcz

@t-b t-b merged commit 9e8a6cd into tango-controls:tango-9-lts Dec 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants