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.

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

Update GitHub PR TTM Service Deployment #457

Closed
2 of 3 tasks
chauhankaranraj opened this issue Dec 15, 2021 · 4 comments
Closed
2 of 3 tasks

Update GitHub PR TTM Service Deployment #457

chauhankaranraj opened this issue Dec 15, 2021 · 4 comments
Assignees

Comments

@chauhankaranraj
Copy link
Member

chauhankaranraj commented Dec 15, 2021

Currently the GitHub PR time-to-merge prediction service uses the Seldon server image built from the directory here. Now that custom transformers are added through #391, we can add the aforementioned directory to this repo and build image from here instead.

Related to #380, #236

Acceptance criteria

@MichaelClifford
Copy link
Member

@chauhankaranraj can we close this now?

@chauhankaranraj
Copy link
Member Author

@chauhankaranraj can we close this now?

So acceptance criterion 2 (building image using aicoe-ci, instead of manually) is still outstanding, but we do have a separate issue to track it. The other two items are done. What would you suggest we do?

@MichaelClifford
Copy link
Member

lets split it, close this and move the outstanding to an new issue. : )

@chauhankaranraj
Copy link
Member Author

chauhankaranraj commented Dec 17, 2021

lets split it, close this and move the outstanding to an new issue. : )

Sounds good!

Acceptance criterion 2 is being tracked in #459 since it's a larger task itself and needs further discussion.

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

No branches or pull requests

3 participants