Skip to content

Add Workbench preview release stream#452

Closed
bschwedler wants to merge 1 commit intorelease-stream-filterfrom
workbench-preview-stream
Closed

Add Workbench preview release stream#452
bschwedler wants to merge 1 commit intorelease-stream-filterfrom
workbench-preview-stream

Conversation

@bschwedler
Copy link
Copy Markdown
Contributor

Summary

  • Add WORKBENCH_PREVIEW_URL using the channel-based dailies endpoint (https://dailies.rstudio.com/rstudio/{channel}/index.json)
  • Wire up PREVIEW stream entries for both workbench and workbench-session products
  • The branch-specific JSON uses a flat structure (workbench.platforms) vs the latest endpoint's wrapped structure (products.workbench.platforms), so preview uses separate resolvers

The {channel} placeholder is resolved from devVersion.values.channel in bakery.yaml, overridable at dispatch time via --dev-channel.

Closes #237

Depends on: #436 (dev-stream/dev-channel plumbing)

Consumed by: posit-dev/images-workbench#75 (adds preview devVersion to bakery.yaml)

Test plan

  • bakery ci matrix --dev-versions only --dev-stream preview --value channel=apple-blossom resolves workbench preview version
  • Existing daily stream continues to resolve from latest endpoint unchanged

Add WORKBENCH_PREVIEW_URL using the channel-based
dailies endpoint ({channel}/index.json). Wire up
PREVIEW stream entries for both workbench and
workbench-session products using the branch-specific
JSON structure (flat product keys instead of the
products wrapper used by the latest endpoint).

The channel value comes from devVersion.values in
bakery.yaml, overridable at dispatch time via
--dev-channel. This lets each preview entry specify
its branch codename (e.g. apple-blossom).

Closes #237
@github-actions
Copy link
Copy Markdown

Test Results

1 334 tests  ±0   1 334 ✅ ±0   8m 16s ⏱️ -34s
    1 suites ±0       0 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 3c165b1. ± Comparison against base commit 79a0ffc.

@ianpittwood
Copy link
Copy Markdown
Contributor

Superseded by #456

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