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.

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

fix: seal object validate if secondary SP in_service #561

Closed

Conversation

alexgao001
Copy link
Collaborator

@alexgao001 alexgao001 commented Jan 15, 2024

Description

seal object validate if secondary SP in_service, in case that SP is exiting but still need to severs as a secondary SP.

Rationale

SP not in_service should not serve users new object requests.

Example

add an example CLI or API response...

Changes

Notable changes:

  • add each change in a bullet point here
  • ...

Potential Impacts

  • add potential impacts for other components here
  • ...

@alexgao001 alexgao001 force-pushed the fix-seal-check-spstatus branch from e837c50 to 9dd6e2f Compare January 15, 2024 03:01
@alexgao001 alexgao001 force-pushed the fix-seal-check-spstatus branch from 9dd6e2f to 4470045 Compare January 15, 2024 03:12
unclezoro
unclezoro previously approved these changes Jan 15, 2024
@unclezoro unclezoro changed the base branch from develop to master January 15, 2024 03:14
@unclezoro unclezoro dismissed their stale review January 15, 2024 03:14

The base branch was changed.

Copy link
Collaborator

@unclezoro unclezoro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need more consideration

@unclezoro
Copy link
Collaborator

hold on first

@unclezoro unclezoro marked this pull request as draft January 22, 2024 02:09
@alexgao001 alexgao001 closed this May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants