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

. #33

Closed
wants to merge 28 commits into from
Closed

. #33

wants to merge 28 commits into from

Commits on Dec 14, 2023

  1. fix missing label on server_server_latency metric

    Signed-off-by: Kun Chang <curtis@mail.ustc.edu.cn>
    ckcd authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    6780691 View commit details
    Browse the repository at this point in the history
  2. feat: add linter version checker and workflow

    Signed-off-by: mikeee <hey@mike.ee>
    mikeee authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    23d93a8 View commit details
    Browse the repository at this point in the history
  3. feat: add lint version checker to lint make flow

    Signed-off-by: mikeee <hey@mike.ee>
    mikeee authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    b88ea63 View commit details
    Browse the repository at this point in the history
  4. fix: convert line endings to LF on checkout

    Signed-off-by: mikeee <hey@mike.ee>
    mikeee authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    4ee8516 View commit details
    Browse the repository at this point in the history
  5. fix: rename main branch

    Signed-off-by: mikeee <hey@mike.ee>
    mikeee authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    61efacc View commit details
    Browse the repository at this point in the history
  6. fix: remove erroneous char

    Co-authored-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com>
    Signed-off-by: Mike <hey@mike.ee>
    2 people authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    1c2a616 View commit details
    Browse the repository at this point in the history
  7. fix: set up go using go.mod version

    Signed-off-by: mikeee <hey@mike.ee>
    mikeee authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    86a6b5b View commit details
    Browse the repository at this point in the history
  8. fix: migrate tool into build-tools

    add add cmd and associated flag for running.
    - `path` exists to supply a custom path
    
    Signed-off-by: mikeee <hey@mike.ee>
    mikeee authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    65873af View commit details
    Browse the repository at this point in the history
  9. fix: migrate tooling workflow

    Signed-off-by: mikeee <hey@mike.ee>
    mikeee authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    698d88b View commit details
    Browse the repository at this point in the history
  10. fix: remove strict lint step for tools

    Signed-off-by: mikeee <hey@mike.ee>
    mikeee authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    de761d2 View commit details
    Browse the repository at this point in the history
  11. fix: refactor version comparison function

    Migrate test target to testdata folder
    Implement errors returned by the version comparison
    Exit with non-zero error for failures
    Migrate update instructions
    
    Signed-off-by: mikeee <hey@mike.ee>
    mikeee authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    e3541a5 View commit details
    Browse the repository at this point in the history
  12. Fixing typo.

    Signed-off-by: Artur Souza <asouza.pro@gmail.com>
    artursouza authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    9354ec0 View commit details
    Browse the repository at this point in the history
  13. Metadata API: report actor runtime status (dapr#7040)

    * Metadata API: report actor runtime status
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Added integration tests
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Changed per review feedback
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Address review feedback 1
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Address review feedback 2
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Renamed to WithInMemoryActorStateStore
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Added comments
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    ---------
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    Signed-off-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>
    Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
    Co-authored-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>
    3 people authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    cbba34f View commit details
    Browse the repository at this point in the history
  14. Enhance workflow logs (dapr#7222)

    * Adding more logs for workflow executions
    
    Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
    
    * fix typo
    
    Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
    
    * Adding quotes to be consistent
    
    Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
    
    * capitalizing log start
    
    Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
    
    ---------
    
    Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
    Co-authored-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>
    Co-authored-by: Loong Dai <long.dai@intel.com>
    3 people authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    d54abf6 View commit details
    Browse the repository at this point in the history
  15. add e2e test of workflow parallel execution (dapr#7237)

    * original e2e test about workflow parallel execution
    
    Signed-off-by: MregXN <mregxn@gmail.com>
    
    * fix conlicts
    
    Signed-off-by: MregXN <mregxn@gmail.com>
    
    ---------
    
    Signed-off-by: MregXN <mregxn@gmail.com>
    Signed-off-by: MregXN <46479059+MregXN@users.noreply.github.com>
    Co-authored-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>
    2 people authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    88e05b8 View commit details
    Browse the repository at this point in the history
  16. Updates Go to 1.21, golangci-lint to 1.55.2

    Note that, due to the change in import and init func ordering changing
    in Go 1.21, the grpc proxy proto codec is no longer registered after the
    grpc internal proto codec definition, meaning the dapr custom definition
    will not be registered and will cause errors. The custom proto needs to
    be either re-registered at runtime, or change its name in order to fix
    this. This is done in the gRPC object, and handler_test.go TestMain.
    
    Signed-off-by: joshvanl <me@joshvanl.dev>
    JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    5d5ca2b View commit details
    Browse the repository at this point in the history
  17. Lint code

    Signed-off-by: joshvanl <me@joshvanl.dev>
    JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    2ad8e44 View commit details
    Browse the repository at this point in the history
  18. Disable tagalign linter, don't aline yaml tags

    Signed-off-by: joshvanl <me@joshvanl.dev>
    JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    345bcb7 View commit details
    Browse the repository at this point in the history
  19. Remove gRPC Get nil checkings

    Signed-off-by: joshvanl <me@joshvanl.dev>
    JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    79d1b13 View commit details
    Browse the repository at this point in the history
  20. Add extra env vars to charts (dapr#7183)

    * add extra env vars to charts
    
    Signed-off-by: Filinto Duran <filinto@diagrid.io>
    
    * use map instead of array, add docs to README
    
    Signed-off-by: Filinto Duran <filinto@diagrid.io>
    
    * Fix building on 32-bit systems (dapr#7185)
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    Signed-off-by: Filinto Duran <filinto@diagrid.io>
    
    ---------
    
    Signed-off-by: Filinto Duran <filinto@diagrid.io>
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    Co-authored-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com>
    Co-authored-by: Loong Dai <long.dai@intel.com>
    3 people authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    c353d51 View commit details
    Browse the repository at this point in the history
  21. chore: removed namespace (dapr#7271)

    Signed-off-by: syedsadath-17 <90619459+sadath-12@users.noreply.github.com>
    sadath-12 authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    ef10920 View commit details
    Browse the repository at this point in the history
  22. Updates dapr/kit and dapr/components-contrib to HEAD

    Updates dapr/kit and dapr/components-contrib to HEAD. This primarily
    includes updating the fswatcher from the old `fswatcher.Watch` to the
    new `fs, err := fswatcher.New(fswatcher.Options)` & `fs.Run` pattern.
    
    `pkg/security` Provider now includes a `fswatcherInterval` option which
    is used as the trust bundle fswatcher interval duration. This is the
    interval to batch file events for the same file in fswatcher. This is always
    set to 500ms except for tests where it is set to 0 to speed up the test
    run. We want to keep this at 500ms as the live value to reduce CPU & IO
    usage on trust bundle file event churn which occurs on Kubernetes
    volumes or some text editors on "single" write events.
    
    https://ahmet.im/blog/kubernetes-inotify/
    https://www.extrema.is/blog/2022/03/04/vim-and-inotify
    
    ---
    
    With these changes not in master, other developers wanting to make
    changes to dapr/dapr with an updated dapr/kit branch have to write these
    changes themself which is a pain & adds noise to PRs. I think moving
    forward, we should make an effort to ensure any updates to dapr/kit &
    dapr/components-contrib, particularly breaking changes, are updated in
    the dapr/dapr go.mod and merged into master as soon as possible. We can
    make the go.mod dapr/dapr PR a prereq like we do for docs PRs on new
    Dapr features.
    
    Signed-off-by: joshvanl <me@joshvanl.dev>
    JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    d992df9 View commit details
    Browse the repository at this point in the history
  23. Adds Daprd option --dapr-block-shutdown-duration (dapr#7268)

    * Adds Daprd option `--block-shutdown-seconds`
    
    Closes dapr#4313
    Docs: dapr/docs#3893
    
    PR adds the `--block-shutdown-seconds` CLI flag and corresponding
    `dapr.io/block-shutdown-seconds` Kubernetes annotation which configures
    Daprd to block the graceful shutdown procedure until _either_, the
    block shutdown seconds has elapsed _or_ the application has become
    unhealthy, as according to the normal app health status.
    
    By default, this option is unset, and therefore there is no effect to
    the current behaviour of graceful shutdown. When set, Daprd will
    block the interrupt signal cascading into runtime until the above
    requirements have been met.
    
    The framework process `Cleanup` order has been reversed to mimic
    `t.Cleanup` and allow the `logline` process to function correctly.
    
    Signed-off-by: joshvanl <me@joshvanl.dev>
    
    * Revert if check on killing process exec proc cleanup
    
    Signed-off-by: joshvanl <me@joshvanl.dev>
    
    * Revert error ignore of processes already killed in unix
    
    Signed-off-by: joshvanl <me@joshvanl.dev>
    
    * Skip shutdown/graceful/block/healthy on windows.
    
    * Skip shutdown/block/unhealthy test on windows.
    
    * Linting
    
    Signed-off-by: joshvanl <me@joshvanl.dev>
    
    * Updates `dapr-block-shutdown-seconds` to `dapr-block-shutdown-duration`
    
    Signed-off-by: joshvanl <me@joshvanl.dev>
    
    ---------
    
    Signed-off-by: joshvanl <me@joshvanl.dev>
    Co-authored-by: Loong Dai <long.dai@intel.com>
    JoshVanL and daixiang0 committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    d1df26e View commit details
    Browse the repository at this point in the history
  24. Fix timeouts in HTTP service invocation when resiliency policies with…

    … timeouts are applied (dapr#7270)
    
    * Fix race condition in policy runner when there's a timeout
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Better way to fix the error
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    ---------
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
    2 people authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    5c61a36 View commit details
    Browse the repository at this point in the history
  25. Serialize reminders as protobuf in state store (dapr#7129)

    * Added support for reading/writing reminders as protobuf depending on API level
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Increase target QPS
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Added integration tests
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * 💄
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Updated test code per review feedback
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * 💄
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Mod tidy
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Update tests/integration/suite/actors/reminders/serialization_json.go
    
    Signed-off-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Update tests/integration/framework/process/daprd/options.go
    
    Co-authored-by: Josh van Leeuwen <me@joshvanl.dev>
    Signed-off-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Changed per review feedback
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Lint
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Should fix the test errors
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Lint....
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    * Disable ITs on Windows due to SQLite limitations
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    
    ---------
    
    Signed-off-by: ItalyPaleAle <43508+ItalyPaleAle@users.noreply.github.com>
    Signed-off-by: Alessandro (Ale) Segala <43508+ItalyPaleAle@users.noreply.github.com>
    Co-authored-by: Dapr Bot <56698301+dapr-bot@users.noreply.github.com>
    Co-authored-by: Josh van Leeuwen <me@joshvanl.dev>
    Co-authored-by: Yaron Schneider <schneider.yaron@live.com>
    4 people committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    4f5d1d3 View commit details
    Browse the repository at this point in the history
  26. Adding myself as dapr bot owner (dapr#7233)

    Signed-off-by: ashiquemd <noorani.ashique5@gmail.com>
    Signed-off-by: MD Ashique <noorani.ashique5@gmail.com>
    Co-authored-by: Mukundan Sundararajan <65565396+mukundansundar@users.noreply.github.com>
    2 people authored and JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    a450f61 View commit details
    Browse the repository at this point in the history
  27. Version Skew Integration Tests

    Like the existing e2e version skew tests, this PR adds version skew
    integration tests. It runs the latest release integrations tests on a
    matrix of latest release/master and control plane/daprd. There is also a
    patching mechanism to account for things like accepted breaking changes
    and error string matching.
    
    Like master, the current placement/actor tests are currently failing.
    
    Invoked via `/test-version-skew`.
    
    Signed-off-by: joshvanl <me@joshvanl.dev>
    JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    2275262 View commit details
    Browse the repository at this point in the history
  28. .

    Signed-off-by: joshvanl <me@joshvanl.dev>
    JoshVanL committed Dec 14, 2023
    Configuration menu
    Copy the full SHA
    d07ee1b View commit details
    Browse the repository at this point in the history