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: removed old k8s demo and refer kube-thanos instead, also update… #1600

Merged
merged 1 commit into from
Nov 13, 2019

Conversation

FUSAKLA
Copy link
Member

@FUSAKLA FUSAKLA commented Oct 2, 2019

… release process to update tutorials

As discussed on slack https://cloud-native.slack.com/archives/CL25937SP/p1568848738006400

  • removed old kubernetes-demo
  • added updating of tutorials to release process
  • refer kube-thanos in getting started since we removed the last k8s manifest examples.

@bwplotka @metalmatze @GiedriusS PTAL

Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks!

cc @metalmatze

docs/getting-started.md Outdated Show resolved Hide resolved
@FUSAKLA FUSAKLA force-pushed the fus-release-manifests-update branch from 38151ee to aa71c4b Compare October 3, 2019 08:15
@bwplotka
Copy link
Member

bwplotka commented Oct 7, 2019

cc @FUSAKLA rebase

@FUSAKLA FUSAKLA force-pushed the fus-release-manifests-update branch from aa71c4b to 9d58b69 Compare October 7, 2019 15:45
@FUSAKLA
Copy link
Member Author

FUSAKLA commented Oct 7, 2019

@bwplotka I saw that you covered the kubernetes references in your update so I just removed mine and left the release process update

@bwplotka
Copy link
Member

Hi @FUSAKLA do you mind fixing links now? ): Let's merge this finally!

@FUSAKLA
Copy link
Member Author

FUSAKLA commented Oct 17, 2019

Hi, sorry for the delay 🏖️ :))

I'll update it right away

@bwplotka
Copy link
Member

bwplotka commented Nov 8, 2019

ping @FUSAKLA (:

docs/release-process.md Outdated Show resolved Hide resolved
@FUSAKLA FUSAKLA force-pushed the fus-release-manifests-update branch from 9d58b69 to c3b3ad0 Compare November 11, 2019 06:40
@FUSAKLA FUSAKLA force-pushed the fus-release-manifests-update branch from c3b3ad0 to 6762df0 Compare November 11, 2019 06:46
@FUSAKLA
Copy link
Member Author

FUSAKLA commented Nov 11, 2019

Sorry for the delay, hopefully should be ok now 🤞

Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

Thanks!

@bwplotka bwplotka merged commit 2591455 into thanos-io:master Nov 13, 2019
tianyuansun pushed a commit to tianyuansun/thanos that referenced this pull request Nov 19, 2019
IKSIN pushed a commit to monitoring-tools/thanos that referenced this pull request Nov 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants