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

[pull] dev from AsynkronIT:dev #258

Open
wants to merge 962 commits into
base: dev
Choose a base branch
from
Open

[pull] dev from AsynkronIT:dev #258

wants to merge 962 commits into from

Conversation

pull[bot]
Copy link

@pull pull bot commented Apr 22, 2021

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

@pull pull bot added the ⤵️ pull label Apr 22, 2021
rogeralsing and others added 29 commits July 1, 2022 16:40
* Add Skyrise ActorBenchmark in mini format
* remove metrics from benchmark
* remoting fixes
* enable correct GC for cluster tests
* only pass target id
* only pass sender address if different from remote
* seal apis
* use concurrent bag for shared futures
* ValueTask cluster context
* add tps to skyrise benchmark
* short pids
* Inline cluster context code
* Get rid of WhenAny for cluster context net6
* mailbox is IThreadPoolWorkitem
* fix ci
* fix ci
* working on timeouts in pub sub

* fixed issue with publish timeout, added failure reason to publish response

* moved pub sub config to separate class

* try higher timeout values to prevent failing in CI

* only remove pid subscribers on error

* does disabling parallelism fix CI?

* diagnosing CI tests

* some tests can run in parallel

* is the problem caused by cancel token expiring too late?

* will this now work in parallel?

* switching to at-most-once guarantee

* fix incorrect failed delivery notification

* logging adjustments, delivery actor is now user level actor

* cleanup

* adjust timeouts

* check if disabling concurrent execution fixes CI

* cleanup

* some missing xml docs

* CR fixes
* fixed problem with nullref when loading subscribers

* test for default registration

* limit cluster size to 1 for the empty kv store
* remove 5.0, limit framework installations
* get rid of global.json
* standardize on netcoreapp3.1
* slow - fast split
* remove net5.0 target, examples only target net6.0
* do not run build/nuget on pull request
* move actor tests to fast lane
* cluster request timeout now results in TimeoutException
* move timeout tests to separate classes, reduce timeouts
)

Adds package-namespaced code-generated grains.

Authored-by: Elnur Mammadov <[email protected]>
* Added single node 'cluster' provider, allowing Proto.Cluster to be used in single applications with the cluster / virtual actor abstractions.

This makes it easy to get started, and allows for the same programming model in a single service use case as in a clustered environment.

Updated cluster tests to allow testing single node setups.
* actor system, context and pid

* converted channel publisher/subscriber to an example

* re-used remote channels example instead of the local one, further public api changes

* block list

* timeout names cleanup in cluster config

* cluster changes

* cluster changes

* removed usage of ClusterInit, the msg itself stays for now

* removed unused Tick message

* test CI

* restore CI config, clean up xml docs related warnings

* adjustments after merge
AdamCLarsen and others added 30 commits February 5, 2024 16:21
…etes cluster provider (#2097)

* Add Host to k8s pod labels

* Removed logging of k8s workload kind

* Added new Public API to get K8s Pod FQDN

hardened GetKubeNamespace() so it will not read in more data then kuberneties allows for a namespace, it will also now throw a with a descriptive error exception message vs FileNotFound.
Replace environment variable HOSTNAME with CONTAINER_APP_REPLICA_NAME.
* subscribe to eventstream using typed channel

* assert publish

* rename
Rearrange cluster shutdown code
… the number of messages in the batchingMailbox because of dead loop (#2105)
* Added support for propagating OpenTelemetry baggage

* Updated tests, only set baggage if it's in use
* Switch from System.Data.SqlClient to Microsoft.Data.SqlClient per official recommendation

* use await on using
* Track Spawning
* Track EventStream Publish
* Track EventStream Subscription
* graceful stop of cluster actors
To be a good citizen, it is important that the caller of this method can (and should) call Dispose() on the returned token source. The problem though, is that each time cts.Token was called before, it would try to create a new CancellationToken which means that it could throw an ObjectDisposed exception leading to noisy logs.
* support dotnet8 target

* make sure all the right versions are installed for dotnet 8 testing

* No need for pre-release since it is no longer an argument

* Try with the newest version of the action
In order to write a proper alternative to the DefaultClusterContext, we need these properties to be public
* update copyrights
* add receive timeout
* Upgraded dependencies

* Dependency upgrades & cleanup
* Migrated to centralized package management

* Made the persistence tests ~90 times faster by using fixtures for the external DB's

* Version bumps
…ccur, causing a lockup (#2139)

* Add endpoint manager test to repro thread lockup

fix merge

* add explanation and dockerfile

* Block endpoint while it disposes instead of holding requests behind a lock. this also allows messages to other endpoints while one is disposing, as well as multiple endpoint disposes at the same time.

* Change EndpointManager Stop to StopAsync

* ensure the EndpointReader always finishes up the request after sending the DisconnectRequest, so it doesn't time out during kestrel shutdown

* increase timeout on a couple tests so they fail less often

* increase another timeout on flakey test
* Support adding additional error handling to generated actors

updates for tests

* add delay for flaky test
* Bump OTEL packages

* Moved spawn trace activity metadata from name to tags

* Bump grpc / protobuf versions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.