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

[Backport 5.16] [NC | NSFS | Glacier] filter out failed recalls #8089

Merged

Conversation

tangledbytes
Copy link
Member

Signed-off-by: Utkarsh Srivastava srivastavautkarsh8097@gmail.com
(cherry picked from commit b1c3570)

Explain the changes

Issues: Fixed #xxx / Gap #xxx

Testing Instructions:

  • Doc added/updated
  • Tests added

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

update comments

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

fix linting issue

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

revamp restore code and add tests

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

add more tests

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

add cleanup to the test

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

address self review comments

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

fix partial failure of finalize_restore

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

improving task_show output parsing

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

fix parsing edge case

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

fix formatting

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

add error logging

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
(cherry picked from commit b1c3570)
@nimrod-becker nimrod-becker merged commit 03db21f into noobaa:5.16 May 28, 2024
10 checks passed
@guymguym guymguym added this to the 5.16.z milestone May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants