-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[TT-1376] more stable compatibility pipeline (#13720)
* increase cl node startup timeout * use newer print summary action * use /readyz endpoint not /health, when waiting for CL node to be ready * slightly increase min time between emitting events to avoid 'alreadyKnown' tx error, when processing is slow * add step that gathers resource metric * move where telemetry action runs * use latest CTF * put collection in correct place, really * WIP#1 - doesn't hang on error * WIP#3 - better early exit * use concurrent executor in log poller tests * use stronger runners * use newer CTF * use free runners, limit parallel to 30 * try with 10x max parallelism * do not gather workflow metrics * notify bix-scale-eng about compat failures, use latest tagged ctf
- Loading branch information
Showing
7 changed files
with
26 additions
and
11 deletions.
There are no files selected for viewing
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -75,6 +75,14 @@ func WithNodeEnvVars(ev map[string]string) ClNodeOption { | |
| } | ||
| } | ||
|
|
||
| func WithStartupTimeout(timeout time.Duration) ClNodeOption { | ||
| return func(n *ClNode) { | ||
| if timeout != 0 { | ||
| n.StartupTimeout = timeout | ||
| } | ||
| } | ||
| } | ||
|
|
||
| // Sets custom node container name if name is not empty | ||
| func WithNodeContainerName(name string) ClNodeOption { | ||
| return func(c *ClNode) { | ||
|
|
@@ -130,6 +138,7 @@ func NewClNode(networks []string, imageName, imageVersion string, nodeConfig *ch | |
| ContainerVersion: imageVersion, | ||
| Networks: networks, | ||
| LogStream: logStream, | ||
| StartupTimeout: 3 * time.Minute, | ||
| }, | ||
| UserEmail: "[email protected]", | ||
| UserPassword: "localdevpassword", | ||
|
|
@@ -447,9 +456,9 @@ func (n *ClNode) getContainerRequest(secrets string) ( | |
| "-a", apiCredsPath, | ||
| }, | ||
| Networks: append(n.Networks, "tracing"), | ||
| WaitingFor: tcwait.ForHTTP("/health"). | ||
| WaitingFor: tcwait.ForHTTP("/readyz"). | ||
| WithPort("6688/tcp"). | ||
| WithStartupTimeout(90 * time.Second). | ||
| WithStartupTimeout(n.StartupTimeout). | ||
| WithPollInterval(1 * time.Second), | ||
| Files: []tc.ContainerFile{ | ||
| { | ||
|
|
||
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
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
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
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
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