Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • O OpenLDAP
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Bugzilla
    • Bugzilla
  • Merge requests 7
    • Merge requests 7
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • openldap
  • OpenLDAP
  • Merge requests
  • !58

Fix slaptest in test077

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Ryan Tandy requested to merge ryan/openldap:test077-slaptest into master Apr 26, 2020
  • Overview 4
  • Commits 4
  • Pipelines 7
  • Changes 4

The libtool wrapper scripts lose argv[0] when execing the real binary.

In the CI Docker container, where the build runs as root, this was actually starting a real slapd on the default port, and converting the config as a side-effect.

Outside Docker, running as a non-root user, this slapd just failed to start, and didn't convert the config either.

Using slapd -Tt fixes the issue but also prints a warning from slaptest since the database hasn't been initialized yet.

Dynamic config isn't actually used in this test script, so let's just run slapd off the config file directly.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: test077-slaptest