-
Notifications
You must be signed in to change notification settings - Fork 53
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
feat(bitmap): use enhanced bitmap from go-common-libs #625
Conversation
This pull request is now in conflict. Could you fix it @james-munson? 🙏 |
The big part of this PR is switch from local bitmap to an enhanced one to be added to go-common-libs in longhorn/go-common-libs#60 Marking it as |
5bb27b6
to
8d297b9
Compare
Signed-off-by: James Munson <james.munson@suse.com>
8d297b9
to
af969cf
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a little remainder for us all: in this repo the packages from go-common-libs
are imported as lh
, meanwhile in longhorn-spdk-engine
they are imported as common
. For example we have lhBitmap
and commonBitmap
for the same package, maybe in the future we could align them.
@mergify backport v1.7.x |
✅ Backports have been created
|
I would like to see this fix backported to 1.7.2, so I have set a PR up for that. |
Which issue(s) this PR fixes:
longhorn/longhorn#7802
What this PR does / why we need it:
Switch to use the enhanced bitmap utility to avoid immediate reuse and port collision.
Special notes for your reviewer:
Additional documentation or context