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

Docs: Remove link to a deleted resource from mixin docs #5107

Merged
merged 1 commit into from
Jan 28, 2022

Conversation

matej-g
Copy link
Collaborator

@matej-g matej-g commented Jan 28, 2022

Signed-off-by: Matej Gera matejgera@gmail.com

  • I added CHANGELOG entry for this change.
  • Change is not relevant to the end user.

Changes

Deleting link to mixin design doc (https://docs.google.com/document/d/1A9xvzwqnFVSOZ5fD3blKODXfsat5fg6ZhnKu9LK3lB4/edit#) which apparently does not exist anymore and causes the doc check to fail.

Verification

Local run of make check-docs

Signed-off-by: Matej Gera <matejgera@gmail.com>
Copy link
Member

@GiedriusS GiedriusS left a comment

Choose a reason for hiding this comment

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

Pity that the doc has been deleted for some reason :/ thanks for this! I was about to open up a PR for fixing this 😄

@GiedriusS GiedriusS merged commit b711a0c into thanos-io:main Jan 28, 2022
@matej-g
Copy link
Collaborator Author

matej-g commented Jan 28, 2022

Pity that the doc has been deleted for some reason :/ thanks for this! I was about to open up a PR for fixing this smile

Yeah, I quickly looked around if it has been moved, but no luck. Perhaps if we could reach @tomwilkie who seem to be the author (guessing from https://groups.google.com/g/prometheus-developers/c/ADgiqnaT1wo/m/BEY9sZfbAQAJ) maybe we could get an alternative link.

@tomwilkie
Copy link

Sorry about that, I don't know what happened there! Heres a PDF we cached a while back: https://github.com/monitoring-mixins/docs/blob/master/design.pdf, but I think that doc was written under the Kausal google account, which is long gone now.

@matej-g matej-g mentioned this pull request Feb 4, 2022
2 tasks
@matej-g
Copy link
Collaborator Author

matej-g commented Feb 4, 2022

Awesome, thanks for providing the link @tomwilkie!

Nicholaswang pushed a commit to Nicholaswang/thanos that referenced this pull request Mar 6, 2022
Signed-off-by: Matej Gera <matejgera@gmail.com>
Signed-off-by: Nicholaswang <wzhever@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants