Cinder doc is not reproducible

Bug #1956601 reported by Thomas Goirand
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Incomplete
Low
Unassigned

Bug Description

Hi,

While working on Cinder package reproducibility [1], I noticed that Cinder documentation is not reproducible, because in cinder/cmd/backup.py contains:

backup_cmd_opts = [
    cfg.IntOpt('backup_workers',
               default=1, min=1, max=processutils.get_worker_count(),

The problem is that processutils.get_worker_count() will have a different output depending on the machine where the package is built. It is also kind of wrong, because the cinder.conf will contain a max= value that is likely not going to be matching the number of process that the server is capable of.

Please fix this issue by having a predictable value as max= option.

[1] https://reproducible-builds.org/

Revision history for this message
Thomas Goirand (thomas-goirand) wrote :

FYI, I sent this patch to oslo.config to fix this problem:

I1e4ff0ba57323232736b94e1e49f9e899fc78487

If one has time, I appreciate comments and reviews.

Revision history for this message
Sofia Enriquez (lsofia-enriquez) wrote :

Greetings Thomas Goirand, I can't find the cinder doc you're referring to in the link. Would you mind sharing the documentation link you are using?

Oslo path https://review.opendev.org/c/openstack/oslo.config/+/823710

Changed in cinder:
importance: Undecided → Low
status: New → Incomplete
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.