Skip to content

feat: automatically set copy_option in publish-odr TDE-1872#1187

Merged
paulfouquet merged 1 commit intomasterfrom
feat/auto-copy-option-tde-1872
Apr 13, 2026
Merged

feat: automatically set copy_option in publish-odr TDE-1872#1187
paulfouquet merged 1 commit intomasterfrom
feat/auto-copy-option-tde-1872

Conversation

@paulfouquet
Copy link
Copy Markdown
Collaborator

@paulfouquet paulfouquet commented Apr 13, 2026

Motivation

copy_option can be determined by the fact that the data publishing is a new dataset (--no-clobber, as no data is supposed to exist in the target location) or an existing dataset update (--force-no-clobber, as we want to override some of the existing data to update it).

Modifications

  • Hardcode the copy_options based on whether or not the ODR URL is provided

Verification

Test workflow

@paulfouquet paulfouquet self-assigned this Apr 13, 2026
@paulfouquet paulfouquet marked this pull request as ready for review April 13, 2026 04:36
@paulfouquet paulfouquet requested review from a team as code owners April 13, 2026 04:36
@paulfouquet paulfouquet enabled auto-merge April 13, 2026 04:37
@paulfouquet paulfouquet added this pull request to the merge queue Apr 13, 2026
Merged via the queue into master with commit f93e61f Apr 13, 2026
4 checks passed
@paulfouquet paulfouquet deleted the feat/auto-copy-option-tde-1872 branch April 13, 2026 05:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants