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

Sync upstream #76

Merged
merged 25 commits into from
Oct 4, 2024
Merged

Sync upstream #76

merged 25 commits into from
Oct 4, 2024

Conversation

antonio2368
Copy link
Member

No description provided.

dependabot bot and others added 25 commits April 9, 2024 22:43
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 4.1.0 to 4.2.0.
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](codecov/codecov-action@54bcd87...7afa10e)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 4.2.0 to 4.3.0.
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](codecov/codecov-action@7afa10e...8450866)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…eBay#498)

* auto_fwd_resp_handler should forward resp->get_result_code (eBay#497)

* auto_fwd_resp_handler should forward resp->get_result_code (eBay#497)

---------

Co-authored-by: byronhe <[email protected]>
…n value (eBay#502) (eBay#503)

* fix src/tracer.hxx stack-buffer-overflow when using vsnprintf's return value (eBay#502)

* 1. add logger_test to code coverage test 2. logger_test set printTestMessage to true   (eBay#502)

---------

Co-authored-by: byronhe <[email protected]>
Bumps [actions/checkout](https://github.com/actions/checkout) from 4.1.2 to 4.1.4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@9bb5618...0ad4b8f)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* If there is an incoming random message, where `CRC_ON_ENTIRE_MESSAGE`
flag is randomly set, NuRaft tries to read the entire message for CRC
check. That requires allocating a memory blob with the given size
(also a random number) which most likely causes problem.

* Added more sanity checks before the memory allocation.
* Once reading snapshot request fails, the state machine should return
a negative number, not 0.
Bumps [actions/checkout](https://github.com/actions/checkout) from 4.1.4 to 4.1.6.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@0ad4b8f...a5ac7e5)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 4.3.0 to 4.4.1.
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](codecov/codecov-action@8450866...125fc84)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Add timeout to async_connect operation

If the network goes down during a async_connect, we may end up being
stuck forever in this function since there is no timeout to cancel it.

Signed-off-by: Alex Michon <[email protected]>

* Indent and line breaking

---------

Signed-off-by: Alex Michon <[email protected]>
Co-authored-by: Jung-Sang Ahn <[email protected]>
* Add result code to response messages

In some cases, the response is marked as accepted even if there was an
error. Therefore, including the accepted tag in the response message is
not enough: a follower may believe that an auto forwarded request was
successful when it wasn't.

Signed-off-by: Alex Michon <[email protected]>

* [Update PR] Add `int32_t` to `cmd_result_code`

---------

Signed-off-by: Alex Michon <[email protected]>
Co-authored-by: Jung-Sang Ahn <[email protected]>
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 4.4.1 to 4.5.0.
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md)
- [Commits](codecov/codecov-action@125fc84...e28ff12)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/checkout](https://github.com/actions/checkout) from 4.1.6 to 4.1.7.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@a5ac7e5...692973e)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Notify user code about follower loss

* PR review fixes

* Fixed a warning "comparison of integer expressions of different signedness"

* [Update PR] Add comments and put more set_recovered() calls

---------

Co-authored-by: Pavel Yurin <[email protected]>
Co-authored-by: Jung-Sang Ahn <[email protected]>
eBay#521)

* Matching format specifies type to argument type to eliminate compile warnings

* Apply comments

* Fix another place

---------

Co-authored-by: Zexi Liu <[email protected]>
* use leadership_expiry_ to deem nodes unhealthy

* address pr comments
* When a leader invokes `end_of_append_batch`, the first parameter
represents the starting index of the batch. There was missing `+1`.
* Use pthread_exit to terminate raft service.

Allow the process to handle termination of raft service rather than
force process wide `exit()`. This is compile time configurable during CMake configuration.

Addresses eBay#535

---------

Co-authored-by: Jung-Sang Ahn <[email protected]>
* Callback can decide whether to ignore or to die.
@CLAassistant
Copy link

CLAassistant commented Oct 3, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 11 committers have signed the CLA.

✅ azat
✅ antonio2368
❌ greensky00
❌ byronhe
❌ fwhdzh
❌ apostolpav113
❌ amichon-kalray
❌ antoniofilipovic
❌ rockett-m
❌ ZexiLiu
❌ szmyd
You have signed the CLA already but the status is still pending? Let us recheck it.

@antonio2368 antonio2368 merged commit c1d2b1c into master Oct 4, 2024
1 check passed
@antonio2368 antonio2368 deleted the sync-upstream branch October 4, 2024 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.