Commit Graph

9 Commits

Author SHA1 Message Date
Mike Nguyen 0cd0585b64
fix: arguments accept units (#1490)
* fix: arguments accept units
`max-body-size` and `read-buffer-size` now accept units as defined in the docs.

Fixes #1489

Signed-off-by: Mike Nguyen <hey@mike.ee>

* chore: gofumpt

Signed-off-by: Mike Nguyen <hey@mike.ee>

* refactor: modify logic to comply with vetting

Signed-off-by: Mike Nguyen <hey@mike.ee>

* chore: gofumpt -w .

Signed-off-by: Mike Nguyen <hey@mike.ee>

* refactor: set defaults
`max-body-size` is defaulted to 4Mi
`request-buffer-size` is defaulted to 4Ki

This is inline with the runtime.

Signed-off-by: Mike Nguyen <hey@mike.ee>

* fix: set defaults in run and annotate

Signed-off-by: Mike Nguyen <hey@mike.ee>

* chore: gofumpt

Signed-off-by: Mike Nguyen <hey@mike.ee>

* refactor: exit with error rather than panic

Co-authored-by: Anton Troshin <troll.sic@gmail.com>
Signed-off-by: Mike Nguyen <hey@mike.ee>

---------

Signed-off-by: Mike Nguyen <hey@mike.ee>
Co-authored-by: Anton Troshin <troll.sic@gmail.com>
2025-02-21 08:11:11 -08:00
Yaron Schneider ac6822ea69
Remove deprecated flags (#1482)
* remove deprecated flags

Signed-off-by: yaron2 <schneider.yaron@live.com>

* update Dapr version in tests

Signed-off-by: yaron2 <schneider.yaron@live.com>

---------

Signed-off-by: yaron2 <schneider.yaron@live.com>
2025-02-03 13:27:22 -08:00
Mike Nguyen ed0d3af2d0
fix: scheduler host address passed to runtime (#1421)
* fix: scheduler host address passed to runtime

Signed-off-by: mikeee <hey@mike.ee>

* fix: scheduler client stream initialised for 1.14<

Signed-off-by: mikeee <hey@mike.ee>

* fix: modify scheduler host address validation

if the scheduler container is not active, the scheduler flag will
not be passed to the runtime

Signed-off-by: mikeee <hey@mike.ee>

* fix: lint and refactor

Signed-off-by: mikeee <hey@mike.ee>

---------

Signed-off-by: mikeee <hey@mike.ee>
2024-07-10 09:37:34 -07:00
Mukundan Sundararajan 4e3ae81df5
deprecate app-ssl flag in cli (#1269)
* deprecate app-ssl flag in cli

Signed-off-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>

* Update cmd/annotate.go

Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
Signed-off-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>

---------

Signed-off-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
2023-04-12 21:02:10 -07:00
Mukundan Sundararajan 65855d5aa0
Upgrade linter to 1.49.0 (#1051)
* Upgrade linter to 1.49.0

Signed-off-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>

* fix linter error

Signed-off-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>

Signed-off-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>
2022-08-25 16:58:54 +05:30
qiangmzsx f8b48c2a8a
Removing the use of the ioutil package (#1037)
Signed-off-by: qiangmzsx <qiangmzsx@gmail.com>
2022-08-07 17:30:20 +05:30
Shubham Sharma abb267057c
Add new annotations to annotate command (#1020)
* Add new annotations

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>

* Fix lint

Signed-off-by: Shubham Sharma <shubhash@microsoft.com>
2022-07-05 09:31:24 -07:00
Joni Collinge 992b493073
require -k flag to target kubernetes for annotate command (#1016)
Signed-off-by: Joni Collinge <jonathancollinge@live.com>

Co-authored-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>
2022-07-04 21:57:09 -07:00
Joni Collinge 0630996fb4
Add dapr annotate command (#873)
* Add dapr inject command

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* Add additional opts

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* Update comment regarding injection

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* fix some linting issues (more to go)

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* use existing const in tests

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* fix some more linting

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* fix some more linting

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* udpate tests

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* udpate tests

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* add option to run against a URL

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* refactor cmd/inject.go slightly

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* update example text

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* autogenerate appid if not present

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* Update comment

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* Add support for target namespace

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* Update comment

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* Add warning about app id generation to usage

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* fix linting issues

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* rename inject to annotate

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* go mod tidy

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* fix lint

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* update --help URL

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

* add README.md seciton

Signed-off-by: Joni Collinge <jonathancollinge@live.com>

Co-authored-by: Yaron Schneider <schneider.yaron@live.com>
2022-06-02 08:00:16 -07:00