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

Proposed 1.6.0-b7 #3424

Merged
merged 9 commits into from
Jun 1, 2020
Merged

Proposed 1.6.0-b7 #3424

merged 9 commits into from
Jun 1, 2020

Conversation

manojsdoshi
Copy link
Contributor

If merged this PR will:

close #3403
close #3235
close #3365
close #3416
close #3415
close #3332
close #3367

HowardHinnant and others added 9 commits May 29, 2020 14:02
* Gives a summary of the health of the node:
  Healthy, Warning, or Critical

* Last validated ledger age:
  <7s is Healthy,
  7s to 20s is Warning
  > 20s is Critcal

* If amendment blocked, Critical

* Number of peers:
  > 7 is Healthy
  1 to 7 is Warning
  0 is Critical

* server state:
  One of full, validating or proposing is Healthy
  One of syncing, tracking or connected is Warning
  All other states are Critical

* load factor:
  <= 100 is Healthy
  101 to 999 is Warning
  >= 1000 is Critical

* If not Healthy, info field contains data that is considered not
  Healthy.

Fixes: XRPLF#2809
If a port number is not specified in the [ips] or [ips_fixed]
blocks, automatically add the new default peer port which was
registered with IANA: 2459. Also use 2459 if no port is specified
with manually using the `connect` command; previously it was
using 6561, which could have resulted in spurious failures.
This commit, if merged, fixes XRPLF#2861.
 * The latest docker version is not supported by
   artifactory causing the package build to fail.
   Setting the docker version to 19.03.8 to fix the build.
@manojsdoshi manojsdoshi requested a review from nbougalis May 30, 2020 01:10
@carlhua carlhua self-assigned this May 30, 2020
@manojsdoshi manojsdoshi merged commit f43aeda into XRPLF:develop Jun 1, 2020
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.

6 participants