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

Block viewer: Show blocks marked for deletion #3308

Closed
roidelapluie opened this issue Oct 13, 2020 · 25 comments
Closed

Block viewer: Show blocks marked for deletion #3308

roidelapluie opened this issue Oct 13, 2020 · 25 comments

Comments

@roidelapluie
Copy link

Thanos, Prometheus and Golang version used:

Object Storage Provider:

s3

What happened:

I am looking at the blocks viewer and I see overlapping blocks

thanos2

What you expected to happen:

Blocks marked for deletion marked in a special way

thanos

@kakkoyun
Copy link
Member

@Shini31 Could this be the same issue as #3307?

@kakkoyun
Copy link
Member

@prmsrswt Any input on this?

@onprem
Copy link
Member

onprem commented Oct 14, 2020

They are in different rows so they are not necessarily overlapping here. Each unique downsampling resolution - compaction level gets its own row.

Although we are currently not differentiating blocks which are marked for deletions but we have that in our roadmap and hopefully will be implemented during this community bridge period.

@roidelapluie
Copy link
Author

roidelapluie commented Oct 14, 2020

What I mean is to find blocks that have a deletion-mark.json file.

@mcinquin
Copy link

@Shini31 Could this be the same issue as #3307?

Yes, there is.
This feature will be very helpful to determine if there is an overlap issue or not.

@stale
Copy link

stale bot commented Dec 13, 2020

Hello 👋 Looks like there was no activity on this issue for the last two months.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity in the next two weeks, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Dec 13, 2020
@mcinquin
Copy link

up?

@stale stale bot removed the stale label Dec 15, 2020
@onprem
Copy link
Member

onprem commented Dec 18, 2020

Yeah this is on our roadmap, help wanted!

@mcinquin
Copy link

mcinquin commented Jan 2, 2021

Good news!
I can help for the development, but i'm available for testing

@stale
Copy link

stale bot commented Mar 5, 2021

Hello 👋 Looks like there was no activity on this issue for the last two months.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity in the next two weeks, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Mar 5, 2021
@onprem onprem removed the stale label Mar 6, 2021
@stale
Copy link

stale bot commented Jun 2, 2021

Hello 👋 Looks like there was no activity on this issue for the last two months.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity in the next two weeks, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Jun 2, 2021
@kakkoyun kakkoyun removed the stale label Jun 3, 2021
@anuraagbarde
Copy link

Hello there! I have experience with working in React, as this is marked as a good first issue how can I contribute? Thanks.

@onprem
Copy link
Member

onprem commented Jun 28, 2021

We can break this down in some steps:

  1. Extend the blocks API endpoint to expose which blocks are marked for deletion.
    • In Thanos, blocks which are going to get deleted have a file named deletion-mark.json, we can use this information to return an additional boolean in the API response, markedForDeletion: true | false.
  2. Use this information in the React UI to visually distinguish the blocks which are marked for deletion.

The first step will require some Go knowledge, as our backend is written in Go, the second step, however, is purely React work.

@anuraagbarde
Copy link

Ok, I will look into this, thanks for the info.

@stale
Copy link

stale bot commented Aug 28, 2021

Hello 👋 Looks like there was no activity on this issue for the last two months.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity in the next two weeks, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Aug 28, 2021
@mcinquin
Copy link

Up

@stale stale bot removed the stale label Aug 29, 2021
@stale
Copy link

stale bot commented Oct 30, 2021

Hello 👋 Looks like there was no activity on this issue for the last two months.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity in the next two weeks, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Oct 30, 2021
@mcinquin
Copy link

Up

@stale stale bot removed the stale label Oct 30, 2021
@yeya24
Copy link
Contributor

yeya24 commented Nov 9, 2021

This is still nice to have.

@stale
Copy link

stale bot commented Jan 9, 2022

Hello 👋 Looks like there was no activity on this issue for the last two months.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity in the next two weeks, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Jan 9, 2022
@mcinquin
Copy link

Up

@stale stale bot removed the stale label Jan 10, 2022
@keremgocen
Copy link

Hello, is anyone working on this? I'm looking for something to contribute.

@stale
Copy link

stale bot commented Apr 16, 2022

Hello 👋 Looks like there was no activity on this issue for the last two months.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity in the next two weeks, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Apr 16, 2022
@stale
Copy link

stale bot commented Apr 30, 2022

Closing for now as promised, let us know if you need this to be reopened! 🤗

@stale stale bot closed this as completed Apr 30, 2022
@keremgocen
Copy link

Up!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants