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: provides shim for bentoctl #3322

Merged
merged 3 commits into from Dec 7, 2022
Merged

Conversation

aarnphm
Copy link
Member

@aarnphm aarnphm commented Dec 7, 2022

provide backward shim for bentoctl until it migrates to public API
I have tested with sagemaker. with and without conda.

Signed-off-by: Aaron Pham 29749331+aarnphm@users.noreply.github.com

Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
@aarnphm aarnphm requested a review from a team as a code owner December 7, 2022 02:11
@aarnphm aarnphm requested review from jjmachan and ssheng and removed request for a team December 7, 2022 02:11
@codecov
Copy link

codecov bot commented Dec 7, 2022

Codecov Report

Merging #3322 (6a4250f) into main (687c3e0) will decrease coverage by 0.10%.
The diff coverage is 3.03%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3322      +/-   ##
==========================================
- Coverage   33.74%   33.64%   -0.11%     
==========================================
  Files         108      110       +2     
  Lines        9853     9886      +33     
  Branches     1697     1699       +2     
==========================================
+ Hits         3325     3326       +1     
- Misses       6299     6331      +32     
  Partials      229      229              
Impacted Files Coverage Δ
src/bentoml/_internal/bento/gen.py 0.00% <0.00%> (ø)
src/bentoml/_internal/container/generate.py 67.24% <ø> (ø)
src/bentoml/_internal/utils/buildx.py 0.00% <0.00%> (ø)
src/bentoml/_internal/container/base.py 63.55% <100.00%> (+0.34%) ⬆️
src/bentoml/_internal/io_descriptors/pandas.py 42.95% <0.00%> (ø)

ssheng
ssheng previously approved these changes Dec 7, 2022
Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
Signed-off-by: Aaron Pham <29749331+aarnphm@users.noreply.github.com>
@aarnphm aarnphm requested a review from ssheng December 7, 2022 02:38
@aarnphm aarnphm changed the title chore: bentoctl shim chore: provides shim for bentoctl Dec 7, 2022
@aarnphm aarnphm merged commit cc38007 into bentoml:main Dec 7, 2022
@aarnphm aarnphm deleted the chore/bentoctl-shim branch December 7, 2022 03:06
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.

None yet

3 participants