NTP servers list settings allow to specify any data

Bug #1413561 reported by Denis Klepikov
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Unassigned

Bug Description

VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "6.0"
  api: "1.0"
  build_number: "58"
  build_id: "2014-12-26_14-25-46"
  astute_sha: "16b252d93be6aaa73030b8100cf8c5ca6a970a91"
  fuellib_sha: "fde8ba5e11a1acaf819d402c645c731af450aff0"
  ostf_sha: "a9afb68710d809570460c29d6c3293219d3624d4"
  nailgun_sha: "5f91157daa6798ff522ca9f6d34e7e135f150a90"
  fuelmain_sha: "81d38d6f2903b5a8b4bee79ca45a54b76c1361b8"

How to reproduce:
create new cluster, via Fuel UI settings tab change "NTP servers" settings to any data, save.

Add nodes to cluster and deploy it.

look into /etc/astute.yaml

external_ntp:
  ntp_list: "@#$%^&*(), @#$%^&*()!"
  metadata:
    label: Upstream NTP
    weight: 100

Revision history for this message
Denis Klepikov (dklepikov) wrote :
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

This is a Fuel python team bug.

Dmitry Pyzhov (dpyzhov)
no longer affects: fuel/6.1.x
Revision history for this message
Denis Klepikov (dklepikov) wrote :

VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "6.1"
  api: "1.0"
  build_number: "230"
  build_id: "2015-03-25_22-54-44"
  nailgun_sha: "64a3d380cefde4f341cd39be350c4c9f78b59b7d"
  python-fuelclient_sha: "e5e8389d8d481561a4d7107a99daae07c6ec5177"
  astute_sha: "631f96d5a09cc48bfbddcbf056b946c8a80438f0"
  fuellib_sha: "345a98b34dd0cd450a45d405ac47a6a9fa48b6d8"
  ostf_sha: "a4cf5f218c6aea98105b10c97a4aed8115c15867"
  fuelmain_sha: "320b5f46fc1b2798f9e86ed7df51d3bda1686c10"

Dmitry Pyzhov (dpyzhov)
tags: added: feature-validation
Revision history for this message
Vitaly Kramskikh (vkramskikh) wrote :

A regexp should be added for DNS field. I suggest assign this bug to the person who introduced this section.

Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Ivan Kliuk (ivankliuk)
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/170525

Changed in fuel:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/170525
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=0f6a8b8486d01a5de8d0fd52482b48ab14072d3e
Submitter: Jenkins
Branch: master

commit 0f6a8b8486d01a5de8d0fd52482b48ab14072d3e
Author: Ivan Kliuk <email address hidden>
Date: Fri Apr 3 17:51:07 2015 +0300

    Add regex for checking NTP and DNS servers list

      Update openstack.yaml with regex and error message.

    Related to change: https://review.openstack.org/#/c/170810/
    Change-Id: I357aaf79351eef96c70b2138b01f447723bbcad8
    Closes-Bug: 1413561

Changed in fuel:
status: In Progress → Fix Committed
tags: added: support
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.