Skip to content

Add presets.artifacts_dynamic_version setting - #3074

Merged
denik merged 9 commits into
mainfrom
denik/artifacts_use_dynamic_version
Jun 18, 2025
Merged

Add presets.artifacts_dynamic_version setting#3074
denik merged 9 commits into
mainfrom
denik/artifacts_use_dynamic_version

Conversation

@denik

@denikdenik commented Jun 17, 2025

Copy link
Copy Markdown
Contributor

Changes

Add new bundle setting presets.artifacts_dynamic_version. If set to true, all whl artifacts (in a given target) will get dynamic_version: true setting.

Why

We want to set this to true for all artifacts in all targets without repeating every artifact under targets. #3042

Tests

New acceptance test.

Comment threadacceptance/bundle/artifacts/artifacts_use_dynamic_version/script Outdated
@denik
deniktemporarily deployed to test-trigger-is June 17, 2025 18:03 — with GitHub Actions Inactive
@denik
deniktemporarily deployed to test-trigger-is June 17, 2025 18:03 — with GitHub Actions Inactive
@denik
denik enabled auto-merge June 17, 2025 18:04

@pieternpietern left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

The JSON schema needs to be updated.

@denik
denikforce-pushed the denik/artifacts_use_dynamic_version branch from 5f85997 to 65aadedCompareJune 17, 2025 18:26
@denik
deniktemporarily deployed to test-trigger-is June 17, 2025 18:26 — with GitHub Actions Inactive
@denik
deniktemporarily deployed to test-trigger-is June 17, 2025 18:26 — with GitHub Actions Inactive
Comment threadbundle/config/presets.go Outdated
@denik
denik disabled auto-merge June 17, 2025 18:49
@denik
deniktemporarily deployed to test-trigger-is June 17, 2025 19:56 — with GitHub Actions Inactive
@denik
denik requested a review from juliacrawf-dbJune 17, 2025 19:57
@denik
deniktemporarily deployed to test-trigger-is June 17, 2025 20:47 — with GitHub Actions Inactive
@denik
denik enabled auto-merge June 17, 2025 20:47
@denikdenik changed the title Add presets.artifacts_use_dynamic_version settingAdd presets.artifacts_dynamic_version settingJun 18, 2025
@denik
deniktemporarily deployed to test-trigger-is June 18, 2025 07:52 — with GitHub Actions Inactive
@denik
denikforce-pushed the denik/artifacts_use_dynamic_version branch from c568150 to c44c7dbCompareJune 18, 2025 07:52
@denik
deniktemporarily deployed to test-trigger-is June 18, 2025 07:52 — with GitHub Actions Inactive
@denik
denikforce-pushed the denik/artifacts_use_dynamic_version branch from c44c7db to 8329951CompareJune 18, 2025 08:42
@denik
deniktemporarily deployed to test-trigger-is June 18, 2025 08:42 — with GitHub Actions Inactive
@denik
deniktemporarily deployed to test-trigger-is June 18, 2025 09:04 — with GitHub Actions Inactive
@pietern

Copy link
Copy Markdown
Contributor

@denik The PR summary is out of date with the new name.

@denik
denik disabled auto-merge June 18, 2025 11:01
@denik
denik merged commit d0fe679 into mainJun 18, 2025
@denik
denik deleted the denik/artifacts_use_dynamic_version branch June 18, 2025 11:02
deco-sdk-taggingBot added a commit that referenced this pull request Jun 18, 2025
## Release v0.256.0
### Bundles
* When building Python artifacts as part of "bundle deploy" we no longer delete `dist`, `build`, `*egg-info` and `__pycache__` directories ([#2982](#2982))
* When glob for wheels is used, like "\*.whl", it will filter out different version of the same package and will only take the most recent version ([#2982](#2982))
* Add preset `presets.artifacts_dynamic_version` that automatically enables `dynamic_version: true` on all "whl" artifacts ([#3074](#3074))
* Update client version to "2" for the serverless variation of the default-python template ([#3083](#3083))
* Fix reading dashboard contents when the sync root is different than the bundle root ([#3006](#3006))
* Fix variable resolution for lookup variables with other references ([#3054](#3054))
* Allow users to override the Terraform version to use by setting the `DATABRICKS_TF_VERSION` environment variable ([#3069](#3069))
denik added a commit that referenced this pull request May 20, 2026
## Changes
Add new bundle setting presets.artifacts_dynamic_version. If set to
true, all whl artifacts (in a given target) will get dynamic_version:
true setting.
## Why
We want to set this to true for all artifacts in all targets without
repeating every artifact under targets.
#3042
## Tests
New acceptance test.
---------
Co-authored-by: Julia Crawford (Databricks) <julia.crawford@databricks.com>
denik pushed a commit that referenced this pull request May 20, 2026
## Release v0.256.0
### Bundles
* When building Python artifacts as part of "bundle deploy" we no longer delete `dist`, `build`, `*egg-info` and `__pycache__` directories ([#2982](#2982))
* When glob for wheels is used, like "\*.whl", it will filter out different version of the same package and will only take the most recent version ([#2982](#2982))
* Add preset `presets.artifacts_dynamic_version` that automatically enables `dynamic_version: true` on all "whl" artifacts ([#3074](#3074))
* Update client version to "2" for the serverless variation of the default-python template ([#3083](#3083))
* Fix reading dashboard contents when the sync root is different than the bundle root ([#3006](#3006))
* Fix variable resolution for lookup variables with other references ([#3054](#3054))
* Allow users to override the Terraform version to use by setting the `DATABRICKS_TF_VERSION` environment variable ([#3069](#3069))
Sign up for freeto 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.

6 participants

@denik@pietern@lennartkats-db@shreyas-goenka@juliacrawf-db@ilyakuz-db