Fuel-menu can't save settings

Bug #1513494 reported by Fedor Zhadaev on 2015-11-05
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Critical
Fedor Zhadaev

Bug Description

I see the next error when I try to save settings in fuel-menu:

Traceback (most recent call last):
  File "/usr/bin/fuelmenu", line 9, in <module>
    load_entry_point('fuelmenu==8.0.0', 'console_scripts', 'fuelmenu')()
  File "/usr/lib/python2.6/site-packages/fuelmenu/fuelmenu.py", line 476, in main
    setup()
  File "/usr/lib/python2.6/site-packages/fuelmenu/fuelmenu.py", line 340, in setup
    FuelSetup()
  File "/usr/lib/python2.6/site-packages/fuelmenu/fuelmenu.py", line 92, in __init__
    self.main()
  File "/usr/lib/python2.6/site-packages/fuelmenu/fuelmenu.py", line 276, in main
    self.mainloop.run()
  File "/usr/lib64/python2.6/site-packages/urwid/main_loop.py", line 271, in run
    self.screen.run_wrapper(self._run)
  File "/usr/lib64/python2.6/site-packages/urwid/raw_display.py", line 241, in run_wrapper
    return fn()
  File "/usr/lib64/python2.6/site-packages/urwid/main_loop.py", line 336, in _run
    self.event_loop.run()
  File "/usr/lib64/python2.6/site-packages/urwid/main_loop.py", line 707, in run
    self._loop()
  File "/usr/lib64/python2.6/site-packages/urwid/main_loop.py", line 786, in _loop
    self._watch_files[fd]()
  File "/usr/lib64/python2.6/site-packages/urwid/main_loop.py", line 387, in _update
    self.process_input(keys)
  File "/usr/lib64/python2.6/site-packages/urwid/main_loop.py", line 487, in process_input
    k = self._topmost_widget.keypress(self.screen_size, k)
  File "/usr/lib64/python2.6/site-packages/urwid/container.py", line 1102, in keypress
    return self.body.keypress( (maxcol, remaining), key )
  File "/usr/lib64/python2.6/site-packages/urwid/listbox.py", line 986, in keypress
    key = focus_widget.keypress((maxcol,),key)
  File "/usr/lib64/python2.6/site-packages/urwid/container.py", line 2240, in keypress
    key = w.keypress((mc,) + size[1:], key)
  File "/usr/lib64/python2.6/site-packages/urwid/container.py", line 1559, in keypress
    key = self.focus.keypress(tsize, key)
  File "/usr/lib64/python2.6/site-packages/urwid/decoration.py", line 384, in keypress
    return self._original_widget.keypress((maxcol, self.height), key)
  File "/usr/lib64/python2.6/site-packages/urwid/decoration.py", line 836, in keypress
    return self._original_widget.keypress((maxcol,maxrow-top-bottom), key)
  File "/usr/lib64/python2.6/site-packages/urwid/listbox.py", line 986, in keypress
    key = focus_widget.keypress((maxcol,),key)
  File "/usr/lib64/python2.6/site-packages/urwid/wimp.py", line 534, in keypress
    self._emit('click')
  File "/usr/lib64/python2.6/site-packages/urwid/widget.py", line 463, in _emit
    signals.emit_signal(self, name, self, *args)
  File "/usr/lib64/python2.6/site-packages/urwid/signals.py", line 120, in emit
    result |= bool(callback(*args_copy))
  File "/usr/lib/python2.6/site-packages/fuelmenu/modules/saveandquit.py", line 51, in save_and_quit
    if self.save():
  File "/usr/lib/python2.6/site-packages/fuelmenu/modules/saveandquit.py", line 57, in save
    results, modulename = self.parent.global_save()
  File "/usr/lib/python2.6/site-packages/fuelmenu/fuelmenu.py", line 325, in global_save
    if module.apply(None):
  File "/usr/lib/python2.6/site-packages/fuelmenu/modules/cobblerconf.py", line 290, in apply
    responses = self.check(args)
  File "/usr/lib/python2.6/site-packages/fuelmenu/modules/cobblerconf.py", line 275, in check
    if old_range[0] not in new_range:
TypeError: 'set' object does not support indexing

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

Changed in fuel:
status: New → In Progress
Dmitry Pyzhov (dpyzhov) on 2015-11-05
tags: added: regression-8.0
Dmitry Pyzhov (dpyzhov) on 2015-11-05
tags: added: area-python

Reviewed: https://review.openstack.org/242094
Committed: https://git.openstack.org/cgit/openstack/fuel-menu/commit/?id=5ab60f1b7bbd770a204f1d5048a95c9c313ea43f
Submitter: Jenkins
Branch: master

commit 5ab60f1b7bbd770a204f1d5048a95c9c313ea43f
Author: Fedor Zhadaev <email address hidden>
Date: Thu Nov 5 17:48:31 2015 +0300

    Change return-type from set to list

     * Now we are able to get elements by index

    Change-Id: I71fc372ab0b3e410ea397a80e96c4ed83b43d185
    Closes-bug: #1513494

Changed in fuel:
status: In Progress → Fix Committed
tags: added: on-verification
Grigory Mikhailov (gmikhailov) wrote :
tags: removed: on-verification
Anastasia Palkina (apalkina) wrote :

Verified on ISO #327

VERSION:
  feature_groups:
    - mirantis
  production: "docker"
  release: "8.0"
  api: "1.0"
  build_number: "327"
  build_id: "327"
  fuel-nailgun_sha: "ec25ae8df28e1f1d87da653c5aab2711dff729f6"
  python-fuelclient_sha: "7c7a756fb6a3d091851c060003a2965c67aa353d"
  fuel-agent_sha: "d354cbe61b055db848a15ce66fb7ae92178d2c0a"
  fuel-nailgun-agent_sha: "a33a58d378c117c0f509b0e7badc6f0910364154"
  astute_sha: "c56dfde2da034151a7e707b381c4cf9d213b4ba2"
  fuel-library_sha: "bcc3d042a47547f6ad826360a85ef21dcaf25882"
  fuel-ostf_sha: "b2ebf15a3530b5c7b57707acf8642c1c3ac71bd8"
  fuel-mirror_sha: "8100acb3a566358d0d4ecc66de32d39626263028"
  fuelmenu_sha: "2942a85796d37f09ba8c8c6d762d8813292cf0d4"
  shotgun_sha: "cacb93cbc28910ff0dc38f30a855efa9af50d8ce"
  network-checker_sha: "d443ef47abeda58d319bc8d33d5005dd09440a02"
  fuel-upgrade_sha: "718aa3d7021fee2970f0fa6791cf5188578cc516"
  fuelmain_sha: "3faa824728ce60734abe602ff3778976f8a16eed"

Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers