Skip to content
This repository has been archived by the owner on Jun 25, 2023. It is now read-only.

jetjodh solution w/ fastapi #42

Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

jetjodh solution w/ fastapi #42

wants to merge 20 commits into from

Conversation

jetjodh
Copy link

@jetjodh jetjodh commented Jun 22, 2023

No description provided.

Copy link
Collaborator

@rsolovev rsolovev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @jetjodh! Thank you for your interest in this challenge and your solution. Unfortunately, container didnt launch, here is the full log and exit reason from k8s:

lastState:
  terminated:
    exitCode: 0
    reason: Completed
    startedAt: '2023-06-23T10:24:06Z'
    finishedAt: '2023-06-23T10:28:56Z'

@jetjodh
Copy link
Author

jetjodh commented Jun 23, 2023

Hi @jetjodh! Thank you for your interest in this challenge and your solution. Unfortunately, container didnt launch, here is the full log and exit reason from k8s:

lastState:
  terminated:
    exitCode: 0
    reason: Completed
    startedAt: '2023-06-23T10:24:06Z'
    finishedAt: '2023-06-23T10:28:56Z'

@rsolovev I moved the model downloads to runtime due to memory issues at build time. I will resolve this and try again.

@jetjodh jetjodh requested a review from rsolovev June 23, 2023 17:30
@jetjodh
Copy link
Author

jetjodh commented Jun 23, 2023

@rsolovev Resolved by moving runtime commands to a bash script.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants