Skip to content

Add tgz artifact type - #6438

Closed
vinchenzo-db wants to merge 1 commit into
mainfrom
vchen/artifacts-tgz-only
Closed

Add tgz artifact type#6438
vinchenzo-db wants to merge 1 commit into
mainfrom
vchen/artifacts-tgz-only

Conversation

@vinchenzo-db

Copy link
Copy Markdown
Contributor

Introduce tgz as a third artifacts.*.type alongside whl and jar. It is a generic gzipped-tar artifact with no language-specific build defaults or wheel/jar semantics: the user's build command produces the tarball, which is then uploaded and referenced like any other artifact file.

This lets a DAB-native user package arbitrary source code and wire it into an AI Runtime task's code_source_path through the standard artifacts mechanism, consolidating the DAB-native workflow with AIR functionality.

Kept intentionally minimal; native DABs-side tarball building from include paths / a git ref is proposed separately in #6428.

Changes

Why

Tests

Introduce `tgz` as a third `artifacts.*.type` alongside `whl` and `jar`. It is a
generic gzipped-tar artifact with no language-specific build defaults or
wheel/jar semantics: the user's `build` command produces the tarball, which is
then uploaded and referenced like any other artifact file.

This lets a DAB-native user package arbitrary source code and wire it into an AI
Runtime task's `code_source_path` through the standard `artifacts` mechanism,
consolidating the DAB-native workflow with AIR functionality.

Kept intentionally minimal; native DABs-side tarball building from `include`
paths / a `git` ref is proposed separately in #6428.

Co-authored-by: Isaac <no-reply@databricks.com>
@github-actions

Copy link
Copy Markdown
Contributor

Approval status: pending

/acceptance/bundle/ - needs approval

Files: acceptance/bundle/validate/strict/output.txt
Suggested: @janniklasrose
Also eligible: @pietern, @shreyas-goenka, @andrewnester, @denik, @anton-107, @lennartkats-db

/bundle/ - needs approval

4 files changed
Suggested: @janniklasrose
Also eligible: @pietern, @shreyas-goenka, @andrewnester, @denik, @anton-107, @lennartkats-db

Any maintainer (@andrewnester, @anton-107, @denik, @pietern, @shreyas-goenka, @simonfaltum, @renaudhartert-db, @janniklasrose, @lennartkats-db, @rugpanov, @rclarey) can approve all areas.
See OWNERS for ownership rules.

@andrewnester

Copy link
Copy Markdown
Contributor

New type is not really needed, this field can be omitted as the type is obsolete

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.

2 participants