-
Notifications
You must be signed in to change notification settings - Fork 45
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
add Connections crosslinks to Existing app tutorial #1103
add Connections crosslinks to Existing app tutorial #1103
Conversation
Signed-off-by: Will Tsai <[email protected]>
@@ -321,4 +321,5 @@ This output shows that Radius has detected the three container resources in the | |||
|
|||
- To learn more about authoring Radius applications, visit the [Authoring applications guide]({{< ref "guides/author-apps" >}}) | |||
- To learn more about deploying applications using Radius, visit the [Deploying applications guide]({{< ref "guides/deploy-apps" >}}) | |||
- To learn more about using [Radius Connections]({{< ref "guides/author-apps/containers/overview#connections" >}}) to connect your containers and resources, visit the [Connect to dependencies guide]({{< ref "guides/author-apps/containers/howto-connect-dependencies" >}}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for updating this. The office hours user did not know how to add connections to existing k8s manifests The connections overview doesn't cover the annotation to use to add connections. I think we should add a simplified version of step 6 from helm tutorial in addition to crosslinking.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the suggestion @Reshrahim - I've updated the cross linking to specifically address the community feedback you've mentioned. However, I think there is a separate issue where the Annotations features aren't documented beyond the tutorials, so I've created this issue to track that work: #1117
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sounds good!
Signed-off-by: Will Tsai <[email protected]>
* Update docs for v0.32.0 * Update environment logic (#1097) Signed-off-by: Aaron Crawfis <[email protected]> * Superbeeny/adding podspec secrets (#1100) * Adding example on how to access secrets and add them to the container environment Signed-off-by: Nick Beenham <[email protected]> * Adding code tabs where the commands differ between linux/windows Signed-off-by: Nick Beenham <[email protected]> * Apply suggestions from code review Co-authored-by: Will <[email protected]> Signed-off-by: Nick Beenham <[email protected]> * Updates requested from PR review Signed-off-by: Nick Beenham <[email protected]> * nit: add a new line for spacing aesthetics Signed-off-by: Will <[email protected]> --------- Signed-off-by: Nick Beenham <[email protected]> Signed-off-by: Will <[email protected]> Co-authored-by: Will <[email protected]> * Make env configurable (#1102) Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Will <[email protected]> * fix(bug): Update index.md (#1113) * fix(bug): Update index.md Fixed a bug in the cleanup script. The command `rad app delete demo -y` results in this output: `Application 'demo' does not exist or has already been deleted.` The command should be: `rad app delete -y` The output will be: `Application first-app deleted` Signed-off-by: Loek Duys <[email protected]> * Update docs/content/getting-started/index.md Signed-off-by: Reshma Abdul Rahim <[email protected]> --------- Signed-off-by: Loek Duys <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> * Fix broken link in faq (#1112) * fix broken link in index.md Signed-off-by: Dan Moseley <[email protected]> * Update index.md Signed-off-by: Dan Moseley <[email protected]> * Update docs/content/concepts/faq/index.md Signed-off-by: Reshma Abdul Rahim <[email protected]> --------- Signed-off-by: Dan Moseley <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> * add Connections crosslinks to Existing app tutorial (#1103) * add Connections crosslinks to Existing app tutorial Signed-off-by: Will Tsai <[email protected]> * add more crosslinks Signed-off-by: Will Tsai <[email protected]> --------- Signed-off-by: Will Tsai <[email protected]> --------- Signed-off-by: Aaron Crawfis <[email protected]> Signed-off-by: Nick Beenham <[email protected]> Signed-off-by: Will <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Loek Duys <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Dan Moseley <[email protected]> Signed-off-by: Will Tsai <[email protected]> Co-authored-by: Radius CI Bot <[email protected]> Co-authored-by: Aaron Crawfis <[email protected]> Co-authored-by: Nick Beenham <[email protected]> Co-authored-by: Will <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Loek Duys <[email protected]> Co-authored-by: Dan Moseley <[email protected]>
* Update docs for v0.32.0 * Update environment logic (#1097) Signed-off-by: Aaron Crawfis <[email protected]> * Superbeeny/adding podspec secrets (#1100) * Adding example on how to access secrets and add them to the container environment Signed-off-by: Nick Beenham <[email protected]> * Adding code tabs where the commands differ between linux/windows Signed-off-by: Nick Beenham <[email protected]> * Apply suggestions from code review Co-authored-by: Will <[email protected]> Signed-off-by: Nick Beenham <[email protected]> * Updates requested from PR review Signed-off-by: Nick Beenham <[email protected]> * nit: add a new line for spacing aesthetics Signed-off-by: Will <[email protected]> --------- Signed-off-by: Nick Beenham <[email protected]> Signed-off-by: Will <[email protected]> Co-authored-by: Will <[email protected]> * Make env configurable (#1102) Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Will <[email protected]> * fix(bug): Update index.md (#1113) * fix(bug): Update index.md Fixed a bug in the cleanup script. The command `rad app delete demo -y` results in this output: `Application 'demo' does not exist or has already been deleted.` The command should be: `rad app delete -y` The output will be: `Application first-app deleted` Signed-off-by: Loek Duys <[email protected]> * Update docs/content/getting-started/index.md Signed-off-by: Reshma Abdul Rahim <[email protected]> --------- Signed-off-by: Loek Duys <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> * Fix broken link in faq (#1112) * fix broken link in index.md Signed-off-by: Dan Moseley <[email protected]> * Update index.md Signed-off-by: Dan Moseley <[email protected]> * Update docs/content/concepts/faq/index.md Signed-off-by: Reshma Abdul Rahim <[email protected]> --------- Signed-off-by: Dan Moseley <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> * update gh to ado sync workflow to use service principals Signed-off-by: Will Tsai <[email protected]> * add Connections crosslinks to Existing app tutorial (#1103) * add Connections crosslinks to Existing app tutorial Signed-off-by: Will Tsai <[email protected]> * add more crosslinks Signed-off-by: Will Tsai <[email protected]> --------- Signed-off-by: Will Tsai <[email protected]> * Update config.toml Signed-off-by: Will <[email protected]> --------- Signed-off-by: Aaron Crawfis <[email protected]> Signed-off-by: Nick Beenham <[email protected]> Signed-off-by: Will <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Loek Duys <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Dan Moseley <[email protected]> Signed-off-by: Will Tsai <[email protected]> Co-authored-by: Radius CI Bot <[email protected]> Co-authored-by: Aaron Crawfis <[email protected]> Co-authored-by: Nick Beenham <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Loek Duys <[email protected]> Co-authored-by: Dan Moseley <[email protected]>
* Update docs for v0.33.0 * update gh to ado sync workflow to use service principals (#1118) * Update docs for v0.32.0 * Update environment logic (#1097) Signed-off-by: Aaron Crawfis <[email protected]> * Superbeeny/adding podspec secrets (#1100) * Adding example on how to access secrets and add them to the container environment Signed-off-by: Nick Beenham <[email protected]> * Adding code tabs where the commands differ between linux/windows Signed-off-by: Nick Beenham <[email protected]> * Apply suggestions from code review Co-authored-by: Will <[email protected]> Signed-off-by: Nick Beenham <[email protected]> * Updates requested from PR review Signed-off-by: Nick Beenham <[email protected]> * nit: add a new line for spacing aesthetics Signed-off-by: Will <[email protected]> --------- Signed-off-by: Nick Beenham <[email protected]> Signed-off-by: Will <[email protected]> Co-authored-by: Will <[email protected]> * Make env configurable (#1102) Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Will <[email protected]> * fix(bug): Update index.md (#1113) * fix(bug): Update index.md Fixed a bug in the cleanup script. The command `rad app delete demo -y` results in this output: `Application 'demo' does not exist or has already been deleted.` The command should be: `rad app delete -y` The output will be: `Application first-app deleted` Signed-off-by: Loek Duys <[email protected]> * Update docs/content/getting-started/index.md Signed-off-by: Reshma Abdul Rahim <[email protected]> --------- Signed-off-by: Loek Duys <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> * Fix broken link in faq (#1112) * fix broken link in index.md Signed-off-by: Dan Moseley <[email protected]> * Update index.md Signed-off-by: Dan Moseley <[email protected]> * Update docs/content/concepts/faq/index.md Signed-off-by: Reshma Abdul Rahim <[email protected]> --------- Signed-off-by: Dan Moseley <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> * update gh to ado sync workflow to use service principals Signed-off-by: Will Tsai <[email protected]> * add Connections crosslinks to Existing app tutorial (#1103) * add Connections crosslinks to Existing app tutorial Signed-off-by: Will Tsai <[email protected]> * add more crosslinks Signed-off-by: Will Tsai <[email protected]> --------- Signed-off-by: Will Tsai <[email protected]> * Update config.toml Signed-off-by: Will <[email protected]> --------- Signed-off-by: Aaron Crawfis <[email protected]> Signed-off-by: Nick Beenham <[email protected]> Signed-off-by: Will <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Loek Duys <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Dan Moseley <[email protected]> Signed-off-by: Will Tsai <[email protected]> Co-authored-by: Radius CI Bot <[email protected]> Co-authored-by: Aaron Crawfis <[email protected]> Co-authored-by: Nick Beenham <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Loek Duys <[email protected]> Co-authored-by: Dan Moseley <[email protected]> * Update index.md (#1121) Fixed typo `-resource-group myResourceGroup` into `--resource-group myResourceGroup` (added missing `-`) Signed-off-by: Loek Duys <[email protected]> * add new logo types (#1122) Signed-off-by: Will Tsai <[email protected]> * Add CNCF website guidelines (#1123) * Add copyrights Signed-off-by: Reshma Abdul Rahim <[email protected]> * Add trademark Signed-off-by: Reshma Abdul Rahim <[email protected]> * Add trademark Signed-off-by: Reshma Abdul Rahim <[email protected]> --------- Signed-off-by: Reshma Abdul Rahim <[email protected]> --------- Signed-off-by: Aaron Crawfis <[email protected]> Signed-off-by: Nick Beenham <[email protected]> Signed-off-by: Will <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Loek Duys <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Dan Moseley <[email protected]> Signed-off-by: Will Tsai <[email protected]> Co-authored-by: Will <[email protected]> Co-authored-by: Aaron Crawfis <[email protected]> Co-authored-by: Nick Beenham <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Loek Duys <[email protected]> Co-authored-by: Dan Moseley <[email protected]>
* Update docs for v0.33.0 * update gh to ado sync workflow to use service principals (#1118) * Update docs for v0.32.0 * Update environment logic (#1097) Signed-off-by: Aaron Crawfis <[email protected]> * Superbeeny/adding podspec secrets (#1100) * Adding example on how to access secrets and add them to the container environment Signed-off-by: Nick Beenham <[email protected]> * Adding code tabs where the commands differ between linux/windows Signed-off-by: Nick Beenham <[email protected]> * Apply suggestions from code review Co-authored-by: Will <[email protected]> Signed-off-by: Nick Beenham <[email protected]> * Updates requested from PR review Signed-off-by: Nick Beenham <[email protected]> * nit: add a new line for spacing aesthetics Signed-off-by: Will <[email protected]> --------- Signed-off-by: Nick Beenham <[email protected]> Signed-off-by: Will <[email protected]> Co-authored-by: Will <[email protected]> * Make env configurable (#1102) Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Will <[email protected]> * fix(bug): Update index.md (#1113) * fix(bug): Update index.md Fixed a bug in the cleanup script. The command `rad app delete demo -y` results in this output: `Application 'demo' does not exist or has already been deleted.` The command should be: `rad app delete -y` The output will be: `Application first-app deleted` Signed-off-by: Loek Duys <[email protected]> * Update docs/content/getting-started/index.md Signed-off-by: Reshma Abdul Rahim <[email protected]> --------- Signed-off-by: Loek Duys <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> * Fix broken link in faq (#1112) * fix broken link in index.md Signed-off-by: Dan Moseley <[email protected]> * Update index.md Signed-off-by: Dan Moseley <[email protected]> * Update docs/content/concepts/faq/index.md Signed-off-by: Reshma Abdul Rahim <[email protected]> --------- Signed-off-by: Dan Moseley <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> * update gh to ado sync workflow to use service principals Signed-off-by: Will Tsai <[email protected]> * add Connections crosslinks to Existing app tutorial (#1103) * add Connections crosslinks to Existing app tutorial Signed-off-by: Will Tsai <[email protected]> * add more crosslinks Signed-off-by: Will Tsai <[email protected]> --------- Signed-off-by: Will Tsai <[email protected]> * Update config.toml Signed-off-by: Will <[email protected]> --------- Signed-off-by: Aaron Crawfis <[email protected]> Signed-off-by: Nick Beenham <[email protected]> Signed-off-by: Will <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Loek Duys <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Dan Moseley <[email protected]> Signed-off-by: Will Tsai <[email protected]> Co-authored-by: Radius CI Bot <[email protected]> Co-authored-by: Aaron Crawfis <[email protected]> Co-authored-by: Nick Beenham <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Loek Duys <[email protected]> Co-authored-by: Dan Moseley <[email protected]> * Update index.md (#1121) Fixed typo `-resource-group myResourceGroup` into `--resource-group myResourceGroup` (added missing `-`) Signed-off-by: Loek Duys <[email protected]> * add new logo types (#1122) Signed-off-by: Will Tsai <[email protected]> * Add CNCF website guidelines (#1123) * Add copyrights Signed-off-by: Reshma Abdul Rahim <[email protected]> * Add trademark Signed-off-by: Reshma Abdul Rahim <[email protected]> * Add trademark Signed-off-by: Reshma Abdul Rahim <[email protected]> --------- Signed-off-by: Reshma Abdul Rahim <[email protected]> * Upmerge --------- Signed-off-by: Aaron Crawfis <[email protected]> Signed-off-by: Nick Beenham <[email protected]> Signed-off-by: Will <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Loek Duys <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Dan Moseley <[email protected]> Signed-off-by: Will Tsai <[email protected]> Co-authored-by: Radius CI Bot <[email protected]> Co-authored-by: Will <[email protected]> Co-authored-by: Aaron Crawfis <[email protected]> Co-authored-by: Nick Beenham <[email protected]> Co-authored-by: Loek Duys <[email protected]> Co-authored-by: Dan Moseley <[email protected]>
* Update docs for v0.33.0 * update gh to ado sync workflow to use service principals (#1118) * Update docs for v0.32.0 * Update environment logic (#1097) Signed-off-by: Aaron Crawfis <[email protected]> * Superbeeny/adding podspec secrets (#1100) * Adding example on how to access secrets and add them to the container environment Signed-off-by: Nick Beenham <[email protected]> * Adding code tabs where the commands differ between linux/windows Signed-off-by: Nick Beenham <[email protected]> * Apply suggestions from code review Co-authored-by: Will <[email protected]> Signed-off-by: Nick Beenham <[email protected]> * Updates requested from PR review Signed-off-by: Nick Beenham <[email protected]> * nit: add a new line for spacing aesthetics Signed-off-by: Will <[email protected]> --------- Signed-off-by: Nick Beenham <[email protected]> Signed-off-by: Will <[email protected]> Co-authored-by: Will <[email protected]> * Make env configurable (#1102) Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Will <[email protected]> * fix(bug): Update index.md (#1113) * fix(bug): Update index.md Fixed a bug in the cleanup script. The command `rad app delete demo -y` results in this output: `Application 'demo' does not exist or has already been deleted.` The command should be: `rad app delete -y` The output will be: `Application first-app deleted` Signed-off-by: Loek Duys <[email protected]> * Update docs/content/getting-started/index.md Signed-off-by: Reshma Abdul Rahim <[email protected]> --------- Signed-off-by: Loek Duys <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> * Fix broken link in faq (#1112) * fix broken link in index.md Signed-off-by: Dan Moseley <[email protected]> * Update index.md Signed-off-by: Dan Moseley <[email protected]> * Update docs/content/concepts/faq/index.md Signed-off-by: Reshma Abdul Rahim <[email protected]> --------- Signed-off-by: Dan Moseley <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> * update gh to ado sync workflow to use service principals Signed-off-by: Will Tsai <[email protected]> * add Connections crosslinks to Existing app tutorial (#1103) * add Connections crosslinks to Existing app tutorial Signed-off-by: Will Tsai <[email protected]> * add more crosslinks Signed-off-by: Will Tsai <[email protected]> --------- Signed-off-by: Will Tsai <[email protected]> * Update config.toml Signed-off-by: Will <[email protected]> --------- Signed-off-by: Aaron Crawfis <[email protected]> Signed-off-by: Nick Beenham <[email protected]> Signed-off-by: Will <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Loek Duys <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Dan Moseley <[email protected]> Signed-off-by: Will Tsai <[email protected]> Co-authored-by: Radius CI Bot <[email protected]> Co-authored-by: Aaron Crawfis <[email protected]> Co-authored-by: Nick Beenham <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Loek Duys <[email protected]> Co-authored-by: Dan Moseley <[email protected]> * Update index.md (#1121) Fixed typo `-resource-group myResourceGroup` into `--resource-group myResourceGroup` (added missing `-`) Signed-off-by: Loek Duys <[email protected]> * add new logo types (#1122) Signed-off-by: Will Tsai <[email protected]> * Add CNCF website guidelines (#1123) * Add copyrights Signed-off-by: Reshma Abdul Rahim <[email protected]> * Add trademark Signed-off-by: Reshma Abdul Rahim <[email protected]> * Add trademark Signed-off-by: Reshma Abdul Rahim <[email protected]> --------- Signed-off-by: Reshma Abdul Rahim <[email protected]> * Update docs for v0.34.0 * Update command to view deployed container in tutorial (#1128) * Update command to view deployed container in tutorial Signed-off-by: Filipe Revez <[email protected]> * Fixed another wrong rad app connections command Signed-off-by: Filipe Revez <[email protected]> --------- Signed-off-by: Filipe Revez <[email protected]> * fix ado sync workflow (#1130) Signed-off-by: Will Tsai <[email protected]> * Revert "Merge branch 'v0.33' into v0.34" (#1133) This reverts commit bfba3f3, reversing changes made to e36f280. * fix dead links in contribution guide (#1135) Signed-off-by: Will Tsai <[email protected]> * Rename app connections (#1129) Signed-off-by: Reshma Abdul Rahim <[email protected]> * revert changes Signed-off-by: Will Tsai <[email protected]> --------- Signed-off-by: Aaron Crawfis <[email protected]> Signed-off-by: Nick Beenham <[email protected]> Signed-off-by: Will <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Loek Duys <[email protected]> Signed-off-by: Reshma Abdul Rahim <[email protected]> Signed-off-by: Dan Moseley <[email protected]> Signed-off-by: Will Tsai <[email protected]> Signed-off-by: Filipe Revez <[email protected]> Co-authored-by: Radius CI Bot <[email protected]> Co-authored-by: Aaron Crawfis <[email protected]> Co-authored-by: Nick Beenham <[email protected]> Co-authored-by: Reshma Abdul Rahim <[email protected]> Co-authored-by: Loek Duys <[email protected]> Co-authored-by: Dan Moseley <[email protected]> Co-authored-by: Filipe Revez <[email protected]>
Thank you for helping make the Radius documentation better!
Please follow this checklist before submitting:
In addition, please fill out the following to help reviewers understand this pull request:
Description
Small change to Existing application tutorial that adds crosslinks to the Radius Connections materials.
Issue reference
This addresses some feedback collected from the community during office hours.