Skip to content

chore: add cloud environment configuration - #290

Closed
ArnavBalyan wants to merge 2 commits into
mainfrom
chore/company-environment
Closed

ArnavBalyan wants to merge 2 commits into
mainfrom
chore/company-environment

Conversation

@ArnavBalyan

Copy link
Copy Markdown
Contributor

This PR configures .company/environment.json for OpenLake in accordance with the environment specification schema at https://api.thecompany.company/schemas/v1/environment.json.

Environment Configuration

  • scripts.install:
    • Installs required Linux packages (build-essential, pkg-config, clang, cmake, libhwloc-dev, libudev-dev, libssl-dev).
    • Installs rustup with the pinned toolchain 1.91.1 along with rustfmt and clippy.
    • Installs helm for rendering and linting OpenLake Kubernetes Helm charts.
    • Runs cargo fetch to cache workspace dependencies in the snapshot.
  • scripts.prepare:
    • Runs cargo fetch after checkout to reconcile new or changed dependencies.
  • env:
    • Configures RUST_BACKTRACE=1 for detailed backtraces on crashes.
  • Omitted unnecessary lifecycle phases (start is not needed as OpenLake does not require persistent background services for development/testing).

Verification

Verified that both scripts.install and scripts.prepare run successfully to completion in this sandbox environment. All schema checks passed validation.

@the-company-company

Copy link
Copy Markdown

Co · session: Configure cloud environment for openlake

  • Auto-fix ; feedback on this pull request flows to the session and gets addressed
  • Review ; new pushes get reviewed

@co <ask> assigns a task · @co mute pauses · @co archive ends the session

@superagent-security superagent-security Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Superagent found 1 security concern(s).

Comment thread .company/environment.json Outdated

@the-company-company the-company-company Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ No issues found

Reviewed commit f799de7.

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.

1 participant