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
adding null check to fix overlapping blocks #7058
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Harsh Pratap Singh <harshpratapsingh8210@gmail.com>
If it looks good I will add the changelog! @MichaHoffmann |
Seems duplicate with #6768? |
Signed-off-by: Harsh Pratap Singh <harshpratapsingh8210@gmail.com>
@harsh-ps-2003 Thanks for adding the test. We don't have to wait for #6768 since it has conflicts. |
@@ -656,3 +658,37 @@ describe('Filtered block pools', () => { | |||
expect(filteredBlockPoolArray[0].thanos.labels).toEqual(filteredBlocks[0].thanos.labels); | |||
}); | |||
}); | |||
|
|||
const newBlockPools = blockPools; |
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.
Any reason all of these variables aren't moved into the test itself?
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.
Ah! moved the constants in the test. thanks
Signed-off-by: Harsh Pratap Singh <harshpratapsingh8210@gmail.com>
Note that react tests failed. |
Signed-off-by: Harsh Pratap Singh harshpratapsingh8210@gmail.com
Changes
fixes #5359
Verification
Manually tested.