Project

General

Profile

Actions

Bug #49957

closed

Error in dashboard iscsi gateways configuration when IPV6 is used

Added by Juan Miguel Olmo Martínez about 3 years ago. Updated about 3 years ago.

Status:
Resolved
Priority:
Normal
Category:
cephadm
Target version:
-
% Done:

100%

Source:
Tags:
Backport:
pacific
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Error appears in the dashboard and it is caused by the current URL format specified for IPv6 addresses:

http://admin:admin@2620:52:0:880:225:90ff:fefc:26f8:5000

urlparse('http://admin:admin@2620:52:0:880:225:90ff:fefc:26f8:5000').hostname

'2620'

urlparse('http://admin:admin@2620:52:0:880:225:90ff:fefc:26f8:5000').port

Traceback (most recent call last):

... File "<stdin>", line 1, in <module>
... File "/usr/lib64/python3.6/urllib/parse.py", line 169, in port
... port = int(port, 10)
... ValueError: invalid literal for int() with base 10: '52:0:880:225:90ff:fefc:26f8:5000'

That's not compliant with the standard IPv6 URL encoding, which requires square brackets enclosing IPv6 literal addresses (https://tools.ietf.org/html/rfc2732):

urlparse('http://admin:admin@[2620:52:0:880:225:90ff:fefc:26f8]:5000').hostname

'2620:52:0:880:225:90ff:fefc:26f8'

urlparse('http://admin:admin@[2620:52:0:880:225:90ff:fefc:26f8]:5000').port

5000

Actions #1

Updated by Juan Miguel Olmo Martínez about 3 years ago

  • % Done changed from 0 to 50
  • Pull request ID set to 40366
Actions #2

Updated by Sage Weil about 3 years ago

  • Status changed from New to Pending Backport
  • Backport set to pacific
Actions #3

Updated by Sage Weil about 3 years ago

  • Status changed from Pending Backport to Resolved

pacific backport merged in https://github.com/ceph/ceph/pull/40355

Actions #4

Updated by Juan Miguel Olmo Martínez about 3 years ago

  • % Done changed from 50 to 100
Actions

Also available in: Atom PDF