Skip to content

Commit

Permalink
linux test image update
Browse files Browse the repository at this point in the history
  • Loading branch information
neonwatty committed Jun 5, 2023
1 parent d4702be commit 62dc61b
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 32 deletions.
29 changes: 0 additions & 29 deletions quick_batch/utilities/manage_images.py
Original file line number Diff line number Diff line change
Expand Up @@ -56,35 +56,6 @@ def pull_and_tag_image(client, image_name, new_tag):
print(f"FAILURE: Error occurred while pulling and tagging the image: {e}")
return False

# @log_exceptions
# def pull_and_tag_image(client, image_name, new_tag):
# try:
# print('INFO: Pulling and tagging image...')

# # Try tagging local image
# try:
# image = client.images.get(image_name)
# tag_success = image.tag(new_tag)

# if tag_success:
# print('SUCCESS: Tagging local image successful!')
# return True
# except docker.errors.ImageNotFound:
# pass

# # Pull the image
# pulled_image = client.images.pull(image_name)

# # Tag the pulled image with a new name
# tagged_image = pulled_image.tag(new_tag)

# print('SUCCESS: Pulling and tagging image complete!')
# return True

# except docker.errors.APIError as e:
# print(f"FAILURE: Error occurred while pulling and tagging the image: {e}")
# return False


def check_requirements_file(requirements_path):
if not os.path.isfile(requirements_path):
Expand Down
3 changes: 1 addition & 2 deletions quick_batch/utilities/manage_queue.py
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,7 @@ def monitor_queue_app_containers(client):
'logs',
'queue_app'],
check=True)



time.sleep(5)

if time.time() > timeout:
Expand Down
2 changes: 1 addition & 1 deletion tests/test_configs/test_1/quick_batch_github.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,5 +11,5 @@ processor:
dockerfile_path: /home/runner/work/quick_batch/quick_batch/tests/test_dockerfiles/test_1/Dockerfile
requirements_path: /home/runner/work/quick_batch/quick_batch/tests/test_requirements/test_1/requirements.txt
processor_path: /home/runner/work/quick_batch/quick_batch/tests/test_processors/test_1/processor.py
image_name: jermwatt/quick_batch_processor_app_test_1
image_name: jermwatt/quick_batch_processor_app_test_1:linux
num_processors: 2

0 comments on commit 62dc61b

Please sign in to comment.