Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • forge forge
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 2
    • Merge requests 2
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • ISC Open Source ProjectsISC Open Source Projects
  • forgeforge
  • Merge requests
  • !354

don't require variables for server2 to be set

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Andrei Pavel requested to merge defaults into master Jan 30, 2023
  • Overview 0
  • Commits 1
  • Pipelines 1
  • Changes 1

server2 is used in a minority of tests, so like all the other configuration settings that are not strictly required, let's also not mandate them because this makes the user that is not interested in HA tests to set them to empty string. By convention, None is for mandatory settings and gets stuck at raise Exception('Cannot find %s in init_all.py' % key), empty string is for settings that are not used in tests.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: defaults