Opened 6 years ago

Closed 6 years ago

Last modified 5 years ago

#13593 closed Patch - Bug Fix (fixed)

Python: Use 'MasterServerName' instead of 'MasterServerIP'

Reported by: rcrdnalor Owned by: rcrdnalor
Priority: minor Milestone: 31.1
Component: Bindings - Python Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Ticket #13024 : Networking Improvements introduced new settings for 'MasterServerName', 'BackendServerAddr' and 'BackendServerPort' and deprecates '!MasterServerIP' and 'MasterServerPort'.

'BackendServerAddr' can be an IPv4 or IPv6 in the classical "dot/colon" notation, or is provided in it's canonical form.

From settings table:

| value             | data                | hostname |
+-------------------+---------------------+----------+
| BackendServerAddr | 192.168.1.100       | myserver |
| BackendServerPort | 6543                | myserver |
| MasterServerName  | myserver            | NULL     |
+-------------------+---------------------+----------+

or

| value             | data                | hostname |
+-------------------+---------------------+----------+
| BackendServerAddr | server.example.com  | myserver |
| BackendServerPort | 6543                | myserver |
| MasterServerName  | myserver            | NULL     |
+-------------------+---------------------+----------+

Thanks to Ross Boylan for pointing this out in the user mailing list and to and to Peter Bennett for additional explanation.

The workaround until this ticket gets fixed is to use only IP addresses in the ".dot" notation and take care, that the "!MasterServerIP" is identical to the "BackendServerAddr".

Attached patch uses these new settings for the Python Bindings instead of the deprecated ones.

Attachments (2)

Python_Replace_deprecated_settings.patch (4.1 KB ) - added by rcrdnalor 6 years ago.
Replace deprecated settings 'MasterServerIP' and 'MasterServerPort'
0001-BackendServerAddr-and-MasterServerName-replaced-Mast.patch (10.0 KB ) - added by rcrdnalor 6 years ago.
Updated patch for replacing deprecated setting 'MasterServerIP'

Download all attachments as: .zip

Change History (7)

by rcrdnalor, 6 years ago

Replace deprecated settings 'MasterServerIP' and 'MasterServerPort'

by rcrdnalor, 6 years ago

Updated patch for replacing deprecated setting 'MasterServerIP'

comment:1 by Roland Ernst <rcrernst@…>, 6 years ago

Resolution: fixed
Status: assignedclosed

In 105faee393/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:2 by Stuart Auchterlonie, 5 years ago

Roland,

does this need to be cherry-picked back to fixes/31?

Regards Stuart

comment:3 by Stuart Auchterlonie, 5 years ago

Milestone: needs_triage32.0

comment:4 by rcrdnalor, 5 years ago

Milestone: 32.031.1

comment:5 by Roland Ernst <rcrernst@…>, 5 years ago

In c8e779649/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available
Note: See TracTickets for help on using tickets.