Skip to content
Snippets Groups Projects

Update example config file and systemd service file for grpc frontend

Merged Konstantina Skovola requested to merge 1077-update-options-for-grpc-frontend into main

Summary

Additional Required Actions

  • Requires manual tests in pre-production: YES/NO
  • Requires a documentation update: YES/NO

References

Closes #1077 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Michael Davis
  • Grepping through the code, we expect the following configuration options:

    cta.archivefile.max_size_gb
    cta.archivefile.zero_length_files_forbidden
    cta.archivefile.zero_length_files_forbidden_vo_exception_list
    cta.catalogue.numberofconnections
    cta.instance_name
    cta.log.format
    cta.log.level
    cta.log.log_header
    cta.log.url
    cta.ns.config
    cta.objectstore.backendpath
    cta.repack.repack_buffer_url
    cta.repack.repack_max_files_to_select
    cta.schedulerdb.enable_repack_requests
    cta.schedulerdb.enable_user_requests
    cta.schedulerdb.numberofthreads
    cta.schedulerdb.retrieve_queue_cache_max_age_secs
    cta.schedulerdb.scheduler_backend_name
    cta.schedulerdb.tape_cache_max_age_secs
    cta.schedulerdb.threadstacksize_mb
    cta.verification.mount_policy
    port
    threads
    TLS
    TlsCert
    TlsChain
    TlsKey

    The following options exist in the code but not in the config file:

    • cta.catalogue.numberofconnections
    • cta.repack.repack_max_files_to_select

    The port and threads options should be scoped. Especially the scope of threads is unclear, as we also have cta.schedulerdb.numberofthreads.

    We have numberofthreads in one case and threads for the other case. Choose one and be consistent.

    The Tls options should be scoped tls.*

  • added 1 commit

    Compare with previous version

  • added 2 commits

    • a8609024 - Use scoped options
    • d5fb5b88 - Install config file with the frontend-grpc rpm

    Compare with previous version

  • added 7 commits

    • d5fb5b88...59c0eec7 - 4 commits from branch main
    • 3889b6eb - Update example config file and systemd service file for grpc frontend
    • 8f64ba8a - Use scoped options
    • 839bab87 - Install config file with the frontend-grpc rpm

    Compare with previous version

  • added 1 commit

    • 35dd3f8f - Install config file with the frontend-grpc rpm

    Compare with previous version

  • Added a commit to also install the example config file with the rpm, since it was pointed out in the community forum issue that it is not installed.

  • added 1 commit

    • f2d2ac73 - Fix improper handing of enable_{repack, user}_requests config options

    Compare with previous version

  • added 5 commits

    • ebc2b163 - 1 commit from branch main
    • 575d9b74 - Update example config file and systemd service file for grpc frontend
    • 690447bb - Use scoped options
    • 014b83bb - Install config file with the frontend-grpc rpm
    • 078fb9b3 - Fix improper handing of enable_{repack, user}_requests config options

    Compare with previous version

  • Konstantina Skovola resolved all threads

    resolved all threads

  • **** added 1 commit

    added 1 commit

    • 31156c2c - Applied clang format to changed lines

    Compare with previous version

  • added 1 commit

    • 529122c4 - Fix improper handing of enable_{repack, user}_requests config options

    Compare with previous version

  • Konstantina Skovola removed review request for @mdavis

    removed review request for @mdavis

  • added 5 commits

    • 3682bffe - 1 commit from branch main
    • f51bd363 - Update example config file and systemd service file for grpc frontend
    • 99295fbc - Use scoped options
    • c1d657be - Install config file with the frontend-grpc rpm
    • 3b1883a5 - Fix improper handing of enable_{repack, user}_requests config options

    Compare with previous version

  • Michael Davis requested review from @mdavis

    requested review from @mdavis

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading