Simplify GetContainerHealth(): we never use unhealthy count
This commit is contained in:
@@ -200,24 +200,18 @@ def GetImageInfo(mySSH, containerName):
|
|||||||
|
|
||||||
def GetContainerHealth(mySSH, containerName):
|
def GetContainerHealth(mySSH, containerName):
|
||||||
if containerName is None:
|
if containerName is None:
|
||||||
return 0, 0
|
return False
|
||||||
unhealthyNb = 0
|
|
||||||
healthyNb = 0
|
|
||||||
time.sleep(5)
|
time.sleep(5)
|
||||||
for _ in range(3):
|
for _ in range(3):
|
||||||
if containerName != 'db_init':
|
if containerName != 'db_init':
|
||||||
result = mySSH.run(f'docker inspect --format="{{{{.State.Health.Status}}}}" {containerName}')
|
result = mySSH.run(f'docker inspect --format="{{{{.State.Health.Status}}}}" {containerName}')
|
||||||
unhealthyNb = result.stdout.count('unhealthy')
|
if result.stdout == 'healthy':
|
||||||
healthyNb = result.stdout.count('healthy') - unhealthyNb
|
return True
|
||||||
if healthyNb == 1:
|
|
||||||
break
|
|
||||||
else:
|
else:
|
||||||
time.sleep(10)
|
time.sleep(10)
|
||||||
else:
|
else:
|
||||||
unhealthyNb = 0
|
return True
|
||||||
healthyNb = 1
|
return False
|
||||||
break
|
|
||||||
return healthyNb, unhealthyNb
|
|
||||||
|
|
||||||
def ReTagImages(mySSH,IMAGES,ranCommitID,ranBranch,ranAllowMerge,displayedNewTags):
|
def ReTagImages(mySSH,IMAGES,ranCommitID,ranBranch,ranAllowMerge,displayedNewTags):
|
||||||
mySSH.run('cp docker-compose.y*ml ci-docker-compose.yml', 5)
|
mySSH.run('cp docker-compose.y*ml ci-docker-compose.yml', 5)
|
||||||
@@ -1000,10 +994,10 @@ class Containerize():
|
|||||||
imagesInfo=""
|
imagesInfo=""
|
||||||
for svcName in services_list:
|
for svcName in services_list:
|
||||||
containerName = GetContainerName(mySSH, svcName)
|
containerName = GetContainerName(mySSH, svcName)
|
||||||
healthyNb,unhealthyNb = GetContainerHealth(mySSH,containerName)
|
healthy = GetContainerHealth(mySSH,containerName)
|
||||||
self.testCase_id = HTML.testCase_id
|
self.testCase_id = HTML.testCase_id
|
||||||
self.eNB_logFile[self.eNB_instance] = f'{svcName}-{self.testCase_id}.log'
|
self.eNB_logFile[self.eNB_instance] = f'{svcName}-{self.testCase_id}.log'
|
||||||
if unhealthyNb:
|
if not healthy:
|
||||||
logging.warning(f"Deployment Failed: Trying to copy container logs {self.eNB_logFile[self.eNB_instance]}")
|
logging.warning(f"Deployment Failed: Trying to copy container logs {self.eNB_logFile[self.eNB_instance]}")
|
||||||
CopyinContainerLog(mySSH,lSourcePath,self.yamlPath[0].split('/'),containerName,self.eNB_logFile[self.eNB_instance])
|
CopyinContainerLog(mySSH,lSourcePath,self.yamlPath[0].split('/'),containerName,self.eNB_logFile[self.eNB_instance])
|
||||||
status = False
|
status = False
|
||||||
|
|||||||
Reference in New Issue
Block a user