reload only chainlink node dbs when performing restart of CL nodes #1346
+41
−21
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.
Below is a summarization created by an LLM (gpt-4-0125-preview). Be mindful of hallucinations and verify accuracy.
Why
Enhancements and refactoring have been made to improve database container labeling and configuration, facilitate dynamic node and database setups, and enhance the node upgrade process in the testing framework. These changes aim to provide more flexibility, better identification, and easier management of containers during tests.
What
framework/components/postgres/postgres.go
os
,strings
,time
imports for better handling of initialization scripts and time-based operations.DBContainerLabel
to standardize labeling of PostgreSQL containers.Input
struct to includeName
, allowing dynamic naming of PostgreSQL containers.DBContainerLabel
andName
field for clearer identification.NewPostgreSQL
function to use updated container naming logic, enhancing readability and container management.framework/components/simple_node_set/node_set.go
slices
,strings
,sync
, anderrgroup
imports for concurrent operations and string manipulations.CLNodeDBContainerLabel
to standardize labeling of Chainlink node databases.sharedDBSetup
function to includeName
inDbInput
, utilizingCLNodeDBContainerLabel
for clearer database container identification.framework/components/simple_node_set/reload.go
fmt
andtime
imports for formatting strings and handling time-based operations.UpgradeNodeSet
function to use a regex pattern that includes bothDBContainerLabel
andCLNodeDBContainerLabel
, improving the targeting of containers for removal during node upgrades. This enhances the upgrade process by ensuring only relevant containers are targeted.