Skip to content
This repository has been archived by the owner on Nov 20, 2023. It is now read-only.
This repository has been archived by the owner on Nov 20, 2023. It is now read-only.

DaprExtension not working with Dapr 0.8.0 ( unable to find components folder) #555

Closed
mtaghavi2005 opened this issue Jun 21, 2020 · 4 comments · Fixed by #556
Closed

DaprExtension not working with Dapr 0.8.0 ( unable to find components folder) #555

mtaghavi2005 opened this issue Jun 21, 2020 · 4 comments · Fixed by #556
Assignees
Labels
bug Something isn't working
Milestone

Comments

@mtaghavi2005
Copy link
Contributor

Describe the bug

DaprExtension is not working with newly released version of Dapr (0.8.0).

To Reproduce

  1. Clone the tye repository and go to tye\samples\dapr\
cd .\samples\dapr\
  1. Update Dapr.AspNetCore package for all project (order, products, store) to 0.8.1
  2. Reassure that you are using the 0.8.1 runtime version of dapr.
dapr --version
// and upgrade if is necessary
dapr uninstall --all
dapr init --runtime-version 0.8.0
  1. Run tye and open dashboard page
 tye run --dashboard
  1. In opened dashboard find the store binding address and open it. after clicking any buy button you will get following error:

tye_samples_dapr_store

  1. In dashboard everything seems to be alright, but if you go to log details page of one dapr proxy service like "store-dapr" you will find following warning:

image

Further technical details

After some investigations I found following problem:

According to one of the latest changes (dapr/dapr@8f91634) in Dapr 0.8.0, if --components-path option is not provided, then $HOME/.dapr/component in Linux/MacOS and %USERPROFILE%.dapr\components for Windows will be selected to used as the default components path instead of "./components". But in DaprExtension there is no possibility to provide -components-path in tye.yaml file. (like config or log-level)

@mtaghavi2005 mtaghavi2005 added the bug Something isn't working label Jun 21, 2020
@mtaghavi2005
Copy link
Contributor Author

Very quick fix :

https://github.com/mtaghavi2005/tye/tree/bugfix/dapr-components-path

If it is applicable, I will submit a PR.
Thank you in advance.

@kimcuhoang
Copy link

Since the new release of Dapr will automatically create %UserProfile%\.dapr\components folder. Therefore, we can work around by copying pubsub.yaml and statestore.yaml to this location.

@jkotalik
Copy link
Contributor

I think supporting the components via the extension is fine. @mtaghavi2005 feel free to submit a PR.

I'd think tye shouldn't take an opinion on the components folder though, it should use the default based on the version of dapr used.

@mtaghavi2005
Copy link
Contributor Author

I think supporting the components via the extension is fine. @mtaghavi2005 feel free to submit a PR.

I'd think tye shouldn't take an opinion on the components folder though, it should use the default based on the version of dapr used.

True, I 'd think the same, but it can just give a chance (providing option via extension) to configure it otherwise won't change the default behavior for dapr.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants