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

chore: use separate docker image for mlflow server #101

Merged
merged 9 commits into from Nov 29, 2022
Merged

chore: use separate docker image for mlflow server #101

merged 9 commits into from Nov 29, 2022

Conversation

ab93
Copy link
Member

@ab93 ab93 commented Nov 29, 2022

  • Use mlflow-skinny instead of mlflow
  • Update example dockerfile to have multi-stage; one for udf and one for mlflow
  • Update docs to use quay.io images, instead of requiring build

@ab93 ab93 self-assigned this Nov 29, 2022
@codecov
Copy link

codecov bot commented Nov 29, 2022

Codecov Report

Merging #101 (2412409) into main (d2c6293) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #101   +/-   ##
=======================================
  Coverage   95.88%   95.88%           
=======================================
  Files          20       20           
  Lines         948      948           
  Branches       92       92           
=======================================
  Hits          909      909           
  Misses         21       21           
  Partials       18       18           
Impacted Files Coverage Δ
numalogic/synthetic/anomalies.py 93.68% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Signed-off-by: Avik Basu <avikbasu93@gmail.com>
Signed-off-by: Avik Basu <avikbasu93@gmail.com>
Signed-off-by: Avik Basu <avikbasu93@gmail.com>
Signed-off-by: Avik Basu <avikbasu93@gmail.com>
Signed-off-by: Avik Basu <avikbasu93@gmail.com>
Signed-off-by: Avik Basu <avikbasu93@gmail.com>
Signed-off-by: Avik Basu <avikbasu93@gmail.com>
@ab93 ab93 marked this pull request as ready for review November 29, 2022 19:16
@ab93 ab93 linked an issue Nov 29, 2022 that may be closed by this pull request
Signed-off-by: Avik Basu <avikbasu93@gmail.com>
Signed-off-by: Avik Basu <avikbasu93@gmail.com>
@ab93 ab93 merged commit f0993d3 into main Nov 29, 2022
@ab93 ab93 deleted the quay-img branch November 29, 2022 19:39
@ab93 ab93 temporarily deployed to production November 29, 2022 19:48 Inactive
ab93 added a commit that referenced this pull request Nov 30, 2022
* Use mlflow-skinny instead of mlflow
* Update example dockerfile to have multi-stage; one for udf and one for mlflow
* Update docs to use quay.io images, instead of requiring build

Signed-off-by: Avik Basu <avikbasu93@gmail.com>
ab93 added a commit that referenced this pull request Nov 30, 2022
* Use mlflow-skinny instead of mlflow
* Update example dockerfile to have multi-stage; one for udf and one for mlflow
* Update docs to use quay.io images, instead of requiring build

Signed-off-by: Avik Basu <avikbasu93@gmail.com>
ab93 added a commit that referenced this pull request Jan 5, 2023
* fix_readme: mention namespace name when applying the pipeline (#88)
* fix: allow only patch updates in torch version due to cuda build errors on mac (#90)
* chore: bump to v0.2.7 for new release (#91)
* docs: updated CHANGELOG.md (#93)
* fix: change example pipeline mlflow port (#96)
* chore: use separate docker image for mlflow server (#101)
* Use mlflow-skinny instead of mlflow
* Update example dockerfile to have multi-stage; one for udf and one for mlflow
* Update docs to use quay.io images, instead of requiring build
* docs: updated CHANGELOG.md (#107)
* fix: sklearn base import for scikit learn v1.2 (#112)
* docs: updated CHANGELOG.md (#113)
* docs: updated CHANGELOG.md (#114)

Signed-off-by: Avik Basu <avikbasu93@gmail.com>
Co-authored-by: s0nicboOm <i.kushalbatra@gmail.com>
Co-authored-by: GitHub <noreply@github.com>
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

Successfully merging this pull request may close these issues.

Multi stage docker builds for examples
2 participants