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

Debugging docker compose instructions #3788

Closed
wants to merge 315 commits into from

Conversation

olitomlinson
Copy link
Contributor

@olitomlinson olitomlinson commented Oct 5, 2023

Thank you for helping make the Dapr documentation better!

Please follow this checklist before submitting:

  • Commits are signed with Developer Certificate of Origin (DCO - learn more)
  • Read the contribution guide
  • Commands include options for Linux, MacOS, and Windows within codetabs
  • New file and folder names are globally unique
  • Page references use shortcodes instead of markdown or URL links
  • Images use HTML style and have alternative text
  • Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:

Description

Issue reference

hhunter-ms and others added 30 commits September 18, 2023 12:16
Signed-off-by: Hannah Hunter <[email protected]>
* Adds `OIDC` authentication to pulsar pubsub options

Signed-off-by: joshvanl <[email protected]>

* Updates the pulsar OIDC authentication docs to use `oauth2` fields.

Signed-off-by: joshvanl <[email protected]>

* Put pulsar authentication options into different sections

Signed-off-by: joshvanl <[email protected]>

* Update daprdocs/content/en/reference/components-reference/supported-pubsub/setup-pulsar.md

Co-authored-by: Mark Fussell <[email protected]>
Signed-off-by: Josh van Leeuwen <[email protected]>

* Update daprdocs/content/en/reference/components-reference/supported-pubsub/setup-pulsar.md

Co-authored-by: Mark Fussell <[email protected]>
Signed-off-by: Josh van Leeuwen <[email protected]>

* Update daprdocs/content/en/reference/components-reference/supported-pubsub/setup-pulsar.md

Co-authored-by: Mark Fussell <[email protected]>
Signed-off-by: Josh van Leeuwen <[email protected]>

---------

Signed-off-by: joshvanl <[email protected]>
Signed-off-by: Josh van Leeuwen <[email protected]>
Co-authored-by: Mark Fussell <[email protected]>
Co-authored-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
* add placement table api doc

Signed-off-by: zhangchao <[email protected]>

* typo fix

Signed-off-by: zhangchao <[email protected]>

* fix review

Signed-off-by: zhangchao <[email protected]>

* fix example

Signed-off-by: zhangchao <[email protected]>

---------

Signed-off-by: zhangchao <[email protected]>
Co-authored-by: Mark Fussell <[email protected]>
Co-authored-by: Hannah Hunter <[email protected]>
hhunter-ms and others added 15 commits November 13, 2023 15:42
Add missing arg value in multi-app-template.md
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
The dapr run command was listed incorrectly for Powershell users. It now has two different options for both Powershell and non-Powershell environments correctly ends in resources-path ../ (done in the get-started-component.md file),
@msfussell msfussell reopened this Nov 16, 2023
@github-actions github-actions bot removed the stale label Nov 17, 2023
Copy link
Member

@msfussell msfussell left a comment

Choose a reason for hiding this comment

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

@olitomlinson - Thanks for this. I added you video at the end. Any chance you can target this on the v1.12 branch now? It would make it easier for us if you can. Thanks


**Note** : It's important to highlight that the `nodeapp` service in the docker compose environment is actually still running, however it has been removed from the docker network so it is effectively orphaned as nothing can communicate to it.


Copy link
Member

Choose a reason for hiding this comment

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

Demo

Watch this video on how to debug local Dapr apps with Docker Compose

<iframe width="560" height="315" src="https://www.youtube-nocookie.com/embed/nWatANwaAik?start=1738" frameborder="0" allow="accelerometer; autoplay; clipboard-write; encrypted-media; gyroscope; picture-in-picture" allowfullscreen></iframe>

@msfussell msfussell added this to the 1.12 milestone Nov 20, 2023
JoshVanL and others added 8 commits November 20, 2023 17:01
…g-docker-compose.md

Co-authored-by: Mark Fussell <[email protected]>
Signed-off-by: Oliver Tomlinson <[email protected]>
…g-docker-compose.md

Co-authored-by: Mark Fussell <[email protected]>
Signed-off-by: Oliver Tomlinson <[email protected]>
…g-docker-compose.md

Co-authored-by: Mark Fussell <[email protected]>
Signed-off-by: Oliver Tomlinson <[email protected]>
…g-docker-compose.md

Co-authored-by: Mark Fussell <[email protected]>
Signed-off-by: Oliver Tomlinson <[email protected]>
…g-docker-compose.md

Co-authored-by: Mark Fussell <[email protected]>
Signed-off-by: Oliver Tomlinson <[email protected]>
Include demo video content

Signed-off-by: Oliver Tomlinson <[email protected]>
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.