Fix syslog hash debug varible at puppet manifest

Bug #1513836 reported by Alexander Kurenyshev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
In Progress
Medium
Richard Berwald
Mitaka
Won't Fix
Medium
Fuel Sustaining

Bug Description

There is a new feature: developer could set additional debug/verbose output per MOS service [1]
But at the logging puppet there is an one mistake in getting hashes from config [2]. Instead of using `hiera('syslog_hash')` we should use `hiera_hash('syslog_hash')` for proper working of debug.

The same things are for ceph-osd.pp [3] and for vcenter.pp [4]

[1] https://bugs.launchpad.net/fuel/+bug/1490744
[2] https://github.com/openstack/fuel-library/blob/5f800ff6c0e0af55d4c39865b83b4d1c343c2068/deployment/puppet/osnailyfacter/modular/logging/logging.pp#L4
[3] https://github.com/openstack/fuel-library/blob/master/deployment/puppet/osnailyfacter/modular/ceph/ceph-osd.pp#L4
[4] https://github.com/openstack/fuel-library/blob/master/deployment/puppet/osnailyfacter/modular/vmware/vcenter.pp#L4

description: updated
Dmitry Klenov (dklenov)
tags: added: area-library
Changed in fuel:
status: New → Confirmed
tags: added: team-bugfix
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 8.0 → 9.0
Revision history for this message
Bug Checker Bot (bug-checker) wrote : Autochecker

(This check performed automatically)
Please, make sure that bug description contains the following sections filled in with the appropriate data related to the bug you are describing:

actual result

version

expected result

steps to reproduce

For more detailed information on the contents of each of the listed sections see https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Here_is_how_you_file_a_bug

tags: added: need-info
no longer affects: fuel/newton
Changed in fuel:
assignee: Michał Górniak (p4cket) → Fuel Sustaining (fuel-sustaining-team)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

Changed in fuel:
assignee: Fuel Sustaining (fuel-sustaining-team) → Richard Berwald (rberwald)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-library (master)

Change abandoned by Andreas Jaeger (<email address hidden>) on branch: master
Review: https://review.opendev.org/380959
Reason: This repo is retired now, no further work will get merged.

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.