X-Ray module link fixes #137
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
We should move these files into the workshop content rather than linking back to github. Or do they need to be in github for the steps themselves? |
|
I am open to discussing, but I would like to expose these folders as something someone could pull down locally, copy and use in their own testing. |
|
we can package the files for download. you just need to create a folder .files under the module and then add a shortcode into the page similar to https://github.com/aws-samples/eks-workshop/blob/master/content/logging . I'm about to go through the module now. I can take care of this |
|
Thanks. Please leave the link to the source code in the module, unless you have a good way to do this in hugo. |
vjsikha
pushed a commit
to vjsikha/eks-workshop
that referenced
this pull request
Nov 21, 2018
brentley
pushed a commit
that referenced
this pull request
Nov 21, 2018
* Content review and edits. Added Challenge Questions * added challenge question for user to retrieve the elb URL * updated the shortcode to fix the launch button for codepipeline * Batch Processing (redux) (#133) * Batch Processing Workshop (all except advanced example) * Add advanced batch example * Updates based on feedback on PR-133 * Change S3 bucket creation to use AWSCLI * Add IAM policy for artifact S3 access * Add removal instructions for S3 IAM policy * Duplicate ACCOUNT_ID creation per code block where it's used * Clean S3 IAM policy steps * Fix error in ConfigMap for artifact repository (manual substitution) * Delete non-empty bucket * Link fixes (#135) * updating to support cloud9 dev environment * updating repo urls for consistency * commit the link fixes * Tracing with X-Ray module (#132) * Tracing with X-Ray module * Change urls to deployed version * Fix links (#137) * added documentation and examples to demo readiness and liveness probes (#131) * added documentation and examples to demo readiness and liveness probes * simplified text boxes for commands, added additional explanation about how probes work and tweaked readiness probe * added step to make readiness probe pass after failure * updated the folder structure to reflect http paths and guidelines * updated the pages to show and explain the yaml files instead of simply downloading the same * minor edits to add clarity * remove duplicate page (using.md/rolling_back.md) (#140) * updating to support cloud9 dev environment * updating repo urls for consistency * remove duplicate instructions page? * Dynamic Volume Provisioning hyperlink (#141) * adding calico chapter * deleting calico in cleanup step * changing notice from note(blue) to tip(green) * changing chapter weight to fit between example microservices and helm * adding uninstall calico text * separated command from the output * reverting changes to package-lock.json * adding all changes except yaml creation * changing calico resources from links to local files * moved cloning the repo from prereqs to calico chapter * changed dynamic volue hyperlink to k8s doc * Update _index.md (#144) minor grammar changes * Challenge question and answer to Monitoring Chapter (#146) * adding monitoring using prometheus & grafana chapter * changing draft: false * minor change to grafana dashboards * adding challenge question & offering solution * formatting changes - removed for re-emphasizing * minor change with relative links * Remove region (#148) * Remove region * fixing backticks * adding a region selector (#154) * Added challenge question to installnginx.md (#145) * Added challenge question to installnginx.md Challenge question provides clues for the correct helm install syntax. Workshop participant can either figure out the syntax themselves or expand the hidden text for the correct command. * Updated Challenge Formatting and Module Wording Changed the formatting of the challenge to match the requested styling. Made the warning about the ELB creation time more obvious. Miscellaneous wording corrections. * Istiotweaks (#153) * Update _index.md Fixing some grammar and typos. * Update introduction.md Fixed some formatting and grammar. * Update install.md Cleaning up formatting and grammar. * Update deploy.md Fixed grammar and formatting. * Update routing.md Grammar and typo fixes. * Update visualize.md fixed some grammar and typos * Update cleanup.md Cleaning up grammar and typos * Update routing.md Fixed error language message to better illustrate what to expect. * Update _index.md Removing callout * fixing typo in health checks (#155) * adding monitoring using prometheus & grafana chapter * changing draft: false * minor change to grafana dashboards * adding challenge question & offering solution * formatting changes - removed for re-emphasizing * minor change with relative links * fixing typo in health checks chapter * minor improvements in health check chapter * update Batch cleanup.md (#142) Added command to map $ROLE_NAME in case this was lost * Simplify CFN stack creation (#156) * Calico logo, note in StatefulSets -- Test scaling and Batch -- simple workflow (#143) * adding calico chapter * deleting calico in cleanup step * changing notice from note(blue) to tip(green) * changing chapter weight to fit between example microservices and helm * adding uninstall calico text * separated command from the output * reverting changes to package-lock.json * adding all changes except yaml creation * changing calico resources from links to local files * moved cloning the repo from prereqs to calico chapter * changed dynamic volue hyperlink to k8s doc * changed calico chapter index image to official project calico logo * added a note for pods scaling duration * replaced workflow name from command with placeholder * improved changes introduced in commit 5f627f6 * added challenge question for user to retrieve the elb URL * resolved conflicts after rebasing from master * added hint to challenge question * Content review and edits. Added Challenge Questions (#147) * fixed typo in role name. moved challenge question. cleaned up typos. * reverted cf-launch.html changes * reverted codepipeline.md changes for launch button
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue #, if available:
NA
Description of changes:
Fix links back to the Git repo.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.