Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

refactor(renterd): assume autopilot always exists #816

Merged
merged 1 commit into from
Nov 22, 2024
Merged

refactor(renterd): assume autopilot always exists #816

merged 1 commit into from
Nov 22, 2024

Conversation

alexfreska
Copy link
Member

@alexfreska alexfreska commented Nov 18, 2024 •
edited
Loading

  • The app was refactored to assume there is a singular autopilot and its API always exists, even if it is disabled.
  • Fixed missing fields and structure of bus and autopilot state types.

@vercel Vercel
Copy link

vercel bot commented Nov 18, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
renterd ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 22, 2024 9:24pm
4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
explorer ⬜️ Ignored (Inspect) Visit Preview Nov 22, 2024 9:24pm
explorer-zen ⬜️ Ignored (Inspect) Visit Preview Nov 22, 2024 9:24pm
hostd ⬜️ Ignored (Inspect) Visit Preview Nov 22, 2024 9:24pm
website ⬜️ Ignored (Inspect) Visit Preview Nov 22, 2024 9:24pm

@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Nov 18, 2024 •
edited
Loading

🦋 Changeset detected

Latest commit: 09e44d2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
@siafoundation/renterd-js Patch
@siafoundation/renterd-react Patch
@siafoundation/renterd-types Patch
renterd Minor
@siafoundation/clusterd Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

This was referenced Nov 18, 2024
Merged
Merged
Merged
Merged
Merged
Closed
@alexfreska Graphite App
Copy link
Member Author

alexfreska commented Nov 18, 2024 •
edited
Loading

This stack of pull requests is managed by Graphite. Learn more about stacking.

@alexfreska alexfreska mentioned this pull request Nov 18, 2024
Closed
@alexfreska alexfreska force-pushed the refactor_renterd_config_only_patch_changed branch from 571da0c to b4300a2 Compare November 19, 2024 14:20
@alexfreska alexfreska force-pushed the refactor_renterd_assume_autopilot_always_exists branch from d0e8345 to cd5cab4 Compare November 19, 2024 14:20
@alexfreska alexfreska force-pushed the refactor_renterd_config_only_patch_changed branch from b4300a2 to 87eaedc Compare November 19, 2024 14:53
@alexfreska alexfreska force-pushed the refactor_renterd_assume_autopilot_always_exists branch from cd5cab4 to c4f9809 Compare November 19, 2024 14:53
@alexfreska alexfreska force-pushed the refactor_renterd_config_only_patch_changed branch from 87eaedc to db3ab39 Compare November 19, 2024 15:09
@alexfreska alexfreska force-pushed the refactor_renterd_assume_autopilot_always_exists branch from c4f9809 to 8c0fb57 Compare November 19, 2024 15:09
@alexfreska alexfreska force-pushed the refactor_renterd_assume_autopilot_always_exists branch from 8c0fb57 to 42a17a7 Compare November 19, 2024 15:17
@alexfreska alexfreska force-pushed the refactor_renterd_config_only_patch_changed branch from db3ab39 to ac27945 Compare November 19, 2024 15:37
@alexfreska alexfreska force-pushed the refactor_renterd_assume_autopilot_always_exists branch from 42a17a7 to 59a867e Compare November 19, 2024 15:37
@alexfreska alexfreska force-pushed the refactor_renterd_config_only_patch_changed branch from ac27945 to 3e1dff4 Compare November 19, 2024 16:08
@alexfreska alexfreska force-pushed the refactor_renterd_assume_autopilot_always_exists branch from 59a867e to de36391 Compare November 19, 2024 16:08
@alexfreska alexfreska force-pushed the refactor_renterd_config_only_patch_changed branch from 3e1dff4 to e0fa0bc Compare November 19, 2024 19:24
@alexfreska alexfreska force-pushed the refactor_renterd_assume_autopilot_always_exists branch 3 times, most recently from 3ef1ee6 to 2eb0559 Compare November 22, 2024 13:49
This was referenced Nov 22, 2024
Open
Open
@alexfreska Graphite App
Copy link
Member Author

alexfreska commented Nov 22, 2024 •
edited
Loading

Merge activity

  • Nov 22, 3:51 PM EST: A user started a stack merge that includes this pull request via Graphite.
  • Nov 22, 3:55 PM EST: Graphite rebased this pull request as part of a merge.
  • Nov 22, 3:56 PM EST: A user merged this pull request with Graphite.

@alexfreska alexfreska changed the base branch from refactor_renterd_remove_contract_sets to graphite-base/816 November 22, 2024 20:51
@alexfreska alexfreska changed the base branch from graphite-base/816 to main November 22, 2024 20:53
@alexfreska alexfreska force-pushed the refactor_renterd_assume_autopilot_always_exists branch from 2eb0559 to 09e44d2 Compare November 22, 2024 20:54
@alexfreska alexfreska merged commit dfde6e1 into main Nov 22, 2024
14 of 33 checks passed
@alexfreska alexfreska deleted the refactor_renterd_assume_autopilot_always_exists branch November 22, 2024 20:56
This was referenced Nov 22, 2024
Open
Open
Open
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@telestrial telestrial telestrial approved these changes

@ChrisSchinnerl ChrisSchinnerl Awaiting requested review from ChrisSchinnerl

@peterjan peterjan Awaiting requested review from peterjan

Assignees

@alexfreska alexfreska

Labels
None yet
Projects
Sia
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@alexfreska @peterjan @telestrial