Sahara

Warning message about missing policy.d folder during Sahara start

Bug #1391694 reported by Andrew Lazarev on 2014-11-12

This bug affects 1 person

	Status	Importance	Assigned to	Milestone
OpenStack Compute (nova)	Fix Released	Low	Davanum Srinivas (DIMS)	OpenStack Compute (nova) 2015.1.0 "kilo"
Sahara	Fix Released	Low	Michael McCune	Sahara 2015.1.0 "kilo"
oslo-incubator	Fix Released	Low	Davanum Srinivas (DIMS)	oslo-incubator 2015.1.0 "kilo"

Bug Description

2014-11-11 16:14:05.786 403 WARNING sahara.openstack.common.policy [-] Can not find policy directories policy.d

Example: https://sahara.mirantis.com/logs/31/133131/2/check/gate-sahara-integration-vanilla-1/9ca6d41/console.html

Policy library from oslo searches for policy in directories specified by 'policy_dirs' parameter and warns if directory doesn't exist. Default value is ['policy.d'].

Need to check what other projects do about this. I have never seen such warnings in other openstack projects.

Tags:

Andrew Lazarev (alazarev) on 2014-11-12

Changed in sahara:
importance:	Undecided → Low
status:	New → Confirmed
milestone:	none → kilo-1
summary:	- Warning message during Sahara start + Warning message about missing policy.d folder during Sahara start

Revision history for this message

melanie witt (melwitt) wrote on 2014-12-02:

Just noticed this in nova:

2014-12-02 00:32:49.506 DEBUG nova.openstack.common.fileutils [req-a6efc5a7-df21-4d4e-9d37-37761c633416 demo demo] Reloading cached file /etc/nova/policy.json from (pid=8046) read_cached_file /opt/stack/nova/nova/openstack/common/fileutils.py:62
2014-12-02 00:32:49.510 DEBUG nova.openstack.common.policy [req-a6efc5a7-df21-4d4e-9d37-37761c633416 demo demo] Rules successfully reloaded from (pid=8046) _load_policy_file /opt/stack/nova/nova/openstack/common/policy.py:267
2014-12-02 00:32:49.511 WARNING nova.openstack.common.policy [req-a6efc5a7-df21-4d4e-9d37-37761c633416 demo demo] Can not find policy directories policy.d

Changed in nova:
importance:	Undecided → Low
status:	New → Confirmed

Deliang Fan (vanderliang) on 2014-12-02

Changed in nova:
assignee:	nobody → Deliang Fan (vanderliang)

Revision history for this message

Hans Lindgren (hanlind) wrote on 2014-12-02:

This is fixed with a sync of oslo-incubator change https://review.openstack.org/137191

Davanum Srinivas (DIMS) (dims-v) on 2014-12-03

Changed in oslo-incubator:
status:	New → Fix Committed
importance:	Undecided → Low
assignee:	nobody → Davanum Srinivas (DIMS) (dims-v)

Sergey Lukjanov (slukjanov) on 2014-12-16

Changed in sahara:
milestone:	kilo-1 → kilo-2

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2014-12-17: Fix proposed to nova (master)

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

Changed in nova:
assignee:	Deliang Fan (vanderliang) → Davanum Srinivas (DIMS) (dims-v)
status:	Confirmed → In Progress

Thierry Carrez (ttx) on 2014-12-22

Changed in oslo-incubator:
milestone:	none → kilo-1
status:	Fix Committed → Fix Released

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-01-19: Fix merged to nova (master)

Reviewed: https://review.openstack.org/142581
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=5359521df843aa1466eb4cede77d16882163fec9
Submitter: Jenkins
Branch: master

commit 5359521df843aa1466eb4cede77d16882163fec9
Author: Davanum Srinivas <email address hidden>
Date: Wed Dec 17 16:12:22 2014 -0500

Sync with latest oslo-incubator

    Changes Include:
    80c4751 Remove unnecessary import of eventlet
    442fc22 Added graceful argument on Service.stop method
    cf429e5 Remove extra white space in log message
    2a67d23 Support non-lowercase uuids in is_uuid_like
    5985b35 Prefer delayed %r formatting over explicit repr use
    07e9b32 Improving docstrings for policy API...
    b19af08 Don't log missing policy.d as a warning...
    2324c77 Add rule overwrite flag to Enforcer class...
    6166a96 Fixed a problem with neutron http policy check...
    dbe8672 Expanding the help text for policy_dirs...
    8018ae2 Reports: Use sig handler traceback for curr thread...
    0c53201 Report: Propagate view_type through collections...
    cfde127 policy: add a missing staticmethod declaration...
    ac4330d Make use_syslog=True log to syslog via /dev/log...
    66bad01 Allow tempest to use new log w/o oslo.i18n...
    0dd719e Use same mask pattern for reports as mask_password

    NOTE: A comment was changed in versionutils.py inadvertently
    in a previous commit (essentially replacing 'oslo-incubator'
    with 'nova-incubator') due to a glitch in the sync script. In
    this commit we are changing that comment back to oslo-incubator
    as well. (Yes, update.py has been fixed in the following change
    Ib7784262abacf262b48e23b7ce96fafe279eea14)

Closes-Bug: #1391694
Change-Id: Ib2b7fec7725ce912535a2e63ff49903545313287

Changed in nova:
status:	In Progress → Fix Committed

Michael McCune (mimccune) on 2015-01-26

Changed in sahara:
assignee:	nobody → Michael McCune (mimccune)

Revision history for this message

Michael McCune (mimccune) wrote on 2015-01-26:

this was fixed in sahara in this review: https://review.openstack.org/#/c/139149/

Changed in sahara:
status:	Confirmed → Fix Released

Sergey Lukjanov (slukjanov) on 2015-01-29

Changed in sahara:
milestone:	kilo-2 → kilo-1

Thierry Carrez (ttx) on 2015-02-05

Changed in nova:
milestone:	none → kilo-2
status:	Fix Committed → Fix Released

Thierry Carrez (ttx) on 2015-04-30

Changed in oslo-incubator:
milestone:	kilo-1 → 2015.1.0

Thierry Carrez (ttx) on 2015-04-30

Changed in sahara:
milestone:	kilo-1 → 2015.1.0

Thierry Carrez (ttx) on 2015-04-30

Changed in nova:
milestone:	kilo-2 → 2015.1.0

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

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