Bug 2217793 - cinder::scheduler::filter class is not called anywhere.
Summary: cinder::scheduler::filter class is not called anywhere.
Keywords:
Status: NEW
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: puppet-tripleo
Version: 17.0 (Wallaby)
Hardware: x86_64
OS: Linux
low
medium
Target Milestone: ---
: ---
Assignee: Cinder Bugs List
QA Contact: Evelina Shames
URL:
Whiteboard:
Depends On:
Blocks: 2222869
TreeView+ depends on / blocked
 
Reported: 2023-06-27 06:08 UTC by Keigo Noha
Modified: 2023-08-04 00:48 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker OSP-26101 0 None None None 2023-06-27 06:08:45 UTC

Description Keigo Noha 2023-06-27 06:08:12 UTC
Description of problem:
puppet-cinder has a class, cinder::scheduler::filter class.
However, the class is not included in puppet-cinder and puppet-tripleo.

Due to that, a user who wants to set the scheduler filter via puppet cannot configure the value to the relevant option

Version-Release number of selected component (if applicable):
OSP17.0 and upstream.

How reproducible:
Everytime when overcloud deploy is run.

Expected results:
The assigned values to the parameters in cinder::scheduler::filter are configured.

Actual results:
The assigned values to the parameters in cinder::scheduler::filter are not configured.


Additional info:
Currently, the user can configure the parameter via cinder_config.

Comment 1 Keigo Noha 2023-07-06 01:28:56 UTC
Hi DFG:Storage team,

Do you have any updates on this bugzilla?

Best regards,
Keigo Noha

Comment 2 Eric Harney 2023-07-11 12:14:38 UTC
Which Cinder options are they wanting to set that can't be set currently?

Comment 3 Alan Bishop 2023-07-11 14:02:29 UTC
The cinder::scheduler::filter class supports a single parameter that configures cinder's scheduler_default_filters.

It's true the class isn't referenced anywhere, and could be added to puppet-tripleo. However, without a corresponding THT paramter, you're forced to use ExtraConfig to set the corresponding hiera data. It's a tradeoff between these approaches.

Current:
========

parameter_defaults:
  ControllerExtraConfig:
    cinder::config::cinder_config:
      DEFAULT/scheduler_default_filters:
        value: <list of scheduler default filters>

Proposed: [1]
=========
parameter_defaults:
  ControllerExtraConfig:
    cinder::scheduler::filter::scheduler_default_filters: <list of scheduler default filters>

[1] If we update puppet_tripleo to include the cinder::scheduler::filter class

Comment 4 Keigo Noha 2023-07-12 04:14:48 UTC
Hi Eric,

I think Alan already replied to your question, the class provides a single interface, scheduler_driver[1].

[1] https://github.com/openstack/puppet-cinder/blob/master/manifests/scheduler.pp

Currently, we have a workaround which uses cinder_config. So, it is not urgent issue but is should be fixed.

Best regards,
Keigo Noha


Note You need to log in before you can comment on or make changes to this bug.