v1.3.0 #1300
Merged
v1.3.0 #1300
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Changes from ssvlabs/ssv-spec#322 * Changes from ssvlabs/ssv-spec#321 * Update spec to 375a276 * Deploy to 21-24 * Deploy to all except 25-28 * Revert "Deploy to all except 25-28" This reverts commit c3faedb. * Revert "Deploy to 21-24" This reverts commit 69e3a2a. * Deploy to all nodes * Revert "Deploy to all nodes" This reverts commit 6f4bd6f. * Deploy to 21-24 * Revert "Deploy to 21-24" This reverts commit a333cd6. * Deploy to all nodes * Revert "Deploy to all nodes" This reverts commit 0eb4f8c. * Deploy to 21-24 * Revert "Deploy to 21-24" This reverts commit f9d9e9d. * Update differ.config.yaml with approved changes --------- Co-authored-by: moshe-blox <moshe@blox.io>
* initial e2e implementation --------- Co-authored-by: olegshmuelov <45327364+olegshmuelov@users.noreply.github.com> Co-authored-by: moshe-blox <89339422+moshe-blox@users.noreply.github.com> Co-authored-by: moshe-blox <moshe@blox.io>
* removed any reference for operator id in node metadata * adjust TestNodeInfo_Marshal_Unmarshal to test backward compatibility * remove OperatorID from node metadata
* checkpoint * test update fee recipient * add tests for GetValidator and SetOperatorData * checkpoint: adding new test for start share * checkpoint: adding new test for start share * add test to cover validator controller * add test to cover validator controller * fix lint * fix lint * change r to dutyRunner for better description * finish setupValidatorsTest * fix lint * code review improvements * fix lint * add test for on share remove * add new test for UpdateValidatorMetadata * add test for validator controller TestSetupNonCommitteeValidators * added tests for second part of UpdateValidatorMetadata * added tests for task executor * add test for create new controller * fix lint * fix test after merge stage * fix test after merge stage * fix test after merge stage * fix test after merge stage * fix test after merge stage * fix test after merge stage * fix test after merge stage * fix lint issue * remove unusage function * remove redundant file and fix tests * sort imports and remove unused functions from P2P * added a few PK checks --------- Co-authored-by: y0sher <lyosher@gmail.com> Co-authored-by: moshe-blox <moshe@blox.io>
* remove test bellatrix blocks * override state comparison in SyncCommitteeAggregatorProofSpecTest * Refactor RecipientData JSON marshaling and unmarshaling * upgrade go-eth2-client * upgrade ekm & spec to latest tag --------- Co-authored-by: moshe-blox <moshe@blox.io>
* fix: (SlotTicker) potential double tick --------- Co-authored-by: Matus Kysel <MatusKysel@users.noreply.github.com>
* Exposed exporters api ports --------- Co-authored-by: stoyan.peev <stoyan.peev@gotoadmins.com> Co-authored-by: moshe-blox <moshe@blox.io>
* fix: timeout in go-eth2-client's `Validators` method
* Fix error messages in checkPrysmDebugEndpoints function
MatusKysel
previously approved these changes
Feb 1, 2024
liorrutenberg
previously approved these changes
Feb 1, 2024
nkryuchkov
previously approved these changes
Feb 2, 2024
moshe-blox
commented
Feb 2, 2024
| @@ -32,21 +31,25 @@ func TestNodeInfo_Seal_Consume(t *testing.T) { | |||
| } | |||
|
|
|||
| func TestNodeInfo_Marshal_Unmarshal(t *testing.T) { | |||
| ni := &NodeInfo{ | |||
| oldSerializedData := []byte(`{"Entries":["", "testnet", "{\"NodeVersion\":\"v0.1.12\",\"ExecutionNode\":\"geth/x\",\"ConsensusNode\":\"prysm/x\",\"OperatorsID\":\"xxx\"}"]}`) | |||
There was a problem hiding this comment.
typo: OperatorsID (should be OperatorID)
* Don't use BeaconState in Validators call
moshe-blox
dismissed stale reviews from nkryuchkov, liorrutenberg, and MatusKysel
via
e3a6629
liorrutenberg
previously approved these changes
Feb 4, 2024
olegshmuelov
reviewed
Feb 4, 2024
e2e/go.mod
Outdated
|
|
||
| require ( | ||
| github.com/attestantio/go-eth2-client v0.16.3 | ||
| github.com/bloxapp/ssv v1.2.1-0.20231204141016-a6afbe837a26 |
There was a problem hiding this comment.
should the e2e be updated as well?
There was a problem hiding this comment.
Since it doesn't do proposals yet then there's a chance that it doesn't need to, right? @y0sher
There was a problem hiding this comment.
gonna try running it, not sure since go-eth2-client has been updated not just in proposals.
…1295) * stop run if any command exited with non-zero exit code. * clean dockers and volumes before starting a new run * clean exited dockers as well
moshe-blox
requested review from
olegshmuelov,
liorrutenberg,
nkryuchkov and
MatusKysel
olegshmuelov
approved these changes
Feb 4, 2024
y0sher
approved these changes
Feb 4, 2024
liorrutenberg
approved these changes
Feb 4, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
v0.3.6#1299)byIndices#1283)BeaconStateinValidatorscall #1301)