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

Fixes to dapr.exe cmdline invocation#967

Merged
jkotalik merged 2 commits into
dotnet:mainfrom
mmisztal1980:bug/995-cmdline-fixes
Mar 12, 2021
Merged

Fixes to dapr.exe cmdline invocation#967
jkotalik merged 2 commits into
dotnet:mainfrom
mmisztal1980:bug/995-cmdline-fixes

Conversation

@mmisztal1980

@mmisztal1980 mmisztal1980 commented Mar 10, 2021

Copy link
Copy Markdown
Contributor

This PR:

All affected arguments have been fixed to match the ones in dapr run --help

dapr --version
CLI version: 1.0.0
Runtime version: 1.0.1

dapr run --help
Run Dapr and (optionally) your application side by side. Supported platforms: Self-hosted

Usage:
  dapr run [flags]

Examples:

# Run a .NET application:
  dapr run --app-id myapp --app-port 5000 -- dotnet run
# Run a Java application:
  dapr run --app-id myapp -- java -jar myapp.jar
# Run a NodeJs application that listens to port 3000:
  dapr run --app-id myapp --app-port 3000 -- node myapp.js
# Run a Python application:
  dapr run --app-id myapp -- python myapp.py
# Run sidecar only:
  dapr run --app-id myapp


Flags:
  -a, --app-id string                   The id for your application, used for service discovery
      --app-max-concurrency int         The concurrency level of the application, otherwise is unlimited (default -1)
  -p, --app-port int                    The port your application is listening on (default -1)
  -P, --app-protocol string             The protocol (gRPC or HTTP) Dapr uses to talk to the application (default "http")
      --app-ssl                         Enable https when Dapr invokes the application
  -d, --components-path string          The path for components directory (default "C:\\Users\\mmisztal\\.dapr\\components")
  -c, --config string                   Dapr configuration file (default "C:\\Users\\mmisztal\\.dapr\\config.yaml")
  -G, --dapr-grpc-port int              The gRPC port for Dapr to listen on (default -1)
  -H, --dapr-http-port int              The HTTP port for Dapr to listen on (default -1)
      --enable-profiling                Enable pprof profiling via an HTTP endpoint
  -h, --help                            Print this help message
      --log-level string                The log verbosity. Valid values are: debug, info, warn, error, fatal, or panic (default "info")
  -M, --metrics-port int                The port of metrics on dapr (default -1)
      --placement-host-address string   The host on which the placement service resides (default "localhost")
      --profile-port int                The port for the profile server to listen on (default -1)

@mmisztal1980 mmisztal1980 changed the title #995 #995 Fixes to dapr.exe cmdline invocation Mar 10, 2021
@mmisztal1980

mmisztal1980 commented Mar 10, 2021

Copy link
Copy Markdown
Contributor Author

@rynowak @jkotalik please have a look

Managed to get this working locally
image

@mmisztal1980 mmisztal1980 changed the title #995 Fixes to dapr.exe cmdline invocation Fixes to dapr.exe cmdline invocation Mar 10, 2021
Base automatically changed from master to main March 11, 2021 20:50
@jkotalik jkotalik merged commit 63b03f9 into dotnet:main Mar 12, 2021
@jkotalik

Copy link
Copy Markdown
Contributor

Thanks!

@mmisztal1980

Copy link
Copy Markdown
Contributor Author

@jkotalik can you let me know when this makes it out of CI? :)

@mmisztal1980 mmisztal1980 deleted the bug/995-cmdline-fixes branch March 12, 2021 21:49
@mmisztal1980

Copy link
Copy Markdown
Contributor Author

@jkotalik I've tried running the latest nightly, unfortunately the change didn't make it into the build

@jkotalik

Copy link
Copy Markdown
Contributor

Should be updated now.

@mmisztal1980

Copy link
Copy Markdown
Contributor Author

@jkotalik I've managed to verify that this works on my Mac OS out of the box now :)

jkotalik added a commit that referenced this pull request May 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants