Skip to content
This repository was archived by the owner on Mar 10, 2021. It is now read-only.

add network fqdn/alias table to known issues #359

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

sergirubio
Copy link

No description provided.

Copy link
Collaborator

@piogor piogor left a comment

Choose a reason for hiding this comment

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

Thanks, @sergirubio, there is a need for a small correction to avoid the only warning ;-).

Comment on lines +70 to +72
- **known workaround**: it is possible to setup each device on a machine to use
an specific IP address (to run servers on different IP addresses he server host
must have multiple interfaces, tipically virtual ones).
Copy link
Collaborator

Choose a reason for hiding this comment

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

It seem the identation has to be corrected (/home/travis/build/tango-controls/tango-doc/source/development/advanced/known-issues.rst:71: WARNING: Bullet list ends without a blank line; unexpected unindent.)

Suggested change
- **known workaround**: it is possible to setup each device on a machine to use
an specific IP address (to run servers on different IP addresses he server host
must have multiple interfaces, tipically virtual ones).
- **known workaround**: it is possible to setup each device on a machine to use
an specific IP address (to run servers on different IP addresses he server host
must have multiple interfaces, tipically virtual ones).

Copy link
Author

Choose a reason for hiding this comment

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

hi piotr, I agree with the change
thanks

Copy link
Collaborator

Choose a reason for hiding this comment

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

@sergirubio, I think it is you who should apply the suggestion as I do not have write permission to ALBA repo.

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.

2 participants