wrong regex for ntp server list

Bug #1442668 reported by Igor Gajsin
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Nick Bogdanov

Bug Description

There is "NTP server list" field on Settings tab on FuelUI. When I try to set ip addres to it I got message "Invalid NTP server list".

I think that it's wrong and at least such behavior may confuse a user.

VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "6.1"
  openstack_version: "2014.2-6.1"
  api: "1.0"
  build_number: "302"
  build_id: "2015-04-10_08-54-57"
  nailgun_sha: "d6e351189666e8afa01003e643e63216ef7abd26"
  python-fuelclient_sha: "9208ff4a08dcb674ce2df132399a5aa3ddfac21c"
  astute_sha: "5041b2fb508e6860c3cb96474ca31ec97e549e8b"
  fuellib_sha: "e9c3ba332b05120c967b20260c7b223afc1b4f1a"
  ostf_sha: "4bda5bbf9ea033189f16518032c063d43e4d0e5c"
  fuelmain_sha: "8daac234aea6ac0a98f27871deec039f74f6fdab"

Tags: ui
Changed in fuel:
status: New → Confirmed
Changed in fuel:
assignee: nobody → Fuel Python Team (fuel-python)
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Fuel UI Team (fuel-ui)
importance: Medium → High
Changed in fuel:
assignee: Fuel UI Team (fuel-ui) → Nick Bogdanov (nikolay-bogdanov)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

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

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

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

Reviewed: https://review.openstack.org/173805
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=61a115c3c6f986001bb08b29ea8d5c7472c8e457
Submitter: Jenkins
Branch: master

commit 61a115c3c6f986001bb08b29ea8d5c7472c8e457
Author: Nick Bogdanov <email address hidden>
Date: Wed Apr 15 15:49:43 2015 +0300

    Makes it possible to add IPs to NTP list

    Change-Id: Ief357d66fa4459444b403649d40b8ff43b4170b5
    Fixes-Bug: #1442668

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Anastasia Palkina (apalkina) wrote :

Verified on ISO #323

"build_id": "2015-04-20_07-52-58", "ostf_sha": "4bab9b975ace8d9a305d6e0f112b734de587f847", "build_number": "323", "release_versions": {"2014.2-6.1": {"VERSION": {"build_id": "2015-04-20_07-52-58", "ostf_sha": "4bab9b975ace8d9a305d6e0f112b734de587f847", "build_number": "323", "api": "1.0", "nailgun_sha": "5a4556ea0cf943edffa33788994b7fb7abe150b3", "openstack_version": "2014.2-6.1", "production": "docker", "python-fuelclient_sha": "b4f1ddffd5263489090b65e662173e9e11aafd94", "astute_sha": "bf1751a4fe0d912325e3b4af629126a59c1b2b51", "feature_groups": ["mirantis"], "release": "6.1", "fuelmain_sha": "991fa8feca4afcf05ce85a1e7ebd7c7503b3f212", "fuellib_sha": "36f30ae7f19092a61eebb0522ca20d27468b4cbf"}}}, "auth_required": true, "api": "1.0", "nailgun_sha": "5a4556ea0cf943edffa33788994b7fb7abe150b3", "openstack_version": "2014.2-6.1", "production": "docker", "python-fuelclient_sha": "b4f1ddffd5263489090b65e662173e9e11aafd94", "astute_sha": "bf1751a4fe0d912325e3b4af629126a59c1b2b51", "feature_groups": ["mirantis"], "release": "6.1", "fuelmain_sha": "991fa8feca4afcf05ce85a1e7ebd7c7503b3f212", "fuellib_sha": "36f30ae7f19092a61eebb0522ca20d27468b4cbf"

Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-web (master)

Change abandoned by Vitaly Kramskikh (<email address hidden>) on branch: master
Review: https://review.openstack.org/173776

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

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