-
Notifications
You must be signed in to change notification settings - Fork 670
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
vms/platformvm/service: nits (preallocate address slice, error msg) #1477
Merged
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
Signed-off-by: Gyuho Lee <[email protected]>
dhrubabasu
approved these changes
May 5, 2023
StephenButtolph
approved these changes
May 6, 2023
hexfusion
pushed a commit
to hexfusion/avalanchego
that referenced
this pull request
Jun 22, 2023
Co-authored-by: Dan Laine <[email protected]>
joshua-kim
added a commit
to joshua-kim/avalanchego
that referenced
this pull request
Jun 26, 2023
commit a056efc Author: Patrick O'Grady <[email protected]> Date: Fri Jun 23 06:08:09 2023 -0700 [x/merkledb] Remove useless `err` check (ava-labs#1650) commit 607489d Author: aaronbuchwald <[email protected]> Date: Thu Jun 22 11:27:41 2023 -0400 Update coreth to v0.12.4-rc.0 (ava-labs#1646) commit d2899e6 Author: Stephen Buttolph <[email protected]> Date: Wed Jun 21 23:50:41 2023 -0400 Remove GetTx from the DAGVM interface (ava-labs#1642) commit 2fc0d3b Author: Stephen Buttolph <[email protected]> Date: Wed Jun 21 20:33:03 2023 -0400 Remove PendingTxs from the DAGVM interface (ava-labs#1641) commit cc73cd5 Author: Stephen Buttolph <[email protected]> Date: Wed Jun 21 19:39:44 2023 -0400 Update all AVM tests for post-linearization (ava-labs#1631) commit 0073875 Author: Stephen Buttolph <[email protected]> Date: Wed Jun 21 19:10:10 2023 -0400 Remove `dagState` and `GetUTXOFromID` (ava-labs#1632) commit 94302bc Author: Dan Laine <[email protected]> Date: Wed Jun 21 10:35:38 2023 -0400 Add daily fuzzing action (ava-labs#1635) commit ac3f2b4 Author: Sam Batschelet <[email protected]> Date: Tue Jun 20 10:59:43 2023 -0400 Remove MaxConnectionAge gRPC StreamID overflow mitigation (ava-labs#1388) commit c7e1c6a Author: felipemadero <[email protected]> Date: Mon Jun 19 10:56:57 2023 -0300 Improve delegation error message to specify invalid times or over delegated (ava-labs#1606) commit b85b31c Author: Ikko Eltociear Ashimine <[email protected]> Date: Sun Jun 18 23:32:49 2023 +0900 Fix typo in binary_snowflake.go (ava-labs#1630) commit 9725fe9 Author: Dhruba Basu <[email protected]> Date: Thu Jun 15 20:06:56 2023 -0700 Ban usage of `t.Fatal` and `t.Error` (ava-labs#1453) commit df6228b Author: Dhruba Basu <[email protected]> Date: Thu Jun 15 14:41:23 2023 -0700 Improve `database/` tests with `require` (ava-labs#1506) commit 94b9ce6 Author: Dhruba Basu <[email protected]> Date: Thu Jun 15 13:08:15 2023 -0700 Improve `vms/` tests with `require` (ava-labs#1505) commit f458045 Author: Dan Laine <[email protected]> Date: Thu Jun 15 11:28:19 2023 -0400 Fix flaky `TestFindNextKeyRandom` test (ava-labs#1624) commit 689aec6 Author: Dhruba Basu <[email protected]> Date: Thu Jun 15 06:59:32 2023 -0700 Improve `x/` tests with `require` (ava-labs#1454) commit 764c456 Author: Dhruba Basu <[email protected]> Date: Wed Jun 14 23:04:19 2023 -0700 Improve `snow/` tests with `require` (ava-labs#1503) commit bddbbec Author: Stephen Buttolph <[email protected]> Date: Thu Jun 15 01:47:10 2023 -0400 Reduce resource log level (ava-labs#1622) commit 5317357 Author: marun <[email protected]> Date: Wed Jun 14 20:03:37 2023 -0700 e2e: Support testing on MacOS without requiring firewall exceptions (ava-labs#1613) commit eb6e797 Author: Stephen Buttolph <[email protected]> Date: Wed Jun 14 17:35:26 2023 -0400 Update versions for v1.10.3 (ava-labs#1605) Co-authored-by: Aaron Buchwald <[email protected]> commit 9374b56 Author: Stephen Buttolph <[email protected]> Date: Tue Jun 13 22:25:38 2023 -0400 Fix `--http-host` flag to support IPv6 (ava-labs#1620) commit cc69f03 Author: Stephen Buttolph <[email protected]> Date: Tue Jun 13 21:21:54 2023 -0400 Remove old networking metric (ava-labs#1619) commit f41e627 Author: Stephen Buttolph <[email protected]> Date: Tue Jun 13 16:19:28 2023 -0400 Update CodeQL to v2 (ava-labs#1616) commit a3436f1 Author: Gyuho Lee <[email protected]> Date: Tue Jun 13 12:53:40 2023 -0700 snow/engine/snowman: instantiate voter after issuer (ava-labs#1610) commit 31db450 Author: Dhruba Basu <[email protected]> Date: Tue Jun 13 10:09:05 2023 -0700 Add `local-prefixes` setting for `goimports` (ava-labs#1612) commit b157612 Author: Dan Laine <[email protected]> Date: Mon Jun 12 16:18:53 2023 -0400 `merkledb` and `sync` -- use time based rand seed (ava-labs#1607) Signed-off-by: Dan Laine <[email protected]> Co-authored-by: Stephen Buttolph <[email protected]> commit 13863e3 Author: Stephen Buttolph <[email protected]> Date: Mon Jun 12 14:49:54 2023 -0400 Rename license header file to avoid unintended license indexing (ava-labs#1608) commit d0fe28c Author: Dan Laine <[email protected]> Date: Fri Jun 9 14:06:10 2023 -0400 `sync` -- explain algorithm in readme (ava-labs#1600) Co-authored-by: Ron Kuris <[email protected]> commit 54d1022 Author: Dan Laine <[email protected]> Date: Fri Jun 9 12:58:45 2023 -0400 `merkleDB` -- add inner heap type to syncWorkHeap (ava-labs#1582) commit 6dad1d4 Author: Sam Batschelet <[email protected]> Date: Thu Jun 8 20:08:32 2023 -0400 Log unexpected errors during GetValidatorSet (ava-labs#1592) commit 400dd66 Author: morrisettjohn <[email protected]> Date: Thu Jun 8 19:32:37 2023 -0400 Fix unreadable message errors (ava-labs#1585) commit cdf86ae Author: David Boehm <[email protected]> Date: Thu Jun 8 16:36:33 2023 -0400 Merkle db iterator (ava-labs#1533) Signed-off-by: David Boehm <[email protected]> Co-authored-by: Qian (Tony) Zhang <[email protected]> Co-authored-by: Dan Laine <[email protected]> Co-authored-by: Darioush Jalali <[email protected]> commit 925230d Author: Ceyhun Onur <[email protected]> Date: Thu Jun 8 22:55:30 2023 +0300 Separate subnet stake connected health and metrics from P-chain (ava-labs#1358) Co-authored-by: Stephen Buttolph <[email protected]> commit efb7d90 Author: Stephen Buttolph <[email protected]> Date: Thu Jun 8 15:35:11 2023 -0400 Separate health checks by tags (ava-labs#1579) Co-authored-by: Ceyhun Onur <[email protected]> commit 26242ce Author: Anusha <[email protected]> Date: Thu Jun 8 12:14:51 2023 -0700 Improve metrics error message (ava-labs#1598) commit 7b9912d Author: Stephen Buttolph <[email protected]> Date: Thu Jun 8 14:47:23 2023 -0400 Cleanup fx interface compliance (ava-labs#1599) commit 0f6b09d Author: Alberto Benegiamo <[email protected]> Date: Thu Jun 8 18:32:47 2023 +0200 Fix P-Chain GetValidatorSet BLS Keys for Subnets (ava-labs#1584) Co-authored-by: Joshua Kim <[email protected]> Co-authored-by: Sam Batschelet <[email protected]> Co-authored-by: Darioush Jalali <[email protected]> Co-authored-by: Stephen Buttolph <[email protected]> commit 1c90eee Author: Sam Batschelet <[email protected]> Date: Thu Jun 8 11:10:27 2023 -0400 Add buf-push github workflow (ava-labs#1556) commit 110bb61 Author: Sam Batschelet <[email protected]> Date: Wed Jun 7 17:21:20 2023 -0400 Clarify break on error during ancestors lookup (ava-labs#1580) commit 9026e30 Author: Dan Laine <[email protected]> Date: Wed Jun 7 14:53:07 2023 -0400 `x/sync` -- Add `SyncableDB` proto (ava-labs#1559) commit b456e16 Author: Stephen Buttolph <[email protected]> Date: Tue Jun 6 17:46:42 2023 -0400 Revert P-Chain height indexing (ava-labs#1591) commit 00e61d8 Author: Dan Laine <[email protected]> Date: Tue Jun 6 13:23:27 2023 -0400 `MerkleDB` -- fix `onEvictCache.Flush` (ava-labs#1589) commit 268f5a9 Author: Dan Laine <[email protected]> Date: Tue Jun 6 11:04:25 2023 -0400 `MerkleDB` -- add eviction batch size config (ava-labs#1586) commit aed31ae Author: David Boehm <[email protected]> Date: Tue Jun 6 09:51:10 2023 -0400 Remove version db from merkle db (ava-labs#1534) Co-authored-by: Dan Laine <[email protected]> commit ab20b7d Author: Stephen Buttolph <[email protected]> Date: Mon Jun 5 10:39:00 2023 -0400 Remove list from AcceptedFrontier message (ava-labs#1578) commit c2ff5ff Author: Stephen Buttolph <[email protected]> Date: Fri Jun 2 12:55:10 2023 -0400 Fix proposervm.GetAncestors test flake (ava-labs#1572) commit 842a6ab Author: Stephen Buttolph <[email protected]> Date: Thu Jun 1 23:39:58 2023 -0400 Reduce the number of test health checks (ava-labs#1571) commit c32f0d6 Author: Meaghan FitzGerald <[email protected]> Date: Thu Jun 1 22:51:42 2023 -0400 Fix typo (ava-labs#1570) commit d63aa56 Author: Chloe <[email protected]> Date: Thu Jun 1 21:46:27 2023 -0500 Add more X-chain tests (ava-labs#1487) Co-authored-by: Stephen Buttolph <[email protected]> commit bdfa043 Author: Stephen Buttolph <[email protected]> Date: Thu Jun 1 20:03:50 2023 -0400 Remove lists from Chits messages (ava-labs#1412) commit 2cd81c6 Author: Stephen Buttolph <[email protected]> Date: Thu Jun 1 18:23:49 2023 -0400 Re-add upgrade tests (ava-labs#1410) commit 6cff7b6 Author: Dan Laine <[email protected]> Date: Thu Jun 1 17:01:13 2023 -0400 Reduce number of test iterations in merkledb (ava-labs#1568) commit 277d223 Author: Stephen Buttolph <[email protected]> Date: Thu Jun 1 15:42:06 2023 -0400 Add GetBalance examples for the P-chain and X-chain wallets (ava-labs#1569) commit 4debc0e Author: Joshua Kim <[email protected]> Date: Thu Jun 1 13:25:59 2023 -0400 Rename `APIAllowedOrigins` to `HTTPAllowedOrigins` (ava-labs#1567) commit 484b735 Author: Joshua Kim <[email protected]> Date: Thu Jun 1 13:17:14 2023 -0400 Only send `PushQuery` messages after building the block (ava-labs#1428) commit b923ef7 Author: Dan Laine <[email protected]> Date: Thu Jun 1 12:30:14 2023 -0400 `x/sync` -- Use proto for sending Change Proofs (ava-labs#1541) Co-authored-by: dboehm-avalabs <[email protected]> Co-authored-by: Ron Kuris <[email protected]> commit bfaa7f7 Author: Joshua Kim <[email protected]> Date: Thu Jun 1 12:29:44 2023 -0400 Add allowed http hosts configuration (ava-labs#1566) commit 8fb8afe Author: Dan Laine <[email protected]> Date: Wed May 31 21:10:20 2023 -0400 Use `http.Error` instead of separately writing error code and message (ava-labs#1564) commit e8b6a5e Author: marun <[email protected]> Date: Wed May 31 17:31:51 2023 -0700 Randomize unit test execution order to identify unwanted dependencies (ava-labs#1565) commit d6004f2 Author: Stephen Buttolph <[email protected]> Date: Wed May 31 12:52:26 2023 -0400 Add test to ensure that database packing produces sorted values (ava-labs#1560) commit e17a6ca Author: Dan Laine <[email protected]> Date: Wed May 31 09:53:44 2023 -0400 `x/sync` -- use for sending Range Proofs (ava-labs#1537) Co-authored-by: dboehm-avalabs <[email protected]> Co-authored-by: Ron Kuris <[email protected]> commit d77e409 Author: Dhruba Basu <[email protected]> Date: Wed May 31 00:37:55 2023 -0400 Add P-chain `GetBlockByHeight` API method (ava-labs#1448) commit db27133 Author: Dhruba Basu <[email protected]> Date: Wed May 31 00:16:42 2023 -0400 Add P-chain height indexing (ava-labs#1447) Co-authored-by: Stephen Buttolph <[email protected]> commit 6ba90f7 Author: Gyuho Lee <[email protected]> Date: Tue May 30 14:31:26 2023 -0700 Rename beacon to boostrapper, define bootstrappers in JSON file for cross-language compatiblity (ava-labs#1439) Co-authored-by: Stephen Buttolph <[email protected]> commit 243e313 Author: Dan Laine <[email protected]> Date: Tue May 30 15:18:24 2023 -0400 `x/sync` / `x/merkledb` -- add `SyncableDB` interface (ava-labs#1555) commit b66e25e Author: Dhruba Basu <[email protected]> Date: Tue May 30 12:58:19 2023 -0400 Enforce inlining functions with a single error return in `require.NoError` (ava-labs#1500) commit 7403188 Author: Joshua Kim <[email protected]> Date: Wed May 31 00:52:54 2023 +0900 Add workflow to mark stale issues and PRs (ava-labs#1443) commit c374c39 Author: Ceyhun Onur <[email protected]> Date: Sat May 27 04:57:01 2023 +0300 Add ping uptimes test (ava-labs#1550) commit eaf5256 Author: Alberto Benegiamo <[email protected]> Date: Thu May 25 22:55:00 2023 +0200 Minor pchain UTs cleanup (ava-labs#1554) commit 4b52f82 Author: Sam Batschelet <[email protected]> Date: Thu May 25 14:56:19 2023 -0400 Bump Protobuf and tooling and add section to proto docs outlining buf publishing (ava-labs#1552) commit a99a809 Author: David Boehm <[email protected]> Date: Thu May 25 11:41:46 2023 -0400 `x/sync` -- Add proto for P2P messages (ava-labs#1472) Co-authored-by: Ron Kuris <[email protected]> Co-authored-by: Dan Laine <[email protected]> commit f7307d5 Author: Stephen Buttolph <[email protected]> Date: Tue May 23 22:32:22 2023 -0400 Update versions for v1.10.2 (ava-labs#1544) commit 581a673 Author: Dan Laine <[email protected]> Date: Tue May 23 14:02:34 2023 -0400 add interface for MerkleDB (ava-labs#1519) commit 14b8c98 Author: Stephen Buttolph <[email protected]> Date: Mon May 22 23:37:01 2023 -0400 Log chain shutdown duration (ava-labs#1545) commit a2ae740 Author: Stephen Buttolph <[email protected]> Date: Mon May 22 20:55:41 2023 -0400 Add serialization tests for transactions added in Banff (ava-labs#1513) commit ffde992 Author: Stephen Buttolph <[email protected]> Date: Mon May 22 20:34:50 2023 -0400 Fetch process resource stats as best-effort (ava-labs#1543) commit 9e6bf96 Author: Chloe <[email protected]> Date: Mon May 22 19:13:27 2023 -0500 Add CPU cycles and number of disk reads/writes metrics by pid (ava-labs#1334) commit b459661 Author: Ceyhun Onur <[email protected]> Date: Sat May 20 02:00:54 2023 +0300 Deprecate uptimes in pong messages (ava-labs#1362) Co-authored-by: Stephen Buttolph <[email protected]> commit 37b5735 Author: Stephen Buttolph <[email protected]> Date: Fri May 19 14:00:30 2023 -0400 Fix network compression type flag usage (ava-labs#1532) commit 61e7aa6 Author: Ceyhun Onur <[email protected]> Date: Fri May 19 03:19:09 2023 +0300 Rename `StakingEnabled` to `SybilProtectionEnabled` (ava-labs#1441) commit 85c1d24 Author: Ceyhun Onur <[email protected]> Date: Thu May 18 21:36:43 2023 +0300 Standardize config keys (ava-labs#1370) commit 49b71b4 Author: Alberto Benegiamo <[email protected]> Date: Thu May 18 20:32:21 2023 +0200 P-chain validator management refactor (ava-labs#1284) Co-authored-by: Stephen Buttolph <[email protected]> commit b25c22b Author: Alberto Benegiamo <[email protected]> Date: Thu May 18 19:02:55 2023 +0200 Fix incorrect test refactor (ava-labs#1526) commit bce0c92 Author: Gyuho Lee <[email protected]> Date: Fri May 19 00:58:40 2023 +0800 utils/bag: print type of bag elements (ava-labs#1507) commit 396f975 Author: Stephen Buttolph <[email protected]> Date: Wed May 17 22:40:52 2023 -0400 Delete duplicate test var definitions (ava-labs#1518) commit 0c391e8 Author: Gyuho Lee <[email protected]> Date: Thu May 18 10:14:47 2023 +0800 Add missing verbo logs checks (ava-labs#1504) Co-authored-by: dhrubabasu <[email protected]> commit 467b905 Author: Gyuho Lee <[email protected]> Date: Thu May 18 09:53:18 2023 +0800 engine/snowman: clean up comments in `bubbleVotes` unit tests (ava-labs#1444) commit c930483 Author: Alberto Benegiamo <[email protected]> Date: Thu May 18 03:15:01 2023 +0200 Convert P-chain Tx cache to be byte based (ava-labs#1517) Co-authored-by: Stephen Buttolph <[email protected]> commit ec147ab Author: Dhruba Basu <[email protected]> Date: Tue May 16 23:58:26 2023 -0400 Ban usage of `nil` in `require` functions (ava-labs#1498) commit 0eb61fb Author: Dhruba Basu <[email protected]> Date: Tue May 16 23:37:03 2023 -0400 Ban usage of `require.Equal` when testing for length (ava-labs#1497) commit d146232 Author: Dhruba Basu <[email protected]> Date: Tue May 16 23:03:14 2023 -0400 Ban usage of `require.Len` when testing for length `0` (ava-labs#1496) commit 1bcab1f Author: Stephen Buttolph <[email protected]> Date: Tue May 16 22:41:49 2023 -0400 Remove comment referencing old IP based tracking (ava-labs#1509) commit 824c3b2 Author: David Boehm <[email protected]> Date: Tue May 16 16:24:07 2023 -0400 MerkleDB Cleanup (ava-labs#1465) Signed-off-by: Dan Laine <[email protected]> Co-authored-by: Dan Laine <[email protected]> Co-authored-by: Darioush Jalali <[email protected]> commit 2e44364 Author: David Boehm <[email protected]> Date: Tue May 16 14:56:37 2023 -0400 Clean up MerkleDVB Sync Close lock (ava-labs#1469) Co-authored-by: Dan Laine <[email protected]> commit 9f6c371 Author: Dhruba Basu <[email protected]> Date: Mon May 15 10:17:00 2023 -0400 Ban `require.Equal` when testing for `0` (ava-labs#1495) commit 9ac856a Author: Gyuho Lee <[email protected]> Date: Sat May 13 02:07:07 2023 +0800 config: disallow `ThrottlerConfig.MaxRecheckDelay` < 1 ms (ava-labs#1435) commit 3d2537b Author: Gyuho Lee <[email protected]> Date: Sat May 13 00:45:07 2023 +0800 codec: remove `SetMaxSize` from `Manager` (ava-labs#1481) commit e2b4d9a Author: Dhruba Basu <[email protected]> Date: Fri May 12 12:21:34 2023 -0400 Enforce the use of a blank identifier for interface compliance (ava-labs#1493) commit f0a86cc Author: Dhruba Basu <[email protected]> Date: Fri May 12 12:02:01 2023 -0400 Fix license header CI checks (ava-labs#1492) commit a16d9fb Author: Dhruba Basu <[email protected]> Date: Fri May 12 10:04:28 2023 -0400 Standardize single import formats (ava-labs#1466) commit 7b8bbd6 Author: Dhruba Basu <[email protected]> Date: Wed May 10 01:21:42 2023 -0400 ban function params for `require.ErrorIs` (ava-labs#1486) commit b870515 Author: Gyuho Lee <[email protected]> Date: Tue May 9 03:28:21 2023 +0800 chains: move "msgChan" closer to the first use (readability) (ava-labs#1484) commit 0a0e1bb Author: Gyuho Lee <[email protected]> Date: Sat May 6 20:19:45 2023 +0800 Replace deprecated "golang.org/x/crypto/ssh/terminal" with "golang.org/x/term" (ava-labs#1464) commit 755ad40 Author: Dhruba Basu <[email protected]> Date: Sat May 6 07:48:47 2023 -0400 Use `require` in `api` and `utils/password` packages (ava-labs#1471) commit 1b64bbe Author: Dhruba Basu <[email protected]> Date: Sat May 6 06:51:16 2023 -0400 Ban `require.NotEqualValues` (ava-labs#1470) commit 8903335 Author: Gyuho Lee <[email protected]> Date: Sat May 6 18:45:22 2023 +0800 vms/platformvm/service: preallocate address slice and improve error msg (ava-labs#1477) commit b3a07d8 Author: Dhruba Basu <[email protected]> Date: Wed May 3 13:34:55 2023 -0400 Use `require.IsType` for type assertions in tests (ava-labs#1458) commit eb8b52a Author: Dhruba Basu <[email protected]> Date: Wed May 3 13:34:32 2023 -0400 Remove zstd Cortina check (ava-labs#1459) commit 850c6fd Author: Gyuho Lee <[email protected]> Date: Thu May 4 00:59:51 2023 +0800 chains: do not hold write subnetsLock in health checks (ava-labs#1460) commit c125bc1 Author: Dhruba Basu <[email protected]> Date: Wed May 3 12:59:03 2023 -0400 Ban `require.EqualValues` (ava-labs#1457) commit a583991 Author: Gyuho Lee <[email protected]> Date: Thu May 4 00:58:03 2023 +0800 utils/logging: add "Enabled" method to remove redundant verbo logs (ava-labs#1461) commit 99f35bd Author: Dhruba Basu <[email protected]> Date: Wed May 3 01:26:51 2023 -0400 Fix style nits in vm clients (ava-labs#1449) commit 0d8c59e Author: Dhruba Basu <[email protected]> Date: Mon May 1 12:59:48 2023 -0400 Use `require` library functions in more tests (ava-labs#1451)
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.
Why this should be merged
bytes to string conversion never fails, if formatting encoding fails, it should be from the encoding format.
How this works
Accurate error message.
How this was tested
N/A