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.

Sign up for GitHub

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

Jump to bottom

Merge Sidecar and upgrade Pitaya to v3 #287

Open
wants to merge 96 commits into
base: main
Choose a base branch
from
Open

Merge Sidecar and upgrade Pitaya to v3 #287

wants to merge 96 commits into from

Conversation

GabrielBackes
Copy link

No description provided.

codingcn and others added 30 commits December 26, 2020 08:38
@codingcn @felipejfc
BeforeHandler return context
dc45a18
@felipejfc
fix BeforeHandler.Handlers res
9943b0b
@felipejfc
fix TestBeforeHandler test
2b1b9ff
@victor-carvalho
Migrate from Travis CI to Github Actions (topfreegames#159)
d2bb5f0 
Setup github actions
@victor-carvalho @felipejfc
Update go version
053dc61
@victor-carvalho
Update dependencies (topfreegames#161)
d062a17 
* Fix local tests in OSX
* Update dependencies
@rsafonseca
Update several libraries and dependencies
3ac8d06
@rsafonseca
fix make setup, broken since 1.1.5
d9ceab8
@rsafonseca
Prevent gigantic flood of error messages when etcd channel dies
f16dfdd
@rsafonseca
update test libs
370d7ce
@rsafonseca
update more libs
d28ae61
@rsafonseca
Add 2 tests to statsd metrics
3328b0e
@rsafonseca
Fix session test after nats-2.0 server upgrade in helpers
6662b35
@rsafonseca
fix cluster tests
ae67201
@rsafonseca
fix service test
506edba
@rsafonseca
Fix remaining tests
371658f
@rsafonseca
Remove accidental file addition
fd9cf18
@rsafonseca
Fix error assertion
a0958fc
@rsafonseca
Merge pull request topfreegames#170 from topfreegames/update_deps
bb02946 
Update several dependencies
@rsafonseca
Fix error check in test
a94883f
@rsafonseca
Recover service discovery from etcd client channels closing
8251063
@rsafonseca
Revert "Fix tests (topfreegames#154)"
930e1d4 
Breaks Metric codes
This reverts commit a215e15.
@rsafonseca
Revert to returning original error object
41329c3
@rsafonseca
Don't try to initialize real client during test
22e4a99
@rsafonseca
Add RPC type info to spans
85893ce
@rsafonseca
Don't log nats URL on reconnect since it contains credentials when us…
e487b30 
…ing basic auth
@rsafonseca
Drain NATS server connections instead of closing directly
dbfae28
@rsafonseca
Merge pull request topfreegames#172 from topfreegames/fixbugs
177cdf3 
Fix metric tags, etcd reconnect, improve nats shutdown handling and logging
@rsafonseca
Use single get on syncservers and parallelise parsing only
60c9f16
@rsafonseca
Run gofmt on changed files
540d51f
felipejfc and others added 16 commits July 27, 2021 21:32
@coveralls
Copy link

coveralls commented Oct 20, 2022 •
edited

Pull Request Test Coverage Report for Build 3330939036

  • 219 of 299 (73.24%) changed or added relevant lines in 27 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.09%) to 62.137%
Changes Missing Coverage Covered Lines Changed/Added Lines %
pkg/acceptor/ws_acceptor.go 10 11 90.91%
pkg/agent/agent_remote.go 9 10 90.0%
pkg/modules/unique_session.go 0 1 0.0%
pkg/service/handler.go 24 25 96.0%
pkg/cluster/nats_rpc_client.go 4 6 66.67%
pkg/session/session.go 7 9 77.78%
pkg/service/util.go 0 3 0.0%
pkg/cluster/grpc_rpc_client.go 12 16 75.0%
pkg/app.go 18 24 75.0%
pkg/util/util.go 1 8 12.5%
Totals Coverage Status
Change from base Build 2863670078: -0.09%
Covered Lines: 4692
Relevant Lines: 7551
💛 - Coveralls

@felipejfc felipejfc force-pushed the main branch 3 times, most recently from 1d5b0cf to 98580b5 Compare May 20, 2023 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@GabrielBackes @coveralls @codingcn @victor-carvalho @rsafonseca @felipejfc