nixos/openldap: add bootstrapContents option#92544
Closed
celestefox wants to merge 1 commit intoNixOS:masterfrom
Closed
nixos/openldap: add bootstrapContents option#92544celestefox wants to merge 1 commit intoNixOS:masterfrom
celestefox wants to merge 1 commit intoNixOS:masterfrom
Conversation
Adds a bootstrapContents option, which holds LDIF that is used to initialise the LDAP database if it did not exist previously. Motivation primarily came from that there is currently a problem where you cannot start openldap without an existing database, because of slaptest, but cannot easily create an empty database without running slapd manually.
ofborg
bot
added
6.topic: nixos
10 tasks
|
I marked this as stale due to inactivity. → More info |
stale
bot
added
the
2.status: stale
SuperSandro2000
added
the
2.status: merge conflict
stale
bot
removed
the
2.status: stale
|
I marked this as stale due to inactivity. → More info |
stale
bot
added
the
2.status: stale
stale
bot
removed
the
2.status: stale
Member
|
Sorry. This went under the radar. If you could fix merge conflicts. I would test and merge this. |
|
I marked this as stale due to inactivity. → More info |
stale
bot
added
the
2.status: stale
13 tasks
Contributor
Author
|
While this just got my attention: I have gone way off the path that needs this, and do not intend to make any changes, so I'm closing this pull request. |
kwohlfahrt
added a commit
to kwohlfahrt/nixpkgs
that referenced
this pull request
Jun 6, 2022
This addresses the original concern behind NixOS#92544
Mic92
pushed a commit
to Mic92/nixpkgs
that referenced
this pull request
Jun 29, 2022
This addresses the original concern behind NixOS#92544
Artturin
pushed a commit
to Artturin/nixpkgs
that referenced
this pull request
Jul 27, 2022
This addresses the original concern behind NixOS#92544
Janik-Haag
added
the
12.first-time contribution
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation for this change
Currently, you cannot start openldap without an existing database, because of slaptest being run in preStart, but cannot easily create an empty database without running slapd/slapadd manually, creating directories, fixing permissions, etc. This adds a bootstrapFile, which can be used to accomplish this.
Note: the existing test does not test this addition, but it has been tested in practice with Nixops, as well, albeit on 20.03. This may not be the best way to do any of this, but if so, I don't know what is.
Things done
sandboxinnix.confon non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"./result/bin/)nix path-info -Sbefore and after)