Swift configuration parameter audit

Bug #1353581 reported by OpenStack Infra
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-manuals
Fix Released
Medium
Unassigned

Bug Description

https://review.openstack.org/104760
commit 090baa1fa968d85da51dfdfcded2da6430900d7f
Author: Matthew Oliver <email address hidden>
Date: Fri Jul 4 15:56:45 2014 +1000

    Swift configuration parameter audit

    This change is the result of an audit through the config parameters
    provided by swift and how/if they are addressed in the swift
    documentation. The documentation being the sample config files in
    the /etc directory or the documentation.

    This change is only concerned with the config files in etc/ next
    I will look at the documentation in the doc/ folder.

    This change makes the following assumptions:
      - Unless stated otherwise, the commented out parameter in the
        sample configuration is the default for swift.

      - When the default in the code differs from that of the sample
        configuration, the default in the code is correct.

    Container reconciler:
      Parameter: interval
        - code: 30
        - config: 300
      Result: config = 30

    Object Expirer:
      Parameter: recon_cache_path
        - code: /var/cache/swift
        - config: Parameter missing
      Result: Add parameter

    swift-dispersion-populate && swift-dispersion-report
      Parameter: auth_version
        - code: 1.0
        - config: 2.0 (due to being a confusing example of how to setup
                       version 2.0).
      Result: Added 'auth_version = 1.0' to the right section (showing
              default and make the sample configuration for auth version
              2.0 easier to understand.

    swift-drive-audit:
      Parameter: log_file_pattern
        - code: /var/log/kern.*[!.][!g][!z]
        - config: /var/log/kern*
      Result: config = /var/log/kern.*[!.][!g][!z]

      NOTE: swift-drive-audit uses a parameter called device_dir which
            defaults to '/srv/node'. In all other swift binaries/services
            there is a similar parameter called devices which stores the
            same thing. This is an inconsistency which I haven't fixed
            as this could break existing swift clusters out in the wild.

    Proxy Server:
      Parameter: object_chunk_size
        - code: 65536
        - config: 8192
      Result: config = 65536

      Parameter: client_chunk_size
        - code: 65536
        - config: 8192
      Result: config = 65536

      Parameter: strict_cors_mode
        - code: True
        - config: No parameter
      Result: config = True

    Account and Container replicator configuration confusion:
      NOTES:
        The account and container replicators have parameters:
          - interval
          - run_pause

        Both of these are loaded into the same variable in code:
          self.interval = int(conf.get('interval') or
                              conf.get('run_pause') or 30)

        If a user sets both to different values then interval is used.
      Result: Update the configuration to make this more clear.

    DocImpact
    Change-Id: Iaadbb1a6284f8b3e0801bc343b29772f70f4bf6e

Revision history for this message
Andreas Jaeger (jaegerandi) wrote :

These are changes for the autogenerated tables

tags: added: autogenerate-config-docs
Changed in openstack-manuals:
importance: Undecided → Medium
status: New → Confirmed
milestone: none → juno
Revision history for this message
Tom Fifield (fifieldt) wrote :

These made it in.

Changed in openstack-manuals:
status: Confirmed → Fix Released
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.