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

[RHELC-1376] Remove backup controller partition #1096

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

r0x0d
Copy link
Member

@r0x0d r0x0d commented Feb 12, 2024

The workaround was introduced in #912 and #869, but it is no longer necessary as all legacy rollback items are now ported to be an instance tracked by the backup controller.

Jira Issues: RHELC-1376

Checklist

  • PR has been tested manually in a VM (either author or reviewer)
  • Jira issue has been made public if possible
  • [RHELC-] is part of the PR title
  • GitHub label has been added to help with Release notes
  • PR title explains the change from the user's point of view
  • Code and tests are documented properly
  • The commits are squashed to as few commits as possible (without losing data)
  • When merged: Jira issue has been updated to Release Pending if relevant

Blocked by

@r0x0d r0x0d added the tests/tier0 PR ready to run the essential test suit. Equivalent to `/packit test --labels tier0`. label Feb 12, 2024
@has-bot
Copy link
Member

has-bot commented Feb 12, 2024

/packit test --labels tier0


Comment generated by an automation.

Copy link

codecov bot commented Feb 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.40%. Comparing base (f992a1d) to head (fee331a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1096      +/-   ##
==========================================
- Coverage   95.48%   95.40%   -0.08%     
==========================================
  Files          51       51              
  Lines        4627     4612      -15     
  Branches      820      815       -5     
==========================================
- Hits         4418     4400      -18     
- Misses        134      137       +3     
  Partials       75       75              
Flag Coverage Δ
centos-linux-7 90.48% <100.00%> (-0.10%) ⬇️
centos-linux-8 91.48% <100.00%> (-0.10%) ⬇️
centos-linux-9 91.54% <100.00%> (-0.10%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@hosekadam
Copy link
Member

Looks good, everything about the partition removed. With testing, I'll wait till the blockers get merged.

@r0x0d r0x0d force-pushed the remove-partition-from-backup-controller branch from e38cfe5 to 50649b4 Compare February 22, 2024 20:06
Copy link
Contributor

@pr-watson pr-watson left a comment

Choose a reason for hiding this comment

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

Changes look good, will approve after it's been tested in a VM and integration tests pass

@r0x0d r0x0d force-pushed the remove-partition-from-backup-controller branch from 50649b4 to b3db692 Compare February 28, 2024 16:34
The workaround was introduced in oamg#912 and oamg#869, but it is no longer
necessary as all legacy rollback items are now ported to be an instance
tracked by the backup controller.

Signed-off-by: Rodolfo Olivieri <rolivier@redhat.com>
@r0x0d r0x0d force-pushed the remove-partition-from-backup-controller branch from b3db692 to fee331a Compare March 6, 2024 14:01
@r0x0d
Copy link
Member Author

r0x0d commented Mar 6, 2024

/packit test --labels tier0

@r0x0d r0x0d added the kind/feature New feature or request label Mar 7, 2024
@Venefilyn Venefilyn merged commit 3db00f5 into oamg:main Mar 7, 2024
49 of 62 checks passed
@r0x0d r0x0d deleted the remove-partition-from-backup-controller branch March 7, 2024 12:37
jochapma pushed a commit to jochapma/convert2rhel that referenced this pull request Mar 11, 2024
The workaround was introduced in oamg#912 and oamg#869, but it is no longer
necessary as all legacy rollback items are now ported to be an instance
tracked by the backup controller.

Signed-off-by: Rodolfo Olivieri <rolivier@redhat.com>
@hosekadam hosekadam mentioned this pull request May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature or request tests/tier0 PR ready to run the essential test suit. Equivalent to `/packit test --labels tier0`.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants