Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

dsconf for nsslapd-state #4796

Closed
falon opened this issue Jun 9, 2021 · 4 comments
Closed

dsconf for nsslapd-state #4796

falon opened this issue Jun 9, 2021 · 4 comments
Assignees
@mreynolds389
Labels
CLI CLI tools cockpit Cockpit UI Plugin In JIRA ticket is in JIRA priority_high need urgent fix / highly valuable / easy to fix

Comments

@falon
Copy link

falon commented Jun 9, 2021

Is your feature request related to a problem? Please describe.
With the old console on 389-ds 1.3.x I can manage the nsslapd-state for all suffixes.
In the new cockpit console I can't find an option to manage this attribute. It's always backend.
dsconf doesn't manage this attribute too.

Describe the solution you'd like
At least, a new option in dsconf could be provided. For instance:

dsconf instance backend suffix set --state

Allowed values:

  • backend: The back-end database processes all operations.
  • disabled: The database is not available for processing operations. The server returns a No
    such search object error in response to requests made by client applications.
  • referral: Directory Server returns a referral URL for requests to this suffix.
  • referral on update: The database is used for all operations. Only for update requests is a
    referral sent.

Describe alternatives you've considered
Now I have to force an ldapmodify to set the value:

dn: cn=<suffix name>,cn=mapping tree,cn=config
changeype: modify
replace: nsslapd-state
nsslapd-state: referral on update
@falon falon added the needs triage The issue will be triaged during scrum label Jun 9, 2021
@mreynolds389
Copy link
Contributor

This can definitely be added to the CLI and UI. We are currently working on porting the UI to Patternfly 4 so this is a good time to add this change...

@mreynolds389 mreynolds389 added priority_medium good value but complex/risky/not crucial CLI CLI tools cockpit Cockpit UI Plugin priority_high need urgent fix / highly valuable / easy to fix and removed needs triage The issue will be triaged during scrum priority_medium good value but complex/risky/not crucial labels Jun 30, 2021
@mreynolds389
Copy link
Contributor

Downstream bug:

https://bugzilla.redhat.com/show_bug.cgi?id=1989982

@mreynolds389 mreynolds389 added the In JIRA ticket is in JIRA label Aug 4, 2021
@progier389
Copy link
Contributor

just a point about that one: by default dsconf should not allow to change the state of a replicated backend.
(because replication sometimes overrides the state (and the referrals) and it will be confusing for user.
If we really want to offer the referral on update feature we also need to improve the ns-sate handling within the server
(for example by having 3 attributes: ns-state (which is only readable and computed from nsstate;replica and ns-state;config)
to insure there is no risk to revert in backend mode unexpectedly

@mreynolds389 mreynolds389 self-assigned this Sep 8, 2021
@mreynolds389 mreynolds389 mentioned this issue Sep 10, 2021
Merged
mreynolds389 added a commit to mreynolds389/389-ds-base that referenced this issue Sep 11, 2021
@mreynolds389
Issue 4796 - Add support for nsslapd-state to CLI & UI
7c10c39 
Description:  Add support for nsslapd-state to lib389 and UI.  Also
              added a check to prevent the changing of nsslapd-state
              for replicated suffixes.

              Also did a little UI cleanup where a bottom margin was added
              to the bottom of pages instead of using <hr> to create the gap.

relates: 389ds#4796

Reviewed by: jachapman & spichugi(Thanks!!)
mreynolds389 added a commit that referenced this issue Sep 11, 2021
@mreynolds389
Issue 4796 - Add support for nsslapd-state to CLI & UI
735c5a7 
Description:  Add support for nsslapd-state to lib389 and UI.  Also
              added a check to prevent the changing of nsslapd-state
              for replicated suffixes.

              Also did a little UI cleanup where a bottom margin was added
              to the bottom of pages instead of using <hr> to create the gap.

relates: #4796

Reviewed by: jachapman & spichugi(Thanks!!)
mreynolds389 added a commit that referenced this issue Sep 11, 2021
@mreynolds389
Issue 4796 - Add support for nsslapd-state to CLI & UI
2e4387d 
Description:  Add support for nsslapd-state to lib389 and UI.  Also
              added a check to prevent the changing of nsslapd-state
              for replicated suffixes.

              Also did a little UI cleanup where a bottom margin was added
              to the bottom of pages instead of using <hr> to create the gap.

relates: #4796

Reviewed by: jachapman & spichugi(Thanks!!)
@mreynolds389
Copy link
Contributor

mreynolds389 commented Sep 11, 2021 •
edited

4634ec6..2e4387d 389-ds-base-2.0 -> 389-ds-base-2.0
012a6a3..ef01f6d 389-ds-base-1.4.4 -> 389-ds-base-1.4.4
57b6480..9d0c9b8 389-ds-base-1.4.3 -> 389-ds-base-1.4.3

mreynolds389 added a commit that referenced this issue Sep 11, 2021
@mreynolds389
Issue 4796 - Add support for nsslapd-state to CLI & UI
ef01f6d 
Description:

Add support for nsslapd-state to lib389 and UI. Also added a check to prevent the changing of nsslapd-state for replicated suffixes.

Also did a little UI cleanup where a bottom margin was added to the bottom of pages instead of using "hr" to create the gap.

relates: #4796

Reviewed by: jchapman & spichugi(Thanks!)
mreynolds389 added a commit that referenced this issue Sep 11, 2021
@mreynolds389
Issue 4796 - Add support for nsslapd-state to CLI & UI
9d0c9b8 
Description:

Add support for nsslapd-state to lib389 and UI. Also added a check to prevent the changing of nsslapd-state for replicated suffixes.

Also did a little UI cleanup where a bottom margin was added to the bottom of pages instead of using "hr" to create the gap.

relates: #4796

Reviewed by: jchapman & spichugi(Thanks!)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mreynolds389 mreynolds389

Labels
CLI CLI tools cockpit Cockpit UI Plugin In JIRA ticket is in JIRA priority_high need urgent fix / highly valuable / easy to fix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@falon @mreynolds389 @progier389