[RFE] Allow both RFC3986 and FQDNs in Node Registration 'Name' field

Bug #1609917 reported by Dan Trainor
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Jiri Tomasek

Bug Description

When "Upload From File" is used to Register nodes in the UI, the dialogue provides post-import configuration options. One of these options is for the value of 'Name'

This is likely a limitation of how Ironic stores and maintains node names. According to the Ironic WebAPI V1 spec[1], version 1.10 incorporated the usage of RFC3986 unreserved characters. This was in addition to valid FQDNs. The form field validation is only looking for valid RFC3986 unreserved characters, not both RFC3986 unreserved characters AND valid FQDNs.

The form field validation should support both RFC3986 unreserved characters and valid FQDNs.

[1] http://docs.openstack.org/developer/ironic/webapi/v1.html

Changed in tripleo-ui:
importance: Undecided → Low
status: New → Triaged
summary: - Allow both RFC3986 and FQDNs in Node Registration 'Name' field
+ [RFE] Allow both RFC3986 and FQDNs in Node Registration 'Name' field
tags: added: ui
affects: tripleo-ui → tripleo
Revision history for this message
Julie Pichon (jpichon) wrote :

I wonder if that name validation is appropriate at all, looking at http://developer.openstack.org/api-ref/baremetal/?expanded=create-node-detail it sounds like the 'name' can be pretty-much free-form since 1.5? The URL mentioned in the bug is no longer valid so I'm not sure where to double-check the reference. See also bug 1632631.

Jiri Tomasek (jtomasek)
Changed in tripleo:
assignee: nobody → Jiri Tomasek (jtomasek)
milestone: none → ocata-1
importance: Low → High
tags: added: newton-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-ui (master)

Fix proposed to branch: master
Review: https://review.openstack.org/385418

Changed in tripleo:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-ui (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/386199

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-ui (master)

Reviewed: https://review.openstack.org/385418
Committed: https://git.openstack.org/cgit/openstack/tripleo-ui/commit/?id=228aa2ab6aace295dcb531eccb1528d9edaca045
Submitter: Jenkins
Branch: master

commit 228aa2ab6aace295dcb531eccb1528d9edaca045
Author: Jiri Tomasek <email address hidden>
Date: Wed Oct 12 13:51:07 2016 +0200

    Ignore case in Node name validation regexp

    Closes-Bug: 1609917
    Closes-Bug: 1632631
    Change-Id: Ia298f9f99ce491ff5f963b0936db6ea97da06274

Changed in tripleo:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-ui (stable/newton)

Reviewed: https://review.openstack.org/386199
Committed: https://git.openstack.org/cgit/openstack/tripleo-ui/commit/?id=68e6cfcc9032dbc7eacb56bbe941af03f2a880c3
Submitter: Jenkins
Branch: stable/newton

commit 68e6cfcc9032dbc7eacb56bbe941af03f2a880c3
Author: Jiri Tomasek <email address hidden>
Date: Wed Oct 12 13:51:07 2016 +0200

    Ignore case in Node name validation regexp

    Closes-Bug: 1609917
    Closes-Bug: 1632631
    Change-Id: Ia298f9f99ce491ff5f963b0936db6ea97da06274
    (cherry picked from commit 228aa2ab6aace295dcb531eccb1528d9edaca045)

tags: added: in-stable-newton
Revision history for this message
Dan Trainor (dtrainor) wrote :

This still doesn't resolve the bug.

I am able to use a FQDN as the 'name' element in a json node definition that imports fine using 'openstack baremetal import --json stack.json' on the CLI, but uploading the file in the UI fails with the following error:

Name may only consist of RFC3986 unreserved characters, to wit: ALPHA / DIGIT / "-" / "." / "_" / "~"

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-ui 1.0.4

This issue was fixed in the openstack/tripleo-ui 1.0.4 release.

Revision history for this message
Jason E. Rist (jason-rist) wrote :

Dan can you test again with the 1.0.4 release?

Revision history for this message
Dan Trainor (dtrainor) wrote :

Confirmed working in 1.0.4 release, thank you.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

This issue was fixed in the openstack/tripleo-ui 1.0.4 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-ui 2.0.0

This issue was fixed in the openstack/tripleo-ui 2.0.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.