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-1381] Port RestorablePackageSet to packages module #1055

Merged

Conversation

r0x0d
Copy link
Member

@r0x0d r0x0d commented Jan 30, 2024

RestorablePackageSet is moved to be under the packages module inside the backup folder. This is one part of the refactoring of the backup controller to make it more organized and easier to work with.

Jira Issues: RHELC-1381

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

convert2rhel/pkghandler.py Fixed Show fixed Hide fixed
Copy link

codecov bot commented Jan 30, 2024

Codecov Report

Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (d42de9b) 94.86% compared to head (c22ad7c) 94.92%.

Files Patch % Lines
convert2rhel/backup/packages.py 93.10% 5 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1055      +/-   ##
==========================================
+ Coverage   94.86%   94.92%   +0.05%     
==========================================
  Files          48       49       +1     
  Lines        4539     4548       +9     
  Branches      808      808              
==========================================
+ Hits         4306     4317      +11     
+ Misses        158      156       -2     
  Partials       75       75              
Flag Coverage Δ
centos-linux-7 90.10% <93.40%> (+0.06%) ⬆️
centos-linux-8 91.10% <93.40%> (+0.06%) ⬆️
centos-linux-9 91.15% <93.40%> (+0.06%) ⬆️

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.

@r0x0d r0x0d force-pushed the port-restorable-package-set-backup-controller branch from 91e9d27 to 89ef688 Compare January 30, 2024 14:24
@r0x0d r0x0d added kind/feature New feature or request tests/sanity PR ready to run the sanity test suit. Equivalent to `/packit test --labels sanity`. labels Jan 30, 2024
@has-bot
Copy link
Member

has-bot commented Jan 30, 2024

/packit test --labels sanity


@oamg/conversions-qe please review results and provide ack.

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.

This port looks good to me, approved

@r0x0d
Copy link
Member Author

r0x0d commented Jan 31, 2024

/packit test --labels sanity

@r0x0d r0x0d force-pushed the port-restorable-package-set-backup-controller branch from 89ef688 to 5dcf04a Compare February 2, 2024 12:48
@r0x0d
Copy link
Member Author

r0x0d commented Feb 2, 2024

/packit test --labels sanity

@r0x0d r0x0d requested a review from pr-watson February 2, 2024 17:55
@r0x0d
Copy link
Member Author

r0x0d commented Feb 2, 2024

/packit retest-failed

Copy link
Member

@Venefilyn Venefilyn left a comment

Choose a reason for hiding this comment

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

LGTM

convert2rhel/unit_tests/backup/packages_test.py Outdated Show resolved Hide resolved
convert2rhel/unit_tests/backup/packages_test.py Outdated Show resolved Hide resolved
@r0x0d r0x0d force-pushed the port-restorable-package-set-backup-controller branch from 5dcf04a to 2a102a4 Compare February 5, 2024 16:39
@r0x0d r0x0d requested a review from Venefilyn February 5, 2024 16:49
@r0x0d r0x0d force-pushed the port-restorable-package-set-backup-controller branch from 2a102a4 to 5fc33d2 Compare February 5, 2024 17:44
@r0x0d
Copy link
Member Author

r0x0d commented Feb 5, 2024

/packit test --labels sanity

RestorablePackageSet is moved to be under the packages module inside the
backup folder. This is one part of the refactoring of the backup
controller to make it more organized and easier to work with.

Signed-off-by: Rodolfo Olivieri <rolivier@redhat.com>
Signed-off-by: Rodolfo Olivieri <rolivier@redhat.com>
@r0x0d r0x0d force-pushed the port-restorable-package-set-backup-controller branch from 5fc33d2 to c22ad7c Compare February 6, 2024 13:02
@r0x0d
Copy link
Member Author

r0x0d commented Feb 6, 2024

/packit test --labels sanity

@r0x0d
Copy link
Member Author

r0x0d commented Feb 6, 2024

Unit tests for the missing paths will be added as part of another PR. Since what is in main is already missing, we will work on this later.

@r0x0d r0x0d merged commit 23864b1 into oamg:main Feb 6, 2024
26 of 57 checks passed
@r0x0d r0x0d deleted the port-restorable-package-set-backup-controller branch February 6, 2024 18:52
@Venefilyn Venefilyn mentioned this pull request Feb 22, 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/sanity PR ready to run the sanity test suit. Equivalent to `/packit test --labels sanity`.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants