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

Fixes #35556 - Track reclaimspace task properly as an allowed action #10761

Merged
merged 1 commit into from
Nov 20, 2023

Conversation

sjha4
Copy link
Member

@sjha4 sjha4 commented Oct 5, 2023

What are the changes introduced in this pull request?

Considerations taken when implementing this change?

What are the testing steps for this pull request?

  1. Trigger a reclaim space task on a proxy.
  2. Make sure it fails. Turn off the proxy maybe.
  3. Notice the error notification that should show up on task failure.

@theforeman-bot
Copy link

Issues: #35556

@sjha4 sjha4 force-pushed the reclaim_space_task branch from 33013a4 to c4b3fef Compare November 9, 2023 19:30
@chris1984
Copy link
Member

Code looks fine, starting to test it

Copy link
Member

@chris1984 chris1984 left a comment

Choose a reason for hiding this comment

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

ACK, works well

Screenshot 2023-11-20 at 13-19-06 centos8-proxy-devel area51 example com

@sjha4 sjha4 merged commit 878b5f4 into Katello:master Nov 20, 2023
5 of 6 checks passed
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