ironic-inspector: traceback when 'noauth' plugin is used

Bug #1748263 reported by Markos Chandras
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic Inspector
Fix Released
Critical
Pavlo Shchelokovskyy

Bug Description

When ironic is configured with the 'noauth' plugin the following traceback is reported when ironic-inspector is starting

Unhandled error: MissingRequiredOptions: Auth plugin requires parameters which were not given: auth_url
Traceback (most recent call last):
  File "/opt/stack/bifrost/bin/ironic-inspector", line 11, in <module>
    sys.exit(main())
  File "/opt/stack/bifrost/lib/python2.7/site-packages/ironic_inspector/cmd/all.py", line 26, in main
    server.run()
  File "/opt/stack/bifrost/lib/python2.7/site-packages/ironic_inspector/wsgi_service.py", line 183, in run
    self._init_host()
  File "/opt/stack/bifrost/lib/python2.7/site-packages/ironic_inspector/wsgi_service.py", line 125, in _init_host
    callables=[(driver.get_periodic_sync_task(), None, None),
  File "/opt/stack/bifrost/lib/python2.7/site-packages/ironic_inspector/pxe_filter/base.py", line 191, in get_periodic_sync_task
    ironic = ir_utils.get_client()
  File "/opt/stack/bifrost/lib/python2.7/site-packages/ironic_inspector/common/ironic.py", line 87, in get_client
    IRONIC_SESSION = keystone.get_session('ironic')
  File "/opt/stack/bifrost/lib/python2.7/site-packages/ironic_inspector/common/keystone.py", line 36, in get_session
    auth = loading.load_auth_from_conf_options(CONF, group)
  File "/opt/stack/bifrost/lib/python2.7/site-packages/keystoneauth1/loading/conf.py", line 135, in load_from_conf_options
    return plugin.load_from_options_getter(_getter, **kwargs)
  File "/opt/stack/bifrost/lib/python2.7/site-packages/keystoneauth1/loading/base.py", line 187, in load_from_options_getter
    return self.load_from_options(**kwargs)
  File "/opt/stack/bifrost/lib/python2.7/site-packages/keystoneauth1/loading/base.py", line 162, in load_from_options
    raise exceptions.MissingRequiredOptions(missing_required)
MissingRequiredOptions: Auth plugin requires parameters which were not given: auth_url

Changed in ironic-inspector:
assignee: nobody → Markos Chandras (hwoarang)
status: New → In Progress
Changed in ironic-inspector:
assignee: Markos Chandras (hwoarang) → Pavlo Shchelokovskyy (pshchelo)
Dmitry Tantsur (divius)
Changed in ironic-inspector:
importance: Undecided → Critical
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic-inspector (master)

Reviewed: https://review.openstack.org/542206
Committed: https://git.openstack.org/cgit/openstack/ironic-inspector/commit/?id=fbeb0783e4b2f1fc5f44da7f7b704ce48bfe0087
Submitter: Zuul
Branch: master

commit fbeb0783e4b2f1fc5f44da7f7b704ce48bfe0087
Author: Markos Chandras <email address hidden>
Date: Thu Feb 8 14:03:13 2018 +0000

    ironic_inspector: ironic: Fix 'auth_type' when 'auth_strategy' is used

    We should override the 'auth_type' to 'none' when using the old way of
    setting up authentication with 'auth_strategy' so we can override the
    default 'auth_type' value before getting the session information.
    This fixes the following issue

    Unhandled error: MissingRequiredOptions: Auth plugin requires parameters which were not given: auth_url

    This also adds a new testcase to test the strategy='noauth' scenario.

    Closes-Bug: #1748263
    Change-Id: I875e2b17f5c6829ad81f86c32959cb106bf57e53

Changed in ironic-inspector:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic-inspector (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/543577

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ironic-inspector (stable/queens)

Reviewed: https://review.openstack.org/543577
Committed: https://git.openstack.org/cgit/openstack/ironic-inspector/commit/?id=6d3ea7e9c2ed81e4bf230190351ae94167f841c6
Submitter: Zuul
Branch: stable/queens

commit 6d3ea7e9c2ed81e4bf230190351ae94167f841c6
Author: Markos Chandras <email address hidden>
Date: Thu Feb 8 14:03:13 2018 +0000

    ironic_inspector: ironic: Fix 'auth_type' when 'auth_strategy' is used

    We should override the 'auth_type' to 'none' when using the old way of
    setting up authentication with 'auth_strategy' so we can override the
    default 'auth_type' value before getting the session information.
    This fixes the following issue

    Unhandled error: MissingRequiredOptions: Auth plugin requires parameters which were not given: auth_url

    This also adds a new testcase to test the strategy='noauth' scenario.

    Closes-Bug: #1748263
    Change-Id: I875e2b17f5c6829ad81f86c32959cb106bf57e53
    (cherry picked from commit fbeb0783e4b2f1fc5f44da7f7b704ce48bfe0087)

tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ironic-inspector 7.2.0

This issue was fixed in the openstack/ironic-inspector 7.2.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ironic-inspector 7.3.0

This issue was fixed in the openstack/ironic-inspector 7.3.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.