Skip to content

feat: fix dynamic devfile for non-containerfile repos, add factory URL and Dev Spaces docs#7

Merged
jflowers merged 1 commit intomainfrom
opsx/cde-docs
Apr 13, 2026
Merged

feat: fix dynamic devfile for non-containerfile repos, add factory URL and Dev Spaces docs#7
jflowers merged 1 commit intomainfrom
opsx/cde-docs

Conversation

@jflowers
Copy link
Copy Markdown
Contributor

Summary

Addresses all three gaps identified in Issue #5:

  • Dynamic devfile install fix: Replaced local /projects/scripts/install-uf-tools.sh reference with curl -fsSL from GitHub raw URL. The dynamic devfile now works when any project is cloned (not just the containerfile repo). Added retry logic (3 attempts, 5s backoff, 60s timeout) and graceful error handling — workspace starts even if tool install fails.
  • Factory URL documentation: Added one-click workspace creation patterns for both devfile.yaml (default) and devfile-dynamic.yaml (explicit ?devfilePath=).
  • Red Hat Dev Spaces notes: Documented expected compatibility (honestly: not yet tested), known differences (per-user namespaces, OpenShift OAuth, image registry), and link to Dev Spaces docs.

Review Council

  • 5 Divisor agents invoked (adversary, architect, guard, sre, testing)
  • Verdict: APPROVE WITH ADVISORIES
  • Fixes applied: retry logic + error handling for curl downloads, trade-off comment for mutable URL
  • Advisory: mutable main branch URL is a conscious design trade-off (D1) — bounded by container isolation

Files Changed

File Changes
devfile-dynamic.yaml Curl from GitHub + retry + error handling
README.md +Factory URLs, +Red Hat Dev Spaces sections
openspec/changes/cde-docs/ Proposal, design, delta spec, tasks

Closes #5

…L and Dev Spaces docs

- Replace local install-uf-tools.sh reference with curl from GitHub
  raw URL (works with any cloned project)
- Add retry logic (3 attempts, 5s backoff) and error handling for
  both curl downloads
- Document factory URL patterns for one-click Che workspace creation
- Add Red Hat Dev Spaces compatibility notes (honest: not yet tested)

Closes #5
@jflowers jflowers merged commit e2aad4a into main Apr 13, 2026
3 checks passed
@jflowers jflowers deleted the opsx/cde-docs branch April 13, 2026 18: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.

docs: add factory URL documentation, inline install script, and Dev Spaces notes

1 participant