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

feat: subprocess build #3814

Merged
merged 6 commits into from
May 3, 2023
Merged

feat: subprocess build #3814

merged 6 commits into from
May 3, 2023

Conversation

aarnphm
Copy link
Contributor

@aarnphm aarnphm commented May 2, 2023

What does this PR address?

Initial implementation as a part of #3580

Signed-off-by: Aaron [email protected]

Before submitting:

@aarnphm aarnphm requested a review from a team as a code owner May 2, 2023 07:23
@aarnphm aarnphm requested review from sauyon and removed request for a team May 2, 2023 07:23
ssheng
ssheng previously approved these changes May 2, 2023
src/bentoml_cli/bentos.py Outdated Show resolved Hide resolved
src/bentoml/bentos.py Outdated Show resolved Hide resolved
to show machine-readable tag instead of pretty print

by default it is --output default

Signed-off-by: Aaron <[email protected]>
@aarnphm
Copy link
Contributor Author

aarnphm commented May 2, 2023

I will cherry-pick this pr and fix the test in #3793 as the tests fail is related to #3793

src/bentoml/_internal/bento/build_config.py Show resolved Hide resolved
src/bentoml_cli/bentos.py Outdated Show resolved Hide resolved
src/bentoml_cli/bentos.py Outdated Show resolved Hide resolved
src/bentoml_cli/bentos.py Outdated Show resolved Hide resolved
- revert back previous message
- change machine to tag

Signed-off-by: Aaron <[email protected]>
if --quiet is passed, it should silence all logs

Signed-off-by: Aaron <[email protected]>
@aarnphm
Copy link
Contributor Author

aarnphm commented May 2, 2023

cc @sauyon all comments has been address. PTAL

src/bentoml/bentos.py Show resolved Hide resolved
src/bentoml_cli/bentos.py Outdated Show resolved Hide resolved
ssheng
ssheng previously approved these changes May 2, 2023
Copy link
Collaborator

@ssheng ssheng left a comment

Choose a reason for hiding this comment

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

LGTM. Please coordinate with @sauyon on his comments.

this is already handled within import_service

Fix failing tests

Signed-off-by: Aaron <[email protected]>
@aarnphm aarnphm merged commit 03a6367 into bentoml:main May 3, 2023
@aarnphm aarnphm deleted the feat/subprocess-build branch May 3, 2023 02:55
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.

3 participants