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.

Sign up for GitHub

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

Jump to bottom

Add retry to cluster generation for testing in hack folder #11067

Merged
merged 8 commits into from
Oct 31, 2022
Merged

Add retry to cluster generation for testing in hack folder #11067

Changes from 3 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
3935852
Add retry for cluster generation
todaywasawesome Oct 26, 2022
5d9c2f5
Switch to human readable count
todaywasawesome Oct 26, 2022
f706549
fix logic
todaywasawesome Oct 26, 2022
c7e5dfc
Add retry for cluster generation
todaywasawesome Oct 26, 2022
5be09de
remove superfluous if from previous iteration.
todaywasawesome Oct 30, 2022
46bfb48
remove conflict
todaywasawesome Oct 30, 2022
48ccf78
clean up code for final PR
todaywasawesome Oct 31, 2022
de8eb06
Merge branch 'master' into cluster-gen
pasha-codefresh Oct 31, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
24 changes: 20 additions & 4 deletions hack/gen-resources/generators/cluster_generator.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ func (cg *ClusterGenerator) installVCluster(opts *util.GenerateOpts, namespace s
return err
}
log.Print("Execute helm install command")
_, err = cmd.Freestyle("install", releaseName, "vcluster", "--values", opts.ClusterOpts.ValuesFilePath, "--repo", "https://charts.loft.sh", "--namespace", namespace, "--repository-config", "", "--create-namespace", "--wait")
_, err = cmd.Freestyle("upgrade", "--install", releaseName, "vcluster", "--values", opts.ClusterOpts.ValuesFilePath, "--repo", "https://charts.loft.sh", "--namespace", namespace, "--repository-config", "", "--create-namespace", "--wait")
if err != nil {
return err
}
Expand All @@ -157,11 +157,13 @@ func (cg *ClusterGenerator) getClusterServerUri(namespace string, releaseSuffix
return "", err
}
// TODO: should be moved to service instead pod
log.Printf("Get service for https://" + pod.Status.PodIP + ":8443")
return "https://" + pod.Status.PodIP + ":8443", nil
}

func (cg *ClusterGenerator) retrieveClusterUri(namespace, releaseSuffix string) (string, error) {
for i := 0; i < 3; i++ {
for i := 0; i < 10; i++ {
log.Printf("Attempting to get cluster uri")
uri, err := cg.getClusterServerUri(namespace, releaseSuffix)
if err != nil {
log.Printf("Failed to get cluster uri due to %s", err.Error())
Expand All @@ -174,8 +176,8 @@ func (cg *ClusterGenerator) retrieveClusterUri(namespace, releaseSuffix string)
}

func (cg *ClusterGenerator) Generate(opts *util.GenerateOpts) error {
for i := 0; i < opts.ClusterOpts.Samples; i++ {
log.Printf("Generate cluster #%v", i)
for i := 1; i <= opts.ClusterOpts.Samples; i++ {
log.Printf("Generate cluster #%v of #%v", i, opts.ClusterOpts.Samples)

namespace := opts.ClusterOpts.NamespacePrefix + "-" + util.GetRandomString()

Expand All @@ -193,9 +195,23 @@ func (cg *ClusterGenerator) Generate(opts *util.GenerateOpts) error {

log.Print("Get cluster credentials")
caData, cert, key, err := cg.getClusterCredentials(namespace, releaseSuffix)

for o := 0; o < 5; o++ {
if err != nil {
if o > 0 {
todaywasawesome marked this conversation as resolved.
Show resolved Hide resolved
log.Printf("Failed to get cluster credentials " + releaseSuffix + ", retrying...")
time.Sleep(10 * time.Second)
err = nil
}
caData, cert, key, err = cg.getClusterCredentials(namespace, releaseSuffix)
} else {
break
}
}
if err != nil {
return err
}


log.Print("Get cluster server uri")

Expand Down