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

Added katacoda.yaml file to determine the location of the katacoda scenarios in the git repo. #1359

Merged
merged 1 commit into from
Jul 30, 2019

Conversation

gsoria
Copy link
Contributor

@gsoria gsoria commented Jul 29, 2019

Katacoda added the possibility to define an attribute in katacoda.yaml to determine the location of the katacoda scenarios in the git repo.
We have tested with your repo, and worked fine.
We will be happy to help if you have questions building the scenarios.

Signed-off-by: Gabriela S. Soria [email protected]

Changes

Added katacoda.yaml file setting the property
scenario_root : tutorials/katacoda

Verification

We have tested cloning using the forked repository inside https://katacoda.com

Signed-off-by: Gabriela S. Soria <[email protected]>
@povilasv povilasv merged commit 4ca57b3 into thanos-io:master Jul 30, 2019
@bwplotka
Copy link
Member

This amazing @gsoria! <3 Thanks!

We are unblocked to schedule some tutorials now 👍

paulfantom added a commit to paulfantom/thanos that referenced this pull request Aug 7, 2019
* master:
  iter.go: error message typo correction (thanos-io#1376)
  Fix usage of $GOPATH in Makefile (thanos-io#1379)
  Moved Prometheus 2.11.1 and TSDB to 0.9.1 (thanos-io#1380)
  Store latest config hash and timestamp as metrics (thanos-io#1378)
  pkg/receive/handler.go: log errors (thanos-io#1372)
  receive: Hash-ring metrics (thanos-io#1363)
  receiver: avoid race of hashring (thanos-io#1371)
  feat compact: added readiness Prober (thanos-io#1297)
  Add changelog entry for S3 option (thanos-io#1361)
  Multipart features (thanos-io#1358)
  Added katacoda.yaml (thanos-io#1359)
  Remove deprecated option from example (thanos-io#1351)
  Move suggestion about admin API to appropriate place (thanos-io#1355)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants